Skip to content

fix: use sepolia bootnodes in sepolia config#1068

Open
dieutx wants to merge 1 commit into
base:mainfrom
dieutx:fix/sepolia-bootnodes-chain-config
Open

fix: use sepolia bootnodes in sepolia config#1068
dieutx wants to merge 1 commit into
base:mainfrom
dieutx:fix/sepolia-bootnodes-chain-config

Conversation

@dieutx
Copy link
Copy Markdown

@dieutx dieutx commented May 11, 2026

Summary

  • replace the Base Sepolia consensus bootnodes in .env.sepolia with the chain-correct ENRs
  • update the optional Sepolia snap-sync execution bootnode examples to the matching Sepolia enodes
  • align the sample config with the authoritative Sepolia bootnode list in base/base

Problem

base/node was shipping .env.sepolia with the same bootnode values as .env.mainnet.

That means the default Sepolia config advertised mainnet consensus ENRs whose opstack payload resolves to chain ID 8453, while Sepolia expects 84532. Operators in issue #1041 reported these ENRs being rejected as InvalidChainId(8453), leaving discovery with zero usable bootstrap peers.

Solution

Use the Sepolia bootnode set from base/base's chain config:

  • consensus ENRs from crates/common/chains/src/config.rs for chain 84532
  • matching Sepolia execution bootnodes for the optional commented snap-sync examples

Verification

  • confirmed .env.sepolia previously matched .env.mainnet bootnodes byte-for-byte
  • confirmed base/base has a distinct Sepolia bootnode set for chain 84532
  • updated .env.sepolia to match those authoritative Sepolia values

Closes #1041

@cb-heimdall
Copy link
Copy Markdown
Collaborator

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 1
Sum 2

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.

AwaitingELSyncCompletion

2 participants