From Daniel Erez <derez(a)redhat.com>:
Daniel Erez has submitted this change and it was merged.
Change subject: core: TransferDiskImage validation - get storage domain id from disk
......................................................................
core: TransferDiskImage validation - get storage domain id from disk
TransferDiskImageCommand -> validateImageTransfer:
The StorageDomainValidator should be created using the disk's
storage domain id, as the new disk is already created.
Currently, the domain id is fetched from command's parameters,
which is available only when uplaoding from the WebAdmin.
Change-Id: I801f71079e031367cf87b527ae2bf5c3bf1623e7
Bug-Url:
https://bugzilla.redhat.com/1447023
Signed-off-by: Daniel Erez <danielerez(a)gmail.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/image/TransferDiskImageCommand.java
M
backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/storage/disk/image/TransferDiskImageCommandTest.java
2 files changed, 6 insertions(+), 1 deletion(-)
Approvals:
Daniel Erez: Verified
Jenkins CI: Passed CI tests
Allon Mureinik: Looks good to me, approved
--
To view, visit
https://gerrit.ovirt.org/76593
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I801f71079e031367cf87b527ae2bf5c3bf1623e7
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>