Skip to content

Conversation

@jochenehret
Copy link
Contributor

@jochenehret jochenehret commented Dec 5, 2025

Description of the Change

Enhancement for rfc-0040-service-binding-rotation.

Add parameter strategy to bind-service command. Allowed values are single (default) and multiple. With multiple, the CLI sends and additional parameter "strategy": "multiple" to the CC v3 API:
https://v3-apidocs.cloudfoundry.org/version/3.207.0/index.html#create-a-service-credential-binding

Why Is This PR Valuable?

This PR enhances the CLI so that the multiple service binding feature is enabled for end users. More PRs for other related commands will follow.

Applicable Issues

cloudfoundry/community#1198

How Urgent Is The Change?

Not super urgent.

Other Relevant Parties

Change is backwards compatible, so no regressions expected.

@jochenehret jochenehret marked this pull request as ready for review December 5, 2025 14:16
@jochenehret jochenehret changed the title Add "--strategy" parameter to "bind-service" command [v8] Add "--strategy" parameter to "bind-service" command Dec 5, 2025
* and complete "help" docu integration tests
@jochenehret jochenehret force-pushed the bind-service_strategy_parameter branch from fbcace6 to f1d3052 Compare January 8, 2026 12:49
@jochenehret jochenehret requested a review from anujc25 January 14, 2026 14:19
@anujc25
Copy link
Contributor

anujc25 commented Jan 14, 2026

I merged a minor change around the integration tests. Can you please rebase this PR so that we can see if we can do some verification through integration tests?

Also, Will it be possible to add some integration tests for this change?

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.

LGTM. Thanks for addressing the comments.

@anujc25 anujc25 merged commit 5f38fd4 into cloudfoundry:v8 Jan 15, 2026
16 checks passed
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.

2 participants