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.

Sign up for GitHub

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

Jump to bottom

Set the kubelet cgroup driver to systemd (v2) #818

Merged
merged 4 commits into from
Apr 27, 2022
Merged

Set the kubelet cgroup driver to systemd (v2) #818

merged 4 commits into from
Apr 27, 2022

Conversation

afbjorklund
Copy link
Member

@afbjorklund afbjorklund commented Apr 25, 2022 •
edited
Loading

See https://kubernetes.io/docs/tasks/administer-cluster/kubeadm/configure-cgroup-driver/

NAME       STATUS   ROLES                  AGE     VERSION   INTERNAL-IP    EXTERNAL-IP   OS-IMAGE           KERNEL-VERSION      CONTAINER-RUNTIME
lima-k8s   Ready    control-plane,master   2m32s   v1.23.6   192.168.5.15   <none>        Ubuntu 22.04 LTS   5.15.0-25-generic   containerd://1.6.2

Closes #817

@AkihiroSuda
Copy link
Member

Thanks, but please sign the commit for DCO
https://github.com/apps/dco

(run git commit -a -s --amend, and make sure that the Signed-off-by: NAME <EMAIL> line with your real name is included in the commit message)

@afbjorklund
Copy link
Member Author

afbjorklund commented Apr 25, 2022 •
edited
Loading

I don't think this change is needed, it was already defaulting to cgroups v2 (systemd).

We could re-open it and move to YAML configuration anyway, but it's much more verbose

@afbjorklund afbjorklund reopened this Apr 26, 2022
@afbjorklund
Set the containerd cgroup driver to systemd
c8d6216 
Signed-off-by: Anders F Björklund <anders.f.bjorklund@gmail.com>
@afbjorklund
Convert the kubeadm config from flags to yaml
386d32a 
Signed-off-by: Anders F Björklund <anders.f.bjorklund@gmail.com>
@afbjorklund
Set the kubelet cgroup driver to systemd (v2)
f1d2a2d 
Signed-off-by: Anders F Björklund <anders.f.bjorklund@gmail.com>
@afbjorklund afbjorklund mentioned this pull request Apr 26, 2022
Closed
@AkihiroSuda AkihiroSuda added this to the v0.10.1 (tentative) milestone Apr 27, 2022
@afbjorklund
Make sure that containerd config has version 2
c7be769 
Signed-off-by: Anders F Björklund <anders.f.bjorklund@gmail.com>
AkihiroSuda
AkihiroSuda approved these changes Apr 27, 2022
View reviewed changes
Copy link
Member

@AkihiroSuda AkihiroSuda left a comment

Choose a reason for hiding this comment

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

Thanks

@AkihiroSuda AkihiroSuda merged commit 2b88071 into lima-vm:master Apr 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AkihiroSuda AkihiroSuda AkihiroSuda approved these changes

Assignees
No one assigned
Labels
kubernetes
Projects
None yet
Milestone
v0.11.0
Development

Successfully merging this pull request may close these issues.

k8s.yaml example does not work after upgrade to Ubuntu 22.04
2 participants
@afbjorklund @AkihiroSuda