Maor Lipchuk has submitted this change and it was merged. Change subject: core: Preview snapshot with allow snapshot attribute ...................................................................... core: Preview snapshot with allow snapshot attribute Disk which are not allowed to be snapshot, should not be part of the snapshot operation. Those disks are static and should not be changed, they should only be reflected in the VM whether the VM is in snapshot mode or not. Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com&gt; Change-Id: I4d0e5bd4dff1898022f78e379ffa30824912ed1f --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/snapshots/SnapshotsManager.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ovf/OvfManager.java 2 files changed, 20 insertions(+), 5 deletions(-) Approvals: Michael Kublin: Looks good to me, approved Maor Lipchuk: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/4233 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4d0e5bd4dff1898022f78e379ffa30824912ed1f Gerrit-PatchSet: 15 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Maor Lipchuk <mlipchuk(a)redhat.com&gt; Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com&gt; Gerrit-Reviewer: Michael Kublin <mkublin(a)redhat.com&gt; Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com&gt; Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com&gt;