Change in ovirt-engine[master]: restapi: Add 'placeholder' attribute to VmHostDevice resources

juan.hernandez at redhat.com juan.hernandez at redhat.com
Thu Jul 9 10:43:15 UTC 2015


Juan Hernandez has submitted this change and it was merged.

Change subject: restapi: Add 'placeholder' attribute to VmHostDevice resources
......................................................................


restapi: Add 'placeholder' attribute to VmHostDevice resources

So far the backend only returned the devices that are to
be attached to the VM. Now it returns all host devices
configured for the VM and distinguishes IOMMU placeholder
devices with additional boolean flag.

This patch exposes to the REST users that some host device
is configured on the VM only as a placeholder using the
optional <placeholder>true</placeholder> element.

Change-Id: I76e9945cb680537a83b036b6f87cf5aeb33fa420
Signed-off-by: Martin Betak <mbetak at redhat.com>
---
M backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmHostDeviceResource.java
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmHostDevicesResource.java
M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/AbstractBackendHostDevicesResourceTest.java
M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendHostDeviceResourceTest.java
M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendHostDevicesResourceTest.java
M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendVmHostDeviceResourceTest.java
M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendVmHostDevicesResourceTest.java
M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/HostDeviceMapper.java
9 files changed, 59 insertions(+), 19 deletions(-)

Approvals:
  Juan Hernandez: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Martin Betak: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I76e9945cb680537a83b036b6f87cf5aeb33fa420
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Betak <mbetak at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Martin Betak <mbetak at redhat.com>
Gerrit-Reviewer: Ori Liel <oliel at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list