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

chore: bump all dependencies (mainly penumbra, celestia, tendermint) #582

Merged
merged 7 commits into from
Nov 20, 2023

Conversation

SuperFluffy
Copy link
Member

Summary

Bump all dependencies with a special focus on penumbra, celestia, tendermint

Background

PR #579 requires a newer version of penumbra, which in turn requires updating prost and tonic (and their build crates), and a new minimum rust version of 1.73 (due to libp2p setting that in libp2p/rust-libp2p#4692 which we transitively depend on).

As this change triggers a cascade of updated dependencies it makes sense to update everything in one go.

Changes

  • Set all penumbra dependencies to tag v0.63.1
  • Update all tendermint dependencies to version 0.34
  • Remove the backported tendermint patch to fix certificate resolution
  • Set celestia-node-rs to their recent master (which also updates libp2p and hence demands a msrv of 1.73)
  • Fix the breakage in the celestia-node-rs due to them introducing a wrapper around http/websocket clients
  • Bump tonic-build to 0.10

Testing

No breaking changes in our APIs. Everything should still compile, all tests should run.

Related Issues

Blocked on #581 being merged
Unblocks #579
Closes #580

@SuperFluffy SuperFluffy added the blocked blocked on another PR, dependency, or missing feature label Nov 15, 2023
@SuperFluffy SuperFluffy requested review from a team, joroshiba and noot as code owners November 15, 2023 16:31
@github-actions github-actions bot added ci issues that are related to ci and github workflows conductor pertaining to the astria-conductor crate proto pertaining to the Astria Protobuf spec sequencer pertaining to the astria-sequencer crate sequencer-relayer pertaining to the astria-sequencer-relayer crate docker labels Nov 15, 2023
Base automatically changed from superfluffy/buf-to-prost to main November 20, 2023 16:40
@SuperFluffy
Copy link
Member Author

Using admin privileges to bypass the branch protection rules as the codeowner/author of the containerfile and the rust gh action, which are triggering a review by @astriaorg/infra

The changes are relatively low impact and only increment versions.

@SuperFluffy SuperFluffy merged commit 325da87 into main Nov 20, 2023
28 checks passed
@SuperFluffy SuperFluffy deleted the superfluffy/bump-deps branch November 20, 2023 17:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked blocked on another PR, dependency, or missing feature ci issues that are related to ci and github workflows conductor pertaining to the astria-conductor crate docker proto pertaining to the Astria Protobuf spec sequencer pertaining to the astria-sequencer crate sequencer-relayer pertaining to the astria-sequencer-relayer crate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants