Dan Kenigsberg has submitted this change and it was merged. Change subject: Refactoring storage.misc: moving watchCmd and its deps to vdsm.utils. ......................................................................
Refactoring storage.misc: moving watchCmd and its deps to vdsm.utils. This refactoring is needed to break the depedency of lib/vdsm/qemuImg from vdsm/storage/misc. Modules inside lib/vdsm should not use anything that is not present in the python std lib or in a dependency of the vdsm-python package. Change-Id: I45eaf487b0260863af09087e5114fb836276a750 Signed-off-by: Giuseppe Vallarelli <gvall...@redhat.com> --- M lib/vdsm/qemuImg.py M lib/vdsm/utils.py M tests/main.py M tests/miscTests.py M vdsm/storage/Makefile.am M vdsm/storage/blockSD.py M vdsm/storage/blockVolume.py M vdsm/storage/clusterlock.py M vdsm/storage/fileSD.py M vdsm/storage/fileVolume.py M vdsm/storage/image.py M vdsm/storage/misc.py M vdsm/storage/storage_exception.py M vdsm/storage/volume.py 14 files changed, 105 insertions(+), 94 deletions(-) Approvals: Giuseppe Vallarelli: Verified Dan Kenigsberg: Looks good to me, approved Zhou Zheng Sheng: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/14408 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I45eaf487b0260863af09087e5114fb836276a750 Gerrit-PatchSet: 17 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Giuseppe Vallarelli <gvall...@redhat.com> Gerrit-Reviewer: Antoni Segura Puimedon <asegu...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Federico Simoncelli <fsimo...@redhat.com> Gerrit-Reviewer: Giuseppe Vallarelli <gvall...@redhat.com> Gerrit-Reviewer: Livnat Peer <lp...@redhat.com> Gerrit-Reviewer: Zhou Zheng Sheng <zhshz...@linux.vnet.ibm.com> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
