-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
[Bug]: v0.50.2 upgrade #18733
Comments
Hey! So the issue seems to be that you've migrated the params but to the wrong store key: https://github.com/BitCannaGlobal/bcna/blob/BCNA-v3-way-(sdk-v0.47)/app/app.go#L347 To solve it, in the upgrade handler you need to create a new consensus params keeper using the wrong key again and move the data into the consensus params keeper with the right key. I've tried this and seems to work fine: storesvc := runtime.NewKVStoreService(app.GetKey("upgrade"))
consensuskeeper := consensuskeeper.NewKeeper(
app.appCodec,
storesvc,
app.AccountKeeper.GetAuthority(),
runtime.EventService{},
)
params, err := consensuskeeper.ParamsStore.Get(ctx)
if err != nil {
return nil, err
}
err = app.ConsensusParamsKeeper.ParamsStore.Set(ctx, params)
if err != nil {
return nil, err
} |
Thanks a lot 🙏 @facundomedica, what a missing! that v0.47 branch is not (yet) in production, so we are able to fix it in origin before v.050 comes. |
Is there an existing issue for this?
What happened?
Upgrading from v0.47 to v0.50.2 I found this issue related in https://github.com/cosmos/cosmos-sdk/pull/18577/files & #18644 BUT it persist after the upgrade.
Along the upgrade, we could see this messages twice:
After upgrade and the adding of the new modules, when normal activity is back, the general consensus is:
So first thoughts discussing the issue with @julienrbrt are about that Baseapp keeps not finding consensus params, so upgrade should set them and them are not saved.
Full upgrade log:
Cosmos SDK Version
v0.50.2
How to reproduce?
For more context; the previous upgrade was completed as recommended:
v0.46 -> v0.47 upgrade was successful and new module consensus was added and migrated the params:
https://github.com/BitCannaGlobal/bcna/blob/BCNA-v3-way-(sdk-v0.47)/app/upgrades.go#L123
For to reproduce it:
https://github.com/BitCannaGlobal/bcna/blob/BCNA-v3-way-(sdk-v0.47) (v3.0.0-rc3)
The text was updated successfully, but these errors were encountered: