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@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@redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan@redhat.com> Gerrit-Reviewer: Vered Volansky <vvolansk@redhat.com> Gerrit-Reviewer: automation@ovirt.org