-
Notifications
You must be signed in to change notification settings - Fork 883
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: Update k8s version to 1.30 #3748
Conversation
6ee7b62
to
a548264
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #3748 +/- ##
==========================================
- Coverage 83.96% 83.95% -0.02%
==========================================
Files 162 162
Lines 18508 18508
==========================================
- Hits 15540 15538 -2
- Misses 2106 2107 +1
- Partials 862 863 +1 ☔ View full report in Codecov by Sentry. |
Published E2E Test Results 4 files 4 suites 3h 18m 54s ⏱️ For more details on these failures, see this check. Results for commit 870c629. ♻️ This comment has been updated with latest results. |
Published Unit Test Results2 257 tests 2 257 ✅ 2m 59s ⏱️ Results for commit 870c629. ♻️ This comment has been updated with latest results. |
a548264
to
217c733
Compare
We should remove on on the tail end like 1.26. |
Signed-off-by: nikoshet <[email protected]>
Signed-off-by: Zach Aller <[email protected]>
5f7f06e
to
870c629
Compare
Quality Gate passedIssues Measures |
Any plan in near future to support K8S v1.30. We want to upgrade the k8s and we are blocked. |
@mansing2 I would not let what clients Rollouts uses cause you to not upgrade your k8s cluster. As long as you do not use any of the new pod spec fields if there are any it will be ok we use very little k8s feature set. The only big one being the embedded pod spec into the rollout resource. |
This was resolved in #3858 |
Checklist:
"fix(controller): Updates such and such. Fixes #1234"
.