Moving app migrations back to the automation service now that its getting updated in QA - also making sure that the app migration is always cached, disabling it in dev/QA may hide issues with caching, making sure it always runs the same as it does in prod is safest.

packages/server/src/appMigrations/appMigrationMetadata.ts

@@ -25,8 +25,8 @@ export async function getAppMigrationVersion(appId: string): Promise<string> {
 
   let metadata: AppMigrationDoc | undefined = await cache.get(cacheKey)
 
-  // We don't want to cache in dev or QA in order to be able to tweak it
-  if (metadata && !env.isDev() && !env.isQA()) {
+  // returned cached version if we found one
+  if (metadata?.version) {
     return metadata.version
   }
 

packages/server/src/startup/index.ts

@@ -121,8 +121,6 @@ export async function startup(
   // app migrations and automations on other service
   if (automationsEnabled()) {
     queuePromises.push(automations.init())
-  }
-  if (apiEnabled()) {
     queuePromises.push(appMigrations.init())
   }
   queuePromises.push(initPro())