Merge pull request #5473 from oasisprotocol/kostko/fix/cfg-migration-cdb

go/oasis-node: Remove obsolete db section during config migration
oasisprotocol · Nov 26, 2023 · dab847d · dab847d
2 parents 4ea9010 + 3f4c41e
commit dab847d
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 0 deletions.
diff --git a/.changelog/5473.bugfix.md b/.changelog/5473.bugfix.md
@@ -0,0 +1 @@
+go/oasis-node: Remove obsolete db section during config migration
diff --git a/go/oasis-node/cmd/config/migrate/migrate.go b/go/oasis-node/cmd/config/migrate/migrate.go
@@ -346,6 +346,9 @@ func doMigrateConfig(cmd *cobra.Command, args []string) {
 							m(newCfg["consensus"])["prune"] = prune
 							delete(m(m(tendermint)["abci"]), "prune")
 						}
+					} else if k == "db" {
+						logger.Info("consensus.db.* is no longer needed")
+						delete(m(tendermint), "db")
 					} else {
 						m(newCfg["consensus"])[k] = v
 						delete(m(tendermint), k)

diff --git a/go/oasis-node/cmd/config/migrate/migrate_test.go b/go/oasis-node/cmd/config/migrate/migrate_test.go
@@ -120,6 +120,8 @@ consensus:
       upstream_address:
         - "[email protected]:5678"
     mode: full
+    db:
+      backend: badger
 
     # Validators behind sentry nodes should set sentries as persistent peers.
     p2p: