-
Notifications
You must be signed in to change notification settings - Fork 13.1k
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
Intermittent "Permission denied" on Windows #22628
Comments
I've thought it's basically this: for some reason stdtest hanged at previous test, then during the second build linker fails to open exe since it's still running. |
Ping @alexcrichton: I think you can access to buildbot directly. Could you dump thread/stack status of hanged process (when this occurs again) using Process Explorer? This article explains how to see list of threads and stack trace of each thread. |
Some various failures this caused (most of these today!):
This "intermittent" is really becoming a problem, can we prioritize this? |
@Manishearth suspects d0de2b4 may caused the recent spike. |
I added a patch to buildbot that attempts to kill everything named stdtest before running |
This is how mariadb creates buildbot jobs https://mariadb.com/kb/en/mariadb/buildbot-setup-buildbot-setup-for-windows/ |
@klutzy is correct in the diagnosis. The fact that Note that the fix for all the tests in @klutzy's linked comment was to
I took a peek, and unfortunately all I could deduce was that a thread was blocked in
@brson, why change the max builds? I'm under the impression that the error does not come from this but rather the fact that something just hangs for too long. Killing |
@alexcrichton We can't |
@alexcrichton Max builds has to be one, we're indiscriminately killing all |
It depends how you spawned the thread. If the thread was spawned with
Hm ok. |
I have created a simple Rust wrapper application that uses job objects to ensure all existing processes are completely dead. |
http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/3431
So the diagnosis by @klutzy / @alexcrichton is correct; this test sometimes hangs and stays alive for the next build. (Though stdtest might be a separate problem) We need to integrate dojob somehow ... @brson ? |
If these tests exit while a thread is panicking it often leads to situations like rust-lang#22628.
@Manishearth no the fix for that problem specifically is #22723 |
Yeah, I'm talking about both failures here (stdtest, and the unsafe one). |
…t, r=alexcrichton If these tests exit while a thread is panicking it often leads to situations like rust-lang#22628.
http://buildbot.rust-lang.org/builders/auto-win-32-nopt-t/builds/3449
|
@alexcrichton max_builds=1 because the script indiscrimanently kills anything called stdtest. @retep998 wants me to wrap the make calls with dojob.rs, which should make buildbot kill the whole process tree. Seems like a good idea. I hope that would let us remove the taskkill hack. |
Has anyone seen this issue lately since |
I have not, thanks @retep998! |
Example: http://buildbot.rust-lang.org/builders/auto-win-64-nopt-t/builds/2223
It doesn't seem to happen in the same place at every time, but basically an exe (usually stdtest) can't be opened for whatever reason.
cc @retep998 @klutzy
The text was updated successfully, but these errors were encountered: