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] - Bump discv5 to v0.1.0-beta.9 #2479

Closed
wants to merge 1 commit into from

Conversation

michaelsproul
Copy link
Member

@michaelsproul michaelsproul commented Jul 27, 2021

Bump discv5 to fix the issues with IP filters and removing nodes.

Blocked on an upstream release, and more testnet data.

@michaelsproul michaelsproul added blocked do-not-merge v1.5.0 For inclusion in v1.5.0 release labels Jul 27, 2021
paulhauner added a commit to paulhauner/lighthouse that referenced this pull request Jul 27, 2021
commit 768c784
Author: Michael Sproul <[email protected]>
Date:   Tue Jul 27 10:12:38 2021 +1000

    Bump discv5
paulhauner added a commit to paulhauner/lighthouse that referenced this pull request Jul 27, 2021
commit 768c784
Author: Michael Sproul <[email protected]>
Date:   Tue Jul 27 10:12:38 2021 +1000

    Bump discv5
paulhauner added a commit to paulhauner/lighthouse that referenced this pull request Jul 29, 2021
commit 768c784
Author: Michael Sproul <[email protected]>
Date:   Tue Jul 27 10:12:38 2021 +1000

    Bump discv5
@michaelsproul michaelsproul added ready-for-review The code is ready for review and removed blocked do-not-merge labels Aug 2, 2021
@michaelsproul
Copy link
Member Author

Ready to merge now that beta.9 has been released.

Testnet data is positive. This patch stemmed the tide of panics on altair-devnet-1 and has been running well on Pyrmont and Prater via 768c784 and c040410 respectively (which included some other changes).

@michaelsproul michaelsproul changed the title Bump discv5 Bump discv5 to v0.1.0-beta.9 Aug 2, 2021
Copy link
Member

@paulhauner paulhauner left a comment

Choose a reason for hiding this comment

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

Yay!

bors r+

@paulhauner paulhauner added ready-for-merge This PR is ready to merge. and removed ready-for-review The code is ready for review labels Aug 3, 2021
bors bot pushed a commit that referenced this pull request Aug 3, 2021
Bump discv5 to fix the issues with IP filters and removing nodes.

~~Blocked on an upstream release, and more testnet data.~~
@bors bors bot changed the title Bump discv5 to v0.1.0-beta.9 [Merged by Bors] - Bump discv5 to v0.1.0-beta.9 Aug 3, 2021
@bors bors bot closed this Aug 3, 2021
@michaelsproul michaelsproul deleted the bump-discv5 branch August 3, 2021 02:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-merge This PR is ready to merge. v1.5.0 For inclusion in v1.5.0 release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants