Skip to content

Conversation

@jormundur00
Copy link
Member

@jormundur00 jormundur00 commented Nov 14, 2025

What does this PR do?

In this PR, we make the vulnerability scanner less strict by accepting newer versions of docker images if they have less or equal vulnerabilities to the current version (while currently we only accept if they have strictly less vulnerabilities).

Also in this PR, we fix org.graalvm.internal.tck.GrypeTask#getAllowedImagesFromMaster not working in the GitHub actions CI. Previously, whenever this method is reached in the CI, the task would fail with: fatal: invalid object name 'master'.. This happens because the master branch does not exist in the CI, while origin/master does. The change in this PR just made it so we check the origin/master when we fetch allowed images.

Fixes: #770

@jormundur00 jormundur00 self-assigned this Nov 14, 2025
@jormundur00 jormundur00 added bug Something isn't working docker Pull requests that update docker code labels Nov 14, 2025
@jormundur00 jormundur00 requested a review from matneu as a code owner November 14, 2025 08:09
@jormundur00 jormundur00 force-pushed the jormundur00/vulnerability-scanner-fix branch from 7635d65 to 3e95f70 Compare November 14, 2025 08:12
@jormundur00 jormundur00 requested a review from vjovanov November 14, 2025 08:22
@jormundur00 jormundur00 merged commit 5b57c28 into master Nov 14, 2025
512 of 515 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working docker Pull requests that update docker code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: Vulnerability scanner fails when the updated docker image has the same number of vulnerabilities as the old version

2 participants