Closed adriananeci closed 4 months ago
Dependency Review / govulncheck (pull_request)
PR check failure doesn't seem related to changes from this PR if I read the logs right
/assign @M00nF1sh
/lgtm
Dependency Review / govulncheck (pull_request) PR check failure doesn't seem related to changes from this PR if I read the logs right
@adriananeci, correct, it's another issue we will address later. can you please address the comments so we can merge your PR?
Dependency Review / govulncheck (pull_request) PR check failure doesn't seem related to changes from this PR if I read the logs right
@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 /approve
[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
Issue
Helm doesn't support currently passing user parameters of the parent chart to the subchart (without using global) See https://github.com/helm/helm/issues/6699 and https://github.com/helm/helm/pull/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 https://github.com/aws/eks-charts/pull/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 .
Error: execution error at (test-chart/charts/aws-load-balancer-controller/templates/deployment.yaml:61:28): Chart cannot be installed without a valid clusterName! helm.go:84: [debug] execution error at (/test-chart/charts/aws-load-balancer-controller/templates/deployment.yaml:61:28): Chart cannot be installed without a valid clusterName!