Skip to content

fix(ci): stop the pending-release CVE diff going blind on a gating failure - #1509

Merged
mxsrc merged 1 commit into
mainfrom
security
Oct 20, 2026
Merged

mxsrc merged 1 commit into
mainfrom
security

Conversation

@mxsrc

@mxsrc mxsrc commented Oct 20, 2026

Copy link
Copy Markdown
Collaborator

trivy-branch never ran once grype-branch failed the job, since it had no if: condition of its own -- the step was skipped rather than run, so the pending-release diff treated an empty (not scanned) branch result as "no Trivy findings on branch", misclassifying any still-unfixed Trivy CVE in the release image as merely pending release.

Mark the two gating scans continue-on-error and fail the job explicitly afterwards instead, so a real finding no longer skips the informational scans, the diff, or the SBOM upload.

…ilure

trivy-branch never ran once grype-branch failed the job, since it had no
if: condition of its own -- the step was skipped rather than run, so the
pending-release diff treated an empty (not scanned) branch result as "no
Trivy findings on branch", misclassifying any still-unfixed Trivy CVE in
the release image as merely pending release.

Mark the two gating scans continue-on-error and fail the job explicitly
afterwards instead, so a real finding no longer skips the informational
scans, the diff, or the SBOM upload.
@github-actions github-actions Bot added the target/main Targets the development branch label Oct 20, 2026
@mxsrc
mxsrc requested a review from Hamdy-khader October 20, 2026 12:26
@mxsrc
mxsrc merged commit 919a2c5 into main Oct 20, 2026
21 checks passed
@mxsrc
mxsrc deleted the security branch October 20, 2026 15:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

target/main Targets the development branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants