Skip to content

ci(bench): queue reth-bench jobs and report queue position in PR comment#22318

Merged
shekhirin merged 5 commits intomainfrom
alexey/reth-bench-queue
Feb 18, 2026
Merged

ci(bench): queue reth-bench jobs and report queue position in PR comment#22318
shekhirin merged 5 commits intomainfrom
alexey/reth-bench-queue

Conversation

@shekhirin
Copy link
Member

No description provided.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 18, 2026

⚠️ Changelog not found.

A changelog entry is required before merging. We've generated a suggested changelog based on your changes:

Preview
---
reth-bench: patch
---

Refactored the bench CI workflow to split `reth-bench` into an `reth-bench-ack` acknowledgement job and a separate `reth-bench` run job, enabling proper queue position tracking and status updates. Added live queue position polling and PR comment updates with branch display names instead of placeholder strings.

Add changelog to commit this to your branch.

@paradigmxyz paradigmxyz deleted a comment from github-actions bot Feb 18, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 18, 2026

cc @shekhirin

✅ Benchmark complete! View job

Benchmark Results

Metric main alexey/reth-bench-queue Change
Mean 119.05ms 116.94ms -1.77% ✅ (±0.00%)
StdDev 0.00ms 0.00ms
P50 119.05ms 116.94ms -1.77% ✅ (±0.00%)
P90 119.05ms 116.94ms -1.77% ✅ (±0.00%)
P99 119.05ms 116.94ms -1.77% ✅ (±0.00%)
Mgas/s 434.64 442.49 +1.81% ✅ (±0.00%)

1 blocks

Wait Time Breakdown

Trie Cache Update Wait

Metric main alexey/reth-bench-queue
Mean 0.00ms 0.00ms
P50 0.00ms 0.00ms
P95 0.00ms 0.00ms

Execution Cache Update Wait

Metric main alexey/reth-bench-queue
Mean 0.00ms 0.00ms
P50 0.00ms 0.00ms
P95 0.00ms 0.00ms

Charts

Latency, Throughput & Diff

Latency, Throughput & Diff

Wait Time Breakdown

Wait Time Breakdown

Gas vs Latency

Gas vs Latency

github-actions bot pushed a commit that referenced this pull request Feb 18, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 18, 2026

cc @shekhirin

✅ Benchmark complete! View job

Benchmark Results

Metric main alexey/reth-bench-queue Change
Mean 129.22ms 135.89ms +5.16% ❌ (±0.00%)
StdDev 0.00ms 0.00ms
P50 129.22ms 135.89ms +5.16% ❌ (±0.00%)
P90 129.22ms 135.89ms +5.16% ❌ (±0.00%)
P99 129.22ms 135.89ms +5.16% ❌ (±0.00%)
Mgas/s 400.44 380.78 -4.91% ❌ (±0.00%)

1 blocks

Wait Time Breakdown

Trie Cache Update Wait

Metric main alexey/reth-bench-queue
Mean 0.00ms 0.00ms
P50 0.00ms 0.00ms
P95 0.00ms 0.00ms

Execution Cache Update Wait

Metric main alexey/reth-bench-queue
Mean 0.00ms 0.00ms
P50 0.00ms 0.00ms
P95 0.00ms 0.00ms

Charts

Latency, Throughput & Diff

Latency, Throughput & Diff

Wait Time Breakdown

Wait Time Breakdown

Gas vs Latency

Gas vs Latency

@shekhirin shekhirin added C-benchmark A change that impacts how or what we benchmark A-ci Related to github workflows or other build and lint tools labels Feb 18, 2026
github-actions bot pushed a commit that referenced this pull request Feb 18, 2026
@shekhirin shekhirin marked this pull request as ready for review February 18, 2026 12:50
@github-project-automation github-project-automation bot moved this from Backlog to In Progress in Reth Tracker Feb 18, 2026
@shekhirin shekhirin enabled auto-merge February 18, 2026 12:51
@shekhirin shekhirin added this pull request to the merge queue Feb 18, 2026
Merged via the queue into main with commit e6e0dde Feb 18, 2026
43 checks passed
@shekhirin shekhirin deleted the alexey/reth-bench-queue branch February 18, 2026 13:07
@github-project-automation github-project-automation bot moved this from In Progress to Done in Reth Tracker Feb 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-ci Related to github workflows or other build and lint tools C-benchmark A change that impacts how or what we benchmark

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants