Skip to content

Commit 2e1ba04

Browse files
authored
Merge branch 'main' into fix/build-hpl-fatimage
2 parents bc2f2b3 + bb4d95f commit 2e1ba04

File tree

7 files changed

+27
-8
lines changed

7 files changed

+27
-8
lines changed

.github/workflows/stackhpc.yml

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,7 @@ on:
99
- '**'
1010
- '!dev/**'
1111
- 'dev/setup-env.sh'
12-
- '!docs/**'
13-
- '!README.md'
12+
- '!**.md'
1413
- '!.gitignore'
1514
- '!.github/workflows/'
1615
- '.github/workflows/stackhpc'
@@ -19,8 +18,7 @@ on:
1918
- '**'
2019
- '!dev/**'
2120
- 'dev/setup-env.sh'
22-
- '!docs/**'
23-
- '!README.md'
21+
- '!**.md'
2422
- '!.gitignore'
2523
- '!.github/workflows/'
2624
- '.github/workflows/stackhpc'

ansible/roles/freeipa/README.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,7 @@ Support FreeIPA in the appliance. In production use it is expected the FreeIPA s
3737

3838
- `freeipa_host_password`. Required for initial enrolment only, FreeIPA host password as described above.
3939
- `freeipa_setup_dns`: Optional, whether to use the FreeIPA server as the client's nameserver. Defaults to `true` when `freeipa_server` contains a host, otherwise `false`.
40+
- `freeipa_ca_cert_file`: Optional, path **on the ansible deploy host** to FreeIPA server cert. Else this will be downloaded (insecurely) from the FreeIPA server over http.
4041

4142
See also use of `appliances_state_dir` on the control node as described above.
4243

ansible/roles/freeipa/defaults/main.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,5 +10,5 @@ freeipa_user_defaults:
1010
ipa_pass: "{{ freeipa_admin_password | quote }}"
1111
ipa_user: admin
1212
freeipa_users: [] # see community.general.ipa_user
13-
13+
freeipa_ca_cert_file: ''
1414
_freeipa_keytab_backup_path: "{{ hostvars[groups['control'].0].appliances_state_dir }}/freeipa/{{ inventory_hostname }}/krb5.keytab"

ansible/roles/freeipa/tasks/enrol.yml

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,15 @@
1616
mode: ug=rw,o=
1717
when: '"content" in _slurp_persisted_keytab'
1818

19+
- name: Copy CA Cert to host
20+
ansible.builtin.copy:
21+
src: "{{ freeipa_ca_cert_file }}"
22+
dest: /etc/ipa/ca.crt
23+
mode: u=rw,go=r
24+
owner: root
25+
group: root
26+
when: freeipa_ca_cert_file != ''
27+
1928
- name: Re-enrol with FreeIPA using backed-up keytab
2029
# Re-enrolment requires --force-join and --password, or --keytab
2130
# Re-rolement means:
@@ -31,6 +40,9 @@
3140
--mkhomedir
3241
--enable-dns-updates
3342
--keytab /tmp/krb5.keytab
43+
{% if freeipa_ca_cert_file != '' %}
44+
--ca-cert-file=/etc/ipa/ca.crt
45+
{% endif %}
3446
when: '"content" in _slurp_persisted_keytab'
3547
register: ipa_client_install_keytab
3648
changed_when: ipa_client_install_keytab.rc == 0
@@ -47,6 +59,9 @@
4759
--mkhomedir
4860
--enable-dns-updates
4961
--password '{{ freeipa_host_password }}'
62+
{% if freeipa_ca_cert_file != '' %}
63+
--ca-cert-file=/etc/ipa/ca.crt
64+
{% endif %}
5065
when:
5166
- '"content" not in _slurp_persisted_keytab'
5267
- freeipa_host_password is defined

ansible/roles/ofed/README.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,10 @@
11
# ofed
22

3+
> [!IMPORTANT]
4+
> This role is deprecated - it is not regularly maintained and StackHPC CI
5+
> does not test that it works. Consider using [ansible/roles/doca](../doca/README.md)
6+
> instead.
7+
38
This role installs Mellanox OFED:
49
- It checks that the running kernel is the latest installed one, and errors if not.
510
- Installation uses the `mlnxofedinstall` command, with support for the running kernel

docs/upgrades.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -54,9 +54,9 @@ All other commands should be run on the Ansible deploy host.
5454
Make changes as necessary.
5555

5656
1. Identify image(s) from the relevant [Slurm appliance release](https://github.com/stackhpc/ansible-slurm-appliance/releases), and download
57-
using the link on the release plus the image name, e.g. for an image `openhpc-ofed-RL8-240906-1042-32568dbb`:
57+
using the link on the release plus the image name, e.g. for an image `openhpc-RL9-250708-1547-1494192e`:
5858

59-
wget https://object.arcus.openstack.hpc.cam.ac.uk/swift/v1/AUTH_3a06571936a0424bb40bc5c672c4ccb1/openhpc-images/openhpc-ofed-RL8-240906-1042-32568dbb
59+
wget https://object.arcus.openstack.hpc.cam.ac.uk/swift/v1/AUTH_3a06571936a0424bb40bc5c672c4ccb1/openhpc-images/openhpc-RL9-250708-1547-1494192e
6060

6161
Note that some releases may not include new images. In this case use the image from the latest previous release with new images.
6262

requirements.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
ansible==6.7.0 # cloudalchemy.prometheus uses ansible.builtin.include, removed in ansible-core==2.16 => ansible==9
22
openstacksdk
3-
python-openstackclient==6.6.1 # v7.0.0 has a bug re. rebuild
3+
python-openstackclient==8.0.0
44
python-manilaclient
55
python-ironicclient
66
jmespath

0 commit comments

Comments
 (0)