Skip to content

feat(rpc): Add support for custom Tokio runtime configuration in EthereumAddOns#17693

Merged
mattsse merged 9 commits intoparadigmxyz:mainfrom
mablr:feature/custom-tokio-rt-followup
Aug 14, 2025
Merged

feat(rpc): Add support for custom Tokio runtime configuration in EthereumAddOns#17693
mattsse merged 9 commits intoparadigmxyz:mainfrom
mablr:feature/custom-tokio-rt-followup

Conversation

@mablr
Copy link
Contributor

@mablr mablr commented Jul 31, 2025

Towards #17605 (follow-up for #17611)

This change bubbles up the with_tokio_runtime() method from the RpcServerConfig to EthereumAddOns, allowing to configure nodes with custom Tokio runtimes.

The optional runtime handle is now passed through the RpcAddOns structure and applied to RPC server configuration when provided.

Provides to ability to configure a node with custom Tokio runtime
@mablr mablr force-pushed the feature/custom-tokio-rt-followup branch from 10bfa1d to 0ef6092 Compare July 31, 2025 13:53
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.

great, we can also do this for opaddons

@github-project-automation github-project-automation bot moved this from Backlog to In Progress in Reth Tracker Jul 31, 2025
@mablr mablr marked this pull request as ready for review July 31, 2025 14:50
@mablr mablr requested a review from mattsse August 12, 2025 21:24
@mattsse mattsse requested a review from RomanHodulak as a code owner August 14, 2025 04:58
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.

ty!

I made the functions accept an option this makes this slightly more ergonomic when dealing with this

Comment on lines +77 to +78
// #[tokio::test] can not be used here because we need to create a custom tokio runtime
// and it would be dropped before the test is finished, resulting in a panic.
Copy link
Collaborator

Choose a reason for hiding this comment

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

thanks for flagging this, I also added this to the addons fns

@mattsse mattsse added this pull request to the merge queue Aug 14, 2025
@mattsse mattsse added the A-sdk Related to reth's use as a library label Aug 14, 2025
Merged via the queue into paradigmxyz:main with commit 907448f Aug 14, 2025
41 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in Reth Tracker Aug 14, 2025
lwedge99 pushed a commit to sentioxyz/reth that referenced this pull request Sep 16, 2025
…reumAddOns (paradigmxyz#17693)

Co-authored-by: Matthias Seitz <matthias.seitz@outlook.de>
theochap pushed a commit to ethereum-optimism/optimism that referenced this pull request Jan 22, 2026
…reumAddOns (paradigmxyz/reth#17693)

Co-authored-by: Matthias Seitz <matthias.seitz@outlook.de>
theochap pushed a commit to ethereum-optimism/optimism that referenced this pull request Feb 11, 2026
…reumAddOns (paradigmxyz/reth#17693)

Co-authored-by: Matthias Seitz <matthias.seitz@outlook.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-sdk Related to reth's use as a library

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants