Change in ovirt-hosted-engine-ha[master]: init: Only start the services if the config is there and non...
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(a)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(a)redhat.com>
Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>