-
Notifications
You must be signed in to change notification settings - Fork 113
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
Retry trivy when it fails to download the vulnerability database due to rate-limiting #1700
Retry trivy when it fails to download the vulnerability database due to rate-limiting #1700
Conversation
dd7a753
to
a0871b9
Compare
@HeavyWombat just seeing in your PR this error: Should I do the retry also for this error (and remove the TOOMANYREQUESTS check) ? Edit: is also in my PR. :-) |
a0871b9
to
8dd929f
Compare
8dd929f
to
c9a5816
Compare
…to rate-limiting Signed-off-by: Sascha Schwarze <schwarzs@de.ibm.com>
c9a5816
to
caaac89
Compare
This looks promising, only one failure in the integration test suite which is unrelated and which is a flaky test that I address here: #1701 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: HeavyWombat The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Changes
Part of #1689
This changes the image processing step to perform a simple retry in case of Trivy failures due to rate limiting.
Submitter Checklist
See the contributor guide
for details on coding conventions, github and prow interactions, and the code review process.
Release Notes