Support for updating service instances

[ldmberman]

Hi,

I am going to work on adding support for updating service instances. From the start, I would like to discuss things to be sure I keep following the right direction.

These are the public-facing changes I am going to make:

• Update brokerapi.Service with a PlanUpdatable field:

    ...
    PlanUpdatable bool `json:"plan_updateable"`

• Extend ServiceBroker with an Update method:

    ...
    Update(instanceID string, details UpdateDetails, asyncAllowed bool) (IsAsync, error)

• With UpdateDetails declared as:

type UpdateDetails struct {
    ServiceID string `json:"service_id"`
    PlanID string `json:"plan_id"`
    Parameters map[string]interface{} `json:"parameters"`
}

Both the public-facing part and the work to be done under the hood seem very similar to what is done for instance provisioning, and I do not expect to encounter any pitfalls for now.

Looking for your thoughts.

[cf-gitbot]

We have created an issue in Pivotal Tracker to manage this. You can view the current status of your issue at: https://www.pivotaltracker.com/story/show/109435680.

[craigfurman]

Hi @ldmberman, it turns out we recently committed exactly the same thing.

Unfortunately, we didn't see this suggestion before today.

[ldmberman]

Anyway, thank you for pushing this work forward.