remove delimiter support

Author: AdarshK15

community/modules/compute/gke-nodeset/metadata.yaml

@@ -30,12 +30,12 @@ ghpc:
       vars: [network_storage]
       min: 1
       max: 1
-      length_check: true
+      length_check: true # Validates the list's length instead of its elements.
     error_message: "The 'network_storage' list must contain exactly one element."
   - validator: range
     inputs:
       vars: [filestore_id]
       min: 1
       max: 1
-      length_check: true
+      length_check: true # Validates the list's length instead of its elements.
     error_message: "The 'filestore_id' list must contain exactly one element."

community/modules/compute/mig/metadata.yaml

@@ -24,5 +24,5 @@ ghpc:
     inputs:
       vars: [versions]
       min: 1
-      length_check: true
+      length_check: true # Validates the list's length instead of its elements.
     error_message: "The 'versions' list must contain at least one version."

community/modules/compute/notebook/metadata.yaml

@@ -20,11 +20,8 @@ spec:
     - storage.googleapis.com
 ghpc:
   validators:
-  - validator: range
+  - validator: regex
     inputs:
       vars: [mount_runner.args]
-      min: 5
-      max: 5
-      length_check: true
-      delimiter: " "
+      pattern: "^[^ ]+( [^ ]+){4}$"
     error_message: "The 'mount_runner.args' field must contain exactly 5 elements."

community/modules/database/slurm-cloudsql-federation/metadata.yaml

@@ -24,5 +24,5 @@ ghpc:
   - validator: regex
     inputs:
       vars: [network_id]
-      pattern: ^projects/[^/]+/global/networks/[^/]+$
+      pattern: "^projects/[^/]+/global/networks/[^/]+$"
     error_message: "The network id must be provided in the following format: projects/<project_id>/global/networks/<network_name>."

community/modules/internal/slurm-gcp/nodeset_tpu/metadata.yaml

@@ -19,11 +19,9 @@ ghpc:
     inputs:
       vars: [node_count_static]
       min: 0
-      length_check: false
     error_message: "The 'node_count_static' value must be at least 0."
   - validator: range
     inputs:
       vars: [node_count_dynamic_max]
       min: 0
-      length_check: false
     error_message: "The 'node_count_dynamic_max' value must be at least 0."

community/modules/scheduler/htcondor-access-point/metadata.yaml

@@ -24,5 +24,5 @@ ghpc:
     inputs:
       vars: [zones]
       max: 2
-      length_check: true
+      length_check: true # Validates the list's length instead of its elements.
     error_message: "The 'zones' list must contain at most 2 zones."

community/modules/scheduler/schedmd-slurm-gcp-v6-controller/metadata.yaml

@@ -25,7 +25,7 @@ ghpc:
     inputs:
       vars: [static_ips]
       max: 1
-      length_check: true
+      length_check: true # Validates the list's length instead of its elements.
     error_message: "The Slurm modules supports 0 or 1 static IPs on controller instance."
   - validator: regex
     inputs:

docs/blueprint-validation.md

@@ -122,21 +122,20 @@ ghpc:
 ```
 
 ### Range Validator
-The `range` validator ensures that input variables (or their computed lengths/segment counts) fall within a specified numerical minimum and/or maximum. It supports validating individual numeric values, the number of elements in a list, or the count of segments in a delimited string (when an optional delimiter is specified). The optional length_check field (defaulting to false) determines whether to validate the value itself or the count of its elements/segments.
+The `range` validator ensures input variables either their values or lengths fall within specified numerical minimum and/or maximum bounds. It supports validating individual numeric values, lists of numeric values, and the number of elements in a list. The optional length_check field (defaulting to false) determines whether to validate the values themselves or the length of the variable.
 
 **Example definition in `metadata.yaml`:**
 
 ```yaml
 ghpc:
-validators:
+  validators:
   - validator: range
     inputs:
-      vars: [mount_runner.args]
-      min: 5
-      max: 5
-      length_check: true
-      delimiter: " "
-    error_message: "The 'mount_runner.args' field must contain exactly 5 elements."
+      vars: [versions]
+      min: 1
+      max: 8
+      length_check: true # enables validation of the list's length rather than the individual values it contains.
+    error_message: "The 'versions' list must contain at least one version."
 ```
 
 Unlike blueprint-level validators, these are intrinsic to the module and ensure that the module receives data in the exact format required for its internal logic to function.

modules/compute/gke-node-pool/metadata.yaml

@@ -25,3 +25,13 @@ ghpc:
       vars: [name]
       pattern: "^[a-z]([-a-z0-9]{0,34}[a-z0-9])?$"
     error_message: "'name' must start with a lowercase letter, followed by up to 34 lowercase letters, numbers, or hyphens, and cannot end with a hyphen."
+  - validator: range
+    inputs:
+      vars: [local.upgrade_settings.max_surge]
+      min: 0
+    error_message: "'max_surge' value must be at least 0."
+  - validator: range
+    inputs:
+      vars: [local.upgrade_settings.max_unavailable]
+      min: 0
+    error_message: "'max_unavailable' value must be at least 0."

modules/file-system/filestore/metadata.yaml

@@ -24,8 +24,14 @@ ghpc:
     inputs:
       vars: [size_gb]
       min: 1024
-      length_check: false
     error_message: "The 'size_gb' value must be at least 1024 GiB. See https://cloud.google.com/filestore/docs/service-tiers for details."
+
+  - validator: regex
+    inputs:
+      vars: [network_id]
+      pattern: ^projects/[^/]+/global/networks/[^/]+$
+    error_message: "The network id must be provided in the following format: projects/<project_id>/global/networks/<network_name>."
+
   - validator: allowed_enum
     inputs:
       vars: [filestore_tier]