Slurm6. Add support for dynamic nodeset.

[mr0re1]

Unlike normal or TPU nodesets, don't add dedicated modules, but instead expect user to specify them. E.g.:

  - id: savanna
    source: community/modules/compute/schedmd-slurm-gcp-v6-partition
    settings:
      partition_name: savanna
      nodeset_dyn:
      - nodeset_name: giraffe
        nodeset_feature: spots
      - nodeset_name: zebra
        nodeset_feature: stripes

[danielahlin]

Testing this I get the following error:

Error: Resource precondition failed

on .terraform/modules/slurm_controller.slurm_partition/terraform/slurm_cluster/modules/slurm_partition/main.tf line 53, in resource "null_resource" "partition":
53: condition = local.has_node || local.has_dyn || local.has_tpu
├────────────────
│ local.has_dyn is false
│ local.has_node is false
│ local.has_tpu is false

Partition must contain at least one type of nodeset.

Error: Resource precondition failed

on .terraform/modules/slurm_controller.slurm_partition/terraform/slurm_cluster/modules/slurm_partition/main.tf line 57, in resource "null_resource" "partition":
57: condition = ((!local.has_node || !local.has_dyn) && local.has_tpu) || ((local.has_node || local.has_dyn) && !local.has_tpu)
├────────────────
│ local.has_dyn is false
│ local.has_node is false
│ local.has_tpu is false

Partition cannot contain TPU and non-TPU nodesets.

[danielahlin]

Thanks, this results in the correct configuration of the dynamic partition.