Skip to content

Commit

Permalink
CI flakiness fix: reduce numprocesses for integration tests (#769)
Browse files Browse the repository at this point in the history
* Experiment: reduce numprocesses for integration tests

* Make `numprocesses` conditional based on the backend
  • Loading branch information
palinatolmach authored Aug 13, 2024
1 parent 7547212 commit a56c9c4
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions .github/workflows/test-pr.yml
Original file line number Diff line number Diff line change
Expand Up @@ -90,8 +90,12 @@ jobs:
docker exec -u github-user kontrol-ci-integration-${GITHUB_SHA} /bin/bash -c 'CXX=clang++-14 poetry run kdist --verbose build -j`nproc` evm-semantics.haskell kontrol.foundry'
- name: 'Run integration tests'
run: |
TEST_ARGS='--numprocesses=6 -vv -k "not (test_kontrol_cse or test_foundry_minimize_proof or test_kontrol_end_to_end)"'
[ ${{ matrix.backend }} == 'legacy' ] && TEST_ARGS+=' --no-use-booster'
TEST_ARGS='-vv -k "not (test_kontrol_cse or test_foundry_minimize_proof or test_kontrol_end_to_end)"'
if [ ${{ matrix.backend }} == 'legacy' ]; then
TEST_ARGS+=' --no-use-booster --numprocesses=6'
else
TEST_ARGS+=' --numprocesses=4'
fi
docker exec --user github-user kontrol-ci-integration-${GITHUB_SHA} make cov-integration TEST_ARGS="${TEST_ARGS}"
- name: 'Tear down Docker'
if: always()
Expand Down

0 comments on commit a56c9c4

Please sign in to comment.