-
Notifications
You must be signed in to change notification settings - Fork 867
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
chore: bump k8s versions to 1.29 #3494
Conversation
Signed-off-by: Zach Aller <[email protected]>
Go Published Test Results2 160 tests 2 160 ✅ 2m 53s ⏱️ Results for commit 9470afc. ♻️ This comment has been updated with latest results. |
E2E Tests Published Test Results 6 files 6 suites 5h 28m 13s ⏱️ For more details on these failures, see this check. Results for commit 9470afc. ♻️ This comment has been updated with latest results. |
Signed-off-by: Zach Aller <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #3494 +/- ##
==========================================
+ Coverage 81.83% 82.87% +1.03%
==========================================
Files 135 135
Lines 20688 17142 -3546
==========================================
- Hits 16931 14206 -2725
+ Misses 2883 2043 -840
- Partials 874 893 +19 ☔ View full report in Codecov by Sentry. |
Signed-off-by: Zach Aller <[email protected]>
Signed-off-by: Zach Aller <[email protected]>
Signed-off-by: Zach Aller <[email protected]>
Signed-off-by: Zach Aller <[email protected]>
Quality Gate passedIssues Measures |
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
matchLabelKeys |
Bump k8s client libs, had to adjust some timing on one test because the new version of k8s seems a bit faster at creating the job for analysis. The test while checking for 1 is valid though. Had to bump prometheus deps down because of kubernetes/kubernetes#123547 (comment)