Dan Kenigsberg 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).

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]>
---
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
  Dan Kenigsberg: Looks good to me, approved
  Francesco Romani: Verified



-- 
To view, visit https://gerrit.ovirt.org/44812
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I527ca3aa5733a16b2c81c9fa433125bc189c64ca
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Francesco Romani <[email protected]>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: [email protected]
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to