[Engine-devel] RSDL breakage in oVirt upstream

Fri Nov 15 16:14:29 UTC 2013

On 11/15/2013 09:28 AM, Shubhendu Tripathi wrote:
> On 11/14/2013 08:26 PM, Michael Pasternak wrote:
>> Hi Shubhendu,
>>
>> On 11/14/2013 04:32 PM, Shubhendu Tripathi wrote:
>>> Hi Michael/Ori,
>>>
>>> There is payload breakage for RSDL in upstream oVirt.
>> can you elaborate a bit? [1] works for me.
>>
>> [1] http://localhost:8080/api?rsdl
I have raised the patch http://gerrit.ovirt.org/#/c/21308/ resolving the 
same issue.
Kindly have a look and do the needful.
> Hi Michael,
>
> The RSDL gets displayed but there are some issues while showing the 
> payload details for 'glustervolumes'.
> Current value shown in the UI is as below -
>
> <link href="/ovirt-engine/api/clusters/{cluster:id}/glustervolumes" 
> rel="add">
>     <request>
>         <http_method>POST</http_method>
>         <body>
>             <type>GlusterVolume</type>
>         </body>
>     </request>
>     <response>
>         <type>GlusterVolume</type>
>     </response>
> </link>
>
> Attached the RSDl file for reference.
>
> Whereas the correct value should be as below -
>
> <link href="/api/clusters/{cluster:id}/glustervolumes" rel="add">
> <description>add a new gluster volume to the cluster</description>
> <request>
> <http_method>POST</http_method>
> <headers>
> <header required="true">
> <name>Content-Type</name>
> <value>application/xml|json</value>
> </header>
> <header required="false">
> <name>Expect</name>
> <value>201-created</value>
> </header>
> <header required="false">
> <name>Correlation-Id</name>
> <value>any string</value>
> </header>
> </headers>
> <body>
> <type>GlusterVolume</type>
> <parameters_set>
> <description>
> add a new gluster volume to the cluster at the specified gluster fs 
> server
> </description>
> <parameter required="true" type="xs:string">
> <name>gluster_volume.name</name>
> </parameter>
> <parameter required="true" type="xs:string">
> <name>gluster_volume.volume_type</name>
> </parameter>
> <parameter required="true" type="collection">
> <name>gluster_volume.bricks.brick</name>
> <parameters_set>
> <parameter required="true" type="xs:string">
> <name>brick.server_id</name>
> </parameter>
> <parameter required="true" type="xs:string">
> <name>brick.brick_dir</name>
> </parameter>
> </parameters_set>
> </parameter>
> <parameter required="false" type="collection">
> <name>gluster_volume.transport_types</name>
> <parameters_set>
> <parameter required="false" type="xs:string">
> <name>transport_type</name>
> </parameter>
> </parameters_set>
> </parameter>
> <parameter required="false" type="xs:unsignedShort">
> <name>gluster_volume.replica_count</name>
> </parameter>
> <parameter required="false" type="xs:unsignedShort">
> <name>gluster_volume.stripe_count</name>
> </parameter>
> <parameter required="false" type="collection">
> <name>gluster_volume.options.option</name>
> <parameters_set>
> <parameter required="false" type="xs:string">
> <name>option.name</name>
> </parameter>
> <parameter required="false" type="xs:string">
> <name>option.value</name>
> </parameter>
> </parameters_set>
> </parameter>
> </parameters_set>
> </body>
> </request>
> <response>
> <type>GlusterVolume</type>
> </response>
> </link>
>
>
>>
>>> This has happened somewhere between the patches 
>>> gerrit.ovirt.org/#/c/20610/ and http://gerrit.ovirt.org/#/c/15409/.
>>> If I try to test out with patch http://gerrit.ovirt.org/#/c/15409/, 
>>> it does break.
>>>
>>> Not sure what could be the issue. Tried debugging but not able to 
>>> figure it out.
>>>
>>> Kindly help me on the same figuring out what could be the issue.
>>>
>>> Thanks and Regards,
>>> Shubhendu
>>>
>>
>