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

Migrate Kubelet --container-runtime-endpoint to kubelet.config.k8s.io or remove the flag #61674

Closed
mtaufen opened this issue Mar 25, 2018 · 12 comments
Assignees
Labels
area/kubelet kind/feature Categorizes issue or PR as related to a new feature. lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness. priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. sig/node Categorizes an issue or PR as relevant to SIG Node. triage/accepted Indicates an issue or PR is ready to be actively worked on.

Comments

@mtaufen
Copy link
Contributor

mtaufen commented Mar 25, 2018

Flag name: container-runtime-endpoint

Help text:

[Experimental] The endpoint of remote runtime service. Currently unix socket is supported on Linux, and tcp is supported on windows. Examples:'unix:///var/run/dockershim.sock', 'tcp://localhost:3735'


This is part of migrating the Kubelet command-line to a Kubernetes-style API.

The --container-runtime-endpoint flag should either be migrated to the Kubelet's kubelet.config.k8s.io
API group, or simply removed from the Kubelet.

If this could be considered an instance-specific flag, or a descriptor of local topology
managed by the Kubelet, see: #61647.

If this flag is only registered in os-specific builds, see: #61649.

As --container-runtime-endpoint is an alpha/experimental flag, the feature it configures must either be
feature-gated, or graduated from alpha/experimental status prior to the migration.

@sig-node-pr-reviews @sig-node-api-reviews

/assign @mtaufen
/sig node
/kind feature
/priority important-soon
/milestone v1.11
/status approved-for-milestone

@k8s-ci-robot k8s-ci-robot added status/approved-for-milestone sig/node Categorizes an issue or PR as relevant to SIG Node. labels Mar 25, 2018
@k8s-ci-robot k8s-ci-robot added this to the v1.11 milestone Mar 25, 2018
@k8s-ci-robot k8s-ci-robot added kind/feature Categorizes issue or PR as related to a new feature. priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. labels Mar 25, 2018
@k8s-github-robot
Copy link

[MILESTONENOTIFIER] Milestone Issue Needs Attention

@mtaufen @yujuhong @kubernetes/sig-node-misc

Action required: During code slush, issues in the milestone should be in progress.
If this issue is not being actively worked on, please remove it from the milestone.
If it is being worked on, please add the status/in-progress label so it can be tracked with other in-flight issues.

Note: If this issue is not resolved or labeled as priority/critical-urgent by Tuesday, June 5th it will be moved out of the v1.11 milestone.

Issue Labels
  • sig/node: Issue will be escalated to these SIGs if needed.
  • priority/important-soon: Escalate to the issue owners and SIG owner; move out of milestone after several unsuccessful escalation attempts.
  • kind/feature: New functionality.
Help

@mtaufen
Copy link
Contributor Author

mtaufen commented May 31, 2018

/milestone clear

@fejta-bot
Copy link

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/lifecycle stale

@k8s-ci-robot k8s-ci-robot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Aug 29, 2018
@SmileSees
Copy link

SmileSees commented Sep 21, 2018

Hi @mtaufen , I was using --container-runtime-endpoint flag to integrate kubelet with containerd with k8s version 1.10.7.
And I want to upgrade k8s version to 1.11.3, similarly I am using --container-runtime-endpoint flag, but the kubelet service won't start, says:

Unknown lvalue '--runtime-cgroups' in section 'Service'

Unknown lvalue '--container-runtime-endpoint' in section 'Service'

Can you pls tell what is the replacement of this flag? Or how can I integrate kubelet with containerd at k8s 1.11.3 version?

@mtaufen
Copy link
Contributor Author

mtaufen commented Sep 21, 2018

The flag has not been removed from any Kubernetes version yet, so maybe that error is coming from somewhere else?

https://github.com/kubernetes/kubernetes/blob/master/cmd/kubelet/app/options/options.go#L379

@SmileSees
Copy link

Thanks @mtaufen, it seems to be the character encoding cause the issue. It is ok now.

@PatrickLang
Copy link
Contributor

FYI - #69516 adds npipe support (similar to a unix socket) for Windows

@fejta-bot
Copy link

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/lifecycle rotten

@k8s-ci-robot k8s-ci-robot added lifecycle/rotten Denotes an issue or PR that has aged beyond stale and will be auto-closed. and removed lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. labels Nov 14, 2018
@mtaufen
Copy link
Contributor Author

mtaufen commented Nov 16, 2018 via email

@k8s-ci-robot k8s-ci-robot added lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness. and removed lifecycle/rotten Denotes an issue or PR that has aged beyond stale and will be auto-closed. labels Nov 16, 2018
@swatisehgal
Copy link
Contributor

/triage accepted
/area kubelet

@k8s-ci-robot k8s-ci-robot added triage/accepted Indicates an issue or PR is ready to be actively worked on. area/kubelet labels Jun 24, 2021
@ehashman
Copy link
Member

SIG Node Bug Scrub has decided to close the 23 individual flag tracking issues in favour of using the unified tracker in #86843. We will list each issue here that we're closing and the associated flag.

/close

@k8s-ci-robot
Copy link
Contributor

@ehashman: Closing this issue.

In response to this:

SIG Node Bug Scrub has decided to close the 23 individual flag tracking issues in favour of using the unified tracker in #86843. We will list each issue here that we're closing and the associated flag.

/close

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/kubelet kind/feature Categorizes issue or PR as related to a new feature. lifecycle/frozen Indicates that an issue or PR should not be auto-closed due to staleness. priority/important-soon Must be staffed and worked on either currently, or very soon, ideally in time for the next release. sig/node Categorizes an issue or PR as relevant to SIG Node. triage/accepted Indicates an issue or PR is ready to be actively worked on.
Projects
None yet
Development

No branches or pull requests

10 participants