Skip to content

Commit 2d80af3

Browse files
committed
compute nodes must now be marked as available for topo-aware scheduling
1 parent fc48709 commit 2d80af3

File tree

2 files changed

+9
-1
lines changed

2 files changed

+9
-1
lines changed

ansible/slurm.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,9 @@
6363
tasks:
6464
- include_role:
6565
name: topology
66-
when: appliances_mode == 'configure'
66+
# Gated on topology group having compute nodes but role also
67+
# needs to run on control and login nodes
68+
when: appliances_mode == 'configure' and (groups['topology'] | length) > 0
6769
- include_role:
6870
name: stackhpc.openhpc
6971
tasks_from: "{{ 'runtime.yml' if appliances_mode == 'configure' else 'main.yml' }}"

environments/common/inventory/groups

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,12 @@ openhpc
2424
[builder]
2525
# Do not add hosts here manually - used as part of Packer image build pipeline. See packer/README.md.
2626

27+
[topology]
28+
# Compute nodes to be included in the Slurm topology plugin's topology tree
29+
# Should be set to `compute` if enabled
30+
# Note that this feature currently assumes all compute nodes are VMs, enabling
31+
# when the cluster contains baremetal compute nodes may lead to unexpected scheduling behaviour
32+
2733
[podman:children]
2834
# Hosts running containers for below services:
2935
opensearch

0 commit comments

Comments
 (0)