Skip to content
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

system/boot/initrd-network: Add explicit enable option for UDHCPC #147088

Closed
wants to merge 1 commit into from

Conversation

Shados
Copy link
Member

@Shados Shados commented Nov 23, 2021

The default value retains the current behaviour of enabling UDHCPC in initrd if the user has either the global or any per-interface useDHCP option enabled, but making it an explicit option means that the user can enable UDHCPC in initrd under other circumstances.

Motivation for this change

I have a desktop system with an encrypted root, that I want to be able to decrypt over SSH when required, but that I also use networkmanager for normally. Using DHCP in the initrd is more preferable to me than setting an explicit static IP for the initrd.

Things done
  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 21.11 Release Notes (or backporting 21.05 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
    • (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes
  • Fits CONTRIBUTING.md.

@Shados Shados force-pushed the initrd-network-enable-udhcpc branch from 1e51a26 to 3377379 Compare December 5, 2021 08:29
@stale stale bot added the 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md label Jun 12, 2022
@Shados Shados force-pushed the initrd-network-enable-udhcpc branch from 3377379 to d64c8b8 Compare August 25, 2022 03:36
@Shados
Copy link
Member Author

Shados commented Aug 25, 2022

Updated for some bitrot (re. change to markdown doc).

The default value implements the current behaviour of enabling UDHCPC in
initrd if the user has either the global or any per-interface `useDHCP`
option enabled, but making it an explicit option means that the user can
enable UDHCPC in initrd under other circumstances.
@Shados Shados force-pushed the initrd-network-enable-udhcpc branch from d64c8b8 to d4f1749 Compare May 18, 2023 05:26
@stale stale bot removed the 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md label May 18, 2023
@Shados
Copy link
Member Author

Shados commented Aug 13, 2023

...closing because this functionality was merged in #240406.

@Shados Shados closed this Aug 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants