Change in ovirt-hosted-engine-ha[master]: states, broker: Check for already starting VMs on other hosts

[msivak at redhat.com]

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 at 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 at redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfediuck at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Roman Mohr <rmohr at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos at redhat.com>
Gerrit-Reviewer: automation at ovirt.org