Change in ovirt-hosted-engine-ha[v2.1.z]: Fix refreshing vm conf

Code Review gerrit at ovirt.org
Thu Jan 5 09:48:01 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>
(cherry picked from commit d771cf1)
---
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/69615
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I281b42a6cb2c5cfdd5821cd50e3f2608d3ae8896
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-hosted-engine-ha
Gerrit-Branch: v2.1.z
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: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list