Change in ovirt-engine[master]: core: Remove Memory and OVF Volumes on VM Remove

amureini at redhat.com amureini at redhat.com
Sun Sep 28 13:10:31 UTC 2014


Allon Mureinik has submitted this change and it was merged.

Change subject: core: Remove Memory and OVF Volumes on VM Remove
......................................................................


core: Remove Memory and OVF Volumes on VM Remove

When removing a VM that has a snapshot with memory, its memory and OVF
volumes are not removed. The reason for this is the value of the
property removeOnlyIfNotUsedAtAll in RemoveMemoryVolumesParameters.
If we don't set its value to true, by default it's false and then we can
remove the memory and OVF volumes before the VM's disks.

Change-Id: I5303c571e3d8843c03b195ec91763279053bc5de
Bug-Url: https://bugzilla.redhat.com/1145073
Bug-Url: https://bugzilla.redhat.com/1145188
Bug-Url: https://bugzilla.redhat.com/1147085
Bug-Url: https://bugzilla.redhat.com/1145254
Signed-off-by: Idan Shaby <ishaby at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveVmCommand.java
1 file changed, 1 insertion(+), 9 deletions(-)

Approvals:
  Allon Mureinik: Looks good to me, approved
  Idan Shaby: Verified
  Arik Hadas: Looks good to me, but someone else must approve



-- 
To view, visit http://gerrit.ovirt.org/33429
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I5303c571e3d8843c03b195ec91763279053bc5de
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Idan Shaby <ishaby at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas at redhat.com>
Gerrit-Reviewer: Idan Shaby <ishaby at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list