Change in ovirt-engine[master]: restapi: Support skip LUN status check

juan.hernandez at redhat.com juan.hernandez at redhat.com
Tue Apr 19 09:06:30 UTC 2016


Juan Hernandez has submitted this change and it was merged.

Change subject: restapi: Support skip LUN status check
......................................................................


restapi: Support skip LUN status check

Adding support for skipping the LUN status check in
the REST API.
Checking the status of the LUN is an heavy weight operation
and this data is not always needed by the user.

The default is `true` for backward compatibility.
The parameter `report_status` is available both on getting the list of a
host storages or a specific host storage:

GET /hosts/{host:id}/storage
GET /hosts/{host:id}/storage/{storage:id}

For example, to skip the LUN status check, set `report_status` to false:
(The `status` field of the LUN will not be present in the response)

GET /hosts/{host:id}/storage/{storage:id};report_status=false

<host_storage id="360014051136c20574f743bdbd28177fd">
  <logical_units>
    <logical_unit id="360014051136c20574f743bdbd28177fd">
      <lun_mapping>0</lun_mapping>
      <paths>1</paths>
      <product_id>lun0</product_id>
      <serial>SLIO-ORG_lun0_1136c205-74f7-43bd-bd28-177fd5ce6993</serial>
      <size>10737418240</size>
      <vendor_id>LIO-ORG</vendor_id>
      <volume_group_id>O9Du7I-RahN-ECe1-dZ1w-nh0b-64io-MNzIBZ</volume_group_id>
    </logical_unit>
  </logical_units>
  <type>iscsi</type>
  <host id="8bb5ade5-e988-4000-8b93-dbfc6717fe50"/>
</host_storage>

Change-Id: Ia772a0e7d9a67c8364e05c9f36a71a9111d1fdde
Bug-Url: https://bugzilla.redhat.com/1308350
Signed-off-by: Fred Rolland <frolland at redhat.com>
---
M backend/manager/modules/restapi/README.adoc
M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendHostStorageResource.java
3 files changed, 86 insertions(+), 3 deletions(-)

Approvals:
  Juan Hernandez: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Freddy Rolland: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia772a0e7d9a67c8364e05c9f36a71a9111d1fdde
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Fred Rolland <frolland at redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list