Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: update narwhal pointer #4525

Merged
merged 2 commits into from
Sep 8, 2022
Merged

chore: update narwhal pointer #4525

merged 2 commits into from
Sep 8, 2022

Conversation

bmwill
Copy link
Contributor

@bmwill bmwill commented Sep 8, 2022

Update the narwhal pointer and include the change where the primary-to-primary interface was changed to use anemo (QUIC) vs gRPC.

This wasn't expected to be a breaking change but turned out to be one due to the inability of an ipv4 udp socket from being able to send to an ipv6 address. This is due to narwhal unconditionally binding on an ipv4 address (0.0.0.0) while the addresses included in the committee structure are localhost which happens to first be resolved to ::1.

The simple short-term workaround is to change our config generation to always use the ipv4 localhost address of 127.0.0.1.

Fixes: MystenLabs/narwhal#929

Update the narwhal pointer and include the change where the
primary-to-primary interface was changed to use anemo (QUIC) vs gRPC.

This wasn't expected to be a breaking change but turned out to be one
due to the inability of an ipv4 udp socket from being able to send to an
ipv6 address. This is due to narwhal unconditionally binding on an ipv4
address (0.0.0.0) while the addresses included in the committee
structure are localhost which happens to first be resolved to ::1.

The simple short-term workaround is to change our config generation to
always use the ipv4 localhost address of 127.0.0.1.
@bmwill bmwill force-pushed the fix-narwhal branch 2 times, most recently from 38214ee to ba54c3c Compare September 8, 2022 21:56
@bmwill bmwill merged commit f0404c8 into MystenLabs:main Sep 8, 2022
@bmwill bmwill deleted the fix-narwhal branch September 8, 2022 22:14
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.

[network] Sui-breaking change in the switch to Anemo
2 participants