-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
[helm chart] Allow passing template values for clusterName, region and vpcId #3664
Conversation
|
/assign @M00nF1sh |
/lgtm |
@adriananeci, correct, it's another issue we will address later. can you please address the comments so we can merge your PR? |
Done |
thanks for your contribution. /lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: adriananeci, oliviassss The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Issue
Helm doesn't support currently passing user parameters of the parent chart to the subchart (without using global)
See helm/helm#6699 and helm/helm#6876 for more context
Description of changes
While using the
aws-load-balancer-controller
as a dependency for another chart , I found it difficult to pass a global or a dynamic value for theclusterName
,vpcId
,region
and other fields.This PR adds support for passing a template as value for the
clusterName
,vpcId
andregion
field.Initial PR was raised in aws/eks-charts#911
Checklist
README.md
for modified charts)version
inChart.yaml
for the modified chart(s)Testing
I've tested it by passing both static value and dynamic ones as follow:
for static values, nothing has changed, e.g.
values.yaml
:helm template --debug .
for template value:
values.yaml
:helm template --debug --set global.clusterName=ethos000-sbx-va6 .
for empty/unset
clusterName
value:helm template --debug .
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.