Skip to content

Commit 442fc6f

Browse files
committed
mpi4py: enable spawn tests workflow by default
The previous inter-communicator race condition has been fixed. Enable the workflow by default to catch new regressions. Signed-off-by: Wenduo Wang <wenduwan@amazon.com>
1 parent 88302d9 commit 442fc6f

File tree

1 file changed

+2
-23
lines changed

1 file changed

+2
-23
lines changed

.github/workflows/ompi_mpi4py.yaml

Lines changed: 2 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -109,30 +109,9 @@ jobs:
109109
retention-days: 2
110110
name: build-artifacts
111111

112-
#==============================================
113-
114-
run_defaults:
115-
# This whole set of tests run with mpi4py's defaults. As of March
116-
# 2024, this means disabling the spawn and dynamic tests. We want
117-
# this job of tests to pass.
112+
run_tests:
118113
needs: [ build ]
119114
uses: ./.github/workflows/ompi_mpi4py_tests.yaml
120115
with:
121-
# This parameter is required, so send a meaningless
122-
# environment variable name that will not affect the tests at
123-
# all (i.e., the tests will be run with default values).
124-
env_name: MAKE_TODAY_AN_OMPI_DAY
125-
126-
#==============================================
127-
128-
run_spawn:
129-
# This whole set of tests runs explicitly with setting "enable the
130-
# spawn tests". As of March 2024, we know that Open MPI is
131-
# failing these tests.
132-
needs: [ build ]
133-
# Only run if the label "mpi4py" is set on this PR.
134-
if: ${{ contains(github.event.pull_request.labels.*.name, 'mpi4py-all') }}
135-
uses: ./.github/workflows/ompi_mpi4py_tests.yaml
136-
with:
137-
# Enable the spawn tests
138116
env_name: MPI4PY_TEST_SPAWN
117+

0 commit comments

Comments
 (0)