Michael Pasternak has posted comments on this change.

Change subject: restapi:Correct rsdl for add bricks
......................................................................


Patch Set 3: (1 inline comment)

....................................................
File 
backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml
Line 2974:       parameterType: Bricks
Line 2975:       signatures:
Line 2976:       - mandatoryArguments:
Line 2977:           brick--COLLECTION: {brick.server_id: 'xs:string', 
brick.brick_dir: 'xs:string'}
Line 2978:         optionalArguments: {bricks.replica_count: 
'xs:unsignedShort', bricks.stripe_count: 'xs:unsignedShort'}
collection items should be in singular form (unless it's another 
subcollection), e.g  bricks=>brick

(ravi, please verify it's works with your change)
Line 2979:     urlparams: {}
Line 2980:     headers:
Line 2981:       Content-Type: {value: application/xml|json, required: true}
Line 2982:       Expect: {value: 201-created, required: false}


--
To view, visit http://gerrit.ovirt.org/14204
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I2a229faa90c8fcded33b53ab8801c4cb03094798
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Sahina Bose <sab...@redhat.com>
Gerrit-Reviewer: Michael Pasternak <mpast...@redhat.com>
Gerrit-Reviewer: Ravi Nori <rn...@redhat.com>
Gerrit-Reviewer: Sahina Bose <sab...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to