Skip to content

Commit

Permalink
Run all benchmarks in CI
Browse files Browse the repository at this point in the history
In the past we used Git LFS for retrieving large benchmark files and we
didn't want to waste upload/download bandwidth on every CI run. In
addition, we never intended to rely on CI benchmarks, for they are
running in a virtualized and generally unreliable (in terms of
consistent performance) environment.
Despite that, given that by now we switched to pulling said files from a
GitHub hosted Git repository with commit 80f35da ("Stop using Git
LFS for large benchmark files"), it is still somewhat interesting to
have all benchmarks run in CI.
With this change we do just that.

Signed-off-by: Daniel Müller <deso@posteo.net>
  • Loading branch information
d-e-s-o committed Jun 15, 2023
1 parent 8c2b43d commit dc72c90
Showing 1 changed file with 7 additions and 2 deletions.
9 changes: 7 additions & 2 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -145,15 +145,20 @@ jobs:
(echo "!!!! CHECKED IN C HEADER IS OUTDATED !!!!" && false)
bench:
name: Benchmark
runs-on: ubuntu-latest
runs-on: ubuntu-22.04
env:
LLVM_GSYMUTIL: /usr/bin/llvm-gsymutil-14
steps:
- uses: actions/checkout@v3
- uses: actions-rs/toolchain@v1
with:
toolchain: nightly
profile: minimal
override: true
- run: cargo bench --features=nightly,dont-generate-test-files
- name: Install required tools
run: sudo apt-get install -y llvm-14
- uses: Swatinem/rust-cache@v2.5.1
- run: cargo bench --features=nightly,generate-bench-files,dont-generate-test-files
clippy:
name: Lint with clippy
runs-on: ubuntu-latest
Expand Down

0 comments on commit dc72c90

Please sign in to comment.