Skip to content

[v18] Restore cache metrics#56196

Merged
rosstimothy merged 1 commit intobranch/v18from
tross/backport-56096/v18
Jun 30, 2025
Merged

[v18] Restore cache metrics#56196
rosstimothy merged 1 commit intobranch/v18from
tross/backport-56096/v18

Conversation

@rosstimothy
Copy link
Copy Markdown
Contributor

Backport #56096 to branch/v18

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 added backport no-changelog Indicates that a PR does not require a changelog entry labels Jun 27, 2025
@rosstimothy rosstimothy marked this pull request as ready for review June 27, 2025 19:14
@public-teleport-github-review-bot public-teleport-github-review-bot bot removed the request for review from fspmarshall June 30, 2025 14:52
@rosstimothy rosstimothy added this pull request to the merge queue Jun 30, 2025
Merged via the queue into branch/v18 with commit 6dd4cef Jun 30, 2025
47 checks passed
@rosstimothy rosstimothy deleted the tross/backport-56096/v18 branch June 30, 2025 15:20
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