Greg Padgett has uploaded a new change for review. 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 <[email protected]> --- M ovirt_hosted_engine_ha/broker/submonitors/engine_health.py 1 file changed, 8 insertions(+), 2 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-hosted-engine-ha refs/changes/10/20210/1 diff --git a/ovirt_hosted_engine_ha/broker/submonitors/engine_health.py b/ovirt_hosted_engine_ha/broker/submonitors/engine_health.py index 3578c0b..5f7b669 100644 --- a/ovirt_hosted_engine_ha/broker/submonitors/engine_health.py +++ b/ovirt_hosted_engine_ha/broker/submonitors/engine_health.py @@ -63,8 +63,14 @@ self.update_result(None) return vm_status = stats['statsList'][0]['status'] - if vm_status.lower() == 'powering up': - self._log.info("VM powering up") + if (vm_status.lower() == 'powering up' + or vm_status.lower() == 'waitforlaunch' + or vm_status.lower() == 'migration source' + or vm_status.lower() == 'rebootinprogress' + or vm_status.lower() == 'restoringstate' + or vm_status.lower() == 'savingstate' + or vm_status.lower() == 'paused'): + self._log.info("VM status: %s", vm_status) self.update_result('vm-up bad-health-status') return if vm_status.lower() != 'up': -- To view, visit http://gerrit.ovirt.org/20210 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I1b73870306a559505d3441e9a2e57dfb0d191bae Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Greg Padgett <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
