Only check request time for readiness if cache has filled up

Author: ajayyy

src/config.ts

@@ -194,7 +194,8 @@ addDefaults(config, {
     deArrowPaywall: false,
     useCacheForSegmentGroups: false,
     maxConnections: 100,
-    maxResponseTime: 1000
+    maxResponseTime: 1000,
+    maxResponseTimeWhileLoadingCache: 2000
 });
 loadFromEnv(config);
 migrate(config);

src/routes/getReady.ts

@@ -14,9 +14,11 @@ export async function getReady(req: Request, res: Response, server: Server): Pro
     if (!connections
             || (connections < config.maxConnections
                 && (!config.redis || redisStats.activeRequests < config.redis.maxConnections * 0.8)
-                && (!config.redis || redisStats.avgReadTime < config.maxResponseTime || redisStats.activeRequests < 1)
-                && (!config.postgres || postgresStats.activeRequests < config.postgres.maxActiveRequests * 0.8))
-                && (!config.postgres || postgresStats.avgReadTime < config.maxResponseTime)) {
+                && (!config.redis || redisStats.activeRequests < 1 || redisStats.avgReadTime < config.maxResponseTime
+                    || (redisStats.memoryCacheSize < config.redis.clientCacheSize * 0.8 && redisStats.avgReadTime < config.maxResponseTimeWhileLoadingCache))
+                && (!config.postgres || postgresStats.activeRequests < config.postgres.maxActiveRequests * 0.8)
+                && (!config.postgres || postgresStats.avgReadTime < config.maxResponseTime
+                    || (redisStats.memoryCacheSize < config.redis.clientCacheSize * 0.8 && postgresStats.avgReadTime < config.maxResponseTimeWhileLoadingCache)))) {
         return res.sendStatus(200);
     } else {
         return res.sendStatus(500);

src/types/config.model.ts

@@ -107,6 +107,8 @@ export interface SBSConfig {
     deArrowPaywall: boolean,
     useCacheForSegmentGroups: boolean
     maxConnections: number;
+    maxResponseTime: number;
+    maxResponseTimeWhileLoadingCache: number;
 }
 
 export interface WebhookConfig {