Change in ovirt-engine[ovirt-engine-4.1]: engine: Initial size in copy template disk flow

From Tal Nisan <tnisan@redhat.com>:
Tal Nisan has submitted this change and it was merged. Change subject: engine: Initial size in copy template disk flow ...................................................................... engine: Initial size in copy template disk flow The 'determineTotalImageInitialSize' method is use to compute the initial size of a disk image based on the source disk image size, including all the existing snapshots. This method calls 'getTotalActualSizeOfDisk' which has a logic based on the source Storage Domain. In 'ImagesHandler.determineTotalImageInitialSize', the Storage Domain passed to 'getTotalActualSizeOfDisk' was the destination SD instead of the source SD. This fix is relevant for create VM from template (clone) Change-Id: I61e42bdb8c70de6bdfcb2b431c8c8ca59fbd665e Bug-Url: https://bugzilla.redhat.com/1438691 Signed-off-by: Fred Rolland <frolland@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/image/CopyImageGroupWithDataCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/image/ImagesHandler.java 2 files changed, 4 insertions(+), 1 deletion(-) Approvals: Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, approved Freddy Rolland: Verified -- To view, visit https://gerrit.ovirt.org/75360 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I61e42bdb8c70de6bdfcb2b431c8c8ca59fbd665e Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.1 Gerrit-Owner: Freddy Rolland <frolland@redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Fred Rolland <frolland@redhat.com> Gerrit-Reviewer: Freddy Rolland <frolland@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot@redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>
participants (1)
-
Code Review