Change in ovirt-hosted-engine-ha[master]: monitor: recognize additional vm states

gpadgett at redhat.com gpadgett at redhat.com
Fri Oct 25 15:38:27 UTC 2013


Greg Padgett has submitted this change and it was merged.

Change subject: monitor: recognize additional vm states
......................................................................


monitor: recognize additional vm states

The 'waitforlaunch' state was seen on a vm, causing incorrect reporting
that the vm was down.  Upon further investigation, there were several
unhandled vm states.  These will be considered 'vm-up bad-health-status'
by the engine health monitor, giving the host/vm a window of time to
pass through the transient state before any action is taken by the
agent.

Change-Id: I1b73870306a559505d3441e9a2e57dfb0d191bae
Bug-Url: https://bugzilla.redhat.com/1019468
Signed-off-by: Greg Padgett <gpadgett at redhat.com>
---
M ovirt_hosted_engine_ha/broker/submonitors/engine_health.py
1 file changed, 11 insertions(+), 4 deletions(-)

Approvals:
  Sandro Bonazzola: Looks good to me, but someone else must approve
  Alex Lourie: Looks good to me, but someone else must approve
  Greg Padgett: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I1b73870306a559505d3441e9a2e57dfb0d191bae
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-hosted-engine-ha
Gerrit-Branch: master
Gerrit-Owner: Greg Padgett <gpadgett at redhat.com>
Gerrit-Reviewer: Alex Lourie <alourie at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Greg Padgett <gpadgett at redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list