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

Is the 2.14 spec contradictory around orphan mitigation? #563

Closed
williammartin opened this issue Jul 26, 2018 · 1 comment · Fixed by #579
Closed

Is the 2.14 spec contradictory around orphan mitigation? #563

williammartin opened this issue Jul 26, 2018 · 1 comment · Fixed by #579
Assignees

Comments

@williammartin
Copy link
Contributor

Hey all,

I'm trying to understand the orphan mitigation sections of the specification and right now it seems to be contradictory. There may be more cases, but the sections that jumped out to me were 4** status code section in the orphans section and the instructions on what to do under certain response codes from the provision operation

The relevant snippets of text are:

Status Code Of Service Broker Response Platform Interpretation Of Response Platform Initiates Orphan Mitigation?
... ... ...
All other 4xx Request rejected No
... ... ...

vs.

Responses with any other status code MUST be interpreted as a failure and a deprovision request MUST be sent to the Service Broker to prevent an orphan being created on the Service Broker

where status codes defined in the 4** range are 400, 409, and 422.

Am I missing something here like the table is generic and can be overridden by specific endpoint instructions, or is this contradictory?

Thanks,

Will

@Samze
Copy link
Contributor

Samze commented Aug 2, 2018

I think this is a mistake in the provision operation section. In CC we follow the guidance set out in the orphan mitigation table and do not orphan mitigate for 4xx codes (aside from #456 which we're addressing).

duglin pushed a commit to duglin/servicebroker that referenced this issue Aug 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants