Skip to content

Add v6 auto-ENR to Lodestar#2451

Merged
yorickdowne merged 1 commit intoethstaker:mainfrom
yorickdowne:lodestar-v6-enr
Mar 23, 2026
Merged

Add v6 auto-ENR to Lodestar#2451
yorickdowne merged 1 commit intoethstaker:mainfrom
yorickdowne:lodestar-v6-enr

Conversation

@yorickdowne
Copy link
Copy Markdown
Collaborator

What I did

In my testing, v6 auto-ENR now works with Lodestar. Remove the manual ENR setup - which btw was broken anyway because the wget returned too much data

@yorickdowne yorickdowne changed the title Add auto-ENR to Lodestarx Add v6 auto-ENR to Lodestar Jan 29, 2026
@nflaig
Copy link
Copy Markdown
Contributor

nflaig commented Jan 29, 2026

did you test this on an IPv6 only host? What I observed testing this is if it's dual stack then it will only discover the IPv4 address

@yorickdowne
Copy link
Copy Markdown
Collaborator Author

I need to test again. It was dual stack, but it was after a manual address had been set once. It’s possible it cached that.

@yorickdowne
Copy link
Copy Markdown
Collaborator Author

Indeed not ready. Placing into draft. I'll give you an issue for it: I think lodestar is close. The network is ready, as seen with Lighthouse and Grandine

@yorickdowne yorickdowne marked this pull request as draft January 29, 2026 12:52
@nflaig
Copy link
Copy Markdown
Contributor

nflaig commented Jan 29, 2026

I think lodestar is close

so from last time I looked at this, the problem was that if the node is reachable via IPv4 then other nodes will just "pong" us on that interface and not on IPv6 which means we do not reach the minimum threshold requires to discover the external IP, right now it's 10

I am not that deep into the networking/discv5 stack, so definitely having an issue for it would help

I believe this problem would also resolve itself if there are more IPv6 only nodes on the network

@yorickdowne
Copy link
Copy Markdown
Collaborator Author

ChainSafe/lodestar#8808

@yorickdowne yorickdowne marked this pull request as ready for review March 19, 2026 22:30
@yorickdowne
Copy link
Copy Markdown
Collaborator Author

This works from Lodestar 1.41.0

Copy link
Copy Markdown
Contributor

@nflaig nflaig left a comment

Choose a reason for hiding this comment

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

running the same exact changes locally already

@yorickdowne yorickdowne merged commit 211fb68 into ethstaker:main Mar 23, 2026
191 checks passed
@yorickdowne yorickdowne deleted the lodestar-v6-enr branch March 23, 2026 09:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants