Francesco Romani has submitted this change and it was merged. Change subject: virt: introduce virdomain module ......................................................................
virt: introduce virdomain module Add virdomain module to hold the libvirt Domain wrapper that virt code needs, to avoid ugly circular dependencies between periodic.py, sampling.py and vm.py, and last but not least, to shed some load from vm.py This patch does the code move, but it is peppered with trivial rename and trivial fix (inherit from object). Bug-Url: https://bugzilla.redhat.com/1154389 Backport-To: 3.6 Change-Id: I527ca3aa5733a16b2c81c9fa433125bc189c64ca Signed-off-by: Francesco Romani <[email protected]> Reviewed-on: https://gerrit.ovirt.org/44812 Continuous-Integration: Jenkins CI Reviewed-by: Dan Kenigsberg <[email protected]> Reviewed-on: https://gerrit.ovirt.org/45378 Reviewed-by: Martin Polednik <[email protected]> --- M debian/vdsm.install M tests/vmTests.py M vdsm.spec.in M vdsm/virt/Makefile.am M vdsm/virt/sampling.py A vdsm/virt/virdomain.py M vdsm/virt/vm.py 7 files changed, 98 insertions(+), 72 deletions(-) Approvals: Jenkins CI: Passed CI tests Francesco Romani: Verified; Looks good to me, approved Martin Polednik: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/45378 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I527ca3aa5733a16b2c81c9fa433125bc189c64ca Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: ovirt-3.6 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: [email protected] _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
