Dan Kenigsberg has submitted this change and it was merged. Change subject: vdsm-tool: unify ad-hoc execCmd implementations ......................................................................
vdsm-tool: unify ad-hoc execCmd implementations There are various ad-hoc execCmd implementations in the modules of vdsm-tool. Before the parent of this patch, execCmd is in storage.misc, we can not import storage.misc.execCmd without modifying PYTHONPATH. The parent of this patch moved execCmd to vdsm.utils, so this patch makes use of the 'official' execCmd implementation in vdsm.utils for all the modules of vdsm-tool. Change-Id: I8ca64ec5dbfebe5c5090ec8fac46785a538b79d3 Signed-off-by: Zhou Zheng Sheng <[email protected]> --- M lib/vdsm/tool/load_needed_modules.py.in M lib/vdsm/tool/validate_ovirt_certs.py.in 2 files changed, 4 insertions(+), 10 deletions(-) Approvals: Dan Kenigsberg: Looks good to me, approved Zhou Zheng Sheng: Verified -- To view, visit http://gerrit.ovirt.org/13917 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8ca64ec5dbfebe5c5090ec8fac46785a538b79d3 Gerrit-PatchSet: 5 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Zhou Zheng Sheng <[email protected]> Gerrit-Reviewer: Antoni Segura Puimedon <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Douglas Schilling Landgraf <[email protected]> Gerrit-Reviewer: Ewoud Kohl van Wijngaarden <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Mark Wu <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: Zhou Zheng Sheng <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
