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
nixos/network: remove 99-main.network #71790
Conversation
Just maching all network interfaces caused many breakages, see NixOS#18962 and NixOS#71106. We already don't support the global networking.useDHCP, networking.defaultGateway(6) options if networking.useNetworkd is enabled, but direct users to configure the per-device networking.interfaces.<name?>.… options.
d13626e
to
c3098d1
Compare
@GrahamcOfBorg test networking.networkd networking.scripted |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 in principle, but networking.nameservers
will also cease to work. You might want to add that to the release notes.
Also, shouldn't we add assertions to make the use of these options eval errors like networking.useDHCP
instead of silently disregarding them?
We already had assertions about global |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, I wasn't aware the assertions were already in place (but apparently are since 2017) and that configuration bit for resolved. In that case: 👍
I'm not sure about the state of ofborg status suggests it passed on aarch64 (but no logs available, so this might be a bug). It didn't even start on x86_64-linux, so I tried building it locall. It seems the I don't believe this PR broke it, but it was already broken before, at least it broke here while building locally. |
The |
Edit: nvm just read your comment we already have assertions for this |
Okay, then let's merge this in, and fix the |
Another perk: NetworkManager can now configure resolved with the actual DNS servers, as the corresponding interfaces aren't managed anymore. |
Just maching all network interfaces caused many breakages, see #18962
and #71106.
We already don't support the global networking.useDHCP,
networking.defaultGateway(6) options if networking.useNetworkd is
enabled, but direct users to configure the per-device
networking.interfaces.<name?>.… options.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @