Change in ovirt-hosted-engine-ha[master]: Fix refreshing vm conf

Code Review gerrit at ovirt.org
Wed Jan 4 12:25:48 UTC 2017


>From Martin Sivák <msivak at redhat.com>:

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

Change subject: Fix refreshing vm conf
......................................................................


Fix refreshing vm conf

When refreshing the vm configuration on the host
we first try the ovf store, if no content was found
we try the shared storage vm.conf.
When we have the content we can refresh the locally
stored file.
The recent refactor broke this functionality, this patch
fixes it.

Change-Id: I281b42a6cb2c5cfdd5821cd50e3f2608d3ae8896
Bug-Url: https://bugzilla.redhat.com/1301681
Signed-off-by: Jenny Tokar <jtokar at redhat.com>
---
M ovirt_hosted_engine_ha/env/config.py
1 file changed, 8 insertions(+), 3 deletions(-)

Approvals:
  Martin Sivák: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Jenny Tokar: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I281b42a6cb2c5cfdd5821cd50e3f2608d3ae8896
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-hosted-engine-ha
Gerrit-Branch: master
Gerrit-Owner: Jenny Tokar <jtokar at redhat.com>
Gerrit-Reviewer: Andrej Krejcir <akrejcir at redhat.com>
Gerrit-Reviewer: Denis Chaplygin <dchaplyg 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: Phillip Bailey <phbailey at redhat.com>
Gerrit-Reviewer: Roman Mohr <rmohr at redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos at redhat.com>
Gerrit-Reviewer: Yanir Quinn <yquinn at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list