Change in ovirt-engine[master]: HA reservation needs to take the full VM's memory into account

[rgolan at redhat.com]

Roy Golan has submitted this change and it was merged.

Change subject: HA reservation needs to take the full VM's memory into account
......................................................................


HA reservation needs to take the full VM's memory into account

The current HA reservation code only counted the used memory of
HA VMs. But the engine will refuse to migrate the VM when there
is not enough memory when compared to the minimum guaranteed
settings.

This patch updates the computation and uses the minimum guaranteed
memory size or the currently used amount of memory whichever is
higher.

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1220134
Change-Id: Ib3af7e370f004f7c22335c507e4e191b90166f6d
Signed-off-by: Martin Sivak <msivak at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/HaReservationHandling.java
1 file changed, 5 insertions(+), 1 deletion(-)

Approvals:
  Martin Sivák: Verified
  Dudi Maroshi: Looks good to me, but someone else must approve
  Roman Mohr: Looks good to me, but someone else must approve
  Jenkins CI: Passed CI tests
  Roy Golan: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib3af7e370f004f7c22335c507e4e191b90166f6d
Gerrit-PatchSet: 9
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Dudi Maroshi <dudi at redhat.com>
Gerrit-Reviewer: Jenkins CI
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: Tomer Saban <tsaban at redhat.com>
Gerrit-Reviewer: automation at ovirt.org