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

gpadgett at redhat.com gpadgett at redhat.com
Wed Aug 5 23:59:48 UTC 2015


Greg Padgett has posted comments on this change.

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


Patch Set 1: Verified+1

Verified that local storage domains can be created, disconnected, reconnected, etc.  There's no change in behavior except that when creating storage owned by root with perms 0777 there is no error the first time creation is attempted (which was due to the chmod call).

-- 
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: No


More information about the vdsm-patches mailing list