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
Automated cherry pick of #79245: kunsupported cgroup setup causes kubelet to emit a warning #80121
Automated cherry pick of #79245: kunsupported cgroup setup causes kubelet to emit a warning #80121
Conversation
@saschagrunert: This PR is not for the master branch but does not have the To approve the cherry-pick, please ping the kubernetes/patch-release-team in a comment when ready. See also Kubernetes Patch Releases 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. |
Do you think we can change the release notes to something like:
Relates to: #80090 |
/retest |
/test pull-kubernetes-kubemark-e2e-gce-big |
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.
/test pull-kubernetes-kubemark-e2e-gce-big
/lgtm
+1 to the note around updating the release note.
/test pull-kubernetes-kubemark-e2e-gce-big |
1 similar comment
/test pull-kubernetes-kubemark-e2e-gce-big |
/retest |
Hey @kubernetes/sig-release 👋 do you see a way to get this into the next 1.15 patch release? |
/kind bug |
It would help to understand in the future why we felt this change had to be cherry-picked. Production systems should run a unified hierarchy. If its to support testing 1.15 w/ hack/local-up-cluster that is fine, but Kubernetes systems should run unified hierarchy. /approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: derekwaynecarr, saschagrunert 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 |
I am getting this error on version 1.22.0 |
This fixed the error from showing for me. Change Docker from cgroupfs to systemd. Would this be good to add to the docs as a step for installation to do this? Of course, now I have a different error regarding the cni config uninitialized, but that's a different issue. create a file /etc/docker/daemon.json reference: #80121 |
@VR-Architect yeah the systemd cgroup driver should be preferred over cgroupfs. Unfortunately docker shim got deprecated a while ago, but having the information in the docs should be useful. |
Cherry pick of #79245 on release-1.15.
#79245: kunsupported cgroup setup causes kubelet to emit a warning
Release note