Tal Nisan has submitted this change and it was merged.
Change subject: Ignore memory and CPU hotplug errors for hosted engine VM
......................................................................
Ignore memory and CPU hotplug errors for hosted engine VM
The current implementation reverts changes to CPU topology
and memory sizes when hotplug fails. Hosted engine VM falls
back to saving the values to the OVF store and uses them
after HE VM restart.
The old behaviour reset the fields, preventing OVF from
getting the new values.
The new behaviour does not revert and OVF gets the right
values. The new CPU count and memory size are then
automatically applied when the HE VM is restarted.
Change-Id: I101a8fce0bed2fa5dcc3f2bddbf2d58978156d77
Bug-Url:
https://bugzilla.redhat.com/show_bug.cgi?id=1304387
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, 12 insertions(+), 2 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/55932
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I101a8fce0bed2fa5dcc3f2bddbf2d58978156d77
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
Gerrit-Owner: Martin Sivák <msivak(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>