-
Notifications
You must be signed in to change notification settings - Fork 70
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
As a Korifi API user I want to be able to unbind from a managed service instance #3296
Comments
danail-branekov
changed the title
As a Korifi API user I want to be able to unbind from a service instance
As a Korifi API user I want to be able to unbind from a managed service instance
May 23, 2024
uzabanov
added a commit
to uzabanov/korifi
that referenced
this issue
Nov 1, 2024
fixes cloudfoundry#3296 Co-authored-by: Danail Branekov <danailster@gmail.com>
danail-branekov
added a commit
that referenced
this issue
Nov 1, 2024
fixes #3296 Co-authored-by: Danail Branekov <danailster@gmail.com>
github-project-automation
bot
moved this from 🔄 In progress
to ✅ Done
in Korifi - Backlog
Nov 1, 2024
The way we finalize managed service bindings in this story seems to be causing pollution in CI environments. In order to prevent reverting the whole change, we decided to just revert the finalization logic and replace it with a no-op finalizer. Therefore we re-open this story as a reminder to re-introduce finalization logic for cfservicebindings. |
github-project-automation
bot
moved this from ✅ Done
to 🔄 In progress
in Korifi - Backlog
Nov 8, 2024
uzabanov
added a commit
to uzabanov/korifi
that referenced
this issue
Jan 7, 2025
- Added IsAsync flag to UnbindResponse in OSBAPI and the logic in the Unbind method - Added returning Gone error - Implementing the Unbind call in binding controller and also polling the last operation for the unbind response - Refactor binding controller so that the calls for the Bind, Unbind and Provison and Deprovision in the services controller are identical - Implement Setting owner reference with BlockOwnerDeletion=true and added 'foregroundDeletion' finalizer in binding controller fixes cloudfoundry#3296 Co-authored-by: Danail Branekov <danailster@gmail.com>
uzabanov
added a commit
to uzabanov/korifi
that referenced
this issue
Jan 8, 2025
- Added IsAsync flag to UnbindResponse in OSBAPI and the logic in the Unbind method - Added returning Gone error and ignoring in instances and bindings controllers - Implementing the Unbind call in binding controller and also polling the last operation for the unbind response - Refactor binding controller so that the calls for the Bind, Unbind and Provison and Deprovision in the services controller are identical - Implement Setting owner reference with BlockOwnerDeletion=true and added 'foregroundDeletion' finalizer in binding controller fixes cloudfoundry#3296 Co-authored-by: Danail Branekov <danailster@gmail.com>
github-project-automation
bot
moved this from 🔄 In progress
to ✅ Done
in Korifi - Backlog
Jan 8, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
GIVEN I have bound an app to a managed service
WHEN I
DELETE /v3/service_credential_bindings/my-binding-guid
THEN I get a
202 Accepted
response withLocation
header ppointing me to a binding deletion jobTHEN eventually I see the binding deletion job to succeed
THEN binding data should not be available to the app (after a restart/restage) any more
Dev notes:
The implementation should call
Unbind
on the service brokerWhen requesting unbind from the broker, the implementation should send the
accepts_incomplete=true
query parameter to enable async unbindWhen the broker accepts the request (responds either with 200 OK or
202 Accepted
, the implementation must not check for binding last operation and must remove the finalizer.If the operation succeeds, the
CFServiceBinding
along its credentials secrets should be deletedIf the operation fails, the failure should be reflected in the
CFServiceBinding
statusUnbind OSBAPI docs
Synchronous vs asynchronous operations OSBAPI docs
Managed services proposal
Delete service binding docs
The text was updated successfully, but these errors were encountered: