openservicebrokerapi / servicebroker

Open Service Broker API Specification
https://openservicebrokerapi.org/
Apache License 2.0
1.19k stars 434 forks source link

Add Platform recommendations for dealing with "plan_updateable" and "bindable" #446

Closed mattmcneeney closed 6 years ago

mattmcneeney commented 6 years ago

The "plan_updateable" and "bindable" fields in the catalog define what behaviours a Service Broker support. This change adds text stating that Platforms SHOULD NOT make a bind or update service instance call to a Service Broker that sets these fields to false.

This seems especially important for "bindable" as the spec currently says:

Service Brokers that do not offer any bindable services do not need to implement the endpoint for bind requests.
cfdreddbot commented 6 years ago

Hey mattmcneeney!

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.

duglin commented 6 years ago

LGTM

Approved with PullApprove

duglin commented 6 years ago

easy review - 3 needed

fmui commented 6 years ago

LGTM

Approved with PullApprove

n3wscott commented 6 years ago

LGTM

Approved with PullApprove

n3wscott commented 6 years ago

LGTM

Approved with PullApprove

duglin commented 6 years ago

LGTM

Approved with PullApprove

fmui commented 6 years ago

LGTM

Approved with PullApprove

duglin commented 6 years ago

two more needed

duglin commented 6 years ago

LGTM

Approved with PullApprove

zrob commented 6 years ago

lgtm

Approved with PullApprove