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 portmap plugin to CNI conf #888

Merged
merged 1 commit into from
Nov 27, 2017

Conversation

osoriano
Copy link
Contributor

Description

The portmap plugin is needed for hostPorts to work

The CNI portmap capability was added in k8s 1.7. I was able to get past the issue by using the updated kube-flannel.yml on a 1.8.3 cluster.

Closes: #887

Release Note

None required

The portmap plugin is needed for hostPorts to work

Closes: flannel-io#887
@tomdee
Copy link
Contributor

tomdee commented Nov 27, 2017

Looks good

@tomdee tomdee merged commit d608197 into flannel-io:master Nov 27, 2017
@jethrogb
Copy link

10-flannel.conf sorts before 10-flannel.conflist, so if you update you configuration with this change, this change will not be picked up since 10-flannel.conf still exists.

@zcalusic
Copy link

I ended up with both 10-flannel.conf & 10-flannel.conflist with the exactly same contents, and lots of kubelet[29256]: W1227 18:53:27.409233 29256 cni.go:125] Error loading CNI config file /etc/cni/net.d/10-flannel.conf: no 'type'; perhaps this is a .conflist? log spam in journal. Every few seconds... :(

Admittedly, this was applied to 0.9.1 image so maybe some bits & pieces are still missing, possibly will be available in 0.9.2?

For now, I deleted 10-flannel.conf, it fixes the extreme log spam issue. Hopefully everything continues working ok...

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.

4 participants