mirror of
https://github.com/docker/build-push-action.git
synced 2025-08-09 18:12:12 +00:00
src: only prune if buildkitd was spun up
This commit is contained in:
parent
4e97fd3608
commit
f9f71c9f11
2
dist/index.js
generated
vendored
2
dist/index.js
generated
vendored
File diff suppressed because one or more lines are too long
2
dist/index.js.map
generated
vendored
2
dist/index.js.map
generated
vendored
File diff suppressed because one or more lines are too long
34
src/main.ts
34
src/main.ts
@ -400,26 +400,25 @@ actionsToolkit.run(
|
||||
});
|
||||
}
|
||||
|
||||
// Prune buildkit cache to clean up unused layers before shutting down buildkitd.
|
||||
try {
|
||||
core.info('Pruning BuildKit cache');
|
||||
await pruneBuildkitCache();
|
||||
core.info('BuildKit cache pruned');
|
||||
} catch (error) {
|
||||
// Log warning but don't fail the cleanup
|
||||
core.warning(`Error pruning BuildKit cache: ${error.message}`);
|
||||
}
|
||||
|
||||
await leaveTailnet();
|
||||
|
||||
try {
|
||||
const {stdout} = await execAsync('pgrep buildkitd');
|
||||
if (stdout.trim()) {
|
||||
try {
|
||||
core.info('Pruning BuildKit cache');
|
||||
await pruneBuildkitCache();
|
||||
core.info('BuildKit cache pruned');
|
||||
} catch (error) {
|
||||
// Log warning but don't fail the cleanup
|
||||
core.warning(`Error pruning BuildKit cache: ${error.message}`);
|
||||
}
|
||||
|
||||
const buildkitdShutdownStartTime = Date.now();
|
||||
await shutdownBuildkitd();
|
||||
const buildkitdShutdownDurationMs = Date.now() - buildkitdShutdownStartTime;
|
||||
await reporter.reportMetric(Metric_MetricType.BPA_BUILDKITD_SHUTDOWN_DURATION_MS, buildkitdShutdownDurationMs);
|
||||
core.info('Shutdown buildkitd');
|
||||
} else {
|
||||
core.debug('No buildkitd process found running');
|
||||
}
|
||||
} catch (error) {
|
||||
if (error.code === 1) {
|
||||
@ -429,6 +428,8 @@ actionsToolkit.run(
|
||||
core.warning(`Error checking for buildkitd processes: ${error.message}`);
|
||||
}
|
||||
}
|
||||
|
||||
await leaveTailnet();
|
||||
try {
|
||||
// Run sync to flush any pending writes before unmounting.
|
||||
await execAsync('sync');
|
||||
@ -476,7 +477,13 @@ actionsToolkit.run(
|
||||
core.info('buildkitd.log contents:');
|
||||
core.info(buildkitdLog);
|
||||
} catch (error) {
|
||||
core.warning(`Failed to read buildkitd.log: ${error.message}`);
|
||||
// Only log warning if the file was expected to exist (builder setup completed)
|
||||
const sentinelPath = path.join('/tmp', 'builder-setup-complete');
|
||||
if (fs.existsSync(sentinelPath)) {
|
||||
core.warning(`Failed to read buildkitd.log: ${error.message}`);
|
||||
} else {
|
||||
core.debug(`buildkitd.log not found (builder setup incomplete): ${error.message}`);
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
@ -496,7 +503,6 @@ actionsToolkit.run(
|
||||
try {
|
||||
const {stdout} = await execAsync('pgrep buildkitd');
|
||||
if (stdout.trim()) {
|
||||
// Prune buildkit cache to clean up unused layers before shutting down buildkitd.
|
||||
try {
|
||||
core.info('Pruning BuildKit cache');
|
||||
await pruneBuildkitCache();
|
||||
|
Loading…
x
Reference in New Issue
Block a user