Francesco Romani has submitted this change and it was merged. Change subject: vm: make acpiShutdown handle NotConnectedError ......................................................................
vm: make acpiShutdown handle NotConnectedError In the Vm shutdown flow it is possible that the domain is disconnected asynchronously as in any other Vm flow. This patch makes Vm.acpiShutdown() handle the sudden domain disconnections in a graceful manner. vmpowerdown needs to be aware if acpiShutdown failed for this reason, so it can go ahead with the callback chain without wasteful waits, Change-Id: I244f00d62ee24fb42ba3d654961a8fc22f4a6c25 Backport-To: 3.6 Bug-Url: https://bugzilla.redhat.com/1154389 Signed-off-by: Francesco Romani <[email protected]> Reviewed-on: https://gerrit.ovirt.org/34879 Continuous-Integration: Jenkins CI Reviewed-by: Dan Kenigsberg <[email protected]> Reviewed-on: https://gerrit.ovirt.org/45379 Reviewed-by: Martin Polednik <[email protected]> --- M tests/vmTests.py M tests/vmfakelib.py M vdsm/virt/vm.py M vdsm/virt/vmpowerdown.py 4 files changed, 28 insertions(+), 2 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/45379 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I244f00d62ee24fb42ba3d654961a8fc22f4a6c25 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
