Skip to content

Conversation

@yiannistri
Copy link
Contributor

@yiannistri yiannistri commented Nov 5, 2024

What this PR does / why we need it:

We frequently get CI failures when scanning with trivy because there have been too many requests to download the vulnerability db from the rancher org (because many of its repos are using it).

This PR updates the Scan workflow to include additional repositories to be used if trivy fails to retrieve the vulnerability db from the primary source.

Which issue(s) this PR fixes
Issue #708

Special notes for your reviewer:

Source for fix: aquasecurity/trivy#7668 (reply in thread)

Checklist:

  • squashed commits into logical changes
  • includes documentation
  • adds unit tests
  • adds or updates e2e tests
  • backport needed

@yiannistri yiannistri force-pushed the 708-trivy-action-main branch from b50339e to f00d666 Compare November 5, 2024 17:12
@yiannistri yiannistri marked this pull request as ready for review November 5, 2024 17:45
@yiannistri yiannistri requested a review from a team as a code owner November 5, 2024 17:45
Copy link
Contributor

@salasberryfin salasberryfin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @yiannistri, LGTM.

@mjura mjura merged commit c193c48 into rancher:main Nov 6, 2024
6 checks passed
@yiannistri yiannistri deleted the 708-trivy-action-main branch November 6, 2024 09:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants