-
Notifications
You must be signed in to change notification settings - Fork 443
Update all the kubernetes versions to 1.32 #5528
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
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #5528 +/- ##
=======================================
Coverage 53.26% 53.26%
=======================================
Files 272 272
Lines 29529 29529
=======================================
Hits 15730 15730
Misses 12984 12984
Partials 815 815 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
@mboersma are we still building flatcar images?
|
The CAPZ team never did build them, but the Flatcar team was building them in a community gallery. But AFAICT they stopped publishing a while back, once they got their preferred sysext approach working. |
I can revert the changes to these versions but should these tests be dropped if they aren't maintained? |
72ab994
to
f9d9762
Compare
/retest |
/test pull-cluster-api-provider-azure-apiversion-upgrade |
/retest |
I am seeing the same panic in the cloudprovider image: #5523 (comment) It seem that test is buggy /retest |
/retest |
Yes, probably. We could ask the Flatcar team for their opinions--I know they prefer |
failure looks like a flake on the azure cni test |
/retest |
1 similar comment
/retest |
Is the recent run of the test a flake ? Let's retest |
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
LGTM label has been added. Git tree hash: 0d53edd01c193fdc857bfc894fcae4bdefcb0f83
|
/test pull-cluster-api-provider-azure-e2e-workload-upgrade |
New changes are detected. LGTM label has been removed. |
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
8c10b38
to
108453d
Compare
/retest |
Signed-off-by: James Sturtevant <jsturtevant@gmail.com>
Signed-off-by: James Sturtevant <jsturtevant@gmail.com>
Signed-off-by: James Sturtevant <jsturtevant@gmail.com>
108453d
to
cfc3978
Compare
14f4604
to
cfc3978
Compare
/test pull-cluster-api-provider-azure-apiversion-upgrade |
/retest |
1 similar comment
/retest |
@jsturtevant: The following tests failed, say
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
What type of PR is this?
/kind cleanup
What this PR does / why we need it:
1.29 is EOL, this updates all reference to keep things fresh.
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Fixes #
Special notes for your reviewer:
TODOs:
Release note: