-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Update ingress-nginx image to v0.40.2 #9445
Conversation
Codecov Report
@@ Coverage Diff @@
## master #9445 +/- ##
=======================================
Coverage 29.18% 29.18%
=======================================
Files 171 171
Lines 10462 10462
=======================================
Hits 3053 3053
Misses 6985 6985
Partials 424 424 |
/ok-to-test |
kvm2 Driver Times for Minikube (PR 9445): 63.9s 61.2s 60.1s Averages Time Per Log
docker Driver Times for Minikube (PR 9445): 29.3s 29.0s 29.5s Averages Time Per Log
|
/ok-to-test |
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, just want to confirm that no ingress-related integration tests suddenly fail before merging.
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: aledbf, tstromberg 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 |
@tstromberg not sure if the errors are related to the ingress change |
Looks good to me. Thanks! |
No description provided.