Sahina Bose has posted comments on this change.

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


Patch Set 1: (2 inline comments)

Uploading new patchset

....................................................
File 
backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml
Line 6: - name: /api/vms|rel=get
Line 7:   request:
Line 8:     body:
Line 9:       parameterType: null
Line 10:       signatures: []
Done
Line 11:     urlparams: 
Line 12:       search: {context: query, type: 'xs:string', value: 'search 
query', required: false}
Line 13:       case_sensitive: {context: matrix, type: 'xs:boolean', value: 
true|false, required: false}
Line 14:       max: {context: matrix, type: 'xs:int', value: 'max results', 
required: false}


Line 2975:       signatures:
Line 2976:       - mandatoryArguments:
Line 2977:           brick--COLLECTION: {brick.server_id: 'xs:string', 
brick.brick_dir: 'xs:string'}
Line 2978:         optionalArguments:
Line 2979:           brick--COLLECTION: {brick.replica_count: 
'xs:unsignedShort', brick.stripe_count: 'xs:unsignedShort'}
Yes, you're right. Have tested that I need to specify bricks.replica_count
Line 2980:     urlparams: {}
Line 2981:     headers:
Line 2982:       Content-Type: {value: application/xml|json, required: true}
Line 2983:       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: 1
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