Dan Kenigsberg has submitted this change and it was merged. Change subject: BZ#742267 server_connection: checking access is enough ......................................................................
BZ#742267 server_connection: checking access is enough There is no need to verify that a directory/mountpoint is owned by vdsm:kvm. All we need is vdsm accessibility. Ownership requirement was added in downstream commit 9fb05359 for local storage, and later (http://gerrit.usersys.redhat.com/776) extended to all storage types. Change-Id: I799374f13cc9d5fe193d76479f738e0a5fa02836 --- M vdsm/storage/fileUtils.py M vdsm/storage/storage_connection.py 2 files changed, 0 insertions(+), 12 deletions(-) Approvals: Dan Kenigsberg: Looks good to me, approved Igor Lvovsky: Looks good to me, but someone else must approve David Naori: Verified -- To view, visit http://gerrit.usersys.redhat.com/983 To unsubscribe, visit http://gerrit.usersys.redhat.com/settings Gerrit-MessageType: merged Gerrit-Change-Id: I799374f13cc9d5fe193d76479f738e0a5fa02836 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: David Naori <[email protected]> Gerrit-Reviewer: Igor Lvovsky <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://fedorahosted.org/mailman/listinfo/vdsm-patches
