Bug 1332239

Summary: [REST-API] rsdl: wrong parameter name under copy disk template action
Product: [oVirt] ovirt-engine Reporter: Elad <ebenahar>
Component: RestAPIAssignee: Tal Nisan <tnisan>
Status: CLOSED CURRENTRELEASE QA Contact: Kevin Alon Goldblatt <kgoldbla>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 3.6.5CC: acanan, amureini, bugs, sbonazzo, tnisan
Target Milestone: ovirt-4.0.0-betaFlags: rule-engine: ovirt-4.0.0+
rule-engine: planning_ack+
tnisan: devel_ack+
rule-engine: testing_ack+
Target Release: 4.0.0   
Hardware: x86_64   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-07-05 07:39:35 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:

Description Elad 2016-05-02 15:16:23 UTC
Description of problem:
In rsdl, the storage domain parameter should be called 'storage_domain' and not 'storagedomain'. 




Version-Release number of selected component (if applicable):
rhevm-3.6.6-0.1.el6.noarch
rhevm-restapi-3.6.6-0.1.el6.noarch


Actual results:

<type>Action</type>
<parameters_set>
<description>
copy the specified disk attached to the template to a specific storage domain
</description>
<parameter required="true" type="xs:string">
<name>storagedomain.id|name</name>


Expected results:

<type>Action</type>
<parameters_set>
<description>
copy the specified disk attached to the template to a specific storage domain
</description>
<parameter required="true" type="xs:string">
<name>storage_domain.id|name</name>


Additional info:

Comment 1 Yaniv Lavi 2016-05-09 10:53:26 UTC
Moving to first RC, since things should not be targeted to second one at this point.

Comment 2 Kevin Alon Goldblatt 2016-06-08 14:51:24 UTC
Tested with the following code:
------------------------------------------
rhevm-4.0.0.2-0.1.el7ev.noarch
vdsm-4.18.1-11.gita92976e.el7ev.x86_64

Tested with the following result:
------------------------------------------
Output from rsdl
-----------------
Parameter name of storage domain is now called storage_domain for copy template disk 

<description>copy the specified disk attached to the template to a specific storage domain</description><parameter required="true" type="xs:string"><name>action.storage_domain.id|name</name></parameter>

Moving to VERIFIED!

Comment 3 Sandro Bonazzola 2016-07-05 07:39:35 UTC
oVirt 4.0.0 has been released, closing current release.