-
Notifications
You must be signed in to change notification settings - Fork 85
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
Add port 53/udp
by default to netavark_zone
so default networks can use DNS
#1056
Comments
@Luap99 Any thoughts on above ? Maybe we can add |
Is that with using the firewall driver? Because with iptables/nftables we definitely should add the accept rule already. In general the firewalld driver cannot recommend for use as it is pretty broken (see open issues about firewalld), @mheon is working to fix most of them so I guess this would be another? |
DNS should already be fixed on my patch, it's just port forwarding that's broken at this point AFAIK. |
@mheon Is the patch which you are describing is on any of the open PR or merged in recent releases/upstream ? |
I really need to push the latest version with the isolation code added, but I still have not worked out the issues with port forwarding. Hopefully soon? |
Sure I'm not in a rush just asked out of curiosity since I was interested in looking/trying the patch. Please take your time. |
Although given I added tcp support to aardvark-dns we likely need to check that all rules allow 53 udp and tcp. |
I agree if |
It seems on default setups with firewalld
DNS
is not functional at all, would it make sense to add--zone=netavark_zone --add-port=53/udp
?Reproducer
firewalld
Current Output
Expected output
Resolution for default networks should work
The text was updated successfully, but these errors were encountered: