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

As a Korifi API user I want to be able to unbind from a managed service instance #3296

Closed
danail-branekov opened this issue May 23, 2024 · 1 comment · Fixed by #3585 or #3688
Closed

Comments

@danail-branekov
Copy link
Member

danail-branekov commented May 23, 2024

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 with Location header ppointing me to a binding deletion job
THEN 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 broker

  • When requesting unbind from the broker, the implementation should send the accepts_incomplete=true query parameter to enable async unbind

  • When 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 deleted

  • If the operation fails, the failure should be reflected in the CFServiceBinding status

  • Unbind OSBAPI docs

  • Synchronous vs asynchronous operations OSBAPI docs

  • Managed services proposal

  • Delete service binding docs

@github-project-automation github-project-automation bot moved this to 🧊 Icebox in Korifi - Backlog May 23, 2024
@danail-branekov 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
@danail-branekov danail-branekov moved this from 🧊 Icebox to 🇪🇺 To do in Korifi - Backlog May 31, 2024
@georgethebeatle georgethebeatle added this to the v0.13 milestone Sep 20, 2024
@georgethebeatle georgethebeatle modified the milestones: v0.13, v0.14 Oct 16, 2024
@danail-branekov danail-branekov self-assigned this Oct 25, 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 github-project-automation bot moved this from 🔄 In progress to ✅ Done in Korifi - Backlog Nov 1, 2024
@georgethebeatle
Copy link
Member

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 github-project-automation bot moved this from ✅ Done to 🔄 In progress in Korifi - Backlog Nov 8, 2024
@uzabanov uzabanov moved this from 🇪🇺 To do to 🔄 In progress in Korifi - Backlog Dec 18, 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 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