Adds a no-op migration for metadata v3

John Howard · John Howard · commit 8e44270723a6 · 2018-09-12T15:33:42.000-07:00
Signed-off-by: John Howard &lt;jhoward@microsoft.com&gt;
diff --git a/metadata/db.go b/metadata/db.go
@@ -43,7 +43,7 @@ const (
 	// dbVersion represents updates to the schema
 	// version which are additions and compatible with
 	// prior version of the same schema.
-	dbVersion = 2
+	dbVersion = 3
 )
 
 // DB represents a metadata database backed by a bolt
diff --git a/metadata/db_test.go b/metadata/db_test.go
@@ -265,6 +265,16 @@ func TestMigrations(t *testing.T) {
 				return nil
 			},
 		},
+
+		{
+			name: "NoOp",
+			init: func(tx *bolt.Tx) error {
+				return nil
+			},
+			check: func(tx *bolt.Tx) error {
+				return nil
+			},
+		},
 	}
 
 	if len(migrationTests) != len(migrations) {
diff --git a/metadata/migrations.go b/metadata/migrations.go
@@ -45,6 +45,11 @@ var migrations = []migration{
 		version: 2,
 		migrate: migrateIngests,
 	},
+	{
+		schema:  "v1",
+		version: 3,
+		migrate: noOpMigration,
+	},
 }
 
 // addChildLinks Adds children key to the snapshotters to enforce snapshot
@@ -154,3 +159,10 @@ func migrateIngests(tx *bolt.Tx) error {
 
 	return nil
 }
+
+// noOpMigration was for a database change from boltdb/bolt which is no
+// longer being supported, to go.etcd.io/bbolt which is the currently
+// maintained repo for boltdb.
+func noOpMigration(tx *bolt.Tx) error {
+	return nil
+}

Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,7 @@ const (`
`43`	`43`	`// dbVersion represents updates to the schema`
`44`	`44`	`// version which are additions and compatible with`
`45`	`45`	`// prior version of the same schema.`
`46`		`- dbVersion = 2`
	`46`	`+ dbVersion = 3`
`47`	`47`	`)`
`48`	`48`
`49`	`49`	`// DB represents a metadata database backed by a bolt`