-
-
Notifications
You must be signed in to change notification settings - Fork 300
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: bump libp2p deps #6015
chore: bump libp2p deps #6015
Conversation
Performance Report✔️ no performance regression detected Full benchmark results
|
13a9c8d
to
ce0b8f1
Compare
We should confirm that this resolves the active handles issue (related libp2p issues are closed) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are still a bunch of unrelated changes
packages/beacon-node/test/unit/api/impl/validator/produceAttestationData.test.ts
Outdated
Show resolved
Hide resolved
file conflicts |
@@ -19,7 +19,6 @@ describe("gossipsub / worker", function () { | |||
|
|||
function runTests(this: Mocha.Suite, {useWorker}: {useWorker: boolean}): void { | |||
if (this.timeout() < 20 * 1000) this.timeout(150 * 1000); | |||
this.retries(2); // This test fail sometimes, with a 5% rate. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why can this be removed now? Was this just related to libp2p issue?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Because tests are performing more stable now, so we don't need to have retries here.
🎉 This PR is included in v1.12.0 🎉 |
Motivation
CI has been failing due to a bug in
@libp2p/tcp
. That bug should now be fixed in the latest version.Description