Dan Kenigsberg has submitted this change and it was merged. Change subject: virt: make vmdevices a subpackage ......................................................................
virt: make vmdevices a subpackage This patch transforms the vmdevices module into a subpackage, to make it suitable to host the remaining, big devices in nicely separate modules. A few decisions where made for the sake of practicality and to make this patch minimal, although meaningful: - the subpackage retains the 'vm' prefix instead the simpler 'devices', to reduce name clashes. - not-empty __init__, hosting subpackage imports and constants. Nothing else will go there. - 'small' devices moved into a generic module named 'core', which is the shortest yet meaningful name I could find. No changes besides import fixes and code being scatthered around. Change-Id: I66ea868df27b7dd86ba884804016b34721ff7f48 Signed-off-by: Francesco Romani <[email protected]> Reviewed-on: http://gerrit.ovirt.org/35482 Reviewed-by: Dan Kenigsberg <[email protected]> --- M configure.ac M debian/vdsm.install M tests/vmTests.py M vdsm.spec.in M vdsm/API.py M vdsm/clientIF.py M vdsm/virt/Makefile.am M vdsm/virt/vm.py A vdsm/virt/vmdevices/Makefile.am A vdsm/virt/vmdevices/__init__.py R vdsm/virt/vmdevices/core.py R vdsm/virt/vmdevices/hwclass.py 12 files changed, 87 insertions(+), 36 deletions(-) Approvals: Dan Kenigsberg: Looks good to me, approved Francesco Romani: Verified -- To view, visit http://gerrit.ovirt.org/35482 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I66ea868df27b7dd86ba884804016b34721ff7f48 Gerrit-PatchSet: 9 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: Martin Polednik <[email protected]> Gerrit-Reviewer: Vinzenz Feenstra <[email protected]> Gerrit-Reviewer: Vitor de Lima <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
