Change in vdsm[master]: storage: don't chmod local storage symlink

gpadgett at redhat.com gpadgett at redhat.com
Thu Aug 6 20:38:53 UTC 2015


Greg Padgett has posted comments on this change.

Change subject: storage: don't chmod local storage symlink
......................................................................


Patch Set 1:

(1 comment)

https://gerrit.ovirt.org/#/c/44446/1/vdsm/storage/storageServer.py
File vdsm/storage/storageServer.py:

Line 617
Line 618
Line 619
Line 620
Line 621
> Can you verify that we setup permissions properly when creating a storage d
Verified, everything looks good through the full flow including connection.


-- 
To view, visit https://gerrit.ovirt.org/44446
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: If1009033d7cb32add02c9bcc19840c601ea70b4e
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Greg Padgett <gpadgett at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Greg Padgett <gpadgett at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list