-
Notifications
You must be signed in to change notification settings - Fork 123
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
Running kubelet v1.26 fixes (deprecated args) #259
Conversation
Depreacated arguments removed: --log-dir=/var/log/kubelet --logtostderr=false
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.
I have also tested this. Indeed these arguments does not allow kubelet to start on k8s v1.26. @jsturtevant can you take a look as well?
For me it is /lgtm
I see they are marked as deprecated but I don't seem them being removed in the 1.26 docs (https://kubernetes.io/docs/reference/command-line-tools-reference/kubelet/)? Could you link to where they are removed and causing issues starting? Otherwise looks good. |
@jsturtevant They said here that |
[version]$CurrentVersion = $($KubernetesVersion.Split("v") | Select -Index 1) | ||
[version]$V1_24_Version = '1.24' | ||
if ($CurrentVersion -lt $V1_24_Version) { | ||
$cmd_commands = $cmd_commands + "--network-plugin=cni " + "--image-pull-progress-deadline=20m " | ||
} | ||
[version]$V1_26_Version = '1.26' | ||
if ($CurrentVersion -lt $V1_26_Version) { | ||
$cmd_commands += ("--log-dir=/var/log/kubelet ", "--logtostderr=false ") |
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.
nit: ideally, it would have been consistent if the $V1_24_Version
check and appending above.
I see in the v1.26 release notes (https://github.com/kubernetes/kubernetes/blob/master/CHANGELOG/CHANGELOG-1.26.md): /lgtm |
thanks for the links @claudiubelu. looks like the docs need an update /approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: a4099181, fabi200123, jsturtevant 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 |
created kubernetes/website#38599 to track updating the docs |
Reason for PR:
Deprecated arguments removed: --log-dir=/var/log/kubelet --logtostderr=false
Requirements