Kubelet drop in #1299
Kubelet drop in #1299
Conversation
We can now use a proper API for customizations. See follow-up commits. Signed-off-by: Victor Toso <[email protected]>
kubevirt-bot
added
the
dco-signoff: yes
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: 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 |
I've verified that in k8s-1.30, removing all options set in
KUBELET_EXTRA_ARS, would result in the following diff:
@@ -54 +53,0 @@
- "kubeletCgroups": "/systemd/system.slice",
@@ -87 +86 @@
- "failSwapOn": false,
+ "failSwapOn": true,
I've moved both kubeletCgroups and failSwapOn into kubelet's drop-in
config file and verified that it works and config is set.
I assume that --cgroup-driver=systemd is already the default in centos
stream or being set somewhere else, but to avoid regressions I've
included in the config file as well.
I don't know why --runtime-cgroups is not being shown in the node's
configz but it is the only command line option for the initial 4 that
is not marked as DEPRECATED in favor of Config file alternative, so
let's keep it as command line option, together with --config-dir.
Signed-off-by: Victor Toso <[email protected]>
victortoso
force-pushed
the
kubelet-drop-in
branch
from
f72745a to
9e3057c
Compare
|
@victortoso: The following tests failed, say
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-sigs/prow repository. I understand the commands that are listed here. |
What this PR does / why we need it:
We are using deprecated kubelet's command line options which have config alternative options. Seems to be known since b89e9a6. This, combined with the goal of enabling k8s feature gates such as DRA in #1251, the goal of this PR is to improve kubelet's config options and use drop-in feature.
Commit logs clarify further the changes.
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)format, will close the issue(s) when PR gets merged):Fixes #
Special notes for your reviewer:
This PR only focused on k8s-1.30 to validate the change. I can do here or in a new PR for k8s-1.31 too.
/cc @lyarwood
Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.
Release note: