Skip to content

Commit

Permalink
ci(tests): reduce number of workers for tests (#2149)
Browse files Browse the repository at this point in the history
The use of as many workers as logical cores
seems to cause significant increases of some
monitored durations, including on Linux, which
causes irrelevant CI failures.

We revert to using as many workers as actual CPUs.

Also, load balancing of tests is improved using worksteal,
which works far better with our tests which have very different
execution times (integration tests are far longer than unit tests).

---------

Signed-off-by: Sylvain Leclerc <[email protected]>
  • Loading branch information
sylvlecl authored Sep 23, 2024
1 parent 6a4bc9c commit c1f7e96
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ jobs:
pip install -r requirements-dev.txt
- name: Test with pytest
run: |
pytest --cov antarest --cov-report xml -n logical
pytest --cov antarest --cov-report xml -n auto --dist=worksteal
- name: Archive code coverage results
if: matrix.os == 'ubuntu-20.04'
uses: actions/upload-artifact@v4
Expand Down

0 comments on commit c1f7e96

Please sign in to comment.