Change in ovirt-engine[ovirt-engine-3.6]: core: Update the OVF store when hosted engine VM is edited

tnisan at redhat.com tnisan at redhat.com
Thu Jul 14 12:31:04 UTC 2016


Tal Nisan has submitted this change and it was merged.

Change subject: core: Update the OVF store when hosted engine VM is edited
......................................................................


core: Update the OVF store when hosted engine VM is edited

Retrigger the ovf update from the UpdateVmCommand when
the hosted engine VM was edited.

This is needed to persist the hosted engine VM configuration
as soon as possible in case the ovirt-hosted-engine-agent
needs to restart the engine VM with the proper settings.

There is still a very slight chance of hitting a race
condition (when the regularly scheduled OVF updater processes
the hosted engine VM at the exact same moment as the triggered
job), but the VM will be saved during the next OVF update anyway.

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1343991
Change-Id: I121a9f587b2a85bdd4b366f07b040dc85fa515cf
Signed-off-by: Martin Sivak <msivak at redhat.com>
(cherry picked from commit 7e92bd44e17ded6831eceee1610e1b3616c9c505)
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmCommand.java
1 file changed, 26 insertions(+), 0 deletions(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I121a9f587b2a85bdd4b366f07b040dc85fa515cf
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
Gerrit-Owner: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list