Skip to content
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

Node group tags are applied but not recognized during following runs #1961

Closed
gijsdpg opened this issue Mar 24, 2022 · 4 comments
Closed

Node group tags are applied but not recognized during following runs #1961

gijsdpg opened this issue Mar 24, 2022 · 4 comments

Comments

@gijsdpg
Copy link

gijsdpg commented Mar 24, 2022

Description

When I apply this terraform:

https://gist.github.com/gijsdpg/3009309d32c9d298b214d2c7ea615e13

The tags are correctly applied everywhere, but when I rerun terraform it doesn't detect the tags and wants to apply them again for the aws_security_group, aws_launch_template, aws_iam_role and aws_eks_node_group for every node group.

Versions

Tried this with multiple terraform (including v1.1.7), eks module (including 18.11.0) and aws provider (including 4.6.0) versions.

Reproduction

Steps to reproduce the behavior:
not using workspaces, cleared local cache. Just apply this terraform in a new project:

https://gist.github.com/gijsdpg/3009309d32c9d298b214d2c7ea615e13

and reapply, the tags are not properly detected.

Code Snippet to Reproduce

Expected behavior

The tags should be recognized by terraform and not apply again.

Actual behavior

The tags are not recognized by terraform and are applied again.

Terminal Output Screenshot(s)

one of the resources:

  # module.eks.module.eks_managed_node_group["spot1"].aws_security_group.this[0] will be updated in-place
  ~ resource "aws_security_group" "this" {
        id                     = "sg-04abde212566809a3"
        name                   = "spot1-DELETE_ME-eks-node-group-20220323160757995700000006"
      ~ tags                   = {
          + "CNCA"                                = "DELETE_ME"
          + "CreatedBy"                           = "terraform"
          + "Responsible"                         = "recosearch"
          + "Stage"                               = "DELETE_ME"
          + "repository"                          = "https://XXXX"
            # (3 unchanged elements hidden)
        }
        # (9 unchanged attributes hidden)
    }

Plan: 0 to add, 8 to change, 0 to destroy.
@bryantbiggs
Copy link
Member

I have never experienced this before and I deploy this module several times a day. Could you try the following for your cluster definition:

module "eks" {
  source  = "terraform-aws-modules/eks/aws"
  version = "18.14.0"

  cluster_name    = local.cluster_name
  cluster_version = "1.21"

  subnet_ids = [
    data.aws_subnet.private-a.id,
    data.aws_subnet.private-b.id,
    data.aws_subnet.private-c.id
  ]
  vpc_id = data.aws_vpc.prod.id

  cluster_endpoint_private_access = true
  cluster_endpoint_public_access  = true
  enable_irsa                     = true

  cluster_addons = {
    coredns = {
      resolve_conflicts = "OVERWRITE"
      addon_version     = "v1.8.4-eksbuild.1"
    }
    kube-proxy = {
      resolve_conflicts = "OVERWRITE"
      addon_version     = "v1.21.2-eksbuild.2"
    }
    vpc-cni = {
      resolve_conflicts        = "OVERWRITE"
      addon_version            = "v1.10.1-eksbuild.1"
      service_account_role_arn = module.vpc_cni_irsa.iam_role_arn
    }
  }

  eks_managed_node_group_defaults = {
    instance_types = ["m6a.large"]
    capacity_type  = "SPOT"

    iam_role_additional_policies = ["arn:aws:iam::aws:policy/AmazonSSMManagedInstanceCore"]

    tags = {
      "k8s.io/cluster-autoscaler/${local.cluster_name}" = "owned"
      "k8s.io/cluster-autoscaler/enabled"               = "TRUE"
    }
  }

  eks_managed_node_groups = {
    spot0 = {
      name         = "spot0-${local.cluster_name}"
      min_size     = 1
      max_size     = 3
      desired_size = 1
    }

    spot1 = {
      name         = "spot1-${local.cluster_name}"
      min_size     = 1
      max_size     = 3
      desired_size = 1
    }
  }

  tags = local.tags
}

@gijsdpg
Copy link
Author

gijsdpg commented Mar 28, 2022

thanks for the quick answer and sorry for the delay. If I apply that terraform I get:

╷
│ Error: "tags" are identical to those in the "default_tags" configuration block of the provider: please de-duplicate and try again
│
│   with module.eks.aws_cloudwatch_log_group.this[0],
│   on .terraform/modules/eks/main.tf line 70, in resource "aws_cloudwatch_log_group" "this":
│   70: resource "aws_cloudwatch_log_group" "this" {
│
╵
╷
│ Error: "tags" are identical to those in the "default_tags" configuration block of the provider: please de-duplicate and try again
│
│   with module.eks.aws_iam_role.this[0],
│   on .terraform/modules/eks/main.tf line 210, in resource "aws_iam_role" "this":
│  210: resource "aws_iam_role" "this" {

So my guess is that my original issue is also related to some weird interaction between the AWS provider default tag settings and the custom tag settings.

If I remember correctly, the AWS provider default tag doesn't work with ec2 instances and volumes, setting the tag manually solves that issue. But if I leave away default tags the AWS provider many other resources are not tagged.

@bryantbiggs
Copy link
Member

This is an upstream issue that we unfortunately cannot do anything about here hashicorp/terraform-provider-aws#19204

@github-actions
Copy link

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 13, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants