Tal Nisan 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>
(cherry picked from commit 4711d1fdf22265a6c970dc4b2fa3b2eb84646342)
---
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
Jenkins CI: Passed CI tests
Roy Golan: Looks good to me, approved
--
To view, visit
https://gerrit.ovirt.org/46211
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib3af7e370f004f7c22335c507e4e191b90166f6d
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: automation(a)ovirt.org