Change in vdsm[master]: Fix some code didn't use VDSMUSER

danken at redhat.com danken at redhat.com
Sat Apr 7 10:39:47 UTC 2012


Dan Kenigsberg has posted comments on this change.

Change subject: Fix some code didn't use VDSMUSER
......................................................................


Patch Set 2: I would prefer that you didn't submit this

(1 inline comment)

ahh, so much noise...

I'm not sure about changing the hooks, they are unsupported examples, and may be better off left as they are. I really do not know.

minor patch-splitting comment inline.

....................................................
File vdsm/sudoers.vdsm.in
Line 44:     @SETSID_PATH@ @IONICE_PATH@ -c? -n? @SU_PATH@ @IOUSER@ -s /bin/sh -c /usr/libexec/vdsm/spmprotect.sh*, \
this belongs to the former patch!

--
To view, visit http://gerrit.ovirt.org/3363
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ib4f834b1b34ffcdbbf32a1f5cd9e7728976b623c
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Xu He Jie <xuhj at linux.vnet.ibm.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Xu He Jie <xuhj at linux.vnet.ibm.com>


More information about the vdsm-patches mailing list