Change in vdsm[ovirt-3.6]: virt: introduce virdomain module

fromani at redhat.com fromani at redhat.com
Mon Aug 31 11:40:31 UTC 2015


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 <fromani at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/44812
Continuous-Integration: Jenkins CI
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/45378
Reviewed-by: Martin Polednik <mpolednik at redhat.com>
---
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 <fromani at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Polednik <mpolednik at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list