fix partitions for caas and non-rebuilt-enabled clusters

Author: sjpb

environments/.caas/hooks/pre.yml

@@ -14,6 +14,14 @@
   roles:
     - cluster_infra
 
+- name: DEBUG
+  hosts: all
+  tasks:
+    - debug:
+        msg: |
+          openhpc_nodegroups: {{ openhpc_nodegroups }}
+          openhpc_partitions: {{ openhpc_partitions }}
+
 # Ensure that the secrets are generated and persisted on the control host
 - name: Generate and persist secrets
   hosts: control

environments/common/inventory/group_vars/all/openhpc.yml

@@ -17,7 +17,7 @@ openhpc_slurm_control_host: "{{ groups['control'] | first }}" # avoid using host
 openhpc_slurmdbd_host: "{{ openhpc_slurm_control_host }}"
 openhpc_rebuild_partition: # not a role var - could actually add more indirection here for things we're expecting to be modified, e.g. groups and maxtime
   name: rebuild
-  nodegroups: "{{ cluster_compute_groups }}"
+  nodegroups: "{{ cluster_compute_groups | default([]) }}"
   default: NO
   maxtime: 30
   partition_params:
@@ -29,7 +29,7 @@ openhpc_rebuild_partition: # not a role var - could actually add more indirectio
     OverSubscribe: EXCLUSIVE
 openhpc_nodegroups: "{{ cluster_compute_groups | map('community.general.dict_kv', 'name') }}" # create nodegroup for each compute group
 openhpc_user_partitions: "{{ openhpc_nodegroups }}" # create partition for each nodegroup (actually role default) - this is what we'd expect to be changed
-openhpc_partitions: "{{ openhpc_user_partitions + [openhpc_rebuild_partition] if groups['rebuild'] | length > 0 else [] }}" # auto-create rebuild partition if reqd.
+openhpc_partitions: "{{ openhpc_user_partitions + ([openhpc_rebuild_partition] if groups['rebuild'] | length > 0 else []) }}" # auto-create rebuild partition if reqd.
 openhpc_packages_default:
   # system packages
   - podman