bazelbuild / bazelbuild/remote-apis

Cache Capability to indicate that CAS is read-only

Open
#297 2 comments 0 reactions 0 assignees View on GitHub
Dominant language
Go
Stars
445
Forks
141
PR merge metrics
No merged PRs in 30d

Description

We need to signal to the client that in some configurations, the CAS will only accept read requests. All write requests will result in errors.

Given that we are currently advertising

```protobuf
// Describes the server/instance capabilities for updating the action cache.
message ActionCacheUpdateCapabilities {
bool update_enabled = 1;
}
```

in `CacheCapabilities` proto. I was thinking that we could simply add another identical field for CASUpdateCapabilities.

However, such a field would default to `false`, which is troublesome for backward compatibility. And doing `bool is_readonly = 1;` feels out of spec. Looking for feedback on this design.

Contributor guide

Open the contributing guide

Research direction

Review the CacheCapabilities proto and the existing ActionCacheUpdateCapabilities message described in the issue. Determine how CAS read-only capability signaling can preserve backward compatibility, then document the agreed field design and update the API definition accordingly.

Written by the indexing model from the issue text.

Assessment

Domain
api
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.