Skip to content

Commit

Permalink
Run tests with "parallel" feature in CI (#853)
Browse files Browse the repository at this point in the history
* Run tests with "parallel" feature in CI

* Add parallel bench

* Remove --realese flag from run-tests-parallel
  • Loading branch information
alonh5 authored Oct 10, 2024
1 parent ca3d103 commit ce5b975
Showing 1 changed file with 38 additions and 1 deletion.
39 changes: 38 additions & 1 deletion .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -134,7 +134,33 @@ jobs:
summary-always: true
github-token: ${{ secrets.GITHUB_TOKEN }}
comment-on-alert: true
alert-comment-cc-users: "@spapinistarkware"
alert-comment-cc-users: "@shaharsamocha7"

run-avx512-bench-parallel:
runs-on: avx
steps:
- uses: actions/checkout@v4
- uses: dtolnay/rust-toolchain@master
with:
toolchain: nightly-2024-01-04
- name: Run benchmark
run: ./scripts/bench.sh --features="parallel" -- --output-format bencher | tee output.txt
- name: Download previous benchmark data
uses: actions/cache@v4
with:
path: ./cache
key: ${{ runner.os }}-benchmark
- name: Store benchmark result
uses: benchmark-action/github-action-benchmark@v1
with:
tool: "cargo"
output-file-path: output.txt
external-data-json-path: ./cache/benchmark-data.json
fail-on-alert: true
summary-always: true
github-token: ${{ secrets.GITHUB_TOKEN }}
comment-on-alert: true
alert-comment-cc-users: "@shaharsamocha7"

run-tests:
runs-on: ubuntu-latest
Expand All @@ -156,6 +182,16 @@ jobs:
- uses: Swatinem/rust-cache@v2
- run: cargo +nightly-2024-01-04 test --release --features="slow-tests"

run-tests-parallel:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: dtolnay/rust-toolchain@master
with:
toolchain: nightly-2024-01-04
- uses: Swatinem/rust-cache@v2
- run: cargo +nightly-2024-01-04 test --features="parallel"

machete:
runs-on: ubuntu-latest
steps:
Expand All @@ -173,6 +209,7 @@ jobs:
- run-neon-tests
- run-wasm32-wasi-tests
- run-slow-tests
- run-tests-parallel
- machete
steps:
- name: Decide whether all the needed jobs succeeded or failed
Expand Down

0 comments on commit ce5b975

Please sign in to comment.