Mike Kolesnik has submitted this change and it was merged. Change subject: core: Move disk/device deletion to RemoveImage ......................................................................
core: Move disk/device deletion to RemoveImage This command actually deletes the whole disk, so no need for this call to be only if all images are deleted from DB, since they will eventually get all deleted anyway. RestoreSnapshotCommand doesn't need to touch the disk anyway, so it's not relevant there. Change-Id: I846d63d5fe6ef432f3d3d1289f772fa0d841047c --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/BaseImagesCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveImageCommand.java 2 files changed, 4 insertions(+), 15 deletions(-) Approvals: Mike Kolesnik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/2565 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I846d63d5fe6ef432f3d3d1289f772fa0d841047c Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Mike Kolesnik <mkole...@redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkole...@redhat.com> Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
