search

kubernetes-sigs / kubespray

Deploy a Production Ready Kubernetes Cluster
Apache License 2.0
16.2k stars 6.48k forks source link

Feat: kubeadm v1beta4 support #11674

Closed tico88612 closed 2 weeks ago

tico88612 commented 3 weeks ago

What type of PR is this?

/kind feature

What this PR does / why we need it:

Starting from Kubernetes v1.31, kubeadm v1beta4 is supported, and all subsequent versions use v1beta4 for deploying clusters.

Which issue(s) this PR fixes:

Fixes #10935

Special notes for your reviewer:

This is currently a draft PR, and the changes may conflict with #11633, so #11633 needs to be reviewed first.

Does this PR introduce a user-facing change?:

Feat: Kubeadm config API support v1beta4
k8s-ci-robot commented 3 weeks ago

Skipping CI for Draft Pull Request. If you want CI signal for your change, please convert it to an actual PR. You can still manually trigger a test run with /test all

tico88612 commented 2 weeks ago

/label tide/merge-method-merge /ok-to-test

tico88612 commented 2 weeks ago

/retest

tico88612 commented 2 weeks ago

/retest

VannTen commented 2 weeks ago

Looking at that diff makes me think we should definitely not template it this way but juste mostly mirror the data struct and feed it to to_yaml, but oh well, one more TODO :laughing:

/lgtm /approve

k8s-ci-robot commented 2 weeks ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: tico88612, VannTen

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: - ~~[OWNERS](https://github.com/kubernetes-sigs/kubespray/blob/master/OWNERS)~~ [VannTen] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment