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

Add: kube-router cni configuration #1

Closed
wants to merge 0 commits into from
Closed

Add: kube-router cni configuration #1

wants to merge 0 commits into from

Conversation

methodx
Copy link

@methodx methodx commented Oct 8, 2018

Hi

When I try to deploy kubernetes v1.12.0 or v1.12.1 I noticed that kube-router doesn't start at all. If I'm getting it right, we need to create cni configuration file - it should work.

Thanks

@murali-reddy
Copy link

When I try to deploy kubernetes v1.12.0 or v1.12.1 I noticed that kube-router doesn't start at all.

@methodx please see cloudnativelabs/kube-router#547

is it because of toleration?

@methodx
Copy link
Author

methodx commented Oct 8, 2018

@murali-reddy that's it, thank you!
Added that bold option, everything works.

Also, FYI, I added one more thing:

- name: kube-router | Delete kube-proxy pods
  shell: "rm -rf /etc/kubernetes/manifests/kube-proxy.manifest"
  when: kube_router_run_service_proxy

@jjo jjo force-pushed the jjo-add-kube-router-support branch from 241bbbb to 8753a01 Compare October 9, 2018 01:24
@jjo
Copy link
Owner

jjo commented Oct 9, 2018

@murali-reddy that's it, thank you!
Added that bold option, everything works.

Also, FYI, I added one more thing:

- name: kube-router | Delete kube-proxy pods
  shell: "rm -rf /etc/kubernetes/manifests/kube-proxy.manifest"
  when: kube_router_run_service_proxy

Note that this should be already handled by added kube_proxy_remove
(set_fact), it's a bit more complex as it needs to also cope with kubeadm_enabled: true
(which doesn't create kube-proxy manifests at node).
Voiding to write that manifest is specifically handled at https://github.com/jjo/kubespray/blob/jjo-add-kube-router-support/roles/kubernetes/node/tasks/main.yml#L145, wonder
if this isn't working for you ?

@methodx
Copy link
Author

methodx commented Oct 9, 2018

@jjo it works.
I rechecked task main for node. Don't know why, but it looks like I used old version of set_fact for kube_proxy.

@jjo jjo force-pushed the jjo-add-kube-router-support branch from 8753a01 to 5395c6f Compare October 11, 2018 02:24
@methodx methodx closed this Oct 15, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants