Skip to content

Conversation

futreall
Copy link
Contributor

@futreall futreall commented Oct 7, 2025

Description:

Summary

  • keep PayloadBuilderConfig::gas_limit_for at the 30M target that the rest of the stack enforces
  • add unit tests that capture the expected defaults and user overrides so future bumps are coordinated

Background

PR #18779 bumped the payload builder’s default gas limit to 60M in anticipation of a future mainnet target. However, related components (transaction pool defaults, builder config, etc.) still operate with the 30M ceiling, and the consensus layer will currently reject 60M blocks outright. In practice this meant the builder consistently requested an unreachable target and disagreed with the rest of the stack.

Changes

  • return ETHEREUM_BLOCK_GAS_LIMIT_30M for Mainnet/Sepolia/Holesky/Hoodi unless the user overrides it
  • add regression tests that flag any future default change unless the rest of the stack is updated transparently

Testing

  • cargo fmt
  • cargo test -p reth-node-core
  • cargo clippy --workspace --all-targets --all-features

@Rjected
Copy link
Member

Rjected commented Oct 7, 2025

60M block gas limit is intended here

@Rjected Rjected closed this Oct 7, 2025
@github-project-automation github-project-automation bot moved this from Backlog to Done in Reth Tracker Oct 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants