-
-
Notifications
You must be signed in to change notification settings - Fork 15.3k
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
linux_testing_bcachefs: 4.16.2018.08.03 -> 4.18.2018.08.31 #45774
Conversation
@GrahamcOfBorg build bcachefs-tools |
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: bcachefs-tools Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: bcachefs-tools Partial log (click to expand)
|
Let's see if we can make it in time:
|
Success on x86_64-linux (full log) Attempted: bcachefs-tools Partial log (click to expand)
|
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: linux_testing_bcachefs Partial log (click to expand)
|
Failure on aarch64-linux (full log) Attempted: linux_testing_bcachefs Partial log (click to expand)
|
@Chiiruno please disable aarch64 support for this kernel:
|
@Mic92 Done. |
Well, the kernel builds on my |
Timed out, unknown build status on x86_64-linux (full log) Attempted: linux_testing_bcachefs Partial log (click to expand)
|
@vcunat Yes, for me too. |
It is not overloaded probably but there is a one hour limit, this is ok to fail. I can build locally. |
We have tests for the installer and zfs in nixpkgs/nixos/tests. Bcachefs can be added as well. |
For this merge I'm fine if the FS now works for you – you did test that, right? PR with a reasonably written VM test would surely get merged, too. |
@vcunat I only tested compilation and what was checked off in the OP. |
If you want a VM test, it'll have to wait until tomorrow, but chances are you could merge this now and it would be fine. |
I currently cannot test this either if this updates works (manually or automatically). |
The build also works on my machine. |
Then I'd wait until someone runs the FS with this, on bare metal or VM. |
My apologies, I stopped using bcachefs after I lost a lot of data, ironically due to the error this commit supposedly fixes. I'll see what I can do about tests, and at the very least I'll run this in a standard VM now that I have the chance. (Is there a way to easily add an empty disk to a nixos vm, though?) |
@vcunat First time making a test, that was surprisingly easy. |
nixos/tests/bcachefs.nix
Outdated
meta.maintainers = [ pkgs.stdenv.lib.maintainers.chiiruno ]; | ||
}; | ||
in { | ||
stable = makeBcachefsTest "stable" { }; |
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.
if the test is supposed to be run by hydra, it should be also referenced in nixpkgs/nixos/release.nix
@Mic92 All good? |
Motivation for this change
Fixes #45769
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)