Nir Soffer has submitted this change and it was merged. Change subject: lib: utils: consolidate Error class in one place ......................................................................
lib: utils: consolidate Error class in one place Few utilities code have a duplicate Error exception, that they raise when one command run through utils.execCmd fails. Since this Error is closely related to utils.execCmd, we remove the duplicate definition of Error and we move it in one place. Change-Id: Ia490c055e6e5c637eba82ccffb9d2dfc748db8f5 Signed-off-by: Francesco Romani <[email protected]> Reviewed-on: https://gerrit.ovirt.org/47964 Reviewed-by: Piotr Kliczewski <[email protected]> Reviewed-by: Nir Soffer <[email protected]> Continuous-Integration: Jenkins CI --- M lib/vdsm/cmdutils.py M lib/vdsm/taskset.py M lib/vdsm/udevadm.py M tests/tasksetTests.py M vdsm/supervdsmServer 5 files changed, 25 insertions(+), 33 deletions(-) Approvals: Piotr Kliczewski: Looks good to me, but someone else must approve Nir Soffer: Looks good to me, approved Jenkins CI: Passed CI tests Francesco Romani: Verified -- To view, visit https://gerrit.ovirt.org/47964 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia490c055e6e5c637eba82ccffb9d2dfc748db8f5 Gerrit-PatchSet: 8 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Francesco Romani <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Polednik <[email protected]> Gerrit-Reviewer: Milan Zamazal <[email protected]> Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: gerrit-hooks <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
