Skip to content

docs: fix local testnet RPC port numbering and expand running-local-network guide#441

Merged
matthias-wright merged 1 commit into
SeismicSystems:mainfrom
Zhekinmaksim:docs/fix-local-network-ports
Jul 9, 2026
Merged

docs: fix local testnet RPC port numbering and expand running-local-network guide#441
matthias-wright merged 1 commit into
SeismicSystems:mainfrom
Zhekinmaksim:docs/fix-local-network-ports

Conversation

@Zhekinmaksim

Copy link
Copy Markdown
Contributor

Supersedes #438, which was auto-closed when audit-may-2026 was deleted.

The current doc says the local testnet Reth RPC ports are 8545, 8546, 8547, 8548. node/src/bin/testnet.rs assigns them counting down: node0=8545, node1=8544, node2=8543, node3=8542, so step 3 of the guide points readers at ports that are never opened.

This PR fixes the port list and, while touching the file, fills the main gaps for first-time setup, all verified against testnet.rs on this branch: a per-node port table (Reth RPC, Summit RPC, admin RPC, consensus P2P, Prometheus), the testnet bin flags (--nodes, --only-reth, --log-dir, --critical-log-dir), a note on why upstream reth will not work (--enclave.endpoint-port), what reset.sh actually removes, and a short troubleshooting list. The "Running distributed" section is unchanged.

@matthias-wright

Copy link
Copy Markdown
Collaborator

I think you have to rebase the branch

…etwork guide

The doc listed Reth RPC ports as 8545-8548 counting up, but testnet.rs
assigns them counting down from 8545 (node0=8545 .. node3=8542).

Also documents the Summit-side ports per node, the testnet bin flags,
what reset.sh removes, and common failure modes for first-time setup.
@Zhekinmaksim Zhekinmaksim force-pushed the docs/fix-local-network-ports branch from b464f25 to 5fe8536 Compare July 8, 2026 05:35
@Zhekinmaksim

Copy link
Copy Markdown
Contributor Author

Rebased onto the current main and force-pushed the branch. Thanks.

@matthias-wright

Copy link
Copy Markdown
Collaborator

LGTM, ready to merge

@Zhekinmaksim

Copy link
Copy Markdown
Contributor Author

Hi, gentle follow-up on this PR. It is still current from my side, and since it is approved and the checks are green, I would appreciate a merge when convenient.

@matthias-wright

Copy link
Copy Markdown
Collaborator

Hey, sorry, I forgot that external contributors cannot merge PRs after approval.

@matthias-wright matthias-wright merged commit 0f8f006 into SeismicSystems:main Jul 9, 2026
4 checks passed
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