Change in ovirt-hosted-engine-ha[ovirt-hosted-engine-ha-1.3]: Make sure VDSM had enough time to start before failing

[msivak at redhat.com]

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

Change subject: Make sure VDSM had enough time to start before failing
......................................................................


Make sure VDSM had enough time to start before failing

This improves things on two fronts:

- Using After in the systemd unit instructs the init system to
  make sure the hosted engine service is started after the vdsm
  process, eliminating one race condition.

- Moving the initial ServiceNotUp handling to the _run_agent method
  allows us to try multiple times before we fail.

Change-Id: Ie23b110b3018c88656b2119dbeb24d431e922bdf
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1267511
Signed-off-by: Martin Sivak <msivak at redhat.com>
(cherry picked from commit 838fdde0a67ff88a53f12811eba3aaa52464ccc3)
---
M initscripts/ovirt-ha-agent.service
M ovirt_hosted_engine_ha/agent/agent.py
M ovirt_hosted_engine_ha/agent/hosted_engine.py
3 files changed, 11 insertions(+), 11 deletions(-)

Approvals:
  Sandro Bonazzola: Looks good to me, but someone else must approve
  Martin Sivák: Verified; Looks good to me, approved
  Simone Tiraboschi: Looks good to me, but someone else must approve
  Jenkins CI: Passed CI tests



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie23b110b3018c88656b2119dbeb24d431e922bdf
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-hosted-engine-ha
Gerrit-Branch: ovirt-hosted-engine-ha-1.3
Gerrit-Owner: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Sivák <msivak 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