Maor Lipchuk has submitted this change and it was merged. Change subject: core: Add the correlated OVF write logic for Cinder ...................................................................... core: Add the correlated OVF write logic for Cinder We should support diskStorageType for each disk in OVF to support snapshots of cinder disk. We need this for snapshots for the following scenrio: 1. Create a snapshot from a VM with image disk 2. Create a Cinder disk 3. Preview the snapshot 4. Undo the preview The engine then should get the Cinder disk back to the VM as Cinder from the OVF. Change-Id: Idd1b99979e067af41b9d37a982be6da406a9e7e1 Bug-Url: https://bugzilla.redhat.com/1185826 Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com&gt; --- M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ovf/OvfReader.java 1 file changed, 21 insertions(+), 7 deletions(-) Approvals: Jenkins CI: Verified; Looks good to me, but someone else must approve Daniel Erez: Looks good to me, approved Maor Lipchuk: Verified -- To view, visit https://gerrit.ovirt.org/40982 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idd1b99979e067af41b9d37a982be6da406a9e7e1 Gerrit-PatchSet: 21 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: automation(a)ovirt.org