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

[Merged by Bors] - update libp2p #3233

Closed

Conversation

divagant-martian
Copy link
Collaborator

@divagant-martian divagant-martian commented May 31, 2022

Issue Addressed

na

Proposed Changes

Updates libp2p to libp2p/rust-libp2p#2662

Additional Info

From comments on the relevant PRs listed, we should pay attention at peer management consistency, but I don't think anything weird will happen.
This is running in prater tok and sin

@mxinden
Copy link

mxinden commented Jun 3, 2022

From comments on the relevant PRs listed, we should pay attention at peer management consistency, but I don't think anything weird will happen.
This is running in prater tok and sin

User code might behave differently due to polling order being changed.

Let me know in case you see any anomalies.

Thanks for upgrading!

@divagant-martian
Copy link
Collaborator Author

I haven't seen issues with these nodes @AgeManning shall we get this in?

@AgeManning
Copy link
Member

Yeah looks good to me :). Lets do it

bors r+

@michaelsproul michaelsproul added the ready-for-merge This PR is ready to merge. label Jun 7, 2022
bors bot pushed a commit that referenced this pull request Jun 7, 2022
## Issue Addressed
na

## Proposed Changes
Updates libp2p to libp2p/rust-libp2p#2662

## Additional Info
From comments on the relevant PRs listed, we should pay attention at peer management consistency, but I don't think anything weird will happen.
This is running in prater tok and sin
@bors bors bot changed the title update libp2p [Merged by Bors] - update libp2p Jun 7, 2022
@bors bors bot closed this Jun 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Networking ready-for-merge This PR is ready to merge.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants