Change in ovirt-hosted-engine-ha[master]: Do not replace the ovf configuration with the backup vm.conf

Code Review gerrit at ovirt.org
Thu Jan 12 13:49:03 UTC 2017


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

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

Change subject: Do not replace the ovf configuration with the backup vm.conf
......................................................................


Do not replace the ovf configuration with the backup vm.conf

The old code always refreshed the vm.conf during an attempt
to update the local /var/run cached version of it. That unfortunately
replaced the OVF content we prepared earlier.

This splits the method to refresh and publish parts and uses
only the publishing method for OVF cache refresh.

Change-Id: Icc7f33c3369a341191f091bff407e3dfbabc67c4
Bug-Url: https://bugzilla.redhat.com/1411783
Signed-off-by: Martin Sivak <msivak at redhat.com>
---
M ovirt_hosted_engine_ha/env/config.py
1 file changed, 4 insertions(+), 3 deletions(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Icc7f33c3369a341191f091bff407e3dfbabc67c4
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-hosted-engine-ha
Gerrit-Branch: master
Gerrit-Owner: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Jenkins CI
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