Skip to content

Commit cf4ce43

Browse files
committed
ci: use --prepare instead of --setup for DB cleanup
1 parent 658c837 commit cf4ce43

1 file changed

Lines changed: 6 additions & 3 deletions

File tree

.github/workflows/pr_perf_blocks_exec.yaml

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -80,9 +80,12 @@ jobs:
8080
run: |
8181
sudo swapoff -a
8282
BINS="base,head"
83-
hyperfine --setup "./bin/ethrex-base removedb" -w 5 -N -r 10 --show-output --export-markdown "bench_pr_comparison.md" \
84-
-L bin "$BINS" -n "{bin}" \
85-
"./bin/ethrex-{bin} --network fixtures/genesis/perf-ci.json --force import ./fixtures/blockchain/l2-1k-erc20.rlp --removedb"
83+
# Run hyperfine benchmark, removing the database before each run.
84+
# Pipe `yes` into the remove command to automatically confirm.
85+
hyperfine -w 5 -N -r 10 --show-output --export-markdown "bench_pr_comparison.md" \
86+
-L bin "$BINS" -n "{bin}" \
87+
--prepare "yes | ./bin/ethrex-{bin} removedb" \
88+
"./bin/ethrex-{bin} --network fixtures/genesis/perf-ci.json --force import ./fixtures/blockchain/l2-1k-erc20.rlp"
8689
echo -e "## Benchmark Block Execution Results Comparison Against Main\n\n$(cat bench_pr_comparison.md)" > bench_pr_comparison.md
8790
- name: Upload PR results
8891
uses: actions/upload-artifact@v4

0 commit comments

Comments
 (0)