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

laravot at redhat.com laravot at redhat.com
Thu Jun 30 06:56:55 UTC 2016


Liron Aravot 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/1340626
Change-Id: I121a9f587b2a85bdd4b366f07b040dc85fa515cf
Signed-off-by: Martin Sivak <msivak at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmCommand.java
1 file changed, 29 insertions(+), 0 deletions(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I121a9f587b2a85bdd4b366f07b040dc85fa515cf
Gerrit-PatchSet: 26
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot at redhat.com>
Gerrit-Reviewer: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Roman Mohr <rmohr at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo 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