Tal Nisan has submitted this change and it was merged.
Change subject: core: Add memory allocation check to hibernate VM
......................................................................
core: Add memory allocation check to hibernate VM
When hibernating a VM, two volumes are created, one to save memory,
and one for the metadata. When searching for a Storage Domain with
enough space for these volumes, wrong space allocations checks are done.
Added the correct storage allocation checks, replacing existing checks
in HibernateVmCommand.
Added test class to the command - HibernateVmCommandTest, as well as a
test to VmHandlerTest.
Change-Id: I1250e6ea8d67f8026d66cf06589538343d39756a
Bug-Url:
https://bugzilla.redhat.com/1124143
Signed-off-by: Vered Volansky <vvolansk(a)redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/HibernateVmCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmHandler.java
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/memory/MemoryUtils.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/VmHandlerTest.java
4 files changed, 136 insertions(+), 7 deletions(-)
Approvals:
Allon Mureinik: Looks good to me, approved
Vered Volansky: Verified
--
To view, visit
http://gerrit.ovirt.org/32054
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I1250e6ea8d67f8026d66cf06589538343d39756a
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Vered Volansky <vvolansk(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org