Add dbd instance_role in slurm_instance

Adding this instance_role will allow for a future toolkit change to add a separate dbd instance.
GoogleCloudPlatform · Nov 4, 2024 · cb87e64 · cb87e64
1 parent f153fd9
commit cb87e64
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/terraform/slurm_cluster/modules/_slurm_instance/variables.tf b/terraform/slurm_cluster/modules/_slurm_instance/variables.tf
@@ -138,13 +138,13 @@ variable "labels" {
 #########
 
 variable "slurm_instance_role" {
-  description = "Slurm instance type. Must be one of: controller; login; compute."
+  description = "Slurm instance type. Must be one of: controller; login; dbd; compute."
   type        = string
   default     = null
 
   validation {
-    condition     = contains(["controller", "login", "compute"], lower(var.slurm_instance_role))
-    error_message = "Must be one of: controller; login; compute."
+    condition     = contains(["controller", "login", "dbd", "compute"], lower(var.slurm_instance_role))
+    error_message = "Must be one of: controller; login; dbd; compute."
   }
 }
 

diff --git a/terraform/slurm_cluster/modules/slurm_instance_template/variables.tf b/terraform/slurm_cluster/modules/slurm_instance_template/variables.tf
@@ -340,15 +340,15 @@ variable "additional_disks" {
 
 variable "slurm_instance_role" {
   type        = string
-  description = "Slurm instance type. Must be one of: controller; login; compute; or null."
+  description = "Slurm instance type. Must be one of: controller; dbd; login; compute; or null."
   default     = null
 
   validation {
     condition = (
       var.slurm_instance_role == null
       ? true
-    : contains(["controller", "login", "compute"], lower(var.slurm_instance_role)))
-    error_message = "Must be one of: controller; login; compute; or null."
+    : contains(["controller", "dbd", "login", "compute"], lower(var.slurm_instance_role)))
+    error_message = "Must be one of: controller; dbd; login; compute; or null."
   }
 }