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 <gpadg...@redhat.com>
Gerrit-Reviewer: Adam Litke <ali...@redhat.com>
Gerrit-Reviewer: Greg Padgett <gpadg...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: No
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to