Skip to content

fix: use dedicated Base Sepolia bootnode ENRs instead of mainnet's - #1158

Open
Lesnak1 wants to merge 1 commit into
base:mainfrom
Lesnak1:fix/sepolia-dedicated-bootnodes
Open

fix: use dedicated Base Sepolia bootnode ENRs instead of mainnet's#1158
Lesnak1 wants to merge 1 commit into
base:mainfrom
Lesnak1:fix/sepolia-dedicated-bootnodes

Conversation

@Lesnak1

Copy link
Copy Markdown

Summary

Refs #1154 (Bug 1 only — Bug 2 is already covered by #1155)

.env.mainnet and .env.sepolia contain byte-for-byte identical BASE_NODE_P2P_BOOTNODES values, so Sepolia nodes bootstrap consensus-layer discovery against mainnet peers instead of dedicated Sepolia peers.

Fix

Replaced the Sepolia value with the consensus bootnode ENR that base-consensus is actually built with for base-sepolia (chain_id 84532), sourced directly from base/base's crates/common/chains/src/config.rs -> SEPOLIA.bootnodes.consensus.

Verification performed

  • Decoded the ENR and confirmed it is a structurally valid v4 record with IP and opstack chain tags that are disjoint from the mainnet set.
  • Confirmed the new value is a match against base/base's SEPOLIA.bootnodes.consensus.
  • Confirmed only a single line changed (git diff --stat).

Changes

  • .env.sepolia: BASE_NODE_P2P_BOOTNODES updated to the Sepolia-specific ENR.

@cb-heimdall

Copy link
Copy Markdown
Collaborator

🟡 Heimdall Review Status

RequirementStatusMore Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot0
1 if user is external0
2 if repo is sensitive0
From .codeflow.yml1
Additional review requirements
Show calculation
Max0
0
From CODEOWNERS0
Global minimum0
Max 1
1
1 if commit is unverified1
Sum2

@odisseus21odisseus21 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

ı thin so!

@odisseus21

Copy link
Copy Markdown

base very quality!

Sign up for freeto 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.

3 participants

@Lesnak1@cb-heimdall@odisseus21