-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Run nixos tests for each PR #9677
Comments
Github's own runner now also supports kvm btw. |
Nice, thanks for the hint. I just tried out running all the tests in a runner. |
I don't think we can improve nix compilation time magically. Regarding nixos tests, I was expecting no significant slow down since download dependencies should be finished within the 15 minutes window. Can you check if /dev/kvm is present in the build environment? |
Also see the discussion in #7674. |
An extra goal - perhaps separate issue? - is to include tests that Nixpkgs thinks are relevant. |
Is your feature request related to a problem? Please describe.
When a PR breaks a nixos test, there is no obvious feedback about it and it doesn't block the merge.
Describe the solution you'd like
Provide fast feedback about any breaking test to the author of a PR as well as the reviewers.
Describe alternatives you've considered
No quick feedback about breaking nixos tests
-> reduced confidence in changes
-> increased overhead for reviewers
-> harder to track down cause once breakage gets discovered.
Additional context
What's a quick and easy way to get CI checks for the nixos tests?
Ideas:
Priorities
Add 👍 to issues you find important.
The text was updated successfully, but these errors were encountered: