Skip to content

Commit 9f47321

Browse files
committed
fix lustre build for 2.15.5 release candidate
1 parent 410e0ed commit 9f47321

File tree

2 files changed

+20
-8
lines changed

2 files changed

+20
-8
lines changed

ansible/roles/lustre/defaults/main.yml

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
1-
2-
lustre_version: 'b2_15' # TODO: FIXME to 2.15.5 on release see http://lists.lustre.org/pipermail/lustre-discuss-lustre.org/2024-June/019163.html
1+
lustre_version: '2.15.5-RC3' # TODO: FIXME to 2.15.5 on release see http://lists.lustre.org/pipermail/lustre-discuss-lustre.org/2024-June/019163.html
32
lustre_mounts: [{}] # Ensures if all top-level vars are provided then this isn't required
43
#lustre_mgs_nid:
54
#lustre_fs_name:
@@ -32,6 +31,6 @@ lustre_configure_opts:
3231
- --disable-gss-keyring
3332
- --enable-mpitests=no
3433
lustre_rpm_globs: # NB: order is important here, as not installing from a repo
35-
- "kmod-lustre-client-{{ lustre_version }}-*"
36-
- "lustre-client-{{ lustre_version }}-*"
34+
- "kmod-lustre-client-{{ lustre_version | split('.') | first }}*" # only take part of the version as -RC versions produce _RC rpms
35+
- "lustre-client-{{ lustre_version | split('.') | first }}*"
3736
lustre_cleanup_build: true

ansible/roles/lustre/tasks/install.yml

Lines changed: 17 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111

1212
- name: Prepare for lustre configuration
1313
ansible.builtin.command:
14-
cmd: ./autogen.sh
14+
cmd: sh ./autogen.sh
1515
chdir: "{{ lustre_build_dir }}"
1616

1717
- name: Configure lustre build
@@ -43,11 +43,24 @@
4343

4444
- block:
4545
- name: Remove lustre build prerequisites
46-
# NB only remove ones this role installed!
46+
# NB Only remove ones this role installed which weren't upgrades
4747
ansible.builtin.dnf:
48-
name: "{{ item | regex_replace('Installed: ', '') }}"
48+
name: "{{ _new_pkgs }}"
4949
state: absent
50-
loop: "{{ _lustre_dnf_build_packages.results }}"
50+
vars:
51+
_installed_pkgs: |
52+
{{
53+
_lustre_dnf_build_packages.results |
54+
select('match', 'Installed:') |
55+
map('regex_replace', '^Installed: (.+?)-[0-9].*$', '\1')
56+
}}
57+
_removed_pkgs: |
58+
{{
59+
_lustre_dnf_build_packages.results |
60+
select('match', 'Removed:') |
61+
map('regex_replace', '^Removed: (.+?)-[0-9].*$', '\1')
62+
}}
63+
_new_pkgs: "{{ _installed_pkgs | difference(_removed_pkgs) }}"
5164

5265
- name: Delete lustre build dir
5366
file:

0 commit comments

Comments
 (0)