-
-
Notifications
You must be signed in to change notification settings - Fork 13.1k
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
Update path in udev rule of mdadm #6006
Conversation
cc @edolstra |
👍 |
Same should be done for |
udev-md-raid-arrays.rules does not contain any path that needs replacement, at least as far as I can see |
|
@iElectric, you're right, sorry, I didn't see the IMPORT rule. I've updated the PR. |
Update path in udev rule of mdadm
@aszlig, thanks for pointing that out! So what do you suggest? |
@ts468 it's weird that it's not working for you, as it's patched by nixos: https://github.com/NixOS/nixpkgs/blob/master/nixos/modules/services/hardware/udev.nix#L52 |
This reverts commit 1faf610. It causes the size of the initrd to explode from ~6 MB to 19 MB: http://hydra.nixos.org/build/15728196 The reason is that mdadm now contains a self-reference: $ strings ./result/sbin/mdadm | grep /nix/store ... ACTION=="add", SUBSYSTEM=="block", ENV{DEVTYPE}=="%s", ENV{ID_PATH}=="%s", RUN+="/nix/store/y1ap6va0av6jhxf7ddnr8vdwxd7b44gd-mdadm-3.3.2/sbin/mdadm --incremental $env{DEVNAME}" ACTION=="add", SUBSYSTEM=="block", ENV{ID_PATH}=="%s", RUN+="/nix/store/y1ap6va0av6jhxf7ddnr8vdwxd7b44gd-mdadm-3.3.2/sbin/mdadm --incremental $env{DEVNAME}" /nix/store/y1ap6va0av6jhxf7ddnr8vdwxd7b44gd-mdadm-3.3.2/sbin/mdmon (Previously, it referred to /sbin/mdadm.) This causes mdadm, glibc and linux-headers to be added to the initrd.
Could you please check if that change makes sense? Something like that was required by nixos-rebuild.