-
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
feat: Allow setting set_values
through helm_config
for AWS load balancer controller addon
#834
Conversation
…alancer controller addon
<<-EOT | ||
clusterName: ${module.eks_blueprints.eks_cluster_id} | ||
region: ${local.region} | ||
vpcId: ${module.vpc.vpc_id} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we previously had to replicate the values file here in order to set the vpcId
for running the AWS LBC on Fargate. now we can set just the additional values we need to provide to the AWS LBC addon
depends_on = [ | ||
kubernetes_namespace_v1.sample_app[0] | ||
] | ||
yaml_body = templatefile("sample_app.yaml", { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
flattened to one multi-resource yaml file with templating so that we can map dependencies (by passing namespace from namespace resource)
@@ -6,7 +6,7 @@ locals { | |||
name = local.name | |||
chart = local.name | |||
repository = "https://aws.github.io/eks-charts" | |||
version = "1.4.2" | |||
version = "1.4.3" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1.4.4 is out today
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, we should consider applying the same change to all the other addons following the same motivation mentioned here.
…alancer controller addon (aws-ia#834)
What does this PR do?
set_values
throughhelm_config
for AWS load balancer controller addonMotivation
set_values
allows for the default values file to stay as is as the minimum required values, and users can pass in any additional through thevar.helm_config.set_values
. The motivation for being able to set additional values - to able to set the pod disruption budget on the AWS LBC for better HA (discovered during testing for cluster upgrade guidance)More
pre-commit run -a
with this PRNote: Not all the PRs required examples and docs except a new pattern or add-on added.
For Moderators
Additional Notes