Scale Logging Context

modules/runners/lambdas/runners/src/scale-runners/gh-auth.ts

@@ -11,9 +11,9 @@ import {
 } from '@octokit/auth-app/dist-types/types';
 import { OctokitOptions } from '@octokit/core/dist-types/types';
 import { getParameterValue } from './ssm';
-import { logger as rootLogger } from './logger';
+import { logger as rootLogger, LogFields } from './logger';
 
-const logger = rootLogger.getChildLogger();
+const logger = rootLogger.getChildLogger({ name: 'gh-auth' });
 
 export async function createOctoClient(token: string, ghesApiUrl = ''): Promise<Octokit> {
   const ocktokitOptions: OctokitOptions = {
@@ -55,7 +55,7 @@ async function createAuth(installationId: number | undefined, ghesApiUrl: string
   };
   if (installationId) authOptions = { ...authOptions, installationId };
 
-  logger.debug(`GHES API URL: ${ghesApiUrl}`);
+  logger.debug(LogFields.fields, `GHES API URL: ${ghesApiUrl}`);
   if (ghesApiUrl) {
     authOptions.request = request.defaults({
       baseUrl: ghesApiUrl,

modules/runners/lambdas/runners/src/scale-runners/logger.ts

@@ -5,7 +5,10 @@ export const logger = new Logger({
   displayInstanceName: false,
   maskAnyRegEx: ['--token [A-Z0-9]*'],
   minLevel: process.env.LOG_LEVEL || 'info',
-  name: 'scale-up',
+  name: 'scale-runners',
   overwriteConsole: true,
   type: process.env.LOG_TYPE || 'pretty',
 });
+export class LogFields {
+  static fields: { [key: string]: string } = {};
+}

modules/runners/lambdas/runners/src/scale-runners/runners.ts

@@ -1,7 +1,7 @@
 import { EC2, SSM } from 'aws-sdk';
-import { logger as rootLogger } from './logger';
+import { logger as rootLogger, LogFields } from './logger';
 
-const logger = rootLogger.getChildLogger();
+const logger = rootLogger.getChildLogger({ name: 'runners' });
 
 export interface RunnerList {
   instanceId: string;
@@ -75,16 +75,20 @@ export async function terminateRunner(instanceId: string): Promise<void> {
       InstanceIds: [instanceId],
     })
     .promise();
-  logger.info(`Runner ${instanceId} has been terminated.`);
+  logger.info(LogFields.fields, `Runner ${instanceId} has been terminated.`);
 }
 
 export async function createRunner(runnerParameters: RunnerInputParameters, launchTemplateName: string): Promise<void> {
-  logger.debug('Runner configuration: ' + JSON.stringify(runnerParameters));
+  logger.debug(LogFields.fields, 'Runner configuration: ' + JSON.stringify(runnerParameters));
   const ec2 = new EC2();
   const runInstancesResponse = await ec2
     .runInstances(getInstanceParams(launchTemplateName, runnerParameters))
     .promise();
-  logger.info('Created instance(s): ', runInstancesResponse.Instances?.map((i) => i.InstanceId).join(','));
+  logger.info(
+    LogFields.fields,
+    'Created instance(s): ',
+    runInstancesResponse.Instances?.map((i) => i.InstanceId).join(','),
+  );
   const ssm = new SSM();
   runInstancesResponse.Instances?.forEach(async (i: EC2.Instance) => {
     await ssm

modules/runners/lambdas/runners/src/scale-runners/scale-down.ts

@@ -4,20 +4,20 @@ import { listEC2Runners, RunnerInfo, RunnerList, terminateRunner } from './runne
 import { getIdleRunnerCount, ScalingDownConfig } from './scale-down-config';
 import { createOctoClient, createGithubAppAuth, createGithubInstallationAuth } from './gh-auth';
 import { githubCache, GhRunners } from './cache';
-import { logger as rootLogger } from './logger';
+import { logger as rootLogger, LogFields } from './logger';
 
-const logger = rootLogger.getChildLogger();
+const logger = rootLogger.getChildLogger({ name: 'scale-down' });
 
 async function getOrCreateOctokit(runner: RunnerInfo): Promise<Octokit> {
   const key = runner.owner;
   const cachedOctokit = githubCache.clients.get(key);
 
   if (cachedOctokit) {
-    logger.debug(`[createGitHubClientForRunner] Cache hit for ${key}`);
+    logger.debug(LogFields.fields, `[createGitHubClientForRunner] Cache hit for ${key}`);
     return cachedOctokit;
   }
 
-  logger.debug(`[createGitHubClientForRunner] Cache miss for ${key}`);
+  logger.debug(LogFields.fields, `[createGitHubClientForRunner] Cache miss for ${key}`);
   const ghesBaseUrl = process.env.GHES_URL;
   let ghesApiUrl = '';
   if (ghesBaseUrl) {
@@ -50,11 +50,11 @@ async function listGitHubRunners(runner: RunnerInfo): Promise<GhRunners> {
   const key = runner.owner as string;
   const cachedRunners = githubCache.runners.get(key);
   if (cachedRunners) {
-    logger.debug(`[listGithubRunners] Cache hit for ${key}`);
+    logger.debug(LogFields.fields, `[listGithubRunners] Cache hit for ${key}`);
     return cachedRunners;
   }
 
-  logger.debug(`[listGithubRunners] Cache miss for ${key}`);
+  logger.debug(LogFields.fields, `[listGithubRunners] Cache miss for ${key}`);
   const client = await getOrCreateOctokit(runner);
   const runners =
     runner.type === 'Org'
@@ -102,12 +102,18 @@ async function removeRunner(ec2runner: RunnerInfo, ghRunnerId: number): Promise<
 
     if (result.status == 204) {
       await terminateRunner(ec2runner.instanceId);
-      logger.info(`AWS runner instance '${ec2runner.instanceId}' is terminated and GitHub runner is de-registered.`);
+      logger.info(
+        LogFields.fields,
+        `AWS runner instance '${ec2runner.instanceId}' is terminated and GitHub runner is de-registered.`,
+      );
     } else {
-      logger.error(`Failed to de-register GitHub runner: ${result.status}`);
+      logger.error(LogFields.fields, `Failed to de-register GitHub runner: ${result.status}`);
     }
   } catch (e) {
-    logger.info(`Runner '${ec2runner.instanceId}' cannot be de-registered, most likely the runner is active.`);
+    logger.info(
+      LogFields.fields,
+      `Runner '${ec2runner.instanceId}' cannot be de-registered, most likely the runner is active.`,
+    );
   }
 }
 
@@ -120,26 +126,29 @@ async function evaluateAndRemoveRunners(
 
   for (const ownerTag of ownerTags) {
     const ec2RunnersFiltered = ec2Runners.filter((runner) => runner.owner === ownerTag);
-    logger.debug(`Found: '${ec2RunnersFiltered.length}' active GitHub runners with owner tag: '${ownerTag}'`);
+    logger.debug(
+      LogFields.fields,
+      `Found: '${ec2RunnersFiltered.length}' active GitHub runners with owner tag: '${ownerTag}'`,
+    );
     for (const ec2Runner of ec2RunnersFiltered) {
       const ghRunners = await listGitHubRunners(ec2Runner);
       const ghRunner = ghRunners.find((runner) => runner.name === ec2Runner.instanceId);
       if (ghRunner) {
         if (runnerMinimumTimeExceeded(ec2Runner)) {
           if (idleCounter > 0) {
             idleCounter--;
-            logger.info(`Runner '${ec2Runner.instanceId}' will kept idle.`);
+            logger.info(LogFields.fields, `Runner '${ec2Runner.instanceId}' will kept idle.`);
           } else {
-            logger.info(`Runner '${ec2Runner.instanceId}' will be terminated.`);
+            logger.info(LogFields.fields, `Runner '${ec2Runner.instanceId}' will be terminated.`);
             await removeRunner(ec2Runner, ghRunner.id);
           }
         }
       } else {
         if (bootTimeExceeded(ec2Runner)) {
-          logger.info(`Runner '${ec2Runner.instanceId}' is orphaned and will be removed.`);
+          logger.info(LogFields.fields, `Runner '${ec2Runner.instanceId}' is orphaned and will be removed.`);
           terminateOrphan(ec2Runner.instanceId);
         } else {
-          logger.debug(`Runner ${ec2Runner.instanceId} has not yet booted.`);
+          logger.debug(LogFields.fields, `Runner ${ec2Runner.instanceId} has not yet booted.`);
         }
       }
     }

modules/runners/lambdas/runners/src/scale-runners/scale-up.ts

@@ -2,9 +2,9 @@ import { listEC2Runners, createRunner, RunnerInputParameters } from './runners';
 import { createOctoClient, createGithubAppAuth, createGithubInstallationAuth } from './gh-auth';
 import yn from 'yn';
 import { Octokit } from '@octokit/rest';
-import { logger as rootLogger } from './logger';
+import { logger as rootLogger, LogFields } from './logger';
 
-const logger = rootLogger.getChildLogger();
+const logger = rootLogger.getChildLogger({ name: 'scale-up' });
 
 export interface ActionRequestMessage {
   id: number;
@@ -23,7 +23,16 @@ export async function scaleUp(eventSource: string, payload: ActionRequestMessage
   const environment = process.env.ENVIRONMENT;
   const ghesBaseUrl = process.env.GHES_URL;
 
-  console.info(`Received ${payload.eventType} from ${payload.repositoryOwner}/${payload.repositoryName}`);
+  const runnerType = enableOrgLevel ? 'Org' : 'Repo';
+  const runnerOwner = enableOrgLevel ? payload.repositoryOwner : `${payload.repositoryOwner}/${payload.repositoryName}`;
+
+  LogFields.fields = {};
+  LogFields.fields.runnerType = runnerType;
+  LogFields.fields.runnerOwner = runnerOwner;
+  LogFields.fields.event = payload.eventType;
+  LogFields.fields.id = payload.id.toString();
+
+  console.info(LogFields.fields, `Received event`);
 
   let ghesApiUrl = '';
   if (ghesBaseUrl) {
@@ -50,8 +59,6 @@ export async function scaleUp(eventSource: string, payload: ActionRequestMessage
 
   const ghAuth = await createGithubInstallationAuth(installationId, ghesApiUrl);
   const githubInstallationClient = await createOctoClient(ghAuth.token, ghesApiUrl);
-  const runnerType = enableOrgLevel ? 'Org' : 'Repo';
-  const runnerOwner = enableOrgLevel ? payload.repositoryOwner : `${payload.repositoryOwner}/${payload.repositoryName}`;
 
   const isQueued = await getJobStatus(githubInstallationClient, payload);
   if (isQueued) {
@@ -60,7 +67,7 @@ export async function scaleUp(eventSource: string, payload: ActionRequestMessage
       runnerType,
       runnerOwner,
     });
-    logger.info(`${runnerType} ${runnerOwner} has ${currentRunners.length}/${maximumRunners} runners`);
+    logger.info(LogFields.fields, `Current runners: ${currentRunners.length} of ${maximumRunners}`);
 
     if (currentRunners.length < maximumRunners) {
       console.info(`Attempting to launch a new runner`);
@@ -87,7 +94,7 @@ export async function scaleUp(eventSource: string, payload: ActionRequestMessage
         runnerType,
       });
     } else {
-      logger.info('No runner will be created, maximum number of runners reached.');
+      logger.info(LogFields.fields, 'No runner will be created, maximum number of runners reached.');
     }
   }
 }
@@ -112,7 +119,7 @@ async function getJobStatus(githubInstallationClient: Octokit, payload: ActionRe
     throw Error(`Event ${payload.eventType} is not supported`);
   }
   if (!isQueued) {
-    logger.info(`Job ${payload.id} is not queued`);
+    logger.info(LogFields.fields, `Job not queued`);
   }
   return isQueued;
 }
@@ -121,13 +128,13 @@ export async function createRunnerLoop(runnerParameters: RunnerInputParameters):
   const launchTemplateNames = process.env.LAUNCH_TEMPLATE_NAME?.split(',') as string[];
   let launched = false;
   for (let i = 0; i < launchTemplateNames.length; i++) {
-    logger.info(`Attempt '${i}' to launch instance using ${launchTemplateNames[i]}.`);
+    logger.info(LogFields.fields, `Attempt '${i}' to launch instance using ${launchTemplateNames[i]}.`);
     try {
       await createRunner(runnerParameters, launchTemplateNames[i]);
       launched = true;
       break;
     } catch (error) {
-      logger.debug(`Attempt '${i}' to launch instance using ${launchTemplateNames[i]} FAILED.`);
+      logger.debug(LogFields.fields, `Attempt '${i}' to launch instance using ${launchTemplateNames[i]} FAILED.`);
       logger.error(error);
     }
   }