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
rustc: enable parallel building #46406
Conversation
The upstream issue was closed but not fixed AFAIK. I've never been able to reproduce the problems locally either but hydra builds regularly failed. |
Failure on aarch64-linux (full log) Attempted: rustc Partial log (click to expand)
|
Timed out, unknown build status on x86_64-linux (full log) Attempted: rustc Partial log (click to expand)
|
This was the pull request, where we tried to enable it earlier: #21742 |
But how are they supposed to fix it without seeing the error it produces?
I tried building this using 4 cores on a busy system following a suggestion in one of the previous discussions, but I still couldn't reproduce this. Same with 6 and 8 cores.
Can't we merge this and then revert as soon as hydra fails? We would get the failing log, at the very least.
|
I did build the entire rebuild on one of my machines during the last day. It finished some time this morning. I also just build |
Dealing with intermittent failures that have a large dependency tree like rustc is a pain in hydra. There are still thinks like literal sleeps in the rust makefiles, I'm almost certain we're going to revert this anyway. So this seems a bit pointless to me, even tho I'd like to have it enabled, since there's no indication that the problems might be resolved. I suspect lower the io bandwidth in combination with concurrent builds are triggering problems that can't be reproduced on a machine with SSD or something similar like that. |
If we're not sure about this we could put it in its own little Hydra jobset and restart the build multiple times to see what happens. |
(triage) Anyone knows who to poke to put this PR in its Hydra jobset so that we can give log files to upstream and they can fix the issue? |
In fact, this was not an issue even once on any of my builders since I opened this PR. I'm living happily with thing change applied for almost a year now.
|
@oxij are you ok with closing this PR then? |
I think he meant that the concerns about the problems with this PR are now invalid. |
@GrahamcOfBorg eval |
I've just tried Maybe upstream just forced parallel builds on? |
either way, seems to take about as long as specifying |
Motivation for this change
The upstream issue is closed, all mentions of the issue on NixOS GitHub are old, I rebuilt it several times with this enabled, no issues.
rustc
is on the critical path tofirefox
, without thisfirefox
is always the very last package to be built on my machine when doing a mass rebuild becauserustc
take ages to build on a moderately slow single core.Things done