Merge pull request #3397 from yuvipanda/tag-core

gcp: Tag nodes with their purpose for costing purposes
2i2c-org · Nov 10, 2023 · b50605e · b50605e
2 parents f853017 + 8e6f669
commit b50605e
Showing 1 changed file with 10 additions and 2 deletions.
diff --git a/terraform/gcp/cluster.tf b/terraform/gcp/cluster.tf
@@ -218,6 +218,10 @@ resource "google_container_node_pool" "core" {
     # Faster disks provide faster image pulls!
     disk_type = "pd-balanced"
 
+    resource_labels = {
+      "node-purpose" : "core"
+    }
+
     labels = {
       "hub.jupyter.org/node-purpose" = "core",
       "k8s.dask.org/node-purpose"    = "core"
@@ -336,7 +340,9 @@ resource "google_container_node_pool" "notebook" {
       "https://www.googleapis.com/auth/cloud-platform"
     ]
 
-    resource_labels = each.value.resource_labels
+    resource_labels = merge({
+      "node-purpose" : "notebook"
+    }, each.value.resource_labels)
 
     // Set these values explicitly so they don't "change outside terraform"
     tags = []
@@ -416,7 +422,9 @@ resource "google_container_node_pool" "dask_worker" {
       "https://www.googleapis.com/auth/cloud-platform"
     ]
 
-    resource_labels = each.value.resource_labels
+    resource_labels = merge({
+      "node-purpose" : "dask-worker"
+    }, each.value.resource_labels)
 
     // Set these values explicitly so they don't "change outside terraform"
     tags = []