Skip to content

[v17] Restore cache metrics#56198

Merged
rosstimothy merged 1 commit intobranch/v17from
tross/backport-56096/v17
Jun 30, 2025
Merged

[v17] Restore cache metrics#56198
rosstimothy merged 1 commit intobranch/v17from
tross/backport-56096/v17

Conversation

@rosstimothy
Copy link
Copy Markdown
Contributor

Backport the applicable subset of #56096 to branch/v17

@rosstimothy rosstimothy added backport no-changelog Indicates that a PR does not require a changelog entry labels Jun 27, 2025
@rosstimothy rosstimothy force-pushed the tross/backport-56096/v17 branch from 03df653 to 56c592d Compare June 27, 2025 19:01
During the cache storage refactoring, all of the cache metrics
that were recorded via the backend.Reporter that wrapped the
cache backend were lost. This attempts to restore the metrics
as close to the original meaning as possible, though due to the
changes some labels are different and batch metrics don't apply.
The biggest change is in the top requests metrics which has
historically been based on the backend.Key, but since that
the cache no longer deals with backends, the metric is now labeled
by the resource kind. All existing dashboards and tctl top should
still work as expected.
@rosstimothy rosstimothy force-pushed the tross/backport-56096/v17 branch from 56c592d to 9389527 Compare June 27, 2025 19:07
@rosstimothy rosstimothy marked this pull request as ready for review June 27, 2025 19:28
@rosstimothy rosstimothy added this pull request to the merge queue Jun 30, 2025
Merged via the queue into branch/v17 with commit 917f7e6 Jun 30, 2025
41 checks passed
@rosstimothy rosstimothy deleted the tross/backport-56096/v17 branch June 30, 2025 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport no-changelog Indicates that a PR does not require a changelog entry size/md

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants