Skip to content

Commit 22c2aae

Browse files
authored
ci: run the NVHPC container lanes without MPI (#1822)
1 parent cc20c94 commit 22c2aae

1 file changed

Lines changed: 10 additions & 2 deletions

File tree

.github/workflows/test.yml

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -187,6 +187,7 @@ jobs:
187187
-e "FFLAGS=-tp=px -Kieee -noswitcherror" \
188188
-e CFLAGS=-tp=px \
189189
-e CXXFLAGS=-tp=px \
190+
-e "MFC_NVHPC_TEST_FLAGS=--test-all --no-mpi" \
190191
"$NVHPC_IMAGE" sleep infinity
191192
192193
- name: Setup NVHPC
@@ -289,12 +290,19 @@ jobs:
289290
CHANGED_FILES: ${{ needs.file-changes.outputs.changed_files }}
290291

291292
# ── NVHPC build + test (via docker exec into long-lived container) ──
293+
# MFC_NVHPC_TEST_FLAGS carries --no-mpi: post_process segfaults under
294+
# mpirun in these containers, and every failure the lanes have produced
295+
# has been an MPI test (see PR #1822). The harness skips ppn>1 cases
296+
# without MPI, so the rest of --test-all -- the compile coverage these
297+
# lanes exist for -- is unaffected, as is MPI coverage elsewhere. Set in
298+
# one place so the build and test steps cannot disagree: a no-MPI build
299+
# tested with MPI would run ppn>1 cases against a binary that has none.
292300
- name: Build (NVHPC)
293301
if: matrix.nvhpc && matrix.target == 'cpu'
294302
run: |
295303
docker exec nvhpc bash -c '
296304
source /etc/nvhpc-env.sh
297-
/bin/bash mfc.sh test -v --dry-run -j $(nproc) --test-all
305+
/bin/bash mfc.sh test -v --dry-run -j $(nproc) $MFC_NVHPC_TEST_FLAGS
298306
'
299307
300308
- name: Build (NVHPC GPU)
@@ -336,7 +344,7 @@ jobs:
336344
docker exec nvhpc bash -c '
337345
source /etc/nvhpc-env.sh
338346
ulimit -s unlimited || ulimit -s 65536 || true
339-
/bin/bash mfc.sh test -v --max-attempts 3 -j $(nproc) --test-all
347+
/bin/bash mfc.sh test -v --max-attempts 3 -j $(nproc) $MFC_NVHPC_TEST_FLAGS
340348
'
341349
342350
# ── Cleanup ─────────────────────────────────────────────────────────

0 commit comments

Comments
 (0)