From ac9a85ab88da783163afba1b251c99fcb8d2bab2 Mon Sep 17 00:00:00 2001 From: Zach Wasserman Date: Thu, 28 Jan 2021 09:49:06 -0800 Subject: [PATCH] Expose more information for migration status errors (#240) We saw these errors live today and it would be helpful to have the additional error context for debugging. --- server/datastore/mysql/mysql.go | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/server/datastore/mysql/mysql.go b/server/datastore/mysql/mysql.go index f2bb57f034..81813162c1 100644 --- a/server/datastore/mysql/mysql.go +++ b/server/datastore/mysql/mysql.go @@ -193,22 +193,22 @@ func (d *Datastore) MigrationStatus() (kolide.MigrationStatus, error) { lastTablesMigration, err := tables.MigrationClient.Migrations.Last() if err != nil { - return 0, errors.New("missing tables migrations") + return 0, errors.Wrap(err, "missing tables migrations") } currentTablesVersion, err := tables.MigrationClient.GetDBVersion(d.db.DB) if err != nil { - return 0, errors.New("cannot get table migration status") + return 0, errors.Wrap(err, "cannot get table migration status") } lastDataMigration, err := data.MigrationClient.Migrations.Last() if err != nil { - return 0, errors.New("missing data migrations") + return 0, errors.Wrap(err, "missing data migrations") } currentDataVersion, err := data.MigrationClient.GetDBVersion(d.db.DB) if err != nil { - return 0, errors.New("cannot get table migration status") + return 0, errors.Wrap(err, "cannot get data migration status") } switch {