Skip to content

Commit c6aacea

Browse files
authored
Merge branch 'main' into feature/mig
2 parents bd7e1df + eabf59b commit c6aacea

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

56 files changed

+755
-246
lines changed

.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,3 +5,4 @@ venv
55
*.pyc
66
packer/openhpc2
77
.vscode
8+
requirements.yml.last

README.md

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,8 +25,6 @@ The default configuration in this repository may be used to create a cluster to
2525
- Persistent state backed by an OpenStack volume.
2626
- NFS-based shared file system backed by another OpenStack volume.
2727

28-
Note that the Open OnDemand portal and its remote apps are not usable with this default configuration.
29-
3028
It requires an OpenStack cloud, and an Ansible "deploy host" with access to that cloud.
3129

3230
Before starting ensure that:

ansible/.gitignore

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -92,3 +92,5 @@ roles/*
9292
!roles/alertmanager/**
9393
!roles/slurm_recompile/**
9494
!roles/slurm_recompile/**
95+
!roles/nhc/
96+
!roles/nhc/**

ansible/bootstrap.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -134,6 +134,7 @@
134134

135135
- hosts: dnf_repos
136136
become: yes
137+
tags: dnf_repos
137138
tasks:
138139
- name: Check that creds won't be leaked to users
139140
ansible.builtin.assert:
@@ -143,7 +144,7 @@
143144
- appliances_mode == 'configure'
144145
- not (dnf_repos_allow_insecure_creds | default(false)) # useful for development
145146

146-
- hosts: cacerts:!builder
147+
- hosts: cacerts
147148
tags: cacerts
148149
gather_facts: false
149150
tasks:

ansible/disable-repos.yml

Lines changed: 0 additions & 8 deletions
This file was deleted.

ansible/extras.yml

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -72,17 +72,6 @@
7272
- import_role:
7373
name: persist_hostkeys
7474

75-
76-
- name: Setup NFS export for compute node configuration
77-
hosts: compute_init:!builder
78-
# NB: has to be after eeesi and os-manila-mount
79-
tags: compute_init
80-
become: yes
81-
tasks:
82-
- include_role:
83-
name: compute_init
84-
tasks_from: export.yml
85-
8675
- name: Install k9s
8776
become: yes
8877
hosts: k9s

ansible/fatimage.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -267,7 +267,7 @@
267267
import_playbook: "{{ hook_path if hook_path | exists else 'noop.yml' }}"
268268
when: hook_path | exists
269269

270-
- import_playbook: disable-repos.yml
270+
- import_playbook: final.yml
271271

272272
- hosts: builder
273273
become: yes

ansible/final.yml

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
1+
- hosts: dnf_repos
2+
become: yes
3+
tags: dnf_repos
4+
tasks:
5+
- name: Disable pulp repos
6+
ansible.builtin.include_role:
7+
name: dnf_repos
8+
tasks_from: disable_repos.yml
9+
10+
- name: Setup NFS export for compute_init
11+
hosts: compute_init:!builder
12+
# NB: done last so other roles can prepare configuration etc
13+
tags: compute_init
14+
become: yes
15+
tasks:
16+
- include_role:
17+
name: compute_init
18+
tasks_from: export.yml

ansible/roles/cacerts/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ Configure CA certificates and trusts.
44

55
## Role variables
66

7-
- `ca-certificates`: Optional str. Path to directory containing certificates
7+
- `cacerts_cert_dir`: Optional str. Path to directory containing certificates
88
in PEM or DER format. Any files here will be added to the list of CAs trusted
99
by the system.
1010

ansible/roles/compute_init/README.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -85,6 +85,7 @@ it also requires an image build with the role name added to the
8585
| slurm.yml | openhpc [10] | All slurmd functionality | No |
8686
| slurm.yml | (set memory limits) | Fully supported | No |
8787
| slurm.yml | (block ssh) | Fully supported | No |
88+
| slurm.yml | nhc | Fully supported | No |
8889
| portal.yml | (openondemand server) | Not relevant for compute nodes | n/a |
8990
| portal.yml | (openondemand vnc desktop) | None required - use image build | No |
9091
| portal.yml | (openondemand jupyter server) | None required - use image build | No |

0 commit comments

Comments
 (0)