Change in ovirt-hosted-engine-ha[master]: broker, agent: more robust service startup/shutdown

[msivak at redhat.com]

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

Change subject: broker, agent: more robust service startup/shutdown
......................................................................


broker, agent: more robust service startup/shutdown

The agent and broker had no ordering, and there was no way for an
external program to reliably tell when the broker was ready to
serve requests.

With this change:
- The agent and broker services gain an ordering dependency between
  them in systemd.
- The broker attempts to clean its socket file upon exit, so that the
  existence of the file generally indicates that the broker is ready
  to serve requests.
- The agent will be more patient in attempting to connect to the broker.

Change-Id: I392ddf54503d7b9ef3448d22385b547222c0cc86
Bug-Url: https://bugzilla.redhat.com/1018829
Signed-off-by: Greg Padgett <gpadgett at redhat.com>
---
M initscripts/ovirt-ha-agent.init.in
M initscripts/ovirt-ha-agent.service
M ovirt_hosted_engine_ha/agent/constants.py.in
M ovirt_hosted_engine_ha/agent/hosted_engine.py
M ovirt_hosted_engine_ha/broker/broker.py
M ovirt_hosted_engine_ha/broker/listener.py
M ovirt_hosted_engine_ha/lib/brokerlink.py
7 files changed, 48 insertions(+), 8 deletions(-)

Approvals:
  Sandro Bonazzola: Looks good to me, but someone else must approve
  Martin Sivák: Verified; Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/20561
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I392ddf54503d7b9ef3448d22385b547222c0cc86
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-hosted-engine-ha
Gerrit-Branch: master
Gerrit-Owner: Greg Padgett <gpadgett at redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfediuck at redhat.com>
Gerrit-Reviewer: Greg Padgett <gpadgett at redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server