Record telemetry for cache operation duration

github · Dec 3, 2024 · 7bcf845 · 7bcf845
1 parent 93f3303
commit 7bcf845
Show file tree

Hide file tree

Showing 5 changed files with 17 additions and 6 deletions.
diff --git a/lib/setup-codeql.js b/lib/setup-codeql.js
diff --git a/lib/setup-codeql.js.map b/lib/setup-codeql.js.map
diff --git a/lib/tools-download.js.map b/lib/tools-download.js.map
diff --git a/src/setup-codeql.ts b/src/setup-codeql.ts
@@ -550,7 +550,7 @@ export const downloadCodeQL = async function (
     ? toolcacheInfo.path
     : getTempExtractionDir(tempDir);
 
-  const statusReport = await downloadAndExtract(
+  let statusReport = await downloadAndExtract(
     codeqlURL,
     extractedBundlePath,
     authorization,
@@ -585,11 +585,16 @@ export const downloadCodeQL = async function (
       toolcacheInfo.version,
     );
 
+    const cacheDurationMs = performance.now() - toolcacheStart;
     logger.info(
       `Added CodeQL bundle to the tool cache (${formatDuration(
-        performance.now() - toolcacheStart,
+        cacheDurationMs,
       )}).`,
     );
+    statusReport = {
+      ...statusReport,
+      cacheDurationMs,
+    };
 
     // Defensive check: we expect `cacheDir` to copy the bundle to a new location.
     if (codeqlFolder !== extractedBundlePath) {

diff --git a/src/tools-download.ts b/src/tools-download.ts
@@ -73,6 +73,7 @@ type ToolsDownloadDurations =
   | StreamedToolsDownloadDurations;
 
 export type ToolsDownloadStatusReport = {
+  cacheDurationMs?: number;
   compressionMethod: tar.CompressionMethod;
   toolsUrl: string;
   zstdFailureReason?: string;