Skip to content

[v8] Command "unbind-service" can handle multiple bindings#3665

Open
jochenehret wants to merge 3 commits intocloudfoundry:v8from
sap-contributions:v8_unbind_service_cmd_multi_bindings
Open

[v8] Command "unbind-service" can handle multiple bindings#3665
jochenehret wants to merge 3 commits intocloudfoundry:v8from
sap-contributions:v8_unbind_service_cmd_multi_bindings

Conversation

@jochenehret
Copy link
Contributor

Description of the Change

The "unbind-service" command shall be able to handle multiple bindings per (app, service instance). This PR first retrieves the list of all bindings for a given app and a given service instance name. Then it loops over the list and calls DELETE /v3/service_credential_bindings/:guid for each item. A new parameter "--guid" allows to delete only a specific binding

Why Is This PR Valuable?

Allows users to delete multiple service bindings with one CF CLI command.

Applicable Issues

https://github.com/cloudfoundry/community/blob/main/toc/rfc/rfc-0040-service-binding-rotation.md#cf-cli

How Urgent Is The Change?

Not super-urgent.

Other Relevant Parties

Copy link
Contributor

@Dariquest Dariquest left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@anujc25 anujc25 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just one minor comment about updating the integration tests otherwise LGTM.

@anujc25
Copy link
Contributor

anujc25 commented Feb 4, 2026

Can you please help rebasing this PR once? We fixed the environment creation issue that we were seeing for integration tests.

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 this pull request may close these issues.

3 participants