-
Notifications
You must be signed in to change notification settings - Fork 41
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
Bump Cosmos to v0.46.7-pio-1 (from v0.46.6-pio-1) and create "pear" upgrade #1265
Conversation
… due to the local cosmos import).
…re how to automatically exclude the nft object definitions though.
…er context rather than getting the context from the command to get viper from it multiple times.
…appOpt later. A lot of our nodes just use the default, which viper doesn't know about.
…special call to create the snapshots dir since it's no longer needed.
…LeaveGroup msg fee if it exists.
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.
LGTM
…ated, and more importantly, the tests on it don't mess up other tests.
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #1265 +/- ##
==========================================
+ Coverage 58.46% 58.51% +0.05%
==========================================
Files 206 206
Lines 25264 25256 -8
==========================================
+ Hits 14771 14779 +8
+ Misses 9398 9380 -18
- Partials 1095 1097 +2
|
@SpicyLemon nit: noticed a typo in the title. It should be (from v0.46.6-pio-1) |
customMsgFeeFloor := server.GetServerContextFromCmd(cmd).Viper.GetInt64(CustomMsgFeeFloorPriceFlag) | ||
app.SetConfig(testnet, true) | ||
vpr := server.GetServerContextFromCmd(cmd).Viper | ||
testnet := vpr.GetBool(EnvTypeFlag) |
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.
In this context the EnvTypeFlag overlaps heavily with ENV as a source of vars
func RemoveLeaveGroupMsgFee(ctx sdk.Context, app *App) error { | ||
typeURL := sdk.MsgTypeURL(&group.MsgLeaveGroup{}) | ||
app.Logger().Info("deleting MsgLeaveGroup msg fee if it exists") | ||
err := app.MsgFeesKeeper.RemoveMsgFee(ctx, typeURL) | ||
if err == nil { | ||
return nil | ||
} | ||
if errors.Is(err, msgfeestypes.ErrMsgFeeDoesNotExist) { | ||
return nil | ||
} | ||
app.Logger().Error("error deleting MsgLeaveGroup msg fee", "error", err) | ||
return err | ||
} |
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.
This might not be needed anymore. Waiting a bit to find out.
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.
What's your take on this now @SpicyLemon ?
@@ -14,3 +16,19 @@ message StoreKVPair { | |||
bytes key = 3; | |||
bytes value = 4; | |||
} | |||
|
|||
// BlockMetadata contains all the abci event data of a block | |||
// the file streamer dump them into files together with the state changes. |
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.
I know this is coming from cosmos but this comment can be improved.
I'm going to close this.
Note that |
Description
v0.46.7-pio-1
(fromv0.46.6-pio-1
)pear
upgrade that runs migrations and deletes a msg fee that might be on theMsgLeaveGroup
message.Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.
docs/
) or specification (x/<module>/spec/
)godoc
comments.Unreleased
section inCHANGELOG.md
Files changed
in the Github PR explorerCodecov Report
in the comment section below once CI passes