-
Notifications
You must be signed in to change notification settings - Fork 4.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
Bazel at HEAD gets DEADLINE_EXCEEDED with remote strategy #3248
Comments
FYI @ola-rozenfeld |
The default minute of gRPC deadline should probably be increased, although it's the first time I see it not suffice for src:bazel. I don't think it's a release blocker, actually. But if you do, the best solution is increase the default timeout, imo. |
Something is preventing the remote worker from responding in a timely manner, and we don't think that we should increase the default timeout without understanding what. Our hypothesis is that we're blocking the gRPC threads with action execution. If that's correct, we may need to change it to run actions in a separate thread pool. |
If this is a server-side problem, I don't think it's a release blocker and we should remove the tag. If it's a client-side problem with our RemoteSpawnStrategy, it probably is a blocker for 0.5.2. |
I haven't been able to reproduce so far. |
I wonder if 716b527 may have fixed this accidentally? I saw non-zero exit twice:
|
The non-zero exit is #3251, I believe. I still see it a lot. |
I think this is a release blocker.
Reproduction steps:
The text was updated successfully, but these errors were encountered: