Skip to content

Commit dc1f47d

Browse files
authored
Address code review comments
1 parent 9eed717 commit dc1f47d

File tree

4 files changed

+7
-2
lines changed

4 files changed

+7
-2
lines changed

ansible/roles/compute_init/README.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,7 @@ it also requires an image build with the role name added to the
4040
| hooks/pre.yml | ? | None at present | n/a |
4141
| validate.yml | n/a | Not relevant during boot | n/a |
4242
| bootstrap.yml | (wait for ansible-init) | Not relevant during boot | n/a |
43+
| bootstrap.yml | journald | Fully supported | No |
4344
| bootstrap.yml | resolv_conf | Fully supported | No |
4445
| bootstrap.yml | etc_hosts | Fully supported | No |
4546
| bootstrap.yml | chrony | Fully supported | No |

docs/experimental/isolated-clusters.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,7 @@ See above for definition of "Default" features. In the "Isolated?" column:
4444
| gateway | n/a | n/a - build only |
4545
| grafana | Y | Y |
4646
| hpctests | Y | Y |
47+
| journald | Y | Y |
4748
| k3s_agent | - | ? |
4849
| k3s_server | - | ? |
4950
| k9s | - | ? |

environments/common/inventory/groups

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,9 +26,8 @@ login
2626
openhpc
2727
additional
2828

29-
[journald:childen]
29+
[journald]
3030
# Hosts where journald should be configured. See ansible/roles/journald/README.md.
31-
cluster
3231

3332
[builder]
3433
# Do not add hosts here manually - used as part of Packer image build pipeline. See packer/README.md.

environments/site/inventory/groups

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -81,6 +81,10 @@ cluster
8181
# Hosts to recompile Slurm for - allows supporting Slurm autodetection method 'nvml'
8282
cuda
8383

84+
[journald:childen]
85+
# Hosts where journald should be configured. See ansible/roles/journald/README.md.
86+
cluster
87+
8488
[eessi:children]
8589
# Hosts on which EESSI stack should be configured
8690
openhpc

0 commit comments

Comments
 (0)