From 8c4adb87149c67205c3abdf54249db0aeb312af6 Mon Sep 17 00:00:00 2001 From: aayushshah15 Date: Mon, 28 Sep 2026 05:21:22 +0000 Subject: [PATCH] Log git mirror hydration contention as info instead of a warning Co-authored-by: Codesmith Staging --- dist/index.js | 6 +++--- src/blacksmith-cache.ts | 4 ++-- src/git-source-provider.ts | 2 +- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/dist/index.js b/dist/index.js index 5cc758e..d1d1519 100644 --- a/dist/index.js +++ b/dist/index.js @@ -332,8 +332,8 @@ function setupCache(owner, repo) { // Check if this is a gRPC Aborted error indicating hydration in progress if (error instanceof connect_1.ConnectError && error.code === connect_1.Code.Aborted) { const hydrationMessage = error.message || 'Initial mirror clone is running'; - core.warning(`[git-mirror] Another job is hydrating the git mirror cache: ${hydrationMessage}`); - core.warning('[git-mirror] No sticky disk will be mounted for this run; checkout will clone directly from GitHub onto the runner disk (no mirror cache). The mirror cache will be available on subsequent runs once hydration completes.'); + core.info(`[git-mirror] Another job is hydrating the git mirror cache: ${hydrationMessage}`); + core.info('[git-mirror] No sticky disk will be mounted for this run; checkout will clone directly from GitHub onto the runner disk (no mirror cache). The mirror cache will be available on subsequent runs once hydration completes.'); return { exposeId: '', stickyDiskKey, @@ -3714,7 +3714,7 @@ function getSourceInner(settings, report, onHydrationReport) { report.sticky_disk_setup_ms = Date.now() - setupStart; // Check if hydration is in progress - another job is doing the initial git clone --mirror if (cacheInfo.hydrationInProgress) { - // Warning already logged by setupCache, just fall back to standard checkout + // Already logged by setupCache, just fall back to standard checkout report.serving_mode = 'fallback-contention'; cacheInfo = null; core.endGroup(); diff --git a/src/blacksmith-cache.ts b/src/blacksmith-cache.ts index f24bc4d..f073efc 100644 --- a/src/blacksmith-cache.ts +++ b/src/blacksmith-cache.ts @@ -348,10 +348,10 @@ export async function setupCache( if (error instanceof ConnectError && error.code === Code.Aborted) { const hydrationMessage = error.message || 'Initial mirror clone is running' - core.warning( + core.info( `[git-mirror] Another job is hydrating the git mirror cache: ${hydrationMessage}` ) - core.warning( + core.info( '[git-mirror] No sticky disk will be mounted for this run; checkout will clone directly from GitHub onto the runner disk (no mirror cache). The mirror cache will be available on subsequent runs once hydration completes.' ) return { diff --git a/src/git-source-provider.ts b/src/git-source-provider.ts index 42a4dde..f4c9d45 100644 --- a/src/git-source-provider.ts +++ b/src/git-source-provider.ts @@ -172,7 +172,7 @@ async function getSourceInner( // Check if hydration is in progress - another job is doing the initial git clone --mirror if (cacheInfo.hydrationInProgress) { - // Warning already logged by setupCache, just fall back to standard checkout + // Already logged by setupCache, just fall back to standard checkout report.serving_mode = 'fallback-contention' cacheInfo = null core.endGroup()