Change in ovirt-hosted-engine-ha[master]: Add transition to EngineMaybeAway from EngineStarting

Code Review gerrit at ovirt.org
Mon May 22 10:26:25 UTC 2017


>From Martin Sivák <msivak at redhat.com>:

Martin Sivák has submitted this change and it was merged.

Change subject: Add transition to EngineMaybeAway from EngineStarting
......................................................................


Add transition to EngineMaybeAway from EngineStarting

When the agent tries to start the VM and it is already
running on another host, the VDSM reports an error:
"Failed to acquire lock".

Broker periodically checks the state of the VM,
but it sees the error message only once and then
the VM is destroyed.
The agent may miss the message if it sleeps too long.

This patch adds transition to EngineMaybeAway,
to check if the VM is running elsewhere.

Change-Id: I4d92e021ebf6398d7846a233531f1be5e94dbe5f
Bug-Url: https://bugzilla.redhat.com/1448699
Signed-off-by: Andrej Krejcir <akrejcir at redhat.com>
---
M ovirt_hosted_engine_ha/agent/states.py
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Denis Chaplygin: Looks good to me, but someone else must approve
  Martin Sivák: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Andrej Krejcir: Verified



-- 
To view, visit https://gerrit.ovirt.org/77030
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I4d92e021ebf6398d7846a233531f1be5e94dbe5f
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-hosted-engine-ha
Gerrit-Branch: master
Gerrit-Owner: Andrej Krejcir <akrejcir at redhat.com>
Gerrit-Reviewer: Andrej Krejcir <akrejcir at redhat.com>
Gerrit-Reviewer: Denis Chaplygin <dchaplyg at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Jenny Tokar <jtokar at redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Phillip Bailey <phbailey at redhat.com>
Gerrit-Reviewer: Roman Mohr <rmohr at redhat.com>
Gerrit-Reviewer: Yanir Quinn <yquinn at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list