Skip to content

Commit 74e1526

Browse files
authored
Fix empty nodelist in ondemand apps (#861)
* fix empty nodelist in ondemand apps (reqd. when using topology) * enable topology in CI
1 parent 971728b commit 74e1526

File tree

2 files changed

+7
-0
lines changed

2 files changed

+7
-0
lines changed

environments/.stackhpc/inventory/extra_groups

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,3 +40,6 @@ cluster
4040

4141
[compute_init:children]
4242
compute
43+
44+
[topology:children]
45+
compute

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

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -158,7 +158,9 @@ openondemand_apps_desktop_default:
158158
native:
159159
- <%= "--nodes=1" %>
160160
- <%= "--ntasks=#{num_cores}" %>
161+
<% if node != '' %>
161162
- <%= "--nodelist=#{node}" %>
163+
<% end %>
162164
<% if gres != 'none' %>
163165
- <%= "--gres=#{gres}:#{gres_count}" %>
164166
<% end %>
@@ -218,7 +220,9 @@ openondemand_apps_jupyter_default:
218220
native:
219221
- <%= "--nodes=1" %>
220222
- <%= "--ntasks=#{num_cores}" %>
223+
<% if node != '' %>
221224
- <%= "--nodelist=#{node}" %>
225+
<% end %>
222226
<% if gres != 'none' %>
223227
- <%= "--gres=#{gres}:#{gres_count}" %>
224228
<% end %>

0 commit comments

Comments
 (0)