Skip to content
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

Open
DavHau opened this issue Dec 29, 2023 · 5 comments
Open

Run nixos tests for each PR #9677

DavHau opened this issue Dec 29, 2023 · 5 comments
Labels

Comments

@DavHau
Copy link
Member

DavHau commented Dec 29, 2023

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.

@DavHau DavHau added the feature Feature request or proposal label Dec 29, 2023
@Mic92
Copy link
Member

Mic92 commented Dec 29, 2023

Github's own runner now also supports kvm btw.

@DavHau
Copy link
Member Author

DavHau commented Dec 30, 2023

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.
It seems to be unacceptably slow. I mean it already was before enabling the tests. Compiling nix alone takes around 15 minutes on the linux runner. What can we do to improve this situation?

@Mic92
Copy link
Member

Mic92 commented Dec 31, 2023

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?
There is the magic-nix-cache but it only improves build times if there are some custom packages or overrides that benefit from caching: https://github.com/Nix-Security-WG/nix-security-tracker/blob/cbd2abdef8ec77f0ac9d6ab7b5e5414d972c41c5/.github/workflows/checks.yaml#L12

@zowoq
Copy link

zowoq commented Jan 1, 2024

Also see the discussion in #7674.

@roberth
Copy link
Member

roberth commented Jan 11, 2024

An extra goal - perhaps separate issue? - is to include tests that Nixpkgs thinks are relevant.
We can configure the nix flake to run those, with a little bit of collaboration between the repos.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants