Change in ovirt-hosted-engine-ha[master]: Treat migration state as Up and check engine health normally

msivak at redhat.com msivak at redhat.com
Thu Jul 10 19:18:54 UTC 2014


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 at redhat.com>
---
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



-- 
To view, visit http://gerrit.ovirt.org/29787
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: If3c813461909103b11266c902ccc51b2df2d0d36
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-hosted-engine-ha
Gerrit-Branch: master
Gerrit-Owner: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Jiří Moskovčák <jmoskovc at redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list