Add information for migrate failure (#16803)

lunny · web-flow · commit efaf10943517 · 2021-08-24T18:46:55.000+01:00
Improve the reporting of errors when there is a migration failure
diff --git a/models/migrations/migrations.go b/models/migrations/migrations.go
@@ -429,7 +429,7 @@ Please try upgrading to a lower version first (suggested v1.6.4), then upgrade t
 		// Reset the mapper between each migration - migrations are not supposed to depend on each other
 		x.SetMapper(names.GonicMapper{})
 		if err = m.Migrate(x); err != nil {
-			return fmt.Errorf("do migrate: %v", err)
+			return fmt.Errorf("migration[%d]: %s failed: %v", v+int64(i), m.Description(), err)
 		}
 		currentVersion.Version = v + int64(i) + 1
 		if _, err = x.ID(1).Update(currentVersion); err != nil {

Original file line number	Diff line number	Diff line change
`@@ -429,7 +429,7 @@ Please try upgrading to a lower version first (suggested v1.6.4), then upgrade t`
`429`	`429`	`// Reset the mapper between each migration - migrations are not supposed to depend on each other`
`430`	`430`	`x.SetMapper(names.GonicMapper{})`
`431`	`431`	`if err = m.Migrate(x); err != nil {`
`432`		`- return fmt.Errorf("do migrate: %v", err)`
	`432`	`+ return fmt.Errorf("migration[%d]: %s failed: %v", v+int64(i), m.Description(), err)`
`433`	`433`	`}`
`434`	`434`	`currentVersion.Version = v + int64(i) + 1`
`435`	`435`	`if _, err = x.ID(1).Update(currentVersion); err != nil {`