From Dan Kenigsberg <[email protected]>: Dan Kenigsberg has submitted this change and it was merged.
Change subject: utils: Extract CommandPath from utils module ...................................................................... utils: Extract CommandPath from utils module CommandPath has been relocated to common.cmdutils, to make it clearer that it is being used by multiple subsystems, and verifying that it has no dependencies on on non-common vdsm code. Change-Id: I7cd1021c37205ab2f10c653ab1ee961a0bad919f Signed-off-by: Edward Haas <[email protected]> --- M lib/vdsm/alignmentScan.py A lib/vdsm/common/cmdutils.py M lib/vdsm/gluster/api.py M lib/vdsm/gluster/cli.py M lib/vdsm/gluster/storagedev.py M lib/vdsm/network/configurators/iproute2.py M lib/vdsm/network/ip/dhclient.py M lib/vdsm/network/ipwrapper.py M lib/vdsm/network/nm/networkmanager.py M lib/vdsm/network/ovs/driver/vsctl.py M lib/vdsm/numa.py M lib/vdsm/qemuimg.py M lib/vdsm/storage/blkdiscard.py M lib/vdsm/storage/curlImgWrap.py M lib/vdsm/storage/multipath.py M lib/vdsm/supervdsm_api/containers.py M lib/vdsm/supervdsm_api/systemd.py M lib/vdsm/tool/configurators/lvm.py M lib/vdsm/tool/configurators/passwd.py M lib/vdsm/tool/service.py M lib/vdsm/tool/transient.py M lib/vdsm/udevadm.py M lib/vdsm/utils.py M lib/vdsm/v2v.py M lib/vdsm/virt/containers/command.py M lib/vdsm/virt/containers/docker.py M lib/vdsm/virtsparsify.py M lib/vdsm/virtsysprep.py A tests/common/cmdutils_test.py M tests/containers/conttestlib.py M tests/functional/networkTests.py M tests/functional/virtTests.py M tests/modprobe.py M tests/network/dhcp.py M tests/network/firewall.py M tests/network/nettestlib.py M tests/network/nmnettestlib.py M tests/utils_test.py M tests/v2v_test.py M tests/virttests/seal_job_test.py M vdsm/storage/curl-img-wrap M vdsm_hooks/ethtool_options/ethtool_options.py M vdsm_hooks/openstacknet/openstacknet_utils.py 43 files changed, 240 insertions(+), 188 deletions(-) Approvals: Piotr Kliczewski: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Dan Kenigsberg: Looks good to me, approved Francesco Romani: Looks good to me, but someone else must approve Edward Haas: Verified -- To view, visit https://gerrit.ovirt.org/76894 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7cd1021c37205ab2f10c653ab1ee961a0bad919f Gerrit-PatchSet: 6 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Edward Haas <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Edward Haas <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Irit Goihman <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: Petr Horáček <[email protected]> Gerrit-Reviewer: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: gerrit-hooks <[email protected]> _______________________________________________ vdsm-patches mailing list -- [email protected] To unsubscribe send an email to [email protected]
