Merge pull request #15 from sbomify/fix/always-cleanup

Always run SBOM cleanup for docker/chainguard sources

Author: vpetersson

.github/workflows/sbom-builder.yml

@@ -113,20 +113,28 @@ jobs:
             echo "product_release=[${RELEASE_ARRAY}]" >> $GITHUB_OUTPUT
           fi
 
-          # Build PURL for TEA dedup lookup
+          # Build PURLs for TEA dedup and component PURL
           SOURCE_TYPE=$(yq -r '.source.type // ""' "$CONFIG")
           IMAGE=$(yq -r '.source.image // ""' "$CONFIG")
           REGISTRY=$(yq -r '.source.registry // ""' "$CONFIG")
           REPO=$(yq -r '.source.repo // ""' "$CONFIG")
           case "$SOURCE_TYPE" in
             docker)
-              echo "purl=pkg:docker/${IMAGE}@${VERSION}" >> "$GITHUB_OUTPUT" ;;
+              echo "purl=pkg:docker/${IMAGE}@${VERSION}" >> "$GITHUB_OUTPUT"
+              echo "purl_base=pkg:docker/${IMAGE}" >> "$GITHUB_OUTPUT"
+              ;;
             chainguard)
-              echo "purl=pkg:oci/${REGISTRY}/${IMAGE}@${VERSION}" >> "$GITHUB_OUTPUT" ;;
+              echo "purl=pkg:oci/${REGISTRY}/${IMAGE}@${VERSION}" >> "$GITHUB_OUTPUT"
+              echo "purl_base=pkg:oci/${REGISTRY}/${IMAGE}" >> "$GITHUB_OUTPUT"
+              ;;
             github_release|lockfile)
-              echo "purl=pkg:github/${REPO}@${VERSION}" >> "$GITHUB_OUTPUT" ;;
+              echo "purl=pkg:github/${REPO}@${VERSION}" >> "$GITHUB_OUTPUT"
+              echo "purl_base=pkg:github/${REPO}" >> "$GITHUB_OUTPUT"
+              ;;
             *)
-              echo "purl=" >> "$GITHUB_OUTPUT" ;;
+              echo "purl=" >> "$GITHUB_OUTPUT"
+              echo "purl_base=" >> "$GITHUB_OUTPUT"
+              ;;
           esac
 
       - name: Cache fetched SBOM
@@ -181,6 +189,8 @@ jobs:
           digest=$(cat image-digest.txt)
           echo "image_digest=${digest}" >> "$GITHUB_OUTPUT"
           echo "image_digest_safe=${digest//:/-}" >> "$GITHUB_OUTPUT"
+          purl_base="${{ steps.config.outputs.purl_base }}"
+          echo "component_purl=${purl_base}@${digest}" >> "$GITHUB_OUTPUT"
 
       - name: Upload input artifact
         if: always() && hashFiles('sbom.json') != ''
@@ -200,6 +210,8 @@ jobs:
       - name: Build SBOM (from existing SBOM)
         if: steps.config.outputs.component_id != '' && steps.config.outputs.source_type != 'lockfile'
         uses: sbomify/sbomify-action@master
+        with:
+          component-purl: ${{ steps.digest.outputs.component_purl || steps.config.outputs.purl }}
         env:
           TOKEN: ${{ secrets.SBOMIFY_TOKEN }}
           COMPONENT_ID: ${{ steps.config.outputs.component_id }}
@@ -214,6 +226,8 @@ jobs:
       - name: Build SBOM (from lockfile)
         if: steps.config.outputs.component_id != '' && steps.config.outputs.source_type == 'lockfile'
         uses: sbomify/sbomify-action@master
+        with:
+          component-purl: ${{ steps.config.outputs.purl }}
         env:
           TOKEN: ${{ secrets.SBOMIFY_TOKEN }}
           COMPONENT_ID: ${{ steps.config.outputs.component_id }}
@@ -291,6 +305,8 @@ jobs:
           steps.config.outputs.component_id != '' && steps.config.outputs.source_type != 'lockfile'
           && !inputs.dry_run && steps.upload-decision.outputs.should_upload == 'true'
         uses: sbomify/sbomify-action@master
+        with:
+          component-purl: ${{ steps.digest.outputs.component_purl || steps.config.outputs.purl }}
         env:
           TOKEN: ${{ secrets.SBOMIFY_TOKEN }}
           COMPONENT_ID: ${{ steps.config.outputs.component_id }}
@@ -306,6 +322,8 @@ jobs:
           steps.config.outputs.component_id != '' && steps.config.outputs.source_type == 'lockfile'
           && !inputs.dry_run && steps.upload-decision.outputs.should_upload == 'true'
         uses: sbomify/sbomify-action@master
+        with:
+          component-purl: ${{ steps.config.outputs.purl }}
         env:
           TOKEN: ${{ secrets.SBOMIFY_TOKEN }}
           COMPONENT_ID: ${{ steps.config.outputs.component_id }}
@@ -316,10 +334,9 @@ jobs:
           UPLOAD: true
           PRODUCT_RELEASE: ${{ steps.config.outputs.product_release }}
 
-      - name: Cleanup old SBOMs
+      - name: Cleanup old SBOMs and releases
         if: >-
           steps.config.outputs.component_id != '' && !inputs.dry_run
-          && steps.upload-decision.outputs.should_upload == 'true'
           && (steps.config.outputs.source_type == 'docker'
               || steps.config.outputs.source_type == 'chainguard')
         run: |
@@ -328,6 +345,10 @@ jobs:
           sbomify_cleanup_old_sboms \
             "${{ steps.config.outputs.component_id }}" \
             "${{ steps.digest.outputs.image_digest }}"
+          if [[ -n "${{ steps.config.outputs.product_id }}" ]]; then
+            sbomify_cleanup_versioned_releases \
+              "${{ steps.config.outputs.product_id }}"
+          fi
         env:
           SBOMIFY_TOKEN: ${{ secrets.SBOMIFY_TOKEN }}
 

scripts/lib/sbomify-api.sh

@@ -40,3 +40,21 @@ sbomify_cleanup_old_sboms() {
             "${SBOMIFY_API}/api/v1/sboms/sbom/${sbom_id}" || true
     done
 }
+
+# Remove versioned releases from a product (keep only "latest")
+# Usage: sbomify_cleanup_versioned_releases <product_id>
+sbomify_cleanup_versioned_releases() {
+    local product_id="$1"
+
+    local releases release_ids
+    releases=$(curl -fsSL -H "Authorization: Bearer ${SBOMIFY_TOKEN}" \
+        "${SBOMIFY_API}/api/v1/releases?product_id=${product_id}")
+    release_ids=$(echo "$releases" | jq -r \
+        '.items[] | select(.is_latest != true) | .id')
+
+    for release_id in $release_ids; do
+        log_info "Removing versioned release ${release_id} from product ${product_id}"
+        curl -fsSL -X DELETE -H "Authorization: Bearer ${SBOMIFY_TOKEN}" \
+            "${SBOMIFY_API}/api/v1/releases/${release_id}" || true
+    done
+}