Skip to content

Commit d417ed4

Browse files
fix: leverage gov proposal migrations (#1640)
## Description closes: #1627 --- ### Author Checklist _All items are required. Please add a note to the item if the item is not applicable and please add links to any relevant follow up issues._ I have... - [ ] included the correct [type prefix](https://github.com/commitizen/conventional-commit-types/blob/v3.0.0/index.json) in the PR title - [ ] added `!` to the type prefix if API or client breaking change - [ ] added appropriate labels to the PR - [ ] targeted the correct branch (see [PR Targeting](https://github.com/umee-network/umee/blob/main/CONTRIBUTING.md#pr-targeting)) - [ ] provided a link to the relevant issue or specification - [ ] added a changelog entry to `CHANGELOG.md` - [ ] included comments for [documenting Go code](https://blog.golang.org/godoc) - [ ] updated the relevant documentation or specification - [ ] reviewed "Files changed" and left comments if necessary - [ ] confirmed all CI checks have passed ### 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... - [ ] confirmed the correct [type prefix](https://github.com/commitizen/conventional-commit-types/blob/v3.0.0/index.json) in the PR title - [ ] confirmed all author checklist items have been addressed - [ ] reviewed state machine logic - [ ] reviewed API design and naming - [ ] reviewed documentation is accurate - [ ] reviewed tests and test coverage - [ ] manually tested (if applicable)
1 parent 94492a3 commit d417ed4

File tree

9 files changed

+661
-26
lines changed

9 files changed

+661
-26
lines changed

app/upgrades.go

Lines changed: 38 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ import (
1212
bech32ibctypes "github.com/osmosis-labs/bech32-ibc/x/bech32ibc/types"
1313

1414
"github.com/umee-network/umee/v3/app/upgradev3"
15+
"github.com/umee-network/umee/v3/app/upgradev3x3"
1516
leveragetypes "github.com/umee-network/umee/v3/x/leverage/types"
1617
oracletypes "github.com/umee-network/umee/v3/x/oracle/types"
1718
)
@@ -22,34 +23,60 @@ func (app UmeeApp) RegisterUpgradeHandlers(experimental bool) {
2223
panic(err)
2324
}
2425

25-
app.registerV3_0Upgrade(upgradeInfo)
26-
app.registerV3_1Upgrade(upgradeInfo)
27-
app.registerV3_2Upgrade(upgradeInfo)
26+
app.registerUpgrade3_0(upgradeInfo)
27+
app.registerUpgrade3_1(upgradeInfo)
28+
29+
app.registerUpgrade3_1to3_3(upgradeInfo)
30+
app.registerUpgrade3_2to3_3(upgradeInfo)
31+
}
32+
33+
// performs upgrade from v3.1 -> v3.3 (including the v3.2 chanages)
34+
func (app *UmeeApp) registerUpgrade3_1to3_3(_ upgradetypes.Plan) {
35+
const planName = "v3.1-v3.3"
36+
app.UpgradeKeeper.SetUpgradeHandler(
37+
planName,
38+
func(ctx sdk.Context, plan upgradetypes.Plan, fromVM module.VersionMap) (module.VersionMap, error) {
39+
ctx.Logger().Info("Upgrade handler execution", "name", planName)
40+
ctx.Logger().Info("Run v3.3 migrator")
41+
err := upgradev3x3.Migrator(app.GovKeeper, app.interfaceRegistry)(ctx)
42+
if err != nil {
43+
return fromVM, err
44+
}
45+
ctx.Logger().Info("Run x/bank v0.46.5 migration")
46+
err = bankkeeper.NewMigrator(app.BankKeeper).Migrate3_V046_4_To_V046_5(ctx)
47+
if err != nil {
48+
return fromVM, err
49+
}
50+
ctx.Logger().Info("Run module migrations")
51+
return app.mm.RunMigrations(ctx, app.configurator, fromVM)
52+
})
2853
}
2954

30-
// performs upgrade from v3.1 -> v3.2
31-
func (app *UmeeApp) registerV3_2Upgrade(_ upgradetypes.Plan) {
32-
const planName = "v3.2"
55+
// performs upgrade from v3.2 -> v3.3
56+
func (app *UmeeApp) registerUpgrade3_2to3_3(_ upgradetypes.Plan) {
57+
const planName = "v3.2-v3.3"
3358
app.UpgradeKeeper.SetUpgradeHandler(
3459
planName,
3560
func(ctx sdk.Context, plan upgradetypes.Plan, fromVM module.VersionMap) (module.VersionMap, error) {
3661
ctx.Logger().Info("Upgrade handler execution", "name", planName)
37-
err := bankkeeper.NewMigrator(app.BankKeeper).Migrate3_V046_4_To_V046_5(ctx)
62+
ctx.Logger().Info("Run v3.3 migrator")
63+
err := upgradev3x3.Migrator(app.GovKeeper, app.interfaceRegistry)(ctx)
3864
if err != nil {
3965
return fromVM, err
4066
}
67+
ctx.Logger().Info("Run module migrations")
4168
return app.mm.RunMigrations(ctx, app.configurator, fromVM)
4269
})
4370
}
4471

4572
// performs upgrade from v3.0 -> v3.1
46-
func (app *UmeeApp) registerV3_1Upgrade(_ upgradetypes.Plan) {
73+
func (app *UmeeApp) registerUpgrade3_1(_ upgradetypes.Plan) {
4774
const planName = "v3.1.0"
48-
app.UpgradeKeeper.SetUpgradeHandler(planName, onlyRunMigrations(app, planName))
75+
app.UpgradeKeeper.SetUpgradeHandler(planName, onlyModuleMigrations(app, planName))
4976
}
5077

5178
// performs upgrade from v1->v3
52-
func (app *UmeeApp) registerV3_0Upgrade(upgradeInfo upgradetypes.Plan) {
79+
func (app *UmeeApp) registerUpgrade3_0(upgradeInfo upgradetypes.Plan) {
5380
const planName = "v1.1-v3.0"
5481
app.UpgradeKeeper.SetUpgradeHandler(
5582
planName,
@@ -101,7 +128,7 @@ func (app *UmeeApp) registerV3_0Upgrade(upgradeInfo upgradetypes.Plan) {
101128
}})
102129
}
103130

104-
func onlyRunMigrations(app *UmeeApp, planName string) upgradetypes.UpgradeHandler {
131+
func onlyModuleMigrations(app *UmeeApp, planName string) upgradetypes.UpgradeHandler {
105132
return func(ctx sdk.Context, plan upgradetypes.Plan, fromVM module.VersionMap) (module.VersionMap, error) {
106133
ctx.Logger().Info("Upgrade handler execution", "name", planName)
107134
return app.mm.RunMigrations(ctx, app.configurator, fromVM)

0 commit comments

Comments
 (0)