Skip to content

nixos/network-interfaces: let networkd handle privacy extensions#431967

Merged
RaitoBezarius merged 1 commit intoNixOS:masterfrom
nikstur:networkd-privacy-extensions
Aug 9, 2025
Merged

nixos/network-interfaces: let networkd handle privacy extensions#431967
RaitoBezarius merged 1 commit intoNixOS:masterfrom
nikstur:networkd-privacy-extensions

Conversation

@nikstur
Copy link
Contributor

@nikstur nikstur commented Aug 8, 2025

Let networkd handle the privacy extensions when using networkd. Otherwise still use the custom udev rules.

This removes bash from the mandatory system closure.

Part of #428908

Things done

  • Built on platform:
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested, as applicable:
  • Ran nixpkgs-review on this PR. See nixpkgs-review usage.
  • Tested basic functionality of all binary files, usually in ./result/bin/.
  • Nixpkgs Release Notes
    • Package update: when the change is major or breaking.
  • NixOS Release Notes
    • Module addition: when adding a new NixOS module.
    • Module update: when the change is significant.
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

Add a 👍 reaction to pull requests you find important.

@nikstur nikstur requested review from Ma27 and mweinelt August 8, 2025 09:39
@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS 8.has: module (update) This PR changes an existing module in `nixos/` labels Aug 8, 2025
emilazy
emilazy previously requested changes Aug 8, 2025
Copy link
Member

@emilazy emilazy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should set a default for systemd.network.config.networkConfig.IPv6PrivacyExtensions rather than adding a separate .network, but otherwise seems like a good idea. Scripted networking delenda est.

@nikstur nikstur force-pushed the networkd-privacy-extensions branch from fe44fdc to c516356 Compare August 8, 2025 19:52
@nikstur
Copy link
Contributor Author

nikstur commented Aug 8, 2025

Should set a default for systemd.network.config.networkConfig.IPv6PrivacyExtensions rather than adding a separate .network

Done

@nikstur nikstur requested a review from emilazy August 8, 2025 19:53
Copy link
Member

@emilazy emilazy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you update the defaultText of systemd.network.config.networkConfig as appropriate, and probably move the default into the relevant module? (You can use networkd without networking.useNetworkd, and this default would still apply there, so it probably doesn’t belong in this file.)

This removes bash from the mandatory system closure.
@nikstur nikstur force-pushed the networkd-privacy-extensions branch from c516356 to bb954cd Compare August 8, 2025 22:11
@nikstur
Copy link
Contributor Author

nikstur commented Aug 8, 2025

Moved it into the networkd module.

(You can use networkd without networking.useNetworkd, and this default would still apply there, so it probably doesn’t belong in this file.)

Right now, this is always enabled so this change doesn't change the default NixOS behaviour. The only difference is that if you enable networkd, you now use native networkd features to set these flags.

@nikstur nikstur requested a review from emilazy August 9, 2025 09:59
@RaitoBezarius RaitoBezarius merged commit bcc20ca into NixOS:master Aug 9, 2025
25 of 27 checks passed
@nikstur nikstur deleted the networkd-privacy-extensions branch August 9, 2025 15:55
@nixos-discourse
Copy link

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/did-something-change-with-network-interfaces-during-the-summer/69504/2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS 8.has: module (update) This PR changes an existing module in `nixos/` 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants