Cleanup node group if build node is disabled

convox · May 31, 2023 · 6df011a · 6df011a
1 parent 5d2a118
commit 6df011a
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 1 deletion.
diff --git a/terraform/cluster/aws/main.tf b/terraform/cluster/aws/main.tf
@@ -126,7 +126,7 @@ resource "aws_eks_node_group" "cluster-build" {
     aws_iam_openid_connect_provider.cluster,
   ]
 
-  count = 1
+  count = var.build_node_enabled ? 1 : 0
 
   ami_type        = var.gpu_type ? "AL2_x86_64_GPU" : var.arm_type ? "AL2_ARM_64" : "AL2_x86_64"
   capacity_type   = "ON_DEMAND"
@@ -169,6 +169,8 @@ resource "aws_autoscaling_group_tag" "cluster-build" {
     aws_eks_node_group.cluster-build
   ]
 
+  count = var.build_node_enabled ? 1 : 0
+
   autoscaling_group_name = aws_eks_node_group.cluster-build[0].resources[0].autoscaling_groups[0].name
 
   tag {

diff --git a/terraform/cluster/aws/variables.tf b/terraform/cluster/aws/variables.tf
@@ -6,6 +6,10 @@ variable "availability_zones" {
   default = ""
 }
 
+variable "build_node_enabled" {
+  default = false
+  type    = bool
+}
 
 variable "build_node_type" {
   type = string

diff --git a/terraform/system/aws/main.tf b/terraform/system/aws/main.tf
@@ -47,6 +47,7 @@ module "cluster" {
 
   arm_type                 = local.arm_type
   availability_zones       = var.availability_zones
+  build_node_enabled       = var.build_node_enabled
   build_node_min_count     = var.build_node_min_count
   build_node_type          = var.build_node_type != "" ? var.build_node_type : var.node_type
   cidr                     = var.cidr