src: only resolve buildref on success

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

Hide file tree

Showing 3 changed files with 7 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,6 @@ 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] : []
-          });
           await shutdownBuildkitd();
           core.info('Shutdown buildkitd');
           for (let attempt = 1; attempt <= 3; attempt++) {
@@ -759,6 +755,10 @@ actionsToolkit.run(
           }
           core.info('Unmounted device');
           if (!buildError) {
+            const buildxHistory = new BuildxHistory();
+            const exportRes = await buildxHistory.export({
+              refs: ref ? [ref] : []
+            });
             await reportBuildCompleted(exportRes, builderInfo.buildId, ref, buildDurationSeconds);
           } else {
             try {
@@ -862,4 +862,4 @@ function buildSummaryEnabled(): boolean {
     return Util.parseBool(process.env.DOCKER_BUILD_SUMMARY);
   }
   return true;
-}
+}