Maor Lipchuk has uploaded a new change for review.

Change subject: core: set storage domain id for cinder disks in snapshots.
......................................................................

core: set storage domain id for cinder disks in snapshots.

Remove storage domain id from Cinder disks while adding a snapshot to be
comaptible with the images disks, so in case of a move operation the
images in the snapshot will be updated with the accurate storage domain
id

Change-Id: Icaf67979493885ef2f53ac27afd8383942d242ca
Bug-Url: https://bugzilla.redhat.com/??????
Signed-off-by: Maor Lipchuk <[email protected]>
---
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/snapshots/SnapshotsManager.java
1 file changed, 1 insertion(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/84/40984/1

diff --git 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/snapshots/SnapshotsManager.java
 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/snapshots/SnapshotsManager.java
index 38b7e98..313626c 100644
--- 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/snapshots/SnapshotsManager.java
+++ 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/snapshots/SnapshotsManager.java
@@ -378,6 +378,7 @@
 
         if (disks == null) {
             disks = 
ImagesHandler.filterImageDisks(getDiskDao().getAllForVm(vm.getId()), false, 
true, true);
+            
disks.addAll(ImagesHandler.filterDisksBasedOnCinder(getDiskDao().getAllForVm(vm.getId())));
         }
         for (DiskImage image : disks) {
             image.setStorageIds(null);


-- 
To view, visit https://gerrit.ovirt.org/40984
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Icaf67979493885ef2f53ac27afd8383942d242ca
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Maor Lipchuk <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to