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

Node: Update libp2p #4162

Merged

Conversation

bruce-riley
Copy link
Contributor

@bruce-riley bruce-riley commented Nov 19, 2024

This PR updates the version of libp2p being used by the guardian. It assumes PR #4161 which has been merged.

Did the following commands:

go get github.com/libp2p/[email protected]
go get github.com/libp2p/[email protected]
go get github.com/golang/[email protected]
go mod tidy

This PR also contains a P2P change to disable address discovery if we are advertising our public IP as recommended here.

Testing

  • Go tests pass.
  • Tilt passes.
  • Mainnet spy is seeing VAAs.
  • Generated 1000 VAAs in the load testing environment with 19 guardians.
  • Configured the load testing environment to advertise a specific IP to test the change mentioned above.

@bruce-riley bruce-riley marked this pull request as ready for review November 19, 2024 17:13
@bruce-riley bruce-riley requested a review from pires November 19, 2024 17:13
Copy link
Collaborator

@pires pires left a comment

Choose a reason for hiding this comment

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

<3

@bruce-riley bruce-riley merged commit 79946aa into wormhole-foundation:main Nov 26, 2024
24 checks passed
@bruce-riley bruce-riley deleted the node_update_libp2p branch November 26, 2024 14:06
kakucodes added a commit to kakucodes/wormhole that referenced this pull request Dec 10, 2024
Remove bruce-riley as CODEOWNER of governor module (wormhole-foundation#4166)

Node: Update libp2p (wormhole-foundation#4162)

* Node: Update libp2p

* Node: p2p address discovery change

Node/CCQ: Logging changes in watchers (wormhole-foundation#4173)

tilt: support latest docker desktop

fix clippy::needless-lifetimes warnings

by removing the binders for inferrable lifetimes

tilt: m1 flag

tilt: fix wormchain deploy

Queries/SDK: Update Solana SDK (wormhole-foundation#4180)

tilt: specify amd64 for cosmwasm containers

tilt: sdk-ci-tests default maxWorkers=50%
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.

4 participants