Skip to content

Commit c581fe1

Browse files
committed
removed faulty cleanup and added lustre to extrabuild test
1 parent 59e95de commit c581fe1

File tree

2 files changed

+9
-28
lines changed

2 files changed

+9
-28
lines changed

.github/workflows/extra.yml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,12 +8,14 @@ on:
88
- 'environments/.stackhpc/terraform/cluster_image.auto.tfvars.json'
99
- 'ansible/roles/doca/**'
1010
- 'ansible/roles/cuda/**'
11+
- 'ansible/roles/lustre/**'
1112
- '.github/workflows/extra.yml'
1213
pull_request:
1314
paths:
1415
- 'environments/.stackhpc/terraform/cluster_image.auto.tfvars.json'
1516
- 'ansible/roles/doca/**'
1617
- 'ansible/roles/cuda/**'
18+
- 'ansible/roles/lustre/**'
1719
- '.github/workflows/extra.yml'
1820

1921
jobs:
@@ -29,11 +31,11 @@ jobs:
2931
build:
3032
- image_name: openhpc-extra-RL8
3133
source_image_name_key: RL8 # key into environments/.stackhpc/terraform/cluster_image.auto.tfvars.json
32-
inventory_groups: doca,cuda
34+
inventory_groups: doca,cuda,lustre
3335
volume_size: 30 # needed for cuda
3436
- image_name: openhpc-extra-RL9
3537
source_image_name_key: RL9
36-
inventory_groups: doca,cuda
38+
inventory_groups: doca,cuda,lustre
3739
volume_size: 30 # needed for cuda
3840
env:
3941
ANSIBLE_FORCE_COLOR: True

ansible/roles/lustre/tasks/install.yml

Lines changed: 5 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -41,30 +41,9 @@
4141
ansible.builtin.dnf:
4242
name: "{{ _lustre_find_rpms.files | map(attribute='path')}}"
4343
disable_gpg_check: yes
44-
45-
- block:
46-
- name: Remove lustre build prerequisites
47-
# NB Only remove ones this role installed which weren't upgrades
48-
ansible.builtin.dnf:
49-
name: "{{ _new_pkgs }}"
50-
state: absent
51-
vars:
52-
_installed_pkgs: |
53-
{{
54-
_lustre_dnf_build_packages.results |
55-
select('match', 'Installed:') |
56-
map('regex_replace', '^Installed: (.+?)-[0-9].*$', '\1')
57-
}}
58-
_removed_pkgs: |
59-
{{
60-
_lustre_dnf_build_packages.results |
61-
select('match', 'Removed:') |
62-
map('regex_replace', '^Removed: (.+?)-[0-9].*$', '\1')
63-
}}
64-
_new_pkgs: "{{ _installed_pkgs | difference(_removed_pkgs) }}"
65-
66-
- name: Delete lustre build dir
67-
file:
68-
path: "{{ lustre_build_dir }}"
69-
state: absent
44+
45+
- name: Delete lustre build dir
46+
file:
47+
path: "{{ lustre_build_dir }}"
48+
state: absent
7049
when: lustre_build_cleanup | bool

0 commit comments

Comments
 (0)