This repository has been archived by the owner on Nov 5, 2023. It is now read-only.

fix: `trace_filter` matches all transactions on empty addresses (#5279) #660

	on:
	pull_request:
	merge_group:
	push:
	branches: [main]

	env:
	CARGO_TERM_COLOR: always

	concurrency:
	group: ${{ github.workflow }}-${{ github.head_ref \|\| github.run_id }}
	cancel-in-progress: true

	name: bench
	jobs:
	iai:
	runs-on:
	group: Reth
	# Only run benchmarks in merge groups
	if: github.event_name != 'pull_request'
	steps:
	- name: Checkout main sources
	uses: actions/checkout@v3
	with:
	ref: main
	path: main

	- name: Checkout PR sources
	uses: actions/checkout@v3
	with:
	clean: false
	path: pr

	- name: Install Valgrind
	run: \|
	sudo apt update && sudo apt install valgrind

	- name: Install toolchain
	uses: dtolnay/rust-toolchain@stable

	- uses: Swatinem/rust-cache@v2
	with:
	workspaces: \|
	main -> target
	pr -> target
	cache-on-failure: true

	- name: Generate test vectors
	run: \|
	cargo run --bin reth --manifest-path main/Cargo.toml -- test-vectors tables
	cp -r testdata main
	mv testdata pr

	- name: Set main baseline
	run: cargo bench --package reth-db --bench iai --features test-utils --manifest-path main/Cargo.toml

	- name: Compare PR benchmark
	shell: 'script -q -e -c "bash {0}"' # required to workaround /dev/tty not being available
	run: \|
	./pr/.github/scripts/compare_iai.sh

	bench-success:
	if: always()
	name: bench success
	runs-on: ubuntu-latest
	steps:
	# Note: This check is a dummy because we don't have any bench checks enabled.
	- run: echo OK.

Provide feedback