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

Support managed binding parameters #3698

Merged
merged 1 commit into from
Jan 10, 2025

Conversation

danail-branekov
Copy link
Member

Is there a related GitHub Issue?

#3549

What is this change about?

  • Introduce the optional CFServiceBinding.Spec.Parameters to reference a secret to
    store the binding parameters
  • The parameters secret is created by the binding repository. kubectl
    users should create it themselves if they want to provide such
    parameters
  • The binding parameters are sent to the broker on bind. If
    Spec.Parameters is not set, no parameters are sent to the broker

Tag your pair, your PM, and/or team

@georgethebeatle

* Introduce the optional `CFServiceBinding.Spec.Parameters` to reference a secret to
  store the binding parameters
* The parameters secret is created by the binding repository. `kubectl`
  users should create it themselves if they want to provide such
  parameters
* The binding parameters are sent to the broker on `bind`. If
  `Spec.Parameters` is not set, no parameters are sent to the broker

issue #3549

Co-authored-by: Danail Branekov <[email protected]>
Co-authored-by: Georgi Sabev <[email protected]>
@georgethebeatle georgethebeatle force-pushed the issues/3549-cf-bind-with-parameters branch from c3ce57e to 4ffdd57 Compare January 10, 2025 13:38
@georgethebeatle georgethebeatle enabled auto-merge (rebase) January 10, 2025 13:38
@georgethebeatle georgethebeatle merged commit f0adb5d into main Jan 10, 2025
10 checks passed
@georgethebeatle georgethebeatle deleted the issues/3549-cf-bind-with-parameters branch January 10, 2025 14:03
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