Skip to content

BLA-4010 add build cache operation observers#3

Draft
shreyas-blacksmith wants to merge 1 commit into
shreyas/bla-4006-bazel-cache-prefixingfrom
shreyas/bla-4010-build-cache-metrics-bazel-remote
Draft

BLA-4010 add build cache operation observers#3
shreyas-blacksmith wants to merge 1 commit into
shreyas/bla-4006-bazel-cache-prefixingfrom
shreyas/bla-4010-build-cache-metrics-bazel-remote

Conversation

@shreyas-blacksmith

@shreyas-blacksmith shreyas-blacksmith commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds generic build-cache operation observation hooks to the Blacksmith bazel-remote fork. This is intentionally FA/Web agnostic: bazel-remote only forwards opaque labels and semantic cache outcomes.

Metric contract

  • action_cache_get: hit | miss | error
  • cas_lookup: hit | miss | error
  • backend_upload: error | dropped

Client transfer bytes are not inferred in bazel-remote. FA observes gRPC payloads and emits client_upload / client_download rows.

Safety

  • Nil observer preserves current behavior.
  • Observer panics are swallowed, so metrics cannot affect cache serving.
  • No FA, Web, Laravel, tenant, repo, or job concepts are interpreted in this repo.

Tests

  • env GOCACHE=/private/tmp/codex-go-cache-bazel-remote go test ./cache/...
  • git diff --check

View with Codesmith Autofix with Codesmith
Need help on this PR? Tag /codesmith with what you need. Autofix is disabled.


View with Codesmith
Need help on this PR? Tag /codesmith with what you need. Autofix is enabled. (Staging)

@linear-code

linear-code Bot commented Jun 14, 2026

Copy link
Copy Markdown

BLA-4010

@blacksmith-staging blacksmith-staging Bot force-pushed the shreyas/bla-4010-build-cache-metrics-bazel-remote branch from 8005d87 to a0992bd Compare June 15, 2026 18:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant