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

FR: provide a flag to choose CNI during kubeadm init #2408

Closed
medyagh opened this issue Mar 12, 2021 · 3 comments
Closed

FR: provide a flag to choose CNI during kubeadm init #2408

medyagh opened this issue Mar 12, 2021 · 3 comments
Labels
sig/node Categorizes an issue or PR as relevant to SIG Node.

Comments

@medyagh
Copy link
Member

medyagh commented Mar 12, 2021

not sure if this should be a kubelet issue or kubeadm, please advice I will move to the right repo,
based on this comment
kubernetes/minikube#10788 (comment)

in minikube we noticed that during kubeadm init, kuberentes picks up the first alphabetical CNI that could cause some unexpected issues.

If there are multiple CNI configuration files in the directory, the kubelet
uses the configuration file that comes first by name in lexicographic order.
https://kubernetes.io/docs/concepts/extend-kubernetes/compute-storage-net/network-plugins/#cni

would it be possible to have a flag to pass to kubeadm init to specify the CNI explicitly ?

@medyagh medyagh changed the title provide a flag to choose CNI provide a flag to choose CNI during kubeadm init Mar 12, 2021
@medyagh medyagh changed the title provide a flag to choose CNI during kubeadm init FR:provide a flag to choose CNI during kubeadm init Mar 12, 2021
@medyagh medyagh changed the title FR:provide a flag to choose CNI during kubeadm init FR: provide a flag to choose CNI during kubeadm init Mar 12, 2021
@neolit123
Copy link
Member

@medyagh kubeadm allows the users to configure the kubelet via KubeletConfiguration.
AFAIK the field to control this CNI related behavior is missing there, so that is a kubelet issue.

you can log a ticket for this in k/k and ping me on it, but it's for /sig node and /area kubelet.

thanks
/close

@neolit123
Copy link
Member

/sig node

@k8s-ci-robot k8s-ci-robot added the sig/node Categorizes an issue or PR as relevant to SIG Node. label Mar 13, 2021
@k8s-ci-robot
Copy link
Contributor

@neolit123: Closing this issue.

In response to this:

@medyagh kubeadm allows the users to configure the kubelet via KubeletConfiguration.
AFAIK the field to control this CNI related behavior is missing there, so that is a kubelet issue.

you can log a ticket for this in k/k and ping me on it, but it's for /sig node and /area kubelet.

thanks
/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
sig/node Categorizes an issue or PR as relevant to SIG Node.
Projects
None yet
Development

No branches or pull requests

3 participants