Dan Kenigsberg has submitted this change and it was merged. Change subject: vm: do not use _dom for powerdown ......................................................................
vm: do not use _dom for powerdown We must avoid the proliferation of trivial helper methods, but using private VM fields across modules is a greater evil, even if modules are part of the same package and affine, like vm.py and vmpowerdown.py To avoid that, a new simple method is added to request clean shutdown of a VM through ACPI events. Change-Id: Ia99a8a557b9bbbaf693eb0e89aa9f39c0ecce400 Signed-off-by: Francesco Romani <[email protected]> Reviewed-on: http://gerrit.ovirt.org/31353 Reviewed-by: Dan Kenigsberg <[email protected]> --- M vdsm/virt/vm.py M vdsm/virt/vmpowerdown.py 2 files changed, 4 insertions(+), 1 deletion(-) Approvals: Dan Kenigsberg: Looks good to me, approved Francesco Romani: Verified -- To view, visit http://gerrit.ovirt.org/31353 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia99a8a557b9bbbaf693eb0e89aa9f39c0ecce400 Gerrit-PatchSet: 3 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: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
