Skip to content

[SG-133] upgrading to sdk v0.47 #845

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
merged 60 commits into from
Sep 18, 2023
Merged

[SG-133] upgrading to sdk v0.47 #845

merged 60 commits into from
Sep 18, 2023

Conversation

spoo-bar
Copy link
Contributor

@spoo-bar spoo-bar commented Aug 9, 2023

  • Replacing tendermint with cometbft
  • Updating sdk to 0.47
  • Updating wasmd to 0.41
  • Updating ibc-go to v7
  • Removing REST stuff
  • Replacing ibc-hooks fork with reference to github.com/cosmos/ibc-apps/modules/ibc-hooks/v7
  • All the existing gov props for x/cron and x/globalfee are kept as legacy gov props. Will update them in later PR
  • Fixing simapp references in tests

@linear
Copy link

linear bot commented Aug 9, 2023

SG-133 Migrate to 0.47

@@ -157,7 +157,7 @@ DOCKER_BUF := $(DOCKER) run --rm -v $(CURDIR):/workspace --workdir /workspace $(
###############################################################################
### Protobuf ###
###############################################################################
PROTO_BUILDER_IMAGE=ghcr.io/cosmos/proto-builder:0.9.0
PROTO_BUILDER_IMAGE=ghcr.io/cosmos/proto-builder:0.14.0
Copy link
Member

Choose a reason for hiding this comment

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

seems like the recommended version was 0.11.2 is there any difference with this version?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Currently recommend version is > 0.11.5 for sdk 47.
Using 0.14.0 just to use the most upto date builder.
The latest image has some optimizations as well cosmos/cosmos-sdk#16343

@@ -3,13 +3,12 @@ package statesync
import (
Copy link
Member

Choose a reason for hiding this comment

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

we can remove this custom snapshotter if statesync works in v0.47 just need to check

@jhernandezb jhernandezb marked this pull request as ready for review September 18, 2023 15:22
@jhernandezb jhernandezb merged commit fb4ef5a into main Sep 18, 2023
@jhernandezb jhernandezb deleted the spoorthi/upgrade-sdk branch September 18, 2023 16:07
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