add allowed_enum validators to various modules (#5106)

Author: saara-tyagi27

community/modules/compute/schedmd-slurm-gcp-v6-nodeset-dynamic/metadata.yaml

@@ -18,3 +18,23 @@ spec:
     services: [compute.googleapis.com]
 ghpc:
   inject_module_id: name
+  validators:
+  - validator: allowed_enum
+    inputs:
+      vars: [bandwidth_tier]
+      allowed:
+      - platform_default
+      - virtio_enabled
+      - gvnic_enabled
+      - tier_1_enabled
+      allow_null: false
+    error_message: "Allowed values for bandwidth_tier are 'platform_default', 'virtio_enabled', 'gvnic_enabled', or 'tier_1_enabled'."
+
+  - validator: allowed_enum
+    inputs:
+      vars: [on_host_maintenance]
+      allowed:
+      - MIGRATE
+      - TERMINATE
+      allow_null: false
+    error_message: "Allowed values for on_host_maintenance are 'MIGRATE' or 'TERMINATE'."

community/modules/compute/schedmd-slurm-gcp-v6-nodeset/metadata.yaml

@@ -19,3 +19,22 @@ spec:
 ghpc:
   inject_module_id: name
   has_to_be_used: true
+  validators:
+  - validator: allowed_enum
+    inputs:
+      vars: [bandwidth_tier]
+      allowed:
+      - platform_default
+      - virtio_enabled
+      - gvnic_enabled
+      - tier_1_enabled
+      allow_null: false
+    error_message: "Allowed values for bandwidth_tier are 'platform_default', 'virtio_enabled', 'gvnic_enabled', or 'tier_1_enabled'."
+  - validator: allowed_enum
+    inputs:
+      vars: [on_host_maintenance]
+      allowed:
+      - MIGRATE
+      - TERMINATE
+      allow_null: false
+    error_message: "Allowed values for on_host_maintenance are 'MIGRATE' or 'TERMINATE'."

modules/compute/vm-instance/metadata.yaml

@@ -17,3 +17,23 @@ spec:
   requirements:
     services:
     - compute.googleapis.com
+ghpc:
+  validators:
+  - validator: allowed_enum
+    inputs:
+      vars: [on_host_maintenance]
+      allowed:
+      - MIGRATE
+      - TERMINATE
+      allow_null: true
+    error_message: "on_host_maintenance must be either 'MIGRATE' or 'TERMINATE'."
+
+  - validator: allowed_enum
+    inputs:
+      vars: [bandwidth_tier]
+      allowed:
+      - not_enabled
+      - gvnic_enabled
+      - tier_1_enabled
+      allow_null: false
+    error_message: "Allowed values for bandwidth_tier are 'not_enabled', 'gvnic_enabled', or 'tier_1_enabled'."

modules/file-system/filestore/metadata.yaml

@@ -17,3 +17,36 @@ spec:
   requirements:
     services:
     - file.googleapis.com
+
+ghpc:
+  validators:
+  - validator: allowed_enum
+    inputs:
+      vars: [filestore_tier]
+      allowed:
+      - BASIC_HDD
+      - BASIC_SSD
+      - HIGH_SCALE_SSD
+      - ZONAL
+      - REGIONAL
+      - ENTERPRISE
+      allow_null: false
+    error_message: "Invalid value for filestore_tier. The value must be one of: BASIC_HDD, BASIC_SSD, HIGH_SCALE_SSD, ZONAL, REGIONAL, ENTERPRISE."
+
+  - validator: allowed_enum
+    inputs:
+      vars: [connect_mode]
+      allowed:
+      - DIRECT_PEERING
+      - PRIVATE_SERVICE_ACCESS
+      allow_null: false
+    error_message: "'connect_mode' must be either DIRECT_PEERING or PRIVATE_SERVICE_ACCESS."
+
+  - validator: allowed_enum
+    inputs:
+      vars: [protocol]
+      allowed:
+      - NFS_V3
+      - NFS_V4_1
+      allow_null: false
+    error_message: "Allowed values for protocol are 'NFS_V3' or 'NFS_V4_1'."

modules/file-system/gke-storage/metadata.yaml

@@ -16,3 +16,43 @@
 spec:
   requirements:
     services: []
+ghpc:
+  validators:
+  - validator: allowed_enum
+    inputs:
+      vars: [storage_type]
+      allowed: [parallelstore, hyperdisk-balanced, hyperdisk-throughput, hyperdisk-extreme]
+      case_sensitive: false
+    error_message: "Invalid storage_type. Allowed: Parallelstore, Hyperdisk-balanced, Hyperdisk-throughput, Hyperdisk-extreme."
+
+  - validator: allowed_enum
+    inputs:
+      vars: [access_mode]
+      allowed:
+      - ReadWriteOnce
+      - ReadOnlyMany
+      - ReadWriteMany
+      - ReadWriteOncePod
+      case_sensitive: false
+      allow_null: false
+    error_message: "Allowed values for access_mode are 'ReadWriteOnce', 'ReadOnlyMany', 'ReadWriteMany', or 'ReadWriteOncePod'."
+
+  - validator: allowed_enum
+    inputs:
+      vars: [sc_volume_binding_mode]
+      allowed:
+      - Immediate
+      - WaitForFirstConsumer
+      case_sensitive: false
+      allow_null: false
+    error_message: "Allowed values for sc_volume_binding_mode are 'Immediate' or 'WaitForFirstConsumer'."
+
+  - validator: allowed_enum
+    inputs:
+      vars: [sc_reclaim_policy]
+      allowed:
+      - Retain
+      - Delete
+      case_sensitive: false
+      allow_null: false
+    error_message: "Allowed values for sc_reclaim_policy are 'Retain' or 'Delete'."

modules/scheduler/batch-job-template/metadata.yaml

@@ -20,3 +20,32 @@ spec:
     - compute.googleapis.com
 ghpc:
   inject_module_id: job_id
+  validators:
+  - validator: allowed_enum
+    inputs:
+      vars: [on_host_maintenance]
+      allowed:
+      - MIGRATE
+      - TERMINATE
+      allow_null: true
+    error_message: "on_host_maintenance must be either 'MIGRATE' or 'TERMINATE'. If omitted, the system will choose the appropriate default based on your hardware configuration."
+
+  - validator: allowed_enum
+    inputs:
+      vars: [gcloud_version]
+      allowed:
+      - alpha
+      - beta
+      - ""
+      allow_null: false
+    error_message: "Allowed values for gcloud_version are 'alpha', 'beta', or '' (empty string)."
+
+  - validator: allowed_enum
+    inputs:
+      vars: [log_policy]
+      allowed:
+      - CLOUD_LOGGING
+      - PATH
+      - DESTINATION_UNSPECIFIED
+      allow_null: false
+    error_message: "Allowed values for log_policy are 'CLOUD_LOGGING', 'PATH', or 'DESTINATION_UNSPECIFIED'."