Martin Sivák has submitted this change and it was merged.
Change subject: Treat migration state as Up and check engine health normally
......................................................................
Treat migration state as Up and check engine health normally
We used to treat migration source state as Vm up Health bad.
That is not the correct report as the source Vm still works
properly during live-merge.
This caused a situation when a migration caused other hosts
to try to start the Vm and fail because of sanlock protection.
This moves migration source state to the group of states with
Vm up Health <query the status page> behaviour.
Bug-Url:
https://bugzilla.redhat.com/show_bug.cgi?id=1093366
Change-Id: If3c813461909103b11266c902ccc51b2df2d0d36
Signed-off-by: Martin Sivak <msivak(a)redhat.com>
(cherry picked from commit 0554e4208c4915863baeaa16fa5baeb678bfa0cf)
---
M ovirt_hosted_engine_ha/broker/submonitors/engine_health.py
1 file changed, 9 insertions(+), 10 deletions(-)
Approvals:
Martin Sivák: Looks good to me, approved
Jiří Moskovčák: Verified; Looks good to me, but someone else must approve
--
To view, visit
http://gerrit.ovirt.org/29902
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: If3c813461909103b11266c902ccc51b2df2d0d36
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-hosted-engine-ha
Gerrit-Branch: ovirt-hosted-engine-ha-1.2
Gerrit-Owner: Martin Sivák <msivak(a)redhat.com>
Gerrit-Reviewer: Jiří Moskovčák <jmoskovc(a)redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server