Skip to content

Commit 27dd1f2

Browse files
authored
bench: run codspeed on each pr (#2427)
Regressions get flagged by Codspeed but don't block merging PRs.
1 parent 0a37380 commit 27dd1f2

File tree

3 files changed

+31
-64
lines changed

3 files changed

+31
-64
lines changed

.github/workflows/bench-pr.yml

Lines changed: 0 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -22,38 +22,6 @@ jobs:
2222
with:
2323
labels: benchmark
2424

25-
codspeed_bench:
26-
name: Benchmark with Codspeed
27-
needs: label_trigger
28-
runs-on: ubuntu-latest-large
29-
steps:
30-
- uses: actions/checkout@v4
31-
- uses: ./.github/actions/setup-rust
32-
33-
- name: Install Codspeed
34-
shell: bash
35-
run: cargo install --force cargo-codspeed --locked
36-
37-
- name: Build benchmarks
38-
env:
39-
RUSTFLAGS: "-C target-cpu=native"
40-
run: |
41-
cargo codspeed build --features test-harness \
42-
--exclude bench-vortex \
43-
--exclude vortex-datafusion \
44-
--exclude vortex-tui \
45-
--exclude vortex-fuzz \
46-
--exclude pyvortex \
47-
--exclude xtask \
48-
--workspace \
49-
--profile bench
50-
51-
- name: Run benchmarks
52-
uses: CodSpeedHQ/action@v3
53-
with:
54-
run: cargo codspeed run
55-
token: ${{ secrets.CODSPEED_TOKEN }}
56-
5725
bench-criterion:
5826
needs: label_trigger
5927
strategy:

.github/workflows/bench.yml

Lines changed: 0 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -29,38 +29,6 @@ jobs:
2929
bash scripts/commit-json.sh > new-commit.json
3030
bash scripts/cat-s3.sh vortex-benchmark-results-database commits.json new-commit.json
3131
32-
33-
codspeed_bench:
34-
name: Benchmark with Codspeed
35-
runs-on: ubuntu-latest-large
36-
steps:
37-
- uses: actions/checkout@v4
38-
- uses: ./.github/actions/setup-rust
39-
40-
- name: Install Codspeed
41-
shell: bash
42-
run: cargo install --force cargo-codspeed --locked
43-
44-
- name: Build benchmarks
45-
env:
46-
RUSTFLAGS: "-C target-cpu=native"
47-
run: |
48-
cargo codspeed build --features test-harness \
49-
--exclude bench-vortex \
50-
--exclude vortex-datafusion \
51-
--exclude vortex-tui \
52-
--exclude vortex-fuzz \
53-
--exclude pyvortex \
54-
--exclude xtask \
55-
--workspace \
56-
--profile bench
57-
58-
- name: Run benchmarks
59-
uses: CodSpeedHQ/action@v3
60-
with:
61-
run: cargo codspeed run
62-
token: ${{ secrets.CODSPEED_TOKEN }}
63-
6432
bench-criterion:
6533
strategy:
6634
matrix:

.github/workflows/ci.yml

Lines changed: 31 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -219,6 +219,37 @@ jobs:
219219
with:
220220
file: ${{ env.GRCOV_OUTPUT_FILE }}
221221

222+
bench-codspeed:
223+
name: Benchmark with Codspeed
224+
runs-on: ubuntu-latest-large
225+
steps:
226+
- uses: actions/checkout@v4
227+
- uses: ./.github/actions/setup-rust
228+
229+
- name: Install Codspeed
230+
shell: bash
231+
run: cargo install --force cargo-codspeed --locked
232+
233+
- name: Build benchmarks
234+
env:
235+
RUSTFLAGS: "-C target-cpu=native"
236+
run: |
237+
cargo codspeed build --features test-harness \
238+
--exclude bench-vortex \
239+
--exclude vortex-datafusion \
240+
--exclude vortex-tui \
241+
--exclude vortex-fuzz \
242+
--exclude pyvortex \
243+
--exclude xtask \
244+
--workspace \
245+
--profile bench
246+
247+
- name: Run benchmarks
248+
uses: CodSpeedHQ/action@v3
249+
with:
250+
run: cargo codspeed run
251+
token: ${{ secrets.CODSPEED_TOKEN }}
252+
222253
license-check-and-audit-check:
223254
name: License Check and Audit Check
224255
runs-on: ubuntu-latest

0 commit comments

Comments
 (0)