openservicebrokerapi / servicebroker

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

Orphan mitigation when provisioning asynchronously? #360

Closed kibbles-n-bytes closed 5 years ago

kibbles-n-bytes commented 6 years ago

The orphan mitigation section of the spec only details synchronous situations. However, it seems to me like the platform giving up on an async provisioning operation because it hasn't received a "succeeded" or "failed" state should also trigger orphan mitigation.

duglin commented 6 years ago

The https://github.com/openservicebrokerapi/servicebroker/blob/master/spec.md#polling-interval-and-duration section talks about it going into a "failed" state, but then stops. If we added text in there about the failure triggering the orphan mitigation logic then I think that would be good.

avade commented 6 years ago

SGTM

mattmcneeney commented 6 years ago

SGTM too. @kibbles-n-bytes are you able to open a PR for this?

kibbles-n-bytes commented 6 years ago

@mattmcneeney Yes, I will have one out today tomorrow!

mattmcneeney commented 5 years ago

I believe this is done!