don't build label on missing package digests

dataaxiom · Dec 10, 2024 · 2d58aab · 2d58aab
1 parent 792f00f
commit 2d58aab
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 5 deletions.
diff --git a/dist/index.js b/dist/index.js
@@ -44272,6 +44272,7 @@ class CleanupTask {
                 label = `architecture: ${label}`;
             }
             else {
+                // unknown
                 // check if it's a buildx attestation
                 const manifest = await this.registry.getManifestByDigest(imageManifest.digest);
                 // kinda crude
@@ -44700,7 +44701,9 @@ class CleanupTask {
             if (manifest.manifests) {
                 for (const imageManifest of manifest.manifests) {
                     // call the buildLabel method which will prime manifest if its needed
-                    await this.buildLabel(imageManifest);
+                    if (this.packageRepo.getDigests().has(imageManifest)) {
+                        await this.buildLabel(imageManifest);
+                    }
                 }
             }
             // process tagged digests (referrers)
@@ -44713,7 +44716,9 @@ class CleanupTask {
                         const tagManifest = await this.registry.getManifestByDigest(tagDigest);
                         if (tagManifest.manifests) {
                             for (const manifestEntry of tagManifest.manifests) {
-                                await this.buildLabel(manifestEntry);
+                                if (this.packageRepo.getDigests().has(manifestEntry)) {
+                                    await this.buildLabel(manifestEntry);
+                                }
                             }
                         }
                     }

diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/src/cleanup-task.ts b/src/cleanup-task.ts
@@ -317,6 +317,7 @@ export class CleanupTask {
         }
         label = `architecture: ${label}`
       } else {
+        // unknown
         // check if it's a buildx attestation
         const manifest = await this.registry.getManifestByDigest(
           imageManifest.digest
@@ -814,7 +815,9 @@ export class CleanupTask {
       if (manifest.manifests) {
         for (const imageManifest of manifest.manifests) {
           // call the buildLabel method which will prime manifest if its needed
-          await this.buildLabel(imageManifest)
+          if (this.packageRepo.getDigests().has(imageManifest)) {
+            await this.buildLabel(imageManifest)
+          }
         }
       }
       // process tagged digests (referrers)
@@ -828,7 +831,9 @@ export class CleanupTask {
               await this.registry.getManifestByDigest(tagDigest)
             if (tagManifest.manifests) {
               for (const manifestEntry of tagManifest.manifests) {
-                await this.buildLabel(manifestEntry)
+                if (this.packageRepo.getDigests().has(manifestEntry)) {
+                  await this.buildLabel(manifestEntry)
+                }
               }
             }
           }