Juan Hernandez has posted comments on this change.

Change subject: restapi: CRUD for OpenStackVolume authentication keys
......................................................................


Patch Set 6:

(1 comment)

https://gerrit.ovirt.org/#/c/41969/6/backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml
File 
backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml:

Line 3924:   request:
Line 3925:     body:
Line 3926:       parameterType: OpenstackVolumeAuthenticationKey
Line 3927:       signatures:
Line 3928:       - mandatoryArguments: {}
> When not using an empty list, I get an error in "Generating RSDL files..." 
This shouldn't happen in my environment. I used the master branch, applied your 
patch, removed this empty list and RSDL generation works correctly. Are you 
sure that you tried to remove the above line and then put the dash before the 
"optionalArguments" line?

  - optionalArguments: <-- Note the dash
      ...

Can you share the exact error message that you get during the build?
Line 3929:         optionalArguments:
Line 3930:           openstack_volume_authentication_key.value: xs:string
Line 3931:           openstack_volume_authentication_key.usage_type: xs:string
Line 3932:           openstack_volume_authentication_key.description: xs:string


-- 
To view, visit https://gerrit.ovirt.org/41969
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I1041a9760c29919d86da5479ce29db666d18a924
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Daniel Erez <de...@redhat.com>
Gerrit-Reviewer: Daniel Erez <de...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernan...@redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipc...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to