Change in ovirt-hosted-engine-setup[master]: Format errors when getting and setting configuration

Code Review gerrit at ovirt.org
Wed Feb 15 15:01:22 UTC 2017


>From Simone Tiraboschi <stirabos at redhat.com>:

Simone Tiraboschi has submitted this change and it was merged.

Change subject: Format errors when getting and setting configuration
......................................................................


Format errors when getting and setting configuration

This patch formats and clarifies the errors and gets rid
of the stack trace when setting or getting configuration
on shared storage.

Change-Id: Ibbf1a7ac02f43644750b5820648e6ec13137c96a
Bug-Url: https://bugzilla.redhat.com/1417583
Signed-off-by: Jenny Tokar <jtokar at redhat.com>
---
M src/ovirt_hosted_engine_setup/get_shared_config.py
M src/ovirt_hosted_engine_setup/set_shared_config.py
2 files changed, 44 insertions(+), 0 deletions(-)

Approvals:
  Martin Sivák: Looks good to me, but someone else must approve
  Simone Tiraboschi: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Jenny Tokar: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ibbf1a7ac02f43644750b5820648e6ec13137c96a
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-hosted-engine-setup
Gerrit-Branch: master
Gerrit-Owner: Jenny Tokar <jtokar at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Jenny Tokar <jtokar at redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list