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 danken@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: David Naori dnaori@redhat.com Gerrit-Reviewer: Igor Lvovsky ilvovsky@redhat.com
vdsm-patches@lists.fedorahosted.org