Change in ovirt-engine[master]: core: Validation of copy Template disk should use image id

[mlipchuk at redhat.com]

Maor Lipchuk has submitted this change and it was merged.

Change subject: core: Validation of copy Template disk should use image id
......................................................................


core: Validation of copy Template disk should use image id

The validation for Cinder and LUN disks on copy operation uses the
parameter value of imageGroupId to fetch the disk.

If the operation is being done on a Template, the paramtere of
imageGroupId is not initialized, but instead only with image id is.
That is why the validation does not block the operation of copy Cinder
disk in a Template.

The proposed fix, fetches the disk using the image id when the image
group id is empty or null (which indicates that a copy template disk was
called).

Change-Id: Ibc5f7c228d014a9db2dd74c4edc6d3e13a37a991
Bug-Url: https://bugzilla.redhat.com/1275056
Signed-off-by: Maor Lipchuk <mlipchuk at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CopyImageGroupCommand.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/MoveOrCopyDiskCommandTest.java
2 files changed, 4 insertions(+), 1 deletion(-)

Approvals:
  Jenkins CI: Passed CI tests
  Daniel Erez: Looks good to me, approved
  Maor Lipchuk: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ibc5f7c228d014a9db2dd74c4edc6d3e13a37a991
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: automation at ovirt.org