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

New AMI doesn't have forward enabled #3958

Closed
justinsb opened this issue Nov 29, 2017 · 6 comments
Closed

New AMI doesn't have forward enabled #3958

justinsb opened this issue Nov 29, 2017 · 6 comments
Milestone

Comments

@justinsb
Copy link
Member

The new AMI seems to be defaulting to -P FORWARD deny , at least with kopeio-networking (where I found it). Forwarding is needed for containers to talk to other nodes (for example)

@justinsb justinsb added this to the 1.8.0 milestone Nov 29, 2017
@chrislovecnm
Copy link
Contributor

Isn't this the Ubuntu / Debian default now?

@cgilmour
Copy link
Contributor

Does the new AMI have docker 1.13+ or docker-ce 17.03+?
Those newer versions of Docker change the forward chain's policy to DROP.

@chrislovecnm
Copy link
Contributor

https://github.com/kubernetes/kops/blob/master/nodeup/pkg/model/docker.go

Yes

Is it Docker or new iptables?

@chrislovecnm
Copy link
Contributor

K8s 1.8 is using the new version of Docker to be clear.

@cgilmour
Copy link
Contributor

cgilmour commented Dec 1, 2017

Is it Docker or new iptables?

It's basically this part of docker: https://github.com/moby/moby/blob/v1.13.1/vendor/github.com/docker/libnetwork/drivers/bridge/setup_ip_forwarding.go#L42
Essentially, the setting for EnableIPForward defaults to true. When dockerd launches, it will check if it needs to change /proc/.../ip_forward to enable it. If it needs to enable that, it will also change iptables FORWARD policy to DROP.

Related issues: kubernetes/kubernetes#39823
Many of the CNI providers have already had to deal with this, by adding the necessary entries into the FORWARD chain.

@justinsb
Copy link
Member Author

justinsb commented Dec 2, 2017

Should be fixed by #3977.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants