Bug 855890 - RESTAPI: Direct LUN in rest-api requires size
Summary: RESTAPI: Direct LUN in rest-api requires size
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Enterprise Virtualization Manager
Classification: Red Hat
Component: ovirt-engine-restapi
Version: 3.1.0
Hardware: x86_64
OS: Linux
high
high
Target Milestone: ---
: 3.1.0
Assignee: Ori Liel
QA Contact: Jakub Libosvar
URL:
Whiteboard: storage
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-09-10 14:30 UTC by Jakub Libosvar
Modified: 2016-02-10 16:35 UTC (History)
9 users (show)

Fixed In Version: si21
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-12-04 20:00:15 UTC
oVirt Team: Storage
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Jakub Libosvar 2012-09-10 14:30:19 UTC
Description of problem:
I use this for creating direct lun
url:https://jl-rhevm31.rhev.lab.eng.brq.redhat.com/api/disks 
body:<disk>
    <name>direct_lun</name>
    <interface>virtio</interface>
    <format>cow</format>
    <lunStorage>
        <logical_unit id="36006048c12952607a6254682d950135a">
            <address>10.34.63.200</address>
            <port>3260</port>
            <target>iqn.1992-05.com.emc:ckm001201002300000-5-vnxe</target>
        </logical_unit>
    </lunStorage>
</disk>

which I think should have sufficient information for creating a direct lun. But it fails with 
<fault>
    <reason>Incomplete parameters</reason>
    <detail>Disk [provisionedSize|size] required for add</detail>
</fault>

From rsdl:
<parameter required="true" type="xs:int">
<name>provisioned_size</name>
</parameter>

Version-Release number of selected component (if applicable):
rhevm-restapi-3.1.0-15.el6ev.noarch

How reproducible:
Always

Steps to Reproduce:
1. Create direct lun without specifying size
  
Actual results:
Fails

Expected results:
Succeeds

Additional info:
I think the size should be taken from LUN itself and cannot be even overridden.

Comment 3 Ori Liel 2012-10-09 09:55:23 UTC
patch posted upstream: 
  http://gerrit.ovirt.org/#/c/8331/

patch merged upstream: 
  commit 8dcbf3056127a98289fc168c45ae5a02d3dc882d

patch posted downstream: 
  https://gerrit.eng.lab.tlv.redhat.com/#/c/2502/

Comment 4 Ori Liel 2012-10-15 13:52:32 UTC
patch reposted downstream: 
  https://gerrit.eng.lab.tlv.redhat.com/#/c/2632/

patch merged downstream: 
  commit 621006f266a831cd7af96d77df19b674f0c59a3

Comment 5 Jakub Libosvar 2012-10-22 07:57:17 UTC
Verified rhevm-3.1.0-21.el6ev.noarch


Note You need to log in before you can comment on or make changes to this bug.