Skip to content

Comments

error messages for issue 2238#3897

Merged
edolstra merged 3 commits intoNixOS:masterfrom
bburdette:error-2238
Aug 6, 2020
Merged

error messages for issue 2238#3897
edolstra merged 3 commits intoNixOS:masterfrom
bburdette:error-2238

Conversation

@bburdette
Copy link
Contributor

Added a logError to build-remote to show the failure together with system info; added a check to the existing error site and reworded the error there. Added a link to docs.

Here's the tail end of the result from

sudo nix-store -j0 --realise --option substitute false --builders 'ssh://[email protected] x86_64-linux - - - foo foo' /nix/store/crv8z01l69c3y0scq4cvdgyxrk2ldd1f-rustc-1.45.0.drv 

Selection_052

@bburdette
Copy link
Contributor Author

#2238

@edolstra edolstra merged commit 3321b2b into NixOS:master Aug 6, 2020
@edolstra
Copy link
Member

edolstra commented Aug 6, 2020

Thanks!

@bburdette bburdette deleted the error-2238 branch August 6, 2020 14:07
@B4dM4n
Copy link
Contributor

B4dM4n commented Aug 11, 2020

This PR adds the following error to all nix builds for me, but the builds still succeed:

% nix build --impure --expr 'with import <nixpkgs> { }; runCommand "test" { } "touch $out"'
error: --- Remote build --- build-remote
Failed to find a machine for remote build!

required (system, features): (x86_64-linux, )
2 available machines:
(systems, maxjobs, supportedFeatures, mandatoryFeatures)
(x86_64-darwin, 2, , )
(x86_64-darwin, 2, , )
% realpath result
/nix/store/ml206rqibmzwffbbqssq879fj8jpivkb-test

@bburdette
Copy link
Contributor Author

bburdette commented Aug 11, 2020

This PR adds the following error to all nix builds for me, but the builds still succeed:

Yeah, its only in retrospect that we can say it was a critical error - when neither remote nor local build is possible. In your case it appears to be a normal part of the build. Maybe this is better as an warning, or even just an Info level message.

@nh2
Copy link
Contributor

nh2 commented Aug 13, 2020

This PR adds the following error to all nix builds for me, but the builds still succeed:

You can use --max-jobs 0 to force building on remotes only, see #1572 (comment) (with the exception of #3810).

@nh2
Copy link
Contributor

nh2 commented Aug 13, 2020

I made a follow-up PR to add more details in #3927.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants