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(a)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(a)redhat.com>
Gerrit-Reviewer: Dudi Maroshi <dudi(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
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: Tomer Saban <tsaban(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org