Change in ovirt-engine[ovirt-engine-3.6]: Ignore memory and CPU hotplug errors for hosted engine VM

[tnisan at redhat.com]

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 at 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 at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.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>