Skip to content

Commit 6e87605

Browse files
committed
ci(spack): build default variant (+mpi +chemistry) and run a real case
Previously installed mfc+chemistry~mpi and used a placeholder smoke test because the example case requires parallel_io which requires +mpi. Now installs the default variant (+mpi +chemistry, all three targets) and runs the 1D_sodshocktube example via the shim under --prebuilt-prefix to verify the end-to-end prebuilt-mode bootstrap. This is a heavier install (openmpi + fftw+mpi + hdf5+mpi enter the graph) so the first run after this push will re-warm the cache. Subsequent pushes hit the cache.
1 parent 5614e39 commit 6e87605

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

.github/workflows/spack.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -34,8 +34,8 @@ jobs:
3434
key: spack-store-${{ runner.os }}-${{ hashFiles('spack-packages/repos/spack_repo/builtin/packages/mfc/package.py', 'spack-packages/repos/spack_repo/builtin/packages/cantera/package.py', 'spack-packages/repos/spack_repo/builtin/packages/py_pyrometheus/package.py') }}
3535
restore-keys: spack-store-${{ runner.os }}-
3636

37-
- name: Install mfc+chemistry~mpi
38-
run: spack install --reuse mfc+chemistry~mpi
37+
- name: Install mfc (defaults to +mpi +chemistry +pre_process +simulation +post_process)
38+
run: spack install --reuse mfc
3939

4040
- name: Smoke test
4141
run: |
@@ -51,5 +51,5 @@ jobs:
5151
echo "found shim: $prefix/bin/mfc"
5252
cd "$prefix/share/mfc"
5353
./mfc.sh run --prebuilt-prefix="$prefix" \
54-
--no-mpi --dry-run \
54+
--mpi -N 1 -n 1 --dry-run \
5555
examples/1D_sodshocktube/case.py

0 commit comments

Comments
 (0)