Yaniv Bronhaim has submitted this change and it was merged. Change subject: vm: Fix vm unpausing during recovery ......................................................................
vm: Fix vm unpausing during recovery Domain monitor thread was not reporting when going from unknown state to known state. The thread would start with a bogus valid state, before checking the domain. If the first domain check found the domain as valid, no notification was emitted. If there were paused vms while connecting to the pool, they were not unpaused as they should. This patch fixes various modules: - Domain monitor status lastCheck is initialized now to special NOT_CHECKED_YET value. The monitor thread use this value to detect the first sample, and emit the missing notification. - To get the new event when monitor thread starts, domain state change callbacks are registered now *before* the monitor threads are started. - To get the vm unpause code to work in recovery mode, the Vm.sdIds is initialized in Vm._run. This probably fixes other issues with recovered vms. This is a backport of http://gerrit.ovirt.org/21649. Change-Id: I56c468edb967de5bcf15259029ace84333fc4b7e Bug-Url: https://bugzilla.redhat.com/1003588 Signed-off-by: Nir Soffer <[email protected]> Reviewed-on: http://gerrit.ovirt.org/21782 Reviewed-by: Yaniv Bronhaim <[email protected]> Tested-by: Yaniv Bronhaim <[email protected]> --- M vdsm/storage/domainMonitor.py M vdsm/storage/hsm.py M vdsm/vm.py 3 files changed, 19 insertions(+), 5 deletions(-) Approvals: Yaniv Bronhaim: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/21782 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I56c468edb967de5bcf15259029ace84333fc4b7e Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: ovirt-3.3 Gerrit-Owner: Nir Soffer <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: [email protected] _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
