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.autoUpgrade: optionally allow rebooting the system on kernel change #64267
Conversation
The option has a description, so it should show up. |
I tried something similar but I took a different approach. I always did a switch and then checked if the kernel has changed, if so I did a reboot. I'm not sure if this has some drawbacks compared to your version, but it for sure has less cases as it is just a post step after the normal switch. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall looks good! Left a few style comments.
I'd like also to see a test for this, though I understand it isn't easy to write one.
I'd love to have a test for this as well, but I have no idea where to start writing one, so I'll leave that open. |
So there was kernel update recently and the module worked just fine when hammered into 19.03, so I believe the PR is ready to be merged if there are no objections? |
(cherry picked from commit 8a2555132e0afeded544e90cc3990fab0c63093a)
Motivation for this change
new function for the module: Allows automatic reboots if the new generation uses a different kernel.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)