-
Notifications
You must be signed in to change notification settings - Fork 202
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
v8 Upgrade Handler #683
v8 Upgrade Handler #683
Conversation
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.
Reminder we need to add distributor address for evmos
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.
Looks good to me!
Only outstanding question, is there something we can do in the upgrade handler to print out logs when the Evmos airdrop is being added, and when the claims are being reset? Maybe something like "if index % 500 == 0, print index" ?
app/upgrades/v8/upgrades.go
Outdated
|
||
// Add the evmos airdrop | ||
ctx.Logger().Info("Adding evmos airdrop...") | ||
blockTime := uint64(ctx.BlockTime().Unix()) |
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.
we should pick a blocktime in the future I think
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.
good call! should we shoot for next monday?
app/upgrades/v8/upgrades_test.go
Outdated
|
||
// Check that the params of the evmos airdrop were initialized | ||
s.Require().Equal(v8.EvmosAirdropIdentifier, evmosAirdrop.AirdropIdentifier, "evmos airdrop identifier") | ||
s.Require().Zero(evmosAirdrop.ClaimedSoFar.Int64(), "cevmos laimed so far") |
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.
nit, "evmos claimed so far"
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.
good catch!
Co-authored-by: vish-stride <vishal@stridelabs.co>
Closes: #XXX
Context and purpose of the change
Upgrade handler for v8
Brief Changelog
Author's Checklist
I have...
If skipped any of the tests above, explain.
Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...
Documentation and Release Note
Unreleased
section inCHANGELOG.md
?How is the feature or change documented?
XXX
x/<module>/spec/
)