Skip to content

Conversation

@pacoxu
Copy link
Member

@pacoxu pacoxu commented Nov 4, 2025

@k8s-ci-robot k8s-ci-robot added this to the 1.35 milestone Nov 4, 2025
@netlify
Copy link

netlify bot commented Nov 4, 2025

👷 Deploy Preview for kubernetes-io-vnext-staging processing.

Name Link
🔨 Latest commit 6e6e7b6
🔍 Latest deploy log https://app.netlify.com/projects/kubernetes-io-vnext-staging/deploys/690981394bb4ec0008d07f24

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Nov 4, 2025
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign divya-mohan0209 for approval. For more information see the Code Review Process.

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added language/en Issues or PRs related to English language size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Nov 4, 2025
@netlify
Copy link

netlify bot commented Nov 4, 2025

Pull request preview available for checking

Built without sensitive environment variables

Name Link
🔨 Latest commit 6e6e7b6
🔍 Latest deploy log https://app.netlify.com/projects/kubernetes-io-main-staging/deploys/69098139b06aa20008ce663d
😎 Deploy Preview https://deploy-preview-53082--kubernetes-io-main-staging.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Member

@lmktfy lmktfy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks.

I recommend aligning closer to our style guide.
Also, make it clear that this advice is for Linux nodes. Other OS are available.

If you are using cgroups v1, you should migrate to cgroups v2.
To explicitly enable cgroups v1 support for kubelet v1.35 or newer,
you must set the kubelet configuration option 'FailCgroupV1' to 'false'. You must also explicitly skip this validation.
For more information, see https://git.k8s.io/enhancements/keps/sig-node/5573-remove-cgroup-v1.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should link to the docs about the deprecation, not to the KEP.

If you are using cgroups v1, you should migrate to cgroups v2.
To explicitly enable cgroups v1 support for kubelet v1.35 or newer,
you must set the kubelet configuration option 'FailCgroupV1' to 'false'. You must also explicitly skip this validation.
For more information, see https://git.k8s.io/enhancements/keps/sig-node/5573-remove-cgroup-v1.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should link to the docs about the deprecation, not to the KEP.

### Upgrade kubelet and kubectl

{{< note >}}
FailCgroupV1 is a kubelet configuration option that is set to 'true' by default.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Style and wording nit:

Suggested change
FailCgroupV1 is a kubelet configuration option that is set to 'true' by default.
On Linux nodes, the kubelet defaults to supporting only cgroups v2.
For Kubernetes {{< skew currentVersion >}} the `FailCgroupV1` kubelet configuration option is set to true by default.

### Upgrade kubelet and kubectl

{{< note >}}
FailCgroupV1 is a kubelet configuration option that is set to 'true' by default since v1.35.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Style and wording nit:

Suggested change
FailCgroupV1 is a kubelet configuration option that is set to 'true' by default since v1.35.
On Linux nodes, the kubelet defaults to supporting only cgroups v2.
For Kubernetes {{< skew currentVersion >}} the `FailCgroupV1` kubelet configuration option is set to true by default.

The cgroups v1 support is deprecated and will be removed in a future release.
If you are using cgroups v1, you should migrate to cgroups v2.
To explicitly enable cgroups v1 support for kubelet v1.35 or newer,
you must set the kubelet configuration option 'FailCgroupV1' to 'false'. You must also explicitly skip this validation.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
you must set the kubelet configuration option 'FailCgroupV1' to 'false'. You must also explicitly skip this validation.
you **must** set the kubelet configuration option `FailCgroupV1` to false. You must also explicitly skip this kubeadm validation.

The cgroups v1 support is deprecated and will be removed in a future release.
If you are using cgroups v1, you should migrate to cgroups v2.
To explicitly enable cgroups v1 support for kubelet v1.35 or newer,
you must set the kubelet configuration option 'FailCgroupV1' to 'false'. You must also explicitly skip this validation.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
you must set the kubelet configuration option 'FailCgroupV1' to 'false'. You must also explicitly skip this validation.
you **must** set the kubelet configuration option `FailCgroupV1` to false. You must also explicitly skip this kubeadm validation.


{{< note >}}
FailCgroupV1 is a kubelet configuration option that is set to 'true' by default since v1.35.
If you are using cgroups v1, kubelet will fail to start if the FailCgroupV1 is not set..
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
If you are using cgroups v1, kubelet will fail to start if the FailCgroupV1 is not set..
If you are using cgroups v1 and the `FailCgroupV1` setting is not set to true,
the kubelet reports an error and fails to start.


{{< note >}}
FailCgroupV1 is a kubelet configuration option that is set to 'true' by default.
If you are using cgroups v1, kubelet will fail to start if the FailCgroupV1 is not set.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
If you are using cgroups v1, kubelet will fail to start if the FailCgroupV1 is not set.
If you are using cgroups v1 and the `FailCgroupV1` setting is not set to true,
the kubelet reports an error and fails to start.

Copy link
Member

@neolit123 neolit123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

instead of including similar text in multiple places, can we include a note that links to a central blog post or a document that covers the problem?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. language/en Issues or PRs related to English language size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants