Change in ovirt-hosted-engine-ha[master]: init: Only start the services if the config is there and non...

[msivak at redhat.com]

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

Change subject: init: Only start the services if the config is there and non-empty
......................................................................


init: Only start the services if the config is there and non-empty

Previously we only checked for the presence of the hosted-engine config file.
This could lead to non graceful exits if the config file was there but empty.
Now we also check that the file has some contents before starting the services.

Change-Id: Id762a0bc47f050fc30a9047936767cf783e711be
Signed-off-by: Fabian Deutsch <fabiand at fedoraproject.org>
---
M initscripts/ovirt-ha-agent.service.in
M initscripts/ovirt-ha-broker.service.in
2 files changed, 2 insertions(+), 2 deletions(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Id762a0bc47f050fc30a9047936767cf783e711be
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-hosted-engine-ha
Gerrit-Branch: master
Gerrit-Owner: Fabian Deutsch <fabiand at redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand 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: gerrit-hooks <automation at ovirt.org>