Dan Kenigsberg has submitted this change and it was merged. Change subject: utils: move nice ionice sid and sudo out of execCmd ......................................................................
utils: move nice ionice sid and sudo out of execCmd Utility commands as nice, ionice, setsid and sudo should be reusable with regular Popen calls. In order to make them composable this patch moves them out of execCmd. Change-Id: I442c64e8b9cfb3933dbe9879b1d442d0e96fbe40 Signed-off-by: Federico Simoncelli <[email protected]> Reviewed-on: http://gerrit.ovirt.org/33691 Reviewed-by: Nir Soffer <[email protected]> Reviewed-by: Francesco Romani <[email protected]> Reviewed-by: Adam Litke <[email protected]> Reviewed-by: Dan Kenigsberg <[email protected]> --- M debian/vdsm-python.install M lib/vdsm/Makefile.am A lib/vdsm/cmdutils.py M lib/vdsm/utils.py M vdsm.spec.in 5 files changed, 66 insertions(+), 14 deletions(-) Approvals: Nir Soffer: Looks good to me, but someone else must approve Adam Litke: Looks good to me, but someone else must approve Federico Simoncelli: Verified Dan Kenigsberg: Looks good to me, approved Francesco Romani: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/33691 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I442c64e8b9cfb3933dbe9879b1d442d0e96fbe40 Gerrit-PatchSet: 7 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Adam Litke <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Dima Kuznetsov <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: Saggi Mizrahi <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: Yoav Kleinberger <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
