Skip to content

Commit

Permalink
remove unroutable ipv6 config
Browse files Browse the repository at this point in the history
The current configuration configures the docker daemon to issue ipv6
addresses in an RFC 3849 IPv6 documentation range[1], most Likely copied f
rom the docker documentation[2].  I suspect that theses images dont have
any additional IPv6 prefixes delegated to them so im not sure any value
makes senses here.

I suspect that this is never the desired outcome as it means the docker
instance gets configured with a un-routable global IPv6 address.  This
means outgoing connections will first try to connect to resources via
IPv6 (if a AAAA is avalible) and time out before trying IPv4.  This can
be observed in a beaker job[3] where wget first tries to download the
puppet.deb file over ipv6

In most cases i think one just wants to have the docker image have a
ipv6 loopback and linklocal address which for reasons [4] docker dosn't
give us.  I  don't think anyone  actually needs a global IPv6
address.  As such adding something like the following to[4]
spec_helper_acceptance.rb would produce the desired affect

  shell('sysctl net.ipv6.conf.all.disable_ipv6=0')

[1]https://tools.ietf.org/html/rfc3849
[2]https://docs.docker.com/config/daemon/ipv6/
[3]https://github.com/voxpupuli/puppet-unbound/runs/1595247379?check_suite_focus=true
(line 272)
[4]moby/moby#33099
[5]https://github.com/voxpupuli/puppet-unbound/blob/master/spec/acceptance/unbound_spec.rb#L13
  • Loading branch information
b4ldr committed Dec 22, 2020
1 parent 48044bb commit 9bf0636
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions moduleroot/.github/workflows/ci.yml.erb
Original file line number Diff line number Diff line change
Expand Up @@ -79,10 +79,6 @@ jobs:
-%>
name: <%= name.join(' - ') %>
steps:
- name: Enable IPv6 on docker
run: |
echo '{"ipv6":true,"fixed-cidr-v6":"2001:db8:1::/64"}' | sudo tee /etc/docker/daemon.json
sudo service docker restart
- uses: actions/checkout@v2
- name: Setup ruby
uses: ruby/setup-ruby@v1
Expand Down

0 comments on commit 9bf0636

Please sign in to comment.