Change in ovirt-hosted-engine-ha[master]: Support for unconfigured startup handling and API access

msivak at redhat.com msivak at redhat.com
Thu Aug 6 11:57:51 UTC 2015


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 at 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 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: Roman Mohr <rmohr at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan 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



More information about the Engine-commits mailing list