Revert "Synchronize server startup"

What does this MR do?

Per discussion, the "Synchronize server startup" PR breaks charts due to the behavior of BullMQ's Worker.run() method. The promise returned by this method never resolves, causing the startup to stall just before the chart manager starts. This will be fixed later, but we're reverting for now to un-block the release.

Contribution Guidelines

By submitting this merge request, you agree to follow our Contribution Guidelines

• I agree to follow this project's Contribution Guidelines
• I have made sure to test this merge request

packages/backend/src/boot/common.ts

@@ -19,18 +19,17 @@ export async function server() {
 		logger: new NestLogger(),
 	});
 
+	const serverService = app.get(ServerService);
+	await serverService.launch();
+
 	if (process.env.NODE_ENV !== 'test') {
-		await app.get(ChartManagementService).start();
+		app.get(ChartManagementService).start();
 	}
 	if (!envOption.noDaemons) {
-		await app.get(QueueStatsService).start();
-		await app.get(ServerStatsService).start();
+		app.get(QueueStatsService).start();
+		app.get(ServerStatsService).start();
 	}
 
-	// Start server last so the other services can register hooks first
-	const serverService = app.get(ServerService);
-	await serverService.launch();
-
 	return app;
 }
 
@@ -39,8 +38,8 @@ export async function jobQueue() {
 		logger: new NestLogger(),
 	});
 
-	await jobQueue.get(QueueProcessorService).start();
-	await jobQueue.get(ChartManagementService).start();
+	jobQueue.get(QueueProcessorService).start();
+	jobQueue.get(ChartManagementService).start();
 
 	return jobQueue;
 }