Mike Kolesnik has submitted this change and it was merged. Change subject: core: Restore missing image as ILLEGAL ......................................................................
core: Restore missing image as ILLEGAL If image is missing, this means it got deleted somehow, so restore it as ILLEGAL image so that the user can know that there was a disk in that snapshot. Handling of ILLEGAL image was added to RestoreAllSnapshotsCommand as well since these images need not be restored to, and the can do action should ignore checking for missing images since they can be present. Change-Id: I77f7ebdaa1f558351f657245b6dd395e1adad976 --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RestoreAllSnapshotsCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/snapshots/SnapshotsManager.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/RestoreAllSnapshotCommandTest.java 3 files changed, 41 insertions(+), 101 deletions(-) Approvals: Mike Kolesnik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/2563 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I77f7ebdaa1f558351f657245b6dd395e1adad976 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
