github-mirrors/terraform-aws-eks
1
0
Fork 0
mirror of https://github.com/ysoftdevs/terraform-aws-eks.git synced 2026-01-17 09:07:20 +01:00
Code Issues Packages Projects Releases Wiki Activity

fix: Avoid re-naming the primary security group through a Name tag and leave to the EKS service to manage (#2010)

Browse Source
This commit is contained in:
Bryant Biggs
2022-04-12 04:36:03 -04:00
committed by GitHub
parent 69a815c7df
commit b5ae5daa39
2 changed files with 11 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
examples/eks_managed_node_group/main.tf
View File

@@ -67,6 +67,13 @@ module "eks" {
resources = ["secrets"] resources = ["secrets"]
}] }]
cluster_tags = {
# This should not affect the name of the cluster primary security group
# Ref: https://github.com/terraform-aws-modules/terraform-aws-eks/pull/2006
# Ref: https://github.com/terraform-aws-modules/terraform-aws-eks/pull/2008
Name = local.name
}
vpc_id = module.vpc.vpc_id vpc_id = module.vpc.vpc_id
subnet_ids = module.vpc.private_subnets subnet_ids = module.vpc.private_subnets

5
main.tf
View File

@@ -60,7 +60,10 @@ resource "aws_eks_cluster" "this" {
} }
resource "aws_ec2_tag" "cluster_primary_security_group" { resource "aws_ec2_tag" "cluster_primary_security_group" {
for_each = { for k, v in merge(var.tags, var.cluster_tags) : k => v if local.create } # This should not affect the name of the cluster primary security group
# Ref: https://github.com/terraform-aws-modules/terraform-aws-eks/pull/2006
# Ref: https://github.com/terraform-aws-modules/terraform-aws-eks/pull/2008
for_each = { for k, v in merge(var.tags, var.cluster_tags) : k => v if local.create && k != "Name" }
resource_id = aws_eks_cluster.this[0].vpc_config[0].cluster_security_group_id resource_id = aws_eks_cluster.this[0].vpc_config[0].cluster_security_group_id
key = each.key key = each.key