Skip to content

Commit

Permalink
add node_group_name as output
Browse files Browse the repository at this point in the history
  • Loading branch information
morga471 committed Jul 30, 2024
1 parent 091915f commit bcbf1e3
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 23 deletions.
22 changes: 1 addition & 21 deletions main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -114,7 +114,7 @@ module "cluster" {
}

eks_managed_node_group_defaults = {
ami_type = "AL2023_x86_64_STANDARD"
ami_type = "aws-k8s-1.30"
}

node_security_group_enable_recommended_rules = true
Expand Down Expand Up @@ -149,15 +149,6 @@ module "cluster" {
}
}
}
# taints = {
# # This Taint aims to keep just EKS Addons and Karpenter running on this MNG
# # The pods that do not tolerate this taint should run on nodes created by Karpenter
# addons = {
# key = "CriticalAddonsOnly"
# value = "true"
# effect = "NO_SCHEDULE"
# },
# }
}
}

Expand Down Expand Up @@ -185,14 +176,3 @@ resource "null_resource" "kube_config_create" {
command = "aws eks --region ${data.aws_region.current.name} update-kubeconfig --name ${module.cluster.cluster_name} --profile=${var.profile} && export KUBE_CONFIG_PATH=~/.kube/config && export KUBERNETES_MASTER=~/.kube/config"
}
}


# resource "kubernetes_namespace" "operators" {
# depends_on = [
# module.cluster.eks_managed_node_groups,
# ]

# metadata {
# name = var.operators_ns
# }
# }
7 changes: 5 additions & 2 deletions outputs.tf
Original file line number Diff line number Diff line change
Expand Up @@ -277,6 +277,9 @@ output "self_managed_node_groups_autoscaling_group_names" {
################################################################################
# Additional
################################################################################
output "cluster_autoscaler_role_name" {
value = module.cluster_autoscaler_irsa_role.iam_role_name
# output "cluster_autoscaler_role_name" {
# value = module.cluster_autoscaler_irsa_role.iam_role_name
# }
output "node_group_name" {
value = local.ng_name
}

0 comments on commit bcbf1e3

Please sign in to comment.