-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Enable IPv6 support by default #3680
Comments
I like turning it on by default. But what if a Admin wants this turned off by default? |
@saschagrunert Where do I see this disabled? sudo podman run fedora cat /proc/sys/net/ipv6/conf/all/disable_ipv6 |
I think our first steps here are probably throwing IPv6 support into the default CNI config we ship, and enabling the flag for static IPv6 addresses - which, AFAIK, is still blocked on verifying CNI 0.8.1 is available everywhere and building new test images with it. |
Interestingly, it really seems not disabled by default. I guess the main issue is what @mheon mentioned. A CNI config like this seems not to work right now:
If we leverage the latest OCICNI version then it should be possible to provide the same dual stack IPv6 support like in CRI-O, right? |
Yep, once we have CNI updated properly, we can update OCICNI and get v6 support enabled. |
Alright, I see that we have another issue open on that topic: #3245 I will close this one in favor of the other. |
@mheon Will the future default IPv6 support be implemented through nat6, or will DHCPv6 get a subnet and assign IPv6 address to container? |
Definitely not NAT. Right now, I think our approach might be requiring users to manually configure v6 subnets manually (maybe provide a range in the configuration file that |
/kind feature
Description
IPv6 support needs currently explicitly enabled by
--sysctl net.ipv6.conf.all.disable_ipv6=0
.That's the same behavior like in docker and I'm wondering if we want to change it to a more sane default?
If you're fine with that I'd like to submit a PR, otherwise feel free to close this.
The text was updated successfully, but these errors were encountered: