feat: improve naming node pools (#25)

Co-authored-by: dkool <dkool@dkools-MacBook-Pro.local>
CloudNationHQ · Jan 29, 2024 · c83c82b · c83c82b
1 parent 161744a
commit c83c82b
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/examples/node-pools/locals.tf b/examples/node-pools/locals.tf
@@ -15,6 +15,7 @@ locals {
       node_count     = 2
       zones          = [1, 2]
       mode           = "User"
+      os_type        = "Linux"
       node_labels = {
         "workload" = "database"
       }
@@ -24,6 +25,7 @@ locals {
       vnet_subnet_id = module.network.subnets.cache.id
       node_count     = 2
       zones          = [1]
+      os_type        = "Linux"
       node_labels = {
         "workload" = "cache"
       }

diff --git a/locals.tf b/locals.tf
@@ -8,9 +8,13 @@ locals {
       max_count      = try(pools.max_count, 0)
       min_count      = try(pools.min_count, 0)
       max_surge      = try(pools.max_surge, 50)
-      poolname       = "aks${pools_key}"
       aks_cluster_id = azurerm_kubernetes_cluster.aks.id
 
+      poolname = try(
+        pools.name,
+        pools.os_type == "Linux" ? "npl${pools_key}" : "npw${pools_key}"
+      )
+
       linux_os_config = try(pools.config.linux_os, {
         swap_file_size_mb             = "none"
         transparent_huge_page_defrag  = "madvise"
@@ -33,7 +37,7 @@ locals {
       workload_runtime              = try(pools.workload_runtime, null)
       snapshot_id                   = try(pools.snapshot_id, null)
       priority                      = try(pools.priority, null)
-      os_type                       = try(pools.os_type, null)
+      os_type                       = try(pools.os_type, "Linux")
       os_sku                        = try(pools.os_sku, null)
       node_labels                   = try(pools.node_labels, {})
       node_taints                   = try(pools.node_taints, [])