Skip to content

BLA-4010 add build cache operation observers#6

Merged
shreyas-blacksmith merged 1 commit into
mainfrom
shreyas/bla-4006-bazel-cache-prefixing
Jun 22, 2026
Merged

BLA-4010 add build cache operation observers#6
shreyas-blacksmith merged 1 commit into
mainfrom
shreyas/bla-4006-bazel-cache-prefixing

Conversation

@shreyas-blacksmith

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

Copy link
Copy Markdown
Contributor

Summary

  • Add build cache operation observer plumbing for disk and s3proxy cache operations.
  • Carry request-scoped metrics context through cache operations so downstream observers can attribute operations by org/repo/prefix metadata.
  • Document the cache prefixing and observer behavior in BLACKSMITH.md.

Rebase note

This branch was rewritten onto current main after BLA-4013 bazel cache prefixing (#5) landed. The old BLA-4006/BLA-4013 commits are already represented by main; this PR now contains only the BLA-4010 observer delta.

Testing

  • make gofmt
  • go test ./...

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)

* BLA-4010 add build cache operation observers

* BLA-4010 drop unused RunID metrics label

Co-authored-by: Cursor <cursoragent@cursor.com>

* BLA-4010 drop unused OperationOutcome Kind/HasKind

Co-authored-by: Cursor <cursoragent@cursor.com>

---------

Co-authored-by: Cursor <cursoragent@cursor.com>
@linear-code

linear-code Bot commented Jun 22, 2026

Copy link
Copy Markdown

BLA-4010

BLA-4006

@shreyas-blacksmith shreyas-blacksmith merged commit 6d5a625 into main Jun 22, 2026
4 checks passed
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