Merge pull request #104 from kloia/fix-eks-addon

kloia · Oct 17, 2023 · 8e00389 · 8e00389
2 parents c0ecec3 + 5a63017
commit 8e00389
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 1 deletion.
diff --git a/aws-eks-addons/main.tf b/aws-eks-addons/main.tf
@@ -36,6 +36,7 @@ resource "helm_release" "aws_lb_controller" {
   repository = "https://aws.github.io/eks-charts"
   chart      = "aws-load-balancer-controller"
   namespace  = "kube-system"
+  version    = "1.6.1"
   depends_on = [
     kubernetes_service_account.service-account
   ]

diff --git a/aws-eks-addons/variables.tf b/aws-eks-addons/variables.tf
@@ -145,7 +145,7 @@ variable "deploy_rancher_istio" {
 variable "connect_hostnames_from_alb_ing_prefix" {
   description = "Prefix to give ingress names when connecting alb hostnames"
   type = string
-  default = "ing-{servicename}"
+  default = "ing-servicename"
 }
 
 variable "connect_hostnames_from_alb_to_nginx" {

diff --git a/aws-iam/modules/iam-role-for-service-accounts-eks/policies.tf b/aws-iam/modules/iam-role-for-service-accounts-eks/policies.tf
@@ -651,6 +651,7 @@ data "aws_iam_policy_document" "load_balancer_controller" {
       "elasticloadbalancing:DescribeTargetGroupAttributes",
       "elasticloadbalancing:DescribeTargetHealth",
       "elasticloadbalancing:DescribeTags",
+      "elasticloadbalancing:AddTags",
     ]
     resources = ["*"]
   }