5:10 p.m.
Allon Mureinik has submitted this change and it was merged.
Change subject: api: Add created Disk object on import response
......................................................................
api: Add created Disk object on import response
When doing the following 2 actions:
* ovirt-engine/api/openstackimageproviders/UUID/images/UUID/import
* ovirt-engine/api/storagedomains/UUID/images/UUID/import
with async set to false, the response returns a <disk> element
containing id of the new newly created disk.
Example POST data:
<action>
<async>false</async>
<storage_domain>
<name>mp2</name>
</storage_domain>
</action>
Example POST response:
<action>
<async>false</async>
<storage_domain>
<name>mp2</name>
</storage_domain>
<disk href="/ovirt-engine/api/disks/UUID" id="UUID">
<actions>
<link href="/ovirt-engine/api/disks/UUID/export"
rel="export"/>
<link href="/ovirt-engine/api/disks/UUID/move"
rel="move"/>
<link href="/ovirt-engine/api/disks/UUID/copy"
rel="copy"/>
</actions>
</disk>
<status>
<state>complete</state>
</status>
</action>
This patch also introduces SimpleIdResolver, an EntityResolver that
receives a business logic entity class and an API model calls and
resolves IDs of the business logic entities into model resource objects.
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1169228
Change-Id: Ic7a6203a318763f13fca8f0627bb0188b7eb2984
Signed-off-by: Dima Kuznetsov <dkuznets(a)redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportRepoImageCreateTaskHandler.java
M
backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/AbstractBackendActionableResource.java
M
backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendStorageDomainImageResource.java
M
backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/openstack/BackendOpenStackImageResource.java
4 files changed, 70 insertions(+), 2 deletions(-)
Approvals:
Juan Hernandez: Looks good to me, but someone else must approve
Dima Kuznetsov: Verified
Allon Mureinik: Looks good to me, approved
Oved Ourfali: Looks good to me, but someone else must approve
--
To view, visit https://gerrit.ovirt.org/38255
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ic7a6203a318763f13fca8f0627bb0188b7eb2984
Gerrit-PatchSet: 9
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Dima Kuznetsov <dkuznets(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Dima Kuznetsov <dkuznets(a)redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Mooli Tayer <mtayer(a)redhat.com>
Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server