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(a)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(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com>
Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>