Change in ovirt-hosted-engine-ha[v2.1.z]: Add transition to EngineMaybeAway from EngineStarting

From Martin Sivák <msivak@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@redhat.com> (cherry picked from commit a0f909dbb95aad5d96cf5a671582ee8526f28a58) --- M ovirt_hosted_engine_ha/agent/states.py 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Martin Sivák: Looks good to me, approved Jenkins CI: Passed CI tests Andrej Krejcir: Verified -- To view, visit https://gerrit.ovirt.org/77166 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4d92e021ebf6398d7846a233531f1be5e94dbe5f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: v2.1.z Gerrit-Owner: Andrej Krejcir <akrejcir@redhat.com> Gerrit-Reviewer: Andrej Krejcir <akrejcir@redhat.com> Gerrit-Reviewer: Denis Chaplygin <dchaplyg@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Jenny Tokar <jtokar@redhat.com> Gerrit-Reviewer: Martin Sivák <msivak@redhat.com> Gerrit-Reviewer: Phillip Bailey <phbailey@redhat.com> Gerrit-Reviewer: Roman Mohr <rmohr@redhat.com> Gerrit-Reviewer: Yanir Quinn <yquinn@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>
participants (1)
-
Code Review