Merge pull request #2234 from overleaf/bg-avoid-event-loop-metrics-during-startup

start event loop monitoring when the process is ready

GitOrigin-RevId: 0a7759311b737e99579f61cc4cf90b775c26b76d

services/web/app.js

@@ -49,6 +49,8 @@ if (!module.parent) {
   Server.server.listen(port, host, function() {
     logger.info(`web starting up, listening on ${host}:${port}`)
     logger.info(`${require('http').globalAgent.maxSockets} sockets enabled`)
+    // wait until the process is ready before monitoring the event loop
+    metrics.event_loop.monitor(logger)
     if (argv.user) {
       process.setuid(argv.user)
       logger.info(`Running as user: ${argv.user}`)

services/web/app/src/infrastructure/Server.js

@@ -42,10 +42,6 @@ const STATIC_CACHE_AGE = Settings.cacheStaticAssets
 // Init the session store
 const sessionStore = new RedisStore({ client: sessionsRedisClient })
 
-if (metrics.event_loop != null) {
-  metrics.event_loop.monitor(logger)
-}
-
 const app = express()
 
 const webRouter = express.Router()