Skip to content

Commit

Permalink
ci: Limit runtime to 15 minutes and run only one pypy test at a time
Browse files Browse the repository at this point in the history
Otherwise, when tests hang, they take 6 hours to finish. Our mean time
is 8 minutes, thus 15 should be plenty of headroom

The pypy tests seem to hang on GitHub Actions and we dont' get logs for
what happened. Running them serially seems to gix the problem, let's do
that for now
  • Loading branch information
BenjaminSchubert committed Dec 29, 2023
1 parent caf3b30 commit 7eb6df7
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -81,6 +81,7 @@ jobs:
test:
runs-on: ${{ matrix.os }}
needs: package
timeout-minutes: 15

strategy:
fail-fast: false
Expand Down Expand Up @@ -116,7 +117,7 @@ jobs:
path: ${{ env.PACKAGES_PATH }}

- name: pytest[${{ matrix.python }}]
run: dwas --verbose --only pytest[${{ matrix.python }}] -- --numprocesses auto
run: dwas --verbose --only pytest[${{ matrix.python }}] -- ${{ !startsWith(matrix.python, 'pypy') && '--numprocesses 2' || '' }}

- name: Move the coverage to another place to avoid conflicts
if: runner.os != 'Linux'
Expand Down

0 comments on commit 7eb6df7

Please sign in to comment.