From 170d1ca6e579bf61943965ed93d6b6d539b6dc0d Mon Sep 17 00:00:00 2001 From: Vishal Saugat Date: Wed, 30 Nov 2022 01:17:02 +0530 Subject: [PATCH 1/4] Added tags for cert-manager-irsa policy tags are not being applied policy created for cert manager irsa even when I pass the parameters. See https://github.com/aws-ia/terraform-aws-eks-blueprints/issues/1216 for more info --- modules/kubernetes-addons/cert-manager/main.tf | 2 ++ 1 file changed, 2 insertions(+) diff --git a/modules/kubernetes-addons/cert-manager/main.tf b/modules/kubernetes-addons/cert-manager/main.tf index 7d137ac6c1..fdcf988405 100644 --- a/modules/kubernetes-addons/cert-manager/main.tf +++ b/modules/kubernetes-addons/cert-manager/main.tf @@ -44,4 +44,6 @@ resource "aws_iam_policy" "cert_manager" { name = "${var.addon_context.eks_cluster_id}-${local.helm_config["name"]}-irsa" path = var.addon_context.irsa_iam_role_path policy = data.aws_iam_policy_document.cert_manager_iam_policy_document.json + + tags = var.addon_context.tags } From e628f4838406aa85872aa2064b4dfb40458c6613 Mon Sep 17 00:00:00 2001 From: Vishal Saugat Date: Wed, 30 Nov 2022 12:19:10 +0530 Subject: [PATCH 2/4] Remove trailing line --- modules/kubernetes-addons/cert-manager/main.tf | 1 - 1 file changed, 1 deletion(-) diff --git a/modules/kubernetes-addons/cert-manager/main.tf b/modules/kubernetes-addons/cert-manager/main.tf index fdcf988405..1dc68265f7 100644 --- a/modules/kubernetes-addons/cert-manager/main.tf +++ b/modules/kubernetes-addons/cert-manager/main.tf @@ -44,6 +44,5 @@ resource "aws_iam_policy" "cert_manager" { name = "${var.addon_context.eks_cluster_id}-${local.helm_config["name"]}-irsa" path = var.addon_context.irsa_iam_role_path policy = data.aws_iam_policy_document.cert_manager_iam_policy_document.json - tags = var.addon_context.tags } From 7b43e222827a558bf19e4024e48bca418f2bcdfe Mon Sep 17 00:00:00 2001 From: Vishal Saugat Date: Wed, 30 Nov 2022 13:05:46 +0530 Subject: [PATCH 3/4] Added formatting --- modules/kubernetes-addons/cert-manager/main.tf | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/modules/kubernetes-addons/cert-manager/main.tf b/modules/kubernetes-addons/cert-manager/main.tf index 1dc68265f7..573030c795 100644 --- a/modules/kubernetes-addons/cert-manager/main.tf +++ b/modules/kubernetes-addons/cert-manager/main.tf @@ -44,5 +44,5 @@ resource "aws_iam_policy" "cert_manager" { name = "${var.addon_context.eks_cluster_id}-${local.helm_config["name"]}-irsa" path = var.addon_context.irsa_iam_role_path policy = data.aws_iam_policy_document.cert_manager_iam_policy_document.json - tags = var.addon_context.tags + tags = var.addon_context.tags } From 39f6e1b7b5663f3adf6b7cbeb84105f3e8bd48a0 Mon Sep 17 00:00:00 2001 From: Vishal Saugat Date: Fri, 2 Dec 2022 17:59:53 +0530 Subject: [PATCH 4/4] Added tags --- modules/kubernetes-addons/helm-addon/main.tf | 1 + 1 file changed, 1 insertion(+) diff --git a/modules/kubernetes-addons/helm-addon/main.tf b/modules/kubernetes-addons/helm-addon/main.tf index 0620145364..fe32a31d40 100644 --- a/modules/kubernetes-addons/helm-addon/main.tf +++ b/modules/kubernetes-addons/helm-addon/main.tf @@ -76,4 +76,5 @@ module "irsa" { irsa_iam_permissions_boundary = lookup(var.addon_context, "irsa_iam_permissions_boundary", null) eks_cluster_id = var.addon_context.eks_cluster_id eks_oidc_provider_arn = var.addon_context.eks_oidc_provider_arn + tags = var.addon_context.tags }