Martin Sivák has submitted this change and it was merged.
Change subject: Support for unconfigured startup handling and API access
......................................................................
Support for unconfigured startup handling and API access
Hosted engine correctly shuts down when no configuration can be
found. Missing configuration files generate error message or
debug message depending on the importance of the file.
API call to get the host id returns None in the unconfigured case
and also handles missing config files properly.
Configuration file access errors are reported to a separate
logger (either ovirt_hosted_engine_ha.env.config or the passed logger
with .config added to the end)
Bug-Url:
https://bugzilla.redhat.com/show_bug.cgi?id=1248539
Change-Id: I0b6a60dda3afc97a91f542c2449174fc09131a4b
Signed-off-by: Martin Sivak <msivak(a)redhat.com>
---
M ovirt_hosted_engine_ha/agent/agent.py
M ovirt_hosted_engine_ha/agent/hosted_engine.py
M ovirt_hosted_engine_ha/client/client.py
M ovirt_hosted_engine_ha/env/config.py
4 files changed, 108 insertions(+), 28 deletions(-)
Approvals:
Martin Sivák: Verified; Looks good to me, approved
Roman Mohr: Looks good to me, but someone else must approve
Jenkins CI: Passed CI tests
--
To view, visit
https://gerrit.ovirt.org/44325
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I0b6a60dda3afc97a91f542c2449174fc09131a4b
Gerrit-PatchSet: 8
Gerrit-Project: ovirt-hosted-engine-ha
Gerrit-Branch: master
Gerrit-Owner: Martin Sivák <msivak(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: Roman Mohr <rmohr(a)redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org