Gilad Chaplik has uploaded a new change for review. Change subject: core: set Disk Profile for cloned VM from snapshot ......................................................................
core: set Disk Profile for cloned VM from snapshot Disk Profile was missing in disk image mapping to storage_doamin_image_map. Change-Id: Iab5f79e853a1482ccabf6c287d663da87ab332fe Bug-Url: https://bugzilla.redhat.com/1161939 Signed-off-by: Gilad Chaplik <[email protected]> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmAndCloneImageCommand.java 1 file changed, 7 insertions(+), 2 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/29/35729/1 diff --git a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmAndCloneImageCommand.java b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmAndCloneImageCommand.java index 4379aaf..cdb048d 100644 --- a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmAndCloneImageCommand.java +++ b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmAndCloneImageCommand.java @@ -72,12 +72,14 @@ DiskImage diskImage, Guid srcStorageDomainId, Guid destStorageDomainId, + Guid diskProfileId, VdcActionType parentCommandType) { DiskImage newDiskImage = cloneDiskImage(getVmId(), destStorageDomainId, Guid.newGuid(), Guid.newGuid(), - diskImage); + diskImage, + diskProfileId); ImagesHandler.setDiskAlias(newDiskImage, getVm()); MoveOrCopyImageGroupParameters parameters = createCopyParameters(newDiskImage, srcStorageDomainId, @@ -182,7 +184,8 @@ Guid storageDomainId, Guid newImageGroupId, Guid newImageGuid, - DiskImage srcDiskImage) { + DiskImage srcDiskImage, + Guid diskProfileId) { DiskImage clonedDiskImage = DiskImage.copyOf(srcDiskImage); clonedDiskImage.setImageId(newImageGuid); @@ -196,6 +199,7 @@ ArrayList<Guid> storageIds = new ArrayList<Guid>(); storageIds.add(storageDomainId); clonedDiskImage.setStorageIds(storageIds); + clonedDiskImage.setDiskProfileId(diskProfileId); // If volume information was changed at client , use its volume information. // If volume information was not changed at client - use the volume information of the ancestral image @@ -415,6 +419,7 @@ copyDiskImage(diskImage, diskImage.getStorageIds().get(0), diskInfoDestinationMap.get(diskImage.getId()).getStorageIds().get(0), + diskInfoDestinationMap.get(diskImage.getId()).getDiskProfileId(), getActionType()); numberOfStartedCopyTasks++; } -- To view, visit http://gerrit.ovirt.org/35729 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Iab5f79e853a1482ccabf6c287d663da87ab332fe Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Gilad Chaplik <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
