Xu He Jie has posted comments on this change. Change subject: Fix some code didn't use VDSMUSER ......................................................................
Patch Set 2: (1 inline comment) If changing hooks can make it works when @VDSMUSER@ was changed. That won't make people confuse what happened. Maybe other people have idea for this. .................................................... 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*, \ Second thinking...move this to former patch is better! Send next patch set for this later. -- 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 <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Xu He Jie <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://fedorahosted.org/mailman/listinfo/vdsm-patches
