remote-apis-sdks icon indicating copy to clipboard operation
remote-apis-sdks copied to clipboard

support parsing RemoteExecutionLog

Open asuffield opened this issue 4 years ago • 3 comments

One of the tiny number of things https://github.com/bazelbuild/tools_remote can do and these tools can't is parse the RemoteExecutionLog. It would be nice to figure out a solution to that.

Annoyingly, that proto isn't part of github.com/bazelbuild/remote-apis so this is a more thorny puzzle than just implementing the (fairly simple) features.

asuffield avatar Oct 02 '20 16:10 asuffield

The intersection between remote-apis (which is technically independent of Bazel, despite being hosted there), Bazel, and support tooling has always been tricky. We definitely shouldn't have Bazel-specific logic in this repository. That likely leads to...tools_remote being a wrapper that glues together Bazel-specific log parsing with this SDK? But I think you were just referred away from tools_remote...

bergsieker avatar Oct 02 '20 16:10 bergsieker

Indeed, tools_remote appears to be dead now. Turning this repo into a suitable base for solving this puzzle would be reasonable.

Alternatively, standardising RemoteExecutionLog would also be reasonable, it's not very strongly tied to bazel, and any remote execution client could produce something similar.

I'm not sure what's the most correct direction here.

asuffield avatar Oct 02 '20 16:10 asuffield

Right, the intention was that this repository would be the generic remote client solution that subsumes everything that tools_remote was, except for parts that are entirely Bazel-specific, e.g. help debugging Bazel invocations -- those would be left with tools_remote. And we always thought that RemoteExecutionLog falls in that category.

You have a good point on RemoteExecutionLog being potentially standardizable... but for now, it is a part of Bazel, and defined there. Changing that would not be easy.

Would using tools_remote for Bazel-specific debugging, and using this repository for everything else work for you?

ola-rozenfeld avatar Oct 02 '20 16:10 ola-rozenfeld