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

Detect iptables version and avoid --wait #374

Closed
bprashanth opened this issue Nov 21, 2015 · 6 comments · Fixed by #464
Closed

Detect iptables version and avoid --wait #374

bprashanth opened this issue Nov 21, 2015 · 6 comments · Fixed by #464

Comments

@bprashanth
Copy link

Logs seem to indicate it isn't available on 1.4.14 which comes with a stock gce vm.

E1121 06:03:55.025714 03163 network.go:80] Failed to set up IP Masquerade for network : Failed to create/clear FLANNEL chain in NAT table: exit status 2: iptables v1.4.14: unknown option "--wait"
@jonboulle
Copy link
Contributor

coreos/go-iptables#8

@philips
Copy link
Contributor

philips commented Feb 4, 2016

@jonboulle since coreos/go-iptables#12 is merged can we just bump the dependency to move forward?

@philips philips added this to the v0.6.0 milestone Feb 4, 2016
@jonboulle
Copy link
Contributor

Yes AFAICT

@davidsiefert
Copy link
Contributor

Having enormous trouble with godep to update dependencies... what version of godep are you guys using where it works to update?

@jonboulle
Copy link
Contributor

Almost always want to use the very latest version of godep.

On Fri, Feb 19, 2016 at 12:39 AM, David Siefert [email protected]
wrote:

Having enormous trouble with godep to update dependencies... what version
of dep are you guys using where it works to update?


Reply to this email directly or view it on GitHub
#374 (comment).

@davidsiefert
Copy link
Contributor

Okay I figured out this godep issue (I think) and have another open issue #406 which blocked me from updating the coreos/go-iptables dependency using godep. Once that PR is merged, I have another fix on a branch to update go-iptables with the fix needed. I really need a critical eye on #406 if anybody can help me out here.

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

Successfully merging a pull request may close this issue.

4 participants