Ed Schouten

My recommendation would actually to keep it. Or alternatively, provide another repo where we serve pre-generated copies of this file. Otherwise there is no uniform URL under which this protocol...

I don't really understand what the issue is here. - If an infrastructure failure occurred while running the action, Execute() will return an ExecuteResponse that has a non-null status. Even...

If workers are shutting down gracefully, why are they sending SIGTERM to the build action? That's not graceful. If workers send SIGTERM to an action, they should already record some...

> For completeness, there's an option 3, which is to let the client explicitly mark mandatory outputs in the Command message. (After some spelunking I did yesterday, it turns out...

The issue is that it's virtually impossible for the server to validate the ActionResult. File existence may not be sufficient. For example, I've seen cases where programs terminate with exit...

Exactly what @tjgq says. :-) The AC entry that is created in case of the erroneous action is when taken in isolation valid. You're only touching 1.txt, so you get...

@tjgq Then we're actually going back and forth on this. In the past it wasn't permitted to write AC entries with `exit_code != 0`, but some people outside of Bazel...

> As for @EdSchouten suggestion about enhancing `GetCapabilitiesRequest` for better negotiation. I think that should be done in a separate PR from this one? Wdyt? Why can't we just do...

No, it doesn't. The presence of ZSTD_DICT can be used by the client to determine whether that additional RPC is supported.

Though this proposal may solve certain annoyances, it does not fix some other issues: - The inability to upload files by uploading missing chunks. Given the fact that uploading is...