Dan Kenigsberg has submitted this change and it was merged.
Change subject: execCmd: no need to pass the default sudo=False value
......................................................................
execCmd: no need to pass the default sudo=False value
These are remnants from the days where we had a crazier default.
Cleaned with
sed -i 's/execCmd\(.*\), sudo=False/execCmd\1/'
and a couple of manual additions.
Change-Id: Id8c0e30756113ddb35d2e51984c3ea72bbe45a44
Signed-off-by: Dan Kenigsberg <[email protected]>
Signed-off-by: Vinzenz Feenstra <[email protected]>
Reviewed-on: http://gerrit.ovirt.org/24145
Reviewed-by: Francesco Romani <[email protected]>
Reviewed-by: Nir Soffer <[email protected]>
Reviewed-by: Yaniv Bronhaim <[email protected]>
---
M lib/vdsm/utils.py
M tests/functional/virtTests.py
M tests/miscTests.py
M tests/utilsTests.py
M vdsm/hooks.py
M vdsm/ksm.py
M vdsm/storage/blockSD.py
M vdsm/storage/clusterlock.py
M vdsm/storage/devicemapper.py
M vdsm/storage/fuser.py
M vdsm/storage/misc.py
M vdsm/storage/multipath.py
M vdsm/storage/storage_mailbox.py
M vdsm/storage/volume.py
M vdsm/supervdsmServer
M vdsm/tc.py
M vdsm_hooks/checkimages/before_vm_start.py
M vdsm_hooks/hostusb/after_vm_destroy.py
M vdsm_hooks/hostusb/before_vm_start.py
M vdsm_hooks/scratchpad/before_vm_start.py
20 files changed, 43 insertions(+), 47 deletions(-)
Approvals:
Nir Soffer: Looks good to me, but someone else must approve
Yaniv Bronhaim: Looks good to me, but someone else must approve
Dan Kenigsberg: Verified; Looks good to me, approved
Francesco Romani: Looks good to me, but someone else must approve
--
To view, visit http://gerrit.ovirt.org/24145
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Id8c0e30756113ddb35d2e51984c3ea72bbe45a44
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Francesco Romani <[email protected]>
Gerrit-Reviewer: Nir Soffer <[email protected]>
Gerrit-Reviewer: Vinzenz Feenstra <[email protected]>
Gerrit-Reviewer: Yaniv Bronhaim <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
