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/qemu-vm: Don't explicitly mknod /dev/vda* #44938
Conversation
Don't merge this just yet, the "fixes" is very speculative, since I haven't been able to boot a vm with this just yet... |
I don't have time to look into this right now, as it got more complex. I'll close for now so as to not keep peoples' hope up. |
Well at least the changes here so far sound ObviouslyCorrect to me, so they could be merged regardless of not being a full fix to the original problem. |
@dezgeg That's a good point. Thanks. |
@GrahamcOfBorg test simple |
Success on x86_64-linux (full log) Attempted: tests.simple Partial log (click to expand)
|
I squashed them in and noted in the commit message why these no longer need to be created. 78fb4bd |
Success on aarch64-linux (full log) Attempted: tests.simple Partial log (click to expand)
|
Motivation for this change
Partially addresses #44918
The device node already exists, causing the
mknod
call to fail.The actual vm is still not bootable, but that failure seems to be unrelated,
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)