-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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
[kubernetes] Support Kubernetes v1.32.0 with RHEL8 #11885
[kubernetes] Support Kubernetes v1.32.0 with RHEL8 #11885
Conversation
f8ae2dc
to
d5ad991
Compare
{% if ansible_os_family == "RedHat" and ansible_distribution_major_version == "8" %} | ||
# Workaround for RHEL8: kernel version 4.18 is lower than Kubernetes system verification. | ||
# Remove this when [Deprecate support for RedHat 8 family hosts](https://github.com/kubernetes-sigs/kubespray/issues/11872) | ||
ignorePreflightErrors: | ||
- SystemVerification | ||
{% endif %} |
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.
I don't think we should automate this. This should be in the doc but given the potential problems, it should be an explicit decision by the users.
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.
It has been changed
d5ad991
to
351ba17
Compare
351ba17
to
1950602
Compare
1950602
to
1596d95
Compare
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.
Some suggestions
6244669
to
50ed396
Compare
a8861a8
to
d0b2cdf
Compare
6ae7e0e
to
9778e3e
Compare
/retest |
1 similar comment
/retest |
Signed-off-by: Kay Yan <kay.yan@daocloud.io>
9778e3e
to
fcfd669
Compare
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
Thank you!
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mzaian, tico88612, yankay 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:
Approvers can indicate their approval by writing |
…1885) * [kubernetes] Support Kubernetes v1.32.0 * add workaround for RHEL8 Signed-off-by: Kay Yan <kay.yan@daocloud.io> --------- Signed-off-by: Kay Yan <kay.yan@daocloud.io> Co-authored-by: Mohamed Zaian <mohamedzaian@gmail.com>
…1885) * [kubernetes] Support Kubernetes v1.32.0 * add workaround for RHEL8 Signed-off-by: Kay Yan <kay.yan@daocloud.io> --------- Signed-off-by: Kay Yan <kay.yan@daocloud.io> Co-authored-by: Mohamed Zaian <mohamedzaian@gmail.com>
What type of PR is this?
/kind feature
What this PR does / why we need it:
Similar to #11862 with RHEL8 workaround.
Just skip the kubeadm system verification when the OS is RHEL8.
There are also some discussion about the #11872.
Upgrade kubernetes versions
Add hashes for kubernetes 1.32.0
Make kubernetes v1.32.0 default
Set minimum required version to Kubernetes v1.30.x
Cleanup old hashes
workaround for the RHEL8
Which issue(s) this PR fixes:
Fixes #11856
Does this PR introduce a user-facing change?: