Skip to content

ci: make the vuln scan simpler #2113

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jul 31, 2025
Merged

Conversation

luhring
Copy link
Member

@luhring luhring commented Jul 30, 2025

We were fetching advisory data as an input to a filtered scan, similar to what we'd do in a packages repo, but that's really unnecessary here since we're not concerned with how many vulns are in the output, we just care that the scanning operation itself doesn't blow up. For the same reason, we can remove the strict --require-zero flag that would only serve as a nuisance for melange changes.

cc: @jonjohnsonjr

luhring and others added 2 commits July 30, 2025 15:03
We were fetching advisory data as an input to a filtered scan, similar to what we'd do in a packages repo, but that's really unnecessary here since we're not concerned with how many vulns are in the output, we just care that the scanning operation itself doesn't blow up. For the same reason, we can remove the strict --require-zero flag that would only serve as a nuisance for melange changes.

Signed-off-by: Dan Luhring <[email protected]>
@luhring luhring merged commit 8e0e6af into chainguard-dev:main Jul 31, 2025
60 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants