Martin Sivák has uploaded a new change for review. Change subject: Consider engine to be safe and sound when the state is Running ......................................................................
Consider engine to be safe and sound when the state is Running VDSM should not return this state at all according to it's API documentation. But it does. Running is returned when VM is Up and the guest agent is running. Change-Id: I51507f1ed3c642280b600ee512f4bcc8d5b9bb88 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1034787 Signed-off-by: Martin Sivak <msi...@redhat.com> --- M ovirt_hosted_engine_ha/broker/submonitors/engine_health.py 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-hosted-engine-ha refs/changes/34/21734/1 diff --git a/ovirt_hosted_engine_ha/broker/submonitors/engine_health.py b/ovirt_hosted_engine_ha/broker/submonitors/engine_health.py index c0e714e..60bbfa9 100644 --- a/ovirt_hosted_engine_ha/broker/submonitors/engine_health.py +++ b/ovirt_hosted_engine_ha/broker/submonitors/engine_health.py @@ -78,7 +78,7 @@ extra=log_filter.lf_args('status', 60)) self.update_result('vm-up bad-health-status') return - if vm_status != 'up': + if vm_status not in ('up', 'running'): self._log.info("VM not running on this host, status %s", vm_status, extra=log_filter.lf_args('status', 60)) self.update_result('vm-down') -- To view, visit http://gerrit.ovirt.org/21734 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I51507f1ed3c642280b600ee512f4bcc8d5b9bb88 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Martin Sivák <msi...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
