-
Notifications
You must be signed in to change notification settings - Fork 362
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Service broker update instance schemas #865
Conversation
[#146082145] Signed-off-by: Luis Urraca <[email protected]>
[#146082145] Signed-off-by: Alex Blease <[email protected]>
[#146082143] Signed-off-by: Alex Blease <[email protected]>
[#146082137] Signed-off-by: Alberto Rios <[email protected]>
[#146082133] Signed-off-by: Alberto Rios <[email protected]>
[#149395975] Signed-off-by: Alberto Rios <[email protected]>
[#149396085] Signed-off-by: Luis Urraca <[email protected]>
[#149396075] Signed-off-by: Alex Blease <[email protected]>
Signed-off-by: Alex Blease <[email protected]>
[#149550919] Signed-off-by: Luis Urraca <[email protected]>
* Change Schema length validation to use ActiveModel [#149550919] Signed-off-by: Sam Gunaratne <[email protected]>
[#149550919] Signed-off-by: Sam Gunaratne <[email protected]>
[#149550919] Signed-off-by: Luis Urraca <[email protected]>
[#149550919] Signed-off-by: Luis Urraca <[email protected]>
[#149550919] Signed-off-by: Sam Gunaratne <[email protected]>
Hey lurraca! Thanks for submitting this pull request! All pull request submitters and commit authors must have a Contributor License Agreement (CLA) on-file with us. Please sign the appropriate CLA (individual or corporate). When sending signed CLA please provide your github username in case of individual CLA or the list of github usernames that can make pull requests on behalf of your organization. If you are confident that you're covered under a Corporate CLA, please make sure you've publicized your membership in the appropriate Github Org, per these instructions. Once you've publicized your membership, one of the owners of this repository can close and reopen this pull request, and dreddbot will take another look. |
We have created an issue in Pivotal Tracker to manage this: https://www.pivotaltracker.com/story/show/149890266 The labels on this github issue will be updated when the story is started. |
Hey lurraca! Thanks for submitting this pull request! I'm here to inform the recipients of the pull request that you and the commit authors have already signed the CLA. |
We have created an issue in Pivotal Tracker to manage this: https://www.pivotaltracker.com/story/show/149894190 The labels on this github issue will be updated when the story is started. |
Thanks for the PR @lurraca! @tusing && @ericpromislow - CF CAPI |
Why
The Open Service Broker API is proposing allowing brokers to define JSON schema for their configuration parameters. This will allow tooling to validate parameters and UIs to auto generate forms.
Schemas are to be defined as part of the catalog on a plan and support create/update parameters on a service instance and create parameters on a service binding (update does not exist yet).
The updated spec can be found here.
Example of what a new catalog with schemas will look like:
What
This PR just adds basic support for update instance schemas (create instance schemas functionality already merged in #834). Schemas are parsed during registration, stored in the service plan model and retrieved on the /v2/service_plan/:guid api endpoint. If a broker does not provide a schema, then we default to an empty schema.
update_instance_schema
to plan model objectupdate_instance_schema
Notable things that are not in this PR but are addressed in future stories are:
Notes
Feedback appreciated!
PR
master
branchbundle exec rake
Luis & @Samze