Reintroduce Cluster Autoscaler label and taint tags (#221)

* fix: Add autoscaler label and taint tags to node_group_tags

* fix: Update launch template to use local.node_tags for tags

Author: tionichm

launch-template.tf

@@ -56,7 +56,7 @@ locals {
     }
     vpc_security_group_ids = local.launch_template_vpc_security_group_ids
     user_data              = local.userdata
-    tags                   = local.node_group_tags
+    tags                   = local.node_tags
     cpu_options            = var.cpu_options
     placement              = var.placement
     enclave_options        = var.enclave_enabled ? ["true"] : []

main.tf

@@ -44,14 +44,16 @@ locals {
     "k8s.io/cluster-autoscaler/${var.cluster_name}" = "owned"
     "k8s.io/cluster-autoscaler/enabled"             = "true"
   }
+  */
+  # AWS does not currently apply the autoscaler autodiscovery label/taint tags to the node group.
   autoscaler_kubernetes_label_tags = {
     for label, value in var.kubernetes_labels : format("k8s.io/cluster-autoscaler/node-template/label/%v", label) => value
   }
   autoscaler_kubernetes_taints_tags = {
     for taint in var.kubernetes_taints : format("k8s.io/cluster-autoscaler/node-template/taint/%v", taint.key) =>
     "${taint.value == null ? "" : taint.value}:${local.taint_effect_map[taint.effect]}"
   }
-
+  /*
 
   node_tags = merge(
     module.label.tags,
@@ -65,8 +67,12 @@ locals {
   # because they only matter when applied to the autoscaling group.
   node_group_tags = local.node_tags
   */
-  node_tags       = module.label.tags
-  node_group_tags = module.label.tags
+  node_tags = module.label.tags
+  node_group_tags = merge(
+    module.label.tags,
+    local.autoscaler_kubernetes_label_tags,
+    local.autoscaler_kubernetes_taints_tags
+  )
 }
 
 module "label" {