Maor Lipchuk has submitted this change and it was merged. Change subject: core: Commiting a partial preview of cinder snapshot. ...................................................................... core: Commiting a partial preview of cinder snapshot. Critical snapshot list is list which reflects all the snapshot ids that are being used for all the volumes in the VM. Partial snapshot might contain a volume used for active snapshot. Once the snapshot gets commited the engine should avoid removing the snapshots which are being used by other volumes, indicated in the critical snapshot list Change-Id: I623ac84b1a0a869d25f0d8427ee5cd7631afda3e Bug-Url: https://bugzilla.redhat.com/1293226 Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com&gt; --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/snapshots/RestoreAllSnapshotsCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/connection/CINDERStorageHelper.java 2 files changed, 24 insertions(+), 12 deletions(-) Approvals: Daniel Erez: Looks good to me, approved Maor Lipchuk: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/50786 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I623ac84b1a0a869d25f0d8427ee5cd7631afda3e Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Maor Lipchuk <mlipchuk(a)redhat.com&gt; Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com&gt; Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com&gt; Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org&gt;