Skip to content
This repository has been archived by the owner on Oct 22, 2021. It is now read-only.

fix: use correct secret name when credentials are overridden #1671

Merged
merged 1 commit into from
Jan 5, 2021

Conversation

jandubois
Copy link
Member

@jandubois jandubois commented Jan 4, 2021

kubecf uses the quarks mechanism to override explicit bosh variables defined in cloudfoundry-incubator/quarks-operator#915

This works automatically for bosh variables, but not for explicit references from manually created kube objects. This PR provides a helper template to generate the proper name.

Fixes #1666

@jandubois jandubois requested a review from mook-as January 4, 2021 22:12
@jandubois jandubois changed the title Use correct secret name when credentials are overridden fix: use correct secret name when credentials are overridden Jan 4, 2021
kubecf uses the quarks mechanism to override explicit bosh variables
defined in cloudfoundry-incubator/quarks-operator#915

This works automatically for bosh variables, but not for explicit
references from manually created kube objects. This PR provides a
helper template to generate the proper name.
@jandubois
Copy link
Member Author

Just force-pushed this PR because I realized that the template function will work in mixins/eirini/config/eirini.yaml. It only requires the user values in $.Values and doesn't depend on config files being merged in.

chart/templates/_helpers.tpl Show resolved Hide resolved
chart/templates/_helpers.tpl Show resolved Hide resolved
@viovanov viovanov merged commit 0480a6b into master Jan 5, 2021
@viovanov viovanov deleted the jandubois/vars-secret-names branch January 5, 2021 09:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

apps_dns does not find cf-app-sd-client-tls secret credential
3 participants