Making sure any error that occurs in the app migration system gets logged.

packages/server/src/appMigrations/migrationsProcessor.ts

@@ -12,6 +12,7 @@ export async function processMigrations(
   migrations: AppMigration[]
 ) {
   console.log(`Processing app migration for "${appId}"`)
+  try {
     // have to wrap in context, this gets the tenant from the app ID
     await context.doInAppContext(appId, async () => {
       await locks.doWithLock(
@@ -21,7 +22,6 @@ export async function processMigrations(
           resource: appId,
         },
         async () => {
-        try {
           await context.doInAppMigrationContext(appId, async () => {
             let currentVersion = await getAppMigrationVersion(appId)
 
@@ -55,13 +55,13 @@ export async function processMigrations(
               currentVersion = id
             }
           })
-        } catch (err) {
-          logging.logAlert("Failed to run app migration", err)
-          throw err
-        }
         }
       )
 
       console.log(`App migration for "${appId}" processed`)
     })
+  } catch (err) {
+    logging.logAlert("Failed to run app migration", err)
+    throw err
+  }
 }