-
Notifications
You must be signed in to change notification settings - Fork 104
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
fix: Fix v5 upgrade VE handler #1468
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.
Lgtm
panic(fmt.Sprintf("failed to retrieve existing consensus params in VE upgrade handler: %s", err)) | ||
} | ||
if currentParams.Params.Abci.VoteExtensionsEnableHeight != 0 { | ||
if currentParams.Params.Abci != nil && currentParams.Params.Abci.VoteExtensionsEnableHeight != 0 { |
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 like currentParams.Params.Abci
is never read and we just initialize it below. Should we just remove this code block?
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.
The reason I initially put it in there is because if you've already set a VE enable height that's nonzero it's unknown (to me at least) what happens if you update that again.
Happy to remove it since we assume this will be run once ever.
@mergify backport release/protocol/v5.x |
✅ Backports have been created
|
* Add nil protections on upgrade handler * Fix import name * Address comment (cherry picked from commit 5e002ee)
* Add nil protections on upgrade handler * Fix import name * Address comment
Changelist
Add protections around nil VE ABCI Params
Test Plan
Send to a single validator first on testnet.
Author/Reviewer Checklist
state-breaking
label.indexer-postgres-breaking
label.PrepareProposal
orProcessProposal
, manually add the labelproposal-breaking
.feature:[feature-name]
.backport/[branch-name]
.refactor
,chore
,bug
.