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.

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

kubelet deps #623

Merged
merged 2 commits into from
Nov 8, 2023
Merged

kubelet deps #623

merged 2 commits into from
Nov 8, 2023

Conversation

rrotter
Copy link
Member

@rrotter rrotter commented Oct 13, 2023

Install dependencies and configuration to allow initialization of new vms with kubelet on Debian Bullseye

@rrotter rrotter requested review from daaang, antmoth and umjnega October 13, 2023 23:52
@rrotter rrotter added the use_fast_ci Use large runner for CI label Oct 13, 2023
@rrotter rrotter marked this pull request as draft October 14, 2023 00:09
@rrotter rrotter force-pushed the kubelet_deps branch 2 times, most recently from 25c080f to 7a26cbc Compare October 20, 2023 16:19
@rrotter
Copy link
Member Author

rrotter commented Oct 20, 2023

Test k8s cluster nodes seem happy with this so far. Fine after applying, still fine after a reboot of the nodes (tried at least one each of controller, etcd, and worker nodes).

@rrotter rrotter marked this pull request as ready for review October 20, 2023 17:16
@rrotter rrotter force-pushed the kubelet_deps branch 2 times, most recently from aabec82 to 41f0031 Compare October 25, 2023 19:41
@rrotter
Copy link
Member Author

rrotter commented Oct 25, 2023

This actually wasn't working on the worker nodes. Added the full default containerd config and squashed.

per Kubernetes documentation, kubelet needs:
- br_netfilter kmod loaded
- sysctl settings enabled:
  - sysctl net.bridge.bridge-nf-call-iptables
  - net.bridge.bridge-nf-call-ip6tables
  - net.ipv4.ip_forward

(https://kubernetes.io/docs/setup/production-environment/container-runtimes/)
Default containerd settings, modulo "SystemdCgroup = true". That and
"disabled_plugins = []" are the critical settings, however the many of the
so-called defaults in this file must be set explicitly or containerd does not
work properly. The output of `containerd config default` is known to change
over time, but it's not clear whether those changes are expected to need to be
imported.
@rrotter rrotter merged commit d2b047b into master Nov 8, 2023
1 check passed
@antmoth antmoth deleted the kubelet_deps branch February 5, 2024 20:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
use_fast_ci Use large runner for CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant