Change in ovirt-hosted-engine-ha[master]: states, broker: Check for already starting VMs on other hosts
Martin Sivák has submitted this change and it was merged.
Change subject: states, broker: Check for already starting VMs on other hosts
......................................................................
states, broker: Check for already starting VMs on other hosts
When two or more hosts try to take over when one host goes down, only
one of them can start the hosted engine VM. The others will fail to
acquire the storage lock.
The health monitor now detects when this is the case, so that the host,
which could not acquire the lock, can remove the VM which it could not
start and go back to the down state.
Change-Id: Ia5d321039801a9868f2b1232ce0acfdfe756b477
Bug-Url: https://bugzilla.redhat.com/1207634
Signed-off-by: Roman Mohr <rmohr(a)redhat.com>
---
M ovirt_hosted_engine_ha/agent/states.py
M ovirt_hosted_engine_ha/broker/submonitors/engine_health.py
M ovirt_hosted_engine_ha/lib/engine.py
M ovirt_hosted_engine_ha/lib/util.py
4 files changed, 25 insertions(+), 4 deletions(-)
Approvals:
Martin Sivák: Looks good to me, approved
Roman Mohr: Verified
Jenkins CI: Passed CI tests
--
To view, visit https://gerrit.ovirt.org/45592
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia5d321039801a9868f2b1232ce0acfdfe756b477
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-hosted-engine-ha
Gerrit-Branch: master
Gerrit-Owner: Roman Mohr <rmohr(a)redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com>
Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org