Bug 1308350
| Summary: | [SCALE] Improve GetDeviceList verb call through the REST API to work in scale. | ||
|---|---|---|---|
| Product: | [oVirt] ovirt-engine | Reporter: | Yaniv Lavi <ylavi> |
| Component: | RestAPI | Assignee: | Fred Rolland <frolland> |
| Status: | CLOSED CURRENTRELEASE | QA Contact: | Eldad Marciano <emarcian> |
| Severity: | medium | Docs Contact: | |
| Priority: | unspecified | ||
| Version: | 3.6.0 | CC: | acanan, amureini, bugs, gklein, melewis, sbonazzo |
| Target Milestone: | ovirt-4.0.0-beta | Flags: | ylavi:
ovirt-4.0.0?
rule-engine: planning_ack+ rule-engine: devel_ack+ ylavi: testing_ack? |
| Target Release: | 4.0.0 | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | Enhancement | |
| Doc Text: |
With this update, support has been added for skipping the LUN status check in the REST API as checking the status of the LUN is a heavyweight operation and this data is not always required. To enable this feature set report_status to false. The default for report_status is true to allow for backward compatibility. The report_status parameter is available when getting a list of storage devices visible to a host or when getting a specific storage device via a host, GET /hosts/{host:id}/storage or GET /hosts/{host:id}/storage/{storage:id}. For example:
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-43bd6993</serial>
<size>10737418240</size>
<vendor_id>LIO-ORG</vendor_id>
<volume_group_id>O9Du7I-RahN</volume_group_id>
</logical_unit>
</logical_units>
<type>iscsi</type>
<host id="8bb5ade5-e988-4000-8b93-dbfc6717fe50"/>
</host_storage>
|
Story Points: | --- |
| Clone Of: | Environment: | ||
| Last Closed: | 2016-08-11 15:07:10 UTC | Type: | Bug |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | Storage | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
|
Description
Yaniv Lavi
2016-02-14 14:51:37 UTC
Note that the reporting of the status of the LUN can be removed or made optional in version 4 of the API. But as version 4 of the engine has to support versions 3 and 4 of the API the capability can't be removed from the backend. Thus it is easier to just keep it in version 4 of the API.
I'd suggest to add a parameter that can be used to enable/disable this reporting of the status of the LUN:
GET /hosts/{host:id}/storage;report_status=true|false
The default value of this parameter should be "true" in version 3 of the API, to preserve backwards compatibility. In version 4 of the API it can be "false".
This bug was fixed and is slated to be in the upcoming version. As we are focusing our testing at this phase on severe bugs, this bug was closed without going through its verification step. If you think this bug should be verified by QE, please set its severity to high and move it back to ON_QA Megan, the term "host storage domain" is confusing. Storage domains are DC level entities, not host level entities. The correct term would be something like "The report_status parameter is available when getting a list of storage devices visible to a host or when getting a specific storage device via a host". Hi Allon, Thanks for picking that up. Not sure what I was thinking yesterday. Corrected now. Megan |