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

linux_testing_bcachefs: 4.16.2018.08.03 -> 4.18.2018.08.31 #45774

Merged
merged 3 commits into from Sep 1, 2018
Merged

linux_testing_bcachefs: 4.16.2018.08.03 -> 4.18.2018.08.31 #45774

merged 3 commits into from Sep 1, 2018

Conversation

ghost
Copy link

@ghost ghost commented Aug 30, 2018

Motivation for this change

Fixes #45769

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Fits CONTRIBUTING.md.

Sorry, something went wrong.

@GrahamcOfBorg GrahamcOfBorg added 8.has: clean-up 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 11-100 labels Aug 30, 2018
@Mic92
Copy link
Member

Mic92 commented Aug 30, 2018

@GrahamcOfBorg build bcachefs-tools

@GrahamcOfBorg
Copy link

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)


a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.


@GrahamcOfBorg
Copy link

Success on aarch64-linux (full log)

Attempted: bcachefs-tools

Partial log (click to expand)

strip is /nix/store/y4ymnvgxygpq05h03kyzbj572zmh6zla-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/jafq0fmslgvqihxp0dn6hrbfdyn8d1zp-bcachefs-tools-unstable-2018-08-22/sbin
patching script interpreter paths in /nix/store/jafq0fmslgvqihxp0dn6hrbfdyn8d1zp-bcachefs-tools-unstable-2018-08-22
/nix/store/jafq0fmslgvqihxp0dn6hrbfdyn8d1zp-bcachefs-tools-unstable-2018-08-22/etc/initramfs-tools/scripts/local-premount/bcachefs: interpreter directive changed from "/bin/sh" to "/nix/store/fqm2x6kiay1q4vg7pqp4wp17bdijlyc3-bash-4.4-p23/bin/sh"
/nix/store/jafq0fmslgvqihxp0dn6hrbfdyn8d1zp-bcachefs-tools-unstable-2018-08-22/etc/initramfs-tools/hooks/bcachefs: interpreter directive changed from "/bin/sh" to "/nix/store/fqm2x6kiay1q4vg7pqp4wp17bdijlyc3-bash-4.4-p23/bin/sh"
/nix/store/jafq0fmslgvqihxp0dn6hrbfdyn8d1zp-bcachefs-tools-unstable-2018-08-22/sbin/mkfs.bcachefs: interpreter directive changed from "/bin/sh" to "/nix/store/fqm2x6kiay1q4vg7pqp4wp17bdijlyc3-bash-4.4-p23/bin/sh"
/nix/store/jafq0fmslgvqihxp0dn6hrbfdyn8d1zp-bcachefs-tools-unstable-2018-08-22/sbin/fsck.bcachefs: interpreter directive changed from "/bin/sh" to "/nix/store/fqm2x6kiay1q4vg7pqp4wp17bdijlyc3-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/jafq0fmslgvqihxp0dn6hrbfdyn8d1zp-bcachefs-tools-unstable-2018-08-22...
moving /nix/store/jafq0fmslgvqihxp0dn6hrbfdyn8d1zp-bcachefs-tools-unstable-2018-08-22/sbin/* to /nix/store/jafq0fmslgvqihxp0dn6hrbfdyn8d1zp-bcachefs-tools-unstable-2018-08-22/bin
/nix/store/jafq0fmslgvqihxp0dn6hrbfdyn8d1zp-bcachefs-tools-unstable-2018-08-22

@Mic92
Copy link
Member

Mic92 commented Aug 30, 2018

Let's see if we can make it in time:

@GrahamcOfBorg build linux_testing_bcachefs

@GrahamcOfBorg
Copy link

Success on x86_64-linux (full log)

Attempted: bcachefs-tools

Partial log (click to expand)

strip is /nix/store/h0lbngpv6ln56hjj59i6l77vxq25flbz-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/sf6m7x3m4vr8x69rqz4vj65nx8xc85c0-bcachefs-tools-unstable-2018-08-22/sbin
patching script interpreter paths in /nix/store/sf6m7x3m4vr8x69rqz4vj65nx8xc85c0-bcachefs-tools-unstable-2018-08-22
/nix/store/sf6m7x3m4vr8x69rqz4vj65nx8xc85c0-bcachefs-tools-unstable-2018-08-22/etc/initramfs-tools/scripts/local-premount/bcachefs: interpreter directive changed from "/bin/sh" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/sh"
/nix/store/sf6m7x3m4vr8x69rqz4vj65nx8xc85c0-bcachefs-tools-unstable-2018-08-22/etc/initramfs-tools/hooks/bcachefs: interpreter directive changed from "/bin/sh" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/sh"
/nix/store/sf6m7x3m4vr8x69rqz4vj65nx8xc85c0-bcachefs-tools-unstable-2018-08-22/sbin/fsck.bcachefs: interpreter directive changed from "/bin/sh" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/sh"
/nix/store/sf6m7x3m4vr8x69rqz4vj65nx8xc85c0-bcachefs-tools-unstable-2018-08-22/sbin/mkfs.bcachefs: interpreter directive changed from "/bin/sh" to "/nix/store/czx8vkrb9jdgjyz8qfksh10vrnqa723l-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/sf6m7x3m4vr8x69rqz4vj65nx8xc85c0-bcachefs-tools-unstable-2018-08-22...
moving /nix/store/sf6m7x3m4vr8x69rqz4vj65nx8xc85c0-bcachefs-tools-unstable-2018-08-22/sbin/* to /nix/store/sf6m7x3m4vr8x69rqz4vj65nx8xc85c0-bcachefs-tools-unstable-2018-08-22/bin
/nix/store/sf6m7x3m4vr8x69rqz4vj65nx8xc85c0-bcachefs-tools-unstable-2018-08-22

@GrahamcOfBorg
Copy link

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)

Cannot nix-instantiate `linux_testing_bcachefs' because:
error: while evaluating 'callPackageWith' at /private/var/lib/ofborg/builds/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/lib/customisation.nix:108:35, called from /private/var/lib/ofborg/builds/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/top-level/all-packages.nix:14005:28:
while evaluating 'makeOverridable' at /private/var/lib/ofborg/builds/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/lib/customisation.nix:67:24, called from /private/var/lib/ofborg/builds/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/lib/customisation.nix:112:8:
while evaluating anonymous function at /private/var/lib/ofborg/builds/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/os-specific/linux/kernel/linux-testing-bcachefs.nix:1:1, called from /private/var/lib/ofborg/builds/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/lib/customisation.nix:69:12:
while evaluating 'buildLinux' at /private/var/lib/ofborg/builds/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/top-level/all-packages.nix:14284:16, called from /private/var/lib/ofborg/builds/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/os-specific/linux/kernel/linux-testing-bcachefs.nix:3:1:
while evaluating 'callPackageWith' at /private/var/lib/ofborg/builds/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/lib/customisation.nix:108:35, called from /private/var/lib/ofborg/builds/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/top-level/all-packages.nix:14284:23:
while evaluating 'makeOverridable' at /private/var/lib/ofborg/builds/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/lib/customisation.nix:67:24, called from /private/var/lib/ofborg/builds/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/lib/customisation.nix:112:8:
while evaluating anonymous function at /private/var/lib/ofborg/builds/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/os-specific/linux/kernel/generic.nix:1:1, called from /private/var/lib/ofborg/builds/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/lib/customisation.nix:69:12:
assertion failed at /private/var/lib/ofborg/builds/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/os-specific/linux/kernel/generic.nix:52:1

@GrahamcOfBorg
Copy link

Failure on aarch64-linux (full log)

Attempted: linux_testing_bcachefs

Partial log (click to expand)

  CC [M]  drivers/gpu/drm/nouveau/nv84_fence.o
  CC [M]  drivers/gpu/drm/nouveau/nvc0_fence.o
  LD [M]  drivers/gpu/drm/nouveau/nouveau.o
  AR      drivers/gpu/drm/built-in.a
  AR      drivers/gpu/built-in.a
  AR      drivers/built-in.a
make[1]: *** [Makefile:146: sub-make] Error 2
make: *** [Makefile:24: __sub-make] Error 2
builder for '/nix/store/n4z5jwqy1cggpwrkzvfpv8knvlxyz3qp-linux-4.18.2018.08.21.drv' failed with exit code 2
error: build of '/nix/store/n4z5jwqy1cggpwrkzvfpv8knvlxyz3qp-linux-4.18.2018.08.21.drv' failed

@Mic92
Copy link
Member

Mic92 commented Aug 30, 2018

@Chiiruno please disable aarch64 support for this kernel:

extraMeta.platforms = [ "x86_64-linux" ];

@ghost
Copy link
Author

ghost commented Aug 30, 2018

@Mic92 Done.

@vcunat
Copy link
Member

vcunat commented Aug 30, 2018

Well, the kernel builds on my x86_64-linux, but this is more about using it with the FS :-)

@GrahamcOfBorg
Copy link

Timed out, unknown build status on x86_64-linux (full log)

Attempted: linux_testing_bcachefs

Partial log (click to expand)

  CC [M]  drivers/tty/synclinkmp.o
  CC [M]  drivers/scsi/qla2xxx/qla_mr.o
  CC [M]  drivers/usb/gadget/udc/dummy_hcd.o
  CC [M]  drivers/staging/rtl8192e/rtl8192e/rtl_pm.o
  CC [M]  drivers/staging/rtl8192e/rtl8192e/rtl_ps.o
  CC [M]  drivers/usb/gadget/udc/net2272.o
  CC [M]  drivers/scsi/qla2xxx/qla_nx2.o
  CC [M]  drivers/staging/rtl8192e/rtl8192e/rtl_wx.o
building of '/nix/store/b0w47fp0l4dg46v0fd6pjscz5ndfy9n9-linux-4.18.2018.08.21.drv' timed out after 1800 seconds
error: build of '/nix/store/b0w47fp0l4dg46v0fd6pjscz5ndfy9n9-linux-4.18.2018.08.21.drv' failed

@ghost
Copy link
Author

ghost commented Aug 30, 2018

@vcunat Yes, for me too.
Does nix have a way to streamline filesystem tests in a VM?
Is the build bot overloaded right now by any chance?

@Mic92
Copy link
Member

Mic92 commented Aug 30, 2018

It is not overloaded probably but there is a one hour limit, this is ok to fail. I can build locally.

@Mic92
Copy link
Member

Mic92 commented Aug 30, 2018

We have tests for the installer and zfs in nixpkgs/nixos/tests. Bcachefs can be added as well.

@vcunat
Copy link
Member

vcunat commented Aug 30, 2018

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.

@ghost
Copy link
Author

ghost commented Aug 30, 2018

@vcunat I only tested compilation and what was checked off in the OP.
I'm not savvy enough about nix vm to do that, unfortunately.
I have, however, tested bcachefs before, and it's worked, so I would assume this one does too, but (I hope) this probably means very little.

@ghost
Copy link
Author

ghost commented Aug 30, 2018

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.

@Mic92
Copy link
Member

Mic92 commented Aug 30, 2018

I currently cannot test this either if this updates works (manually or automatically).

@Mic92
Copy link
Member

Mic92 commented Aug 30, 2018

The build also works on my machine.

@vcunat
Copy link
Member

vcunat commented Aug 30, 2018

Then I'd wait until someone runs the FS with this, on bare metal or VM.

@ghost
Copy link
Author

ghost commented Aug 30, 2018

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?)

@ghost
Copy link
Author

ghost commented Aug 30, 2018

@vcunat First time making a test, that was surprisingly easy.
Seems to work, this can be merged I think.

@GrahamcOfBorg GrahamcOfBorg added the 6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS label Aug 30, 2018
meta.maintainers = [ pkgs.stdenv.lib.maintainers.chiiruno ];
};
in {
stable = makeBcachefsTest "stable" { };
Copy link
Member

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

@ghost
Copy link
Author

ghost commented Aug 30, 2018

@Mic92 All good?

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@ghost ghost changed the title linux_testing_bcachefs: 4.16.2018.08.03 -> 4.18.2018.08.21 linux_testing_bcachefs: 4.16.2018.08.03 -> 4.18.2018.08.31 Sep 1, 2018
@vcunat vcunat merged commit 3d1fecd into NixOS:master Sep 1, 2018
vcunat added a commit that referenced this pull request Sep 1, 2018

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@ghost ghost deleted the dev/bcachefs branch September 1, 2018 09:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS 8.has: clean-up 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 11-100
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants