app: register v1.20.2 migration-only upgrade handler - #198
app: register v1.20.2 migration-only upgrade handler#198mateeullahmalik wants to merge 1 commit into
Conversation
Register the coordinated v1.20.2 upgrade boundary for the evmigration fixes. The handler runs module migrations only and declares no store changes or historical state repair.
|
Closing in favour of #199, which owns WhyBoth PRs branch off #196 and both create That matters because this PR's handler is: case upgrade_v1_20_2.UpgradeName:
return UpgradeConfig{
Handler: standardUpgradeHandler(upgrade_v1_20_2.UpgradeName, params),
}, trueNo Every validator crash-looped on a faithful #199's version declares the five EVM store additions, routes through the add-only store loader, and uses a state-driven handler that delegates to the v1.20.0 bring-up when the EVM stack is absent — verified green on both the single-hop Nothing is lostThis PR is 34 lines across 4 files, all of it func TestUpgradeName(t *testing.T) {
require.Equal(t, "v1.20.2", UpgradeName)
}#199 carries 6 tests over the same package, including
So #199 strictly supersedes this branch. No unique work is being dropped. Follow-up#197 (operator runbooks) is stacked on this branch and will be rebased onto #199. |
Behavior change
Registers the
v1.20.2software-upgrade handler and routes it through the standard Cosmos SDK module migration manager.Rationale
Testnet is currently running
v1.20.1. The history-preservation fixes in the parent PR require a namedv1.20.2handler so Cosmovisor can resume block production at the governance-planned upgrade height.State-machine impact
moduleManager.RunMigrationswhen an installed module consensus version requires migration.Risks
Rollback
Before the upgrade height, withdraw/cancel the proposal or deploy a corrected binary. After execution, use the established chain rollback/governance recovery procedure; this PR adds no custom reversible state transform.
Verification
v1.20.1 -> v1.20.2, five validators, continued block productionStack order
v1.20.2handler