remote-apis

remote-apis copied to clipboard

ActionCache: explicit delete operation?

6

The action cache currently does *not* have a delete operation. According to GCP, the RBE permissions include a delete permission. One could consider an update request without a result as...

ulfjack

Action pipelinining and automatic queuing

3

I was speaking to @edef1c this past weekend about some of her work on Nix, and we ended up talking a bit about the REAPI and the differences between Nix's...

alercah

Asset API: Use of Directory instead of Tree for PushDirectory

2

The use of the [Directory](https://github.com/bazelbuild/remote-apis/blob/master/build/bazel/remote/execution/v2/remote_execution.proto#L697-L710) instead of [Tree](https://github.com/bazelbuild/remote-apis/blob/master/build/bazel/remote/execution/v2/remote_execution.proto#L1075-L1084) for [Push](https://github.com/bazelbuild/remote-apis/blob/master/build/bazel/remote/asset/v1/remote_asset.proto#L409-L441)/[Fetch](https://github.com/bazelbuild/remote-apis/blob/master/build/bazel/remote/asset/v1/remote_asset.proto#L287-L315) Directory leads to additional overhead on each Fetch which could instead be performed once by the client when pushing the...

Qinusty

Directory canonicalization does not require non-empty names

2

Hey folks! The definition of "canonical" for a `Directory` does not currently (https://github.com/bazelbuild/remote-apis/blob/f54876595da9f2c2d66c98c318d00b60fd64900b/build/bazel/remote/execution/v2/remote_execution.proto#L634-L653) require that the names of children are non-empty, but in practice this seems like a useful constraint....

stuhood

REv3 idea: CAS.ExpandTree()?

16

Build systems like Bazel currently treat directory outputs as being opaque. There is no support for addressing individual files contained within. If Bazel wants to pass on a directory output...

EdSchouten

Clarify what a client should do on FAILED_PRECONDITION

13

As discussed with @ola-rozenfeld, we need to clarify what the client should do in case of FAILED_PRECONDITION https://github.com/bazelbuild/remote-apis/blob/master/build/bazel/remote/execution/v2/remote_execution.proto#L101 In particular for any blobs that are reported as MISSING, the client...

sstriker

Execute keep alive response

6

Clarify and document the behavior for using an operation response with done=false. This has not been documented with expected behavior by the server or client, and should be detailed as...

werkt

A number of material differences exist for Trees throughout the API: Directories are uploaded individually as blobs. Trees are fetched in a list streaming fashion from getTree OutputDirectories are represented...

werkt

Remove the ActionCache in V3

2

For V3, I think it is completely possible to implement the ActionCache in terms of the Asset API. This would allow it to be more flexible and less specialized. To...

edbaunton

can't import v2: need semantic import versioning

7

with `require github.com/bazelbuild/remote-apis v2.0.0` in `go.mod` go claims ``` go: finding github.com/bazelbuild/remote-apis v2.0.0 go: finding github.com/bazelbuild/remote-apis v2.0.0 go: errors parsing go.mod: /workspace/go.mod:10: require github.com/bazelbuild/remote-apis: version "v2.0.0" invalid: module contains a...

ukai