Skip to content

Commit d474a4f

Browse files
authored
Disable compute-init by default & warn of security issue (#585)
* Disable compute-init by default, warn of security issue * Update compute-init readme now not cookiecutter default * reenable compute-init for stackhpc b/c CI tests it * don't reimage compute nodes twice
1 parent 2de4f84 commit d474a4f

File tree

4 files changed

+11
-7
lines changed

4 files changed

+11
-7
lines changed

.github/workflows/stackhpc.yml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -182,9 +182,8 @@ jobs:
182182
run: |
183183
. venv/bin/activate
184184
. environments/.stackhpc/activate
185-
ansible-playbook -v --limit compute ansible/adhoc/rebuild.yml
186-
ansible-playbook -v ansible/ci/check_slurm.yml
187185
ansible-playbook -v ansible/adhoc/reboot_via_slurm.yml
186+
ansible-playbook -v ansible/ci/check_slurm.yml
188187
189188
- name: Check sacct state survived reimage
190189
run: |

ansible/roles/compute_init/README.md

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,10 +3,13 @@
33
Experimental functionality to allow compute nodes to rejoin the cluster after
44
a reboot without running the `ansible/site.yml` playbook.
55

6+
**CAUTION:** The approach used here of exporting cluster secrets over NFS
7+
is considered to be a security risk due to the potential for cluster users to
8+
mount the share on a user-controlled machine by tunnelling through a login
9+
node. This feature should not be enabled on production clusters at this time.
10+
611
To enable this:
7-
1. Add the `compute` group (or a subset) into the `compute_init` group. This is
8-
the default when using cookiecutter to create an environment, via the
9-
"everything" template.
12+
1. Add the `compute` group (or a subset) into the `compute_init` group.
1013
2. Build an image which includes the `compute_init` group. This is the case
1114
for StackHPC-built release images.
1215
3. Enable the required functionalities during boot, by setting the

environments/.stackhpc/inventory/extra_groups

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -41,3 +41,6 @@ control
4141

4242
[cacerts:children]
4343
cluster
44+
45+
[compute_init:children]
46+
compute

environments/common/layouts/everything

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -93,9 +93,8 @@ cluster
9393
[sshd]
9494
# Hosts where the OpenSSH server daemon should be configured
9595

96-
[compute_init:children]
96+
[compute_init]
9797
# EXPERIMENTAL: Compute hosts to enable joining cluster on boot on
98-
compute
9998

10099
[k3s:children]
101100
# Hosts to run k3s server/agent

0 commit comments

Comments
 (0)