Skip to content

Hoodi support#120

Closed
Angelin01 wants to merge 5 commits into
chainstacklabs:mainfrom
NethermindEth:hoodi-support
Closed

Hoodi support#120
Angelin01 wants to merge 5 commits into
chainstacklabs:mainfrom
NethermindEth:hoodi-support

Conversation

@Angelin01

@Angelin01 Angelin01 commented Jun 10, 2026

Copy link
Copy Markdown

Description

Issues Resolved

Summary by CodeRabbit

  • New Features

    • New load testing profile enabling comprehensive benchmarking of Ethereum RPC endpoints including eth_call, eth_getLogs, eth_estimateGas, eth_getBalance, and additional operations
    • Added support for two new EVM test networks
  • Chores

    • Refactored Docker image publishing workflow to leverage JFrog registry with automated GitHub release generation

@Angelin01 Angelin01 closed this Jun 10, 2026
@coderabbitai

coderabbitai Bot commented Jun 10, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 303d9984-954d-4e5b-ace0-0f257caacad3

📥 Commits

Reviewing files that changed from the base of the PR and between 5d56c2d and 0198549.

📒 Files selected for processing (3)
  • .github/workflows/docker-publish.yml
  • chainbench/profile/surge/general.py
  • chainbench/test_data/evm.py

📝 Walkthrough

Walkthrough

This PR contains two independent changes: a complete refactor of the Docker publish workflow to use a reusable JFrog build/push workflow with simplified version tagging and GitHub release creation; and the addition of Surge and Hoodi testnet configurations plus a new SurgeProfile load testing profile with 16 RPC call tasks.

Changes

Docker Publish Workflow Refactor

Layer / File(s) Summary
Workflow trigger and version tagging
.github/workflows/docker-publish.yml
Workflow trigger changes from release/pull_request events to push on main branch. New version job generates UTC date-based tag combined with github.run_number and exposes it as output for downstream jobs.
Docker image building and JFrog push
.github/workflows/docker-publish.yml
Removes prior direct Docker build/push logic (QEMU, Buildx, Docker Hub login). Replaces with build-and-push job that calls external reusable NethermindEth/github-workflows JFrog workflow, passing group/repo names and additional_tags from version job.
GitHub release creation with registry links
.github/workflows/docker-publish.yml
Adds release job creating GitHub release with softprops/action-gh-release. Release body includes docker pull commands for both generated tag and latest image from JFrog registry.

Surge Network Load Testing

Layer / File(s) Summary
EVM network test configurations
chainbench/test_data/evm.py
Extends EvmNetwork.DATA with two new chain configurations: chain ID 763375 (surge-testnet) and chain ID 560048 (ethereum-hoodi-testnet), both with start_block: 1 and empty contract_addresses.
Surge profile with RPC call tasks
chainbench/profile/surge/general.py
Introduces SurgeProfile Locust user class with constant-paced 1-second wait intervals and 16 @task-decorated RPC call methods covering eth_call, eth_getLogs, eth_estimateGas, eth_getBlockByNumber, eth_getBalance, eth_getTransactionReceipt, eth_getStorageAt, gas price queries, chain identity, sync status, and block/receipt operations using inherited parameter factories and RNG.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Poem

🐰 A workflow springs new with JFrog's grace,
While Surge profiles race through RPC space,
Date-tagged releases bloom on main,
Load tests pour down like springtime rain—
Hoodi and Surge dance side by side!

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants