Tal Nisan has submitted this change and it was merged. Change subject: core: RestoreCinderSnapshot should delete volumes by order. ...................................................................... core: RestoreCinderSnapshot should delete volumes by order. Re-implement restore Cinder snapshot to delete a Cinder volume and all its descendants from the DB by calling RemoveCinderDiskVolume for each volume by order. The order is important for since Cinder can only remove volume after all its snapshots get deleted first. Change-Id: Ife6ac08ca86a4cb2b30174e3de6422d9db5cbb65 Bug-Url: https://bugzilla.redhat.com/1256048 Bug-Url: https://bugzilla.redhat.com/1256049 Signed-off-by: Maor Lipchuk <mlipchuk@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/RestoreFromCinderSnapshotCommand.java D backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/RestoreFromCinderSnapshotCommandCallback.java 2 files changed, 41 insertions(+), 214 deletions(-) Approvals: Tal Nisan: Passed CI tests Daniel Erez: Looks good to me, approved Maor Lipchuk: Verified -- To view, visit https://gerrit.ovirt.org/50067 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ife6ac08ca86a4cb2b30174e3de6422d9db5cbb65 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Maor Lipchuk <mlipchuk@redhat.com> Gerrit-Reviewer: Daniel Erez <derez@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Maor Lipchuk <mlipchuk@redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>