Skip to content

Conversation

@frankudoags
Copy link
Contributor

closes #19236

frankudoags added 2 commits October 23, 2025 08:14
…onfig

Introduce a shareable OpGasLimitConfig (backed by AtomicU64) to manage per-transaction gas
limits with getter/setter semantics (0 => use default). Add a gas_limit_config field to
OpBuilderConfig and update OpBuilderConfig::new signature accordingly. Include unit tests
for gas limit behavior.
…gas_limit metric

Add OpGasLimitConfig field and constructor parameter to OpMinerExtApi, introduce a gas_limit Gauge
and setter in OpMinerMetrics, and restore the placeholder miner_setGasLimit handler (commented).
frankudoags added 3 commits October 23, 2025 08:18
… and payload builder

Wire OpGasLimitConfig into OpNode, OpAddOns, OpAddOnsBuilder, OpPayloadBuilder and OpMinerExtApi.
Add default initialization and with_gas_limit_config helpers, and include gas_limit_config in
OpBuilderConfig passed to the payload builder.
Copy link
Collaborator

@mattsse mattsse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cool,
some nits

@github-project-automation github-project-automation bot moved this from Backlog to In Progress in Reth Tracker Oct 23, 2025
frankudoags added 5 commits October 23, 2025 09:54
feat(optimism/rpc): implement miner_setGasLimit RPC handler

- Update Cargo.toml/Cargo.lock to source op-alloy crates from git rev 47ecf4a.
- Un-comment and implement miner set_gas_limit to apply OpGasLimitConfig and update metrics.
…emove pinned git rev)

Enable op-alloy crates from the registry in Cargo.toml and remove the git rev pins; update Cargo.lock accordingly.
@mattsse mattsse added the A-op-reth Related to Optimism and op-reth label Oct 29, 2025
@mattsse mattsse added this to the v1.9.0 milestone Oct 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-op-reth Related to Optimism and op-reth

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

implement miner_setGasLimit RPC

2 participants