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

update go-libp2p to v0.30.0 #11189

Merged
merged 5 commits into from
Aug 24, 2023
Merged

update go-libp2p to v0.30.0 #11189

merged 5 commits into from
Aug 24, 2023

Conversation

marten-seemann
Copy link
Contributor

@marten-seemann marten-seemann commented Aug 21, 2023

Related Issues

Proposed Changes

  • Update CI to Go 1.20.7. At this point, Go 1.19 is unmaintained by the Go team.
  • Update go-libp2p to v0.30.0

Additional Info

go-libp2p v0.30.0 allows using Go 1.21. It also drops support for Go 1.19.

Note that this PR does NOT bump the Go version in go.mod. I tried that, and it leads to a whole lot of staticcheck complaints. I'll leave this as an exercise for the reader :)

Checklist

Before you mark the PR ready for review, please make sure that:

  • Commits have a clear commit message.
  • PR title is in the form of of <PR type>: <area>: <change being made>
    • example: fix: mempool: Introduce a cache for valid signatures
    • PR type: fix, feat, build, chore, ci, docs, perf, refactor, revert, style, test
    • area, e.g. api, chain, state, market, mempool, multisig, networking, paych, proving, sealing, wallet, deps
  • New features have usage guidelines and / or documentation updates in
  • Tests exist for new functionality or change in behavior
  • CI is green

@marten-seemann marten-seemann changed the title update go-libp2p v0.30.0 update go-libp2p to v0.30.0 Aug 21, 2023
@@ -11,7 +11,8 @@ import (
"sync"
"time"

lru "github.com/hashicorp/golang-lru/v2"
"github.com/hashicorp/golang-lru/arc/v2"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

golang-lru introduced a breaking change in the v2.0.3 patch release. See hashicorp/golang-lru#142 for discussion. The author has his own interpretation of the semver specification, that diverges from everyone else's reading, and seems oblivious to the amount of pain his actions cause in the ecosystem.

@marten-seemann marten-seemann force-pushed the update-go-libp2p-v0300 branch 2 times, most recently from 36f54b2 to dbad439 Compare August 21, 2023 05:09
@marten-seemann marten-seemann marked this pull request as ready for review August 21, 2023 09:06
@marten-seemann marten-seemann requested a review from a team as a code owner August 21, 2023 09:06
@marten-seemann
Copy link
Contributor Author

Not sure why some of the tests are failing. Flaky? Could anyone more familiar with the code base please take a look?

go.mod Outdated Show resolved Hide resolved
Copy link
Contributor

@magik6k magik6k left a comment

Choose a reason for hiding this comment

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

LGTM, seems to run on mainnet just fine

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.

2 participants