Skip to content
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

Tasks to bump to Kubernetes v1.29 #9578

Closed
14 tasks done
killianmuldoon opened this issue Oct 19, 2023 · 19 comments
Closed
14 tasks done

Tasks to bump to Kubernetes v1.29 #9578

killianmuldoon opened this issue Oct 19, 2023 · 19 comments
Assignees
Labels
triage/accepted Indicates an issue or PR is ready to be actively worked on.

Comments

@killianmuldoon
Copy link
Contributor

killianmuldoon commented Oct 19, 2023

This issue is tracking the tasks that should be implemented after the Kubernetes minor release has been released.

Tasks

Prerequisites:

  • Decide which Cluster API release series will support the new Kubernetes version
    • If feasible we usually cherry-pick the changes back to the latest release series.

Supporting managing and running on the new Kubernetes version

This section contains tasks to update our book, e2e testing and CI to use and test the new Kubernetes version
as well as changes to Cluster API that we might have to make to support the new Kubernetes version. All of these
changes should be cherry-picked to all release series that will support the new Kubernetes version.

Using new Kubernetes dependencies

This section contains tasks to update Cluster API to use the latest Kubernetes Go dependencies and related topics
like using the right Go version and build images. These changes are only made on the main branch. We don't
need them in older releases as they are not necessary to manage workload clusters of the new Kubernetes version or
run the Cluster API controllers on the new Kubernetes version.

@k8s-ci-robot k8s-ci-robot added the needs-triage Indicates an issue or PR lacks a `triage/foo` label and requires one. label Oct 19, 2023
@killianmuldoon
Copy link
Contributor Author

Created now mainly to keep track of kubernetes/enhancements#4302

/triage accepted

@chrischdi
Copy link
Member

/assign

@furkatgofurov7
Copy link
Member

@chrischdi Hey, one question: am I correct to assume that once this is completed, we will be supporting k8s 1.29 in the release-1.6 branch, i.e starting from v1.6.x, although there is a note in the book:

When a new Kubernetes minor release is available, we will try to support it in an upcoming Cluster API patch release (although only in the latest supported Cluster API minor release). But this depends on the changes made in Kubernetes, if the corresponding required changes in Cluster API are too invasive we won’t backport the support and users have to wait for the next Cluster API minor release.

meaning it might not be the case always?

@chrischdi
Copy link
Member

chrischdi commented Dec 12, 2023

Yes it is. With this issue we should also add the necessary jobs for the v1.6 release to get test coverage.

Especially via this part:

  • For the release branch of the latest supported Cluster API minor release:
    • periodics & presubmits:
      • Adust the -latest upgrade jobs to upgrade to the new Kubernetes version instead of latest.

If any fixes are required they should get cherry-picked and released with the next round of patch releases (although normally there is no fix necessary).

@chrischdi
Copy link
Member

chrischdi commented Dec 12, 2023

Also note:

All of these changes should be cherry-picked to all release series that will support the new Kubernetes version.

@kashifest
Copy link
Contributor

@chrischdi is there any timeline for the support for k8s v1.29 in CAPI, I think k8s v1.29 lands tomorrow, so can we expect the support in CAPI branches before the xmas or after? And also if theres any other blocker for example controller-runtime release? Meanwhile if any support is needed for the subtasks listed here in the issue, we can help.

@chrischdi
Copy link
Member

@chrischdi is there any timeline for the support for k8s v1.29 in CAPI, I think k8s v1.29 lands tomorrow, so can we expect the support in CAPI branches before the xmas or after? And also if theres any other blocker for example controller-runtime release? Meanwhile if any support is needed for the subtasks listed here in the issue, we can help.

I want this to land (if possible) before xmas, however a supporting release will be v1.6.1 which is planned for January afaik.

@chrischdi
Copy link
Member

Prerequisites:

  • Decide which Cluster API release series will support the new Kubernetes version
    • If feasible we usually cherry-pick the changes back to the latest release series.

All work will be cherry-picked to release-1.6 so hopefully the upcoming release v1.6.1 will be the first one officially supporting Kubernetes v1.29

@chrischdi
Copy link
Member

chrischdi commented Dec 13, 2023

Sumarizing the above open tasks:

Supporting managing and running on the new Kubernetes version

Using new Kubernetes dependencies

After release of v1.6.1 (and if v1.6.1 supports Kubernetes v1.29):

  • bump WorkloadKubernetesVersion for test When testing clusterctl upgrades using ClusterClass (v1.6=>current) [ClusterClass]

After release-28 kubekins image includes go 1.21:

@kashifest
Copy link
Contributor

@chrischdi are waiting for controller-runtime and controller-tools release on this ?

@sbueringer
Copy link
Member

Yes.

@kashifest
Copy link
Contributor

Do we have any expected timeline ?

@sbueringer
Copy link
Member

No

@sbueringer
Copy link
Member

Do we have any expected timeline ?

@kashifest CR has been released and bump PR is now ready for review #9964

@sbueringer
Copy link
Member

sbueringer commented Jan 16, 2024

Looks like we didn't add 1.29 to versions.md on release-1.6 yet (probably intentional).

We should make sure versions.md is updated once #9971 merges on main

(added a sub-task)

@sbueringer
Copy link
Member

@chrischdi I think now we only have one sub-task for docs and "Bump golang on release-1.6 and release-1.5 branches"

I would move the latter to a separate issue so we can close the current one, WDYT?

@chrischdi
Copy link
Member

Sounds good 👍

@sbueringer
Copy link
Member

Moved to #10025

Otherwise we're done. Thx everyone :)

/close

@k8s-ci-robot
Copy link
Contributor

@sbueringer: Closing this issue.

In response to this:

Moved to #10025

Otherwise we're done. Thx everyone :)

/close

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/test-infra repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
triage/accepted Indicates an issue or PR is ready to be actively worked on.
Projects
None yet
Development

No branches or pull requests

6 participants