Skip to content

ci: output build cache hit rate as GHA annotation#50361

Merged
jkleinsc merged 1 commit intomainfrom
ci/cache-hit-rate-gha
Mar 19, 2026
Merged

ci: output build cache hit rate as GHA annotation#50361
jkleinsc merged 1 commit intomainfrom
ci/cache-hit-rate-gha

Conversation

@dsanders11
Copy link
Copy Markdown
Member

@dsanders11 dsanders11 commented Mar 19, 2026 •
edited
Loading

Description of Change

Small quality of life improvement here so you can easily see the effective cache hit rate for any given build without needing to open the 20 MB log file and search for this line.

This is what it will look like, as an annotation at the top of every build job:

Example Screenshot

Checklist

  • PR description included

Release Notes

Notes: none

@dsanders11 dsanders11 added target/39-x-y PR should also be added to the "39-x-y" branch. target/40-x-y PR should also be added to the "40-x-y" branch. target/41-x-y PR should also be added to the "41-x-y" branch. target/42-x-y PR should also be added to the "42-x-y" branch. labels Mar 19, 2026
@dsanders11 dsanders11 marked this pull request as ready for review March 19, 2026 04:05
jkleinsc
jkleinsc approved these changes Mar 19, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@jkleinsc jkleinsc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

❤️ Thanks for adding this @dsanders11!

@jkleinsc jkleinsc merged commit 1ad832a into main Mar 19, 2026
83 checks passed
@jkleinsc jkleinsc deleted the ci/cache-hit-rate-gha branch March 19, 2026 13:41
@release-clerk
Copy link
Copy Markdown

release-clerk bot commented Mar 19, 2026

No Release Notes

This was referenced Mar 19, 2026
Merged
Merged
Merged
Merged
@trop
Copy link
Copy Markdown
Contributor

trop bot commented Mar 19, 2026

I have automatically backported this PR to "40-x-y", please check out #50368

@trop
Copy link
Copy Markdown
Contributor

trop bot commented Mar 19, 2026

I have automatically backported this PR to "39-x-y", please check out #50369

@trop trop bot removed the target/40-x-y PR should also be added to the "40-x-y" branch. label Mar 19, 2026
@trop
Copy link
Copy Markdown
Contributor

trop bot commented Mar 19, 2026

I have automatically backported this PR to "41-x-y", please check out #50370

@trop
Copy link
Copy Markdown
Contributor

trop bot commented Mar 19, 2026

I have automatically backported this PR to "42-x-y", please check out #50371

@trop trop bot added in-flight/39-x-y in-flight/41-x-y in-flight/42-x-y merged/40-x-y PR was merged to the "40-x-y" branch. and removed target/39-x-y PR should also be added to the "39-x-y" branch. target/41-x-y PR should also be added to the "41-x-y" branch. target/42-x-y PR should also be added to the "42-x-y" branch. in-flight/40-x-y in-flight/42-x-y labels Mar 19, 2026
@trop trop bot added merged/42-x-y PR was merged to the "42-x-y" branch. merged/41-x-y PR was merged to the "41-x-y" branch. merged/39-x-y PR was merged to the "39-x-y" branch. and removed in-flight/41-x-y in-flight/39-x-y labels Mar 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jkleinsc jkleinsc jkleinsc approved these changes

Assignees

No one assigned

Labels

merged/39-x-y PR was merged to the "39-x-y" branch. merged/40-x-y PR was merged to the "40-x-y" branch. merged/41-x-y PR was merged to the "41-x-y" branch. merged/42-x-y PR was merged to the "42-x-y" branch. semver/none

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@dsanders11 @jkleinsc