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
Add basic ZFS tests #31618
Add basic ZFS tests #31618
Conversation
We also have a test in |
nixos/tests/zfs/default.nix
Outdated
@@ -0,0 +1,7 @@ | |||
{ | |||
lts_stable = import ./lts_stable.nix { }; | |||
lts_unstable = import ./lts_unstable.nix { }; |
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.
LTS might be redundant. We will most likely see the bug first in latest
.
df1827c
to
471c5aa
Compare
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.
Perhaps factor out the common stuff for both? The duplication sounds like it'll be quite annoying to maintain if we make the tests more sophisticated. It seems like they could live in the same file. See how something like the installer test works, for example
471c5aa
to
59e4599
Compare
@copumpkin Good feedback. It's done 👍 Also implemented tests for zfs encryption. |
It would be nice to have a test that ran
|
Motivation for this change
Add some basic testing for
zfs
to be able to discover regressions in newer kernels more easily.These are all currently failing because of
linuxPackages_4_14
being both lts and latest (but did succeed when testing withlinuxPackages_4_13
).Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)