Skip to content

Commit

Permalink
[remote asset api] mention RESOURCE_EXHAUSTED with other error status…
Browse files Browse the repository at this point in the history
… codes (#239)

This status value was already mentioned in the Fetch service comments,
but was not included in the FetchBlobResponse or FetchDirectoryResponse
status field comments alongside some other similar values. Let's make
this more clear.
  • Loading branch information
mostynb authored Feb 27, 2023
1 parent ec8e5df commit 55153ba
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 0 deletions.
4 changes: 4 additions & 0 deletions build/bazel/remote/asset/v1/remote_asset.pb.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 4 additions & 0 deletions build/bazel/remote/asset/v1/remote_asset.proto
Original file line number Diff line number Diff line change
Expand Up @@ -216,6 +216,8 @@ message FetchBlobResponse {
// requested an asset from a disallowed origin.
// * `ABORTED`: The operation could not be completed, typically due to a
// failed consistency check.
// * `RESOURCE_EXHAUSTED`: There is insufficient quota of some resource to
// perform the requested operation. The client may retry after a delay.
google.rpc.Status status = 1;

// The uri from the request that resulted in a successful retrieval, or from
Expand Down Expand Up @@ -295,6 +297,8 @@ message FetchDirectoryResponse {
// requested an asset from a disallowed origin.
// * `ABORTED`: The operation could not be completed, typically due to a
// failed consistency check.
// * `RESOURCE_EXHAUSTED`: There is insufficient quota of some resource to
// perform the requested operation. The client may retry after a delay.
google.rpc.Status status = 1;

// The uri from the request that resulted in a successful retrieval, or from
Expand Down

0 comments on commit 55153ba

Please sign in to comment.