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
grub2: Properly disable zfs by default #99386
Conversation
@GrahamcOfBorg test zfs |
@Mic92 I don't really understand the logs but it seems like the grub tests didn't run. Should I rebase or something? Also is it normal that the logs for the zfs tests are empty for me? |
Yeah. I think it should be |
f9ff8d1
to
d44c621
Compare
This comment has been minimized.
This comment has been minimized.
which is only like 6 MB difference. Although this is a much less minimal system which is likely the reason for that. (TODO @mohe2015 test with a more minimal system) For the test
I already set the memory higher ( |
Right the installer test seems broken. |
It seems it is now required to set Not sure if improvements can be easily made here, but detecting automatically if |
… grub if they didn't add zfs to boot.initrd.supportedFilesystems See NixOS#99386
Motivation for this change
Make the system derivation smaller.
Minimal closure size change: 773,395648 MB -> 672,400544 MB (If I measured correctly)
Things done
I don't know if this is the proper change or if this breaks peoples system.
https://github.com/NixOS/nixpkgs/blob/master/nixos/modules/tasks/filesystems/zfs.nix#L450 seems to enable zfsSupport if needed. https://github.com/NixOS/nixpkgs/blob/master/nixos/modules/system/boot/loader/grub/grub.nix#L16 overrides zfsSupport to true if needed and the default is used otherwise (https://github.com/NixOS/nixpkgs/blob/master/nixos/modules/system/boot/loader/grub/grub.nix#L21).
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)