src: only resolve buildref on success

useblacksmith · Dec 1, 2024 · 9841eab · 9841eab
1 parent 99d48d0
commit 9841eab
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 7 deletions.
diff --git a/dist/index.js b/dist/index.js
diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/src/main.ts b/src/main.ts
@@ -738,10 +738,13 @@ actionsToolkit.run(
     await core.group('Cleaning up Blacksmith builder', async () => {
       if (builderInfo.addr) {
         try {
-          const buildxHistory = new BuildxHistory();
-          const exportRes = await buildxHistory.export({
-            refs: ref ? [ref] : []
-          });
+          let exportRes;
+          if (!buildError) {
+            const buildxHistory = new BuildxHistory();
+            exportRes = await buildxHistory.export({
+              refs: ref ? [ref] : []
+            });
+          }
           await shutdownBuildkitd();
           core.info('Shutdown buildkitd');
           for (let attempt = 1; attempt <= 3; attempt++) {
@@ -862,4 +865,4 @@ function buildSummaryEnabled(): boolean {
     return Util.parseBool(process.env.DOCKER_BUILD_SUMMARY);
   }
   return true;
-}
+}