Skip to content

[WIP] Add new driver option to enable IPv6#35252

Closed
wrridgwa wants to merge 1 commit into
moby:masterfrom
wrridgwa:25407-natIPv6
Closed

[WIP] Add new driver option to enable IPv6#35252
wrridgwa wants to merge 1 commit into
moby:masterfrom
wrridgwa:25407-natIPv6

Conversation

@wrridgwa

@wrridgwa wrridgwa commented Oct 19, 2017

Copy link
Copy Markdown
Contributor

Fixes issue 25407

This pull request is linked with moby/libnetwork#1992 to enable NAT functionality for IPv6. The issue can be found #25407

This pull request just adds the EnableIPv6 flag to the bridgeConfig options to allow parts of libnetwork to identify that IPv6 is enabled.

When combined with the libnetwork pull request, the fix can be verified after starting a container in bridge mode with port forwarding and IPv6 enabled:

  1. Compare the iptables -t nat -L command with the ip6tables -t nat -L command
  2. Compare iptables -t filter -L command with ip6tables -t filter -L command
  3. Ping6 from within a running container to an outside device.

@GordonTheTurtle

Copy link
Copy Markdown

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:

$ git clone -b "25407-natIPv6" [email protected]:wrridgwa/moby.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

@GordonTheTurtle GordonTheTurtle added dco/no Automatically set by a bot when one of the commits lacks proper signature status/0-triage labels Oct 19, 2017
@GordonTheTurtle GordonTheTurtle removed the dco/no Automatically set by a bot when one of the commits lacks proper signature label Oct 19, 2017
@cpuguy83

Copy link
Copy Markdown
Member

ping @mavenugo

@thaJeztah thaJeztah changed the title Add new driver option to enable IPv6 [WIP] Add new driver option to enable IPv6 Oct 26, 2017
@thaJeztah

Copy link
Copy Markdown
Member

Thanks for contributing @wrridgwa - given that this depends on a feature in libnetwork upstream, I think we should close this one until it's decided on by the libnetwork maintainers. We can re-open this pull request once that step is completed.

Feel free to discuss though, after I closed 👍

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants