-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Why is IPv6 disabled for rootless/slirp4netns containers? #10889
Comments
Probably safe to enable v6 by default, but not massively tested as v4 and there might be some vulns |
This should probably be done in containers.conf then. That way users could have a way to customize. |
You can already set this in containers.conf with @AkihiroSuda How does slirp4netns behave when ipv6 is disabled in the kernel, will it fail or ignore the ipv6 setup? |
Haven't tried |
A friendly reminder that this issue had no activity for 30 days. |
@rugk interested in opening a PR containers/common to change the default? |
Fixes containers/podman#10889 in case that is the correct thing to change.
Hmm, tried with containers/common#741, though I have no idea whether that is the correct thing to change… 😄 Feel free to change it by yourself, you likely better know what exactly to change. |
You need to change default.go, we will carry on the conversation in the PR. |
Fixes containers/podman#10889 Inspired by @rugk Signed-off-by: Daniel J Walsh <[email protected]>
Fixes containers/podman#10889 Inspired by @rugk Signed-off-by: Daniel J Walsh <[email protected]>
Fixes containers/podman#10889 Inspired by @rugk Signed-off-by: Daniel J Walsh <[email protected]>
Fixes containers/podman#10889 Inspired by @rugk Signed-off-by: Daniel J Walsh <[email protected]>
The default was changed by containers#10889
The man page says:
I interpret that as such, that IPv6 is disabled by default and thus containers/ports bound can only be reached via IPv4. Is that true?
If so, why is that so? Given various initiatives to finally get to the day we use IPv6 only, I wonder why you do not enable this by default…?
The text was updated successfully, but these errors were encountered: