Skip to content
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

Clarify the state of an instance after a failed update or delete #566

Open
fmui opened this issue Aug 3, 2018 · 3 comments
Open

Clarify the state of an instance after a failed update or delete #566

fmui opened this issue Aug 3, 2018 · 3 comments
Assignees

Comments

@fmui
Copy link
Member

fmui commented Aug 3, 2018

Update and delete operations may fail for many different reasons. For example:

  • A deletion may fail because other instances or resources depend on the instance.
  • Resources to do the update are not available at this time.
  • A legal hold does not allow an instance to be altered or deleted.

In many cases, when such an operation fails, the service instance is untouched and still usable. From an application point of view nothing changed. (In the legal hold case there MUST be no change.)

For the update operation, the spec contains this sentence:

When the response includes a 4xx status code, the Service Broker MUST NOT apply any of the requested changes to the Service Instance.

That implies the service instance is still usable.

For the delete operation, we have this sentence:

...MUST be interpreted as a failure and the Platform MUST remember the Service Instance.

The word "remember” does not specify whether the instance is still operational or in an unusable state.
We should clarify what that means and maybe add another status code (409?) that let the broker tell the platform that the instance is untouched.

@fmui
Copy link
Member Author

fmui commented Aug 7, 2018

See PR #570.

@fmui
Copy link
Member Author

fmui commented Aug 7, 2018

See also issue #358.

@nilebox
Copy link

nilebox commented Aug 27, 2018

It would also be nice to have similar feature for "create" (provisioning) operation.
Otherwise platform has to always perform orphan mitigation "just in case" after failed async provisioning, even if nothing has been created.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Inbox
Development

No branches or pull requests

2 participants