Skip to content

[v16] Add metric to expose cluster role count#47812

Merged
rosstimothy merged 1 commit intobranch/v16from
tross/backport-47731/v16
Oct 22, 2024
Merged

[v16] Add metric to expose cluster role count#47812
rosstimothy merged 1 commit intobranch/v16from
tross/backport-47731/v16

Conversation

@rosstimothy
Copy link
Copy Markdown
Contributor

Backport #47731 to branch/v16

changelog: Added a new teleport_roles_total metric that exposes the number of roles which exist in a cluster

@aws-amplify-us-west-2
Copy link
Copy Markdown

This pull request is automatically being deployed by Amplify Hosting (learn more).

Access this pull request here: https://pr-47812.d212ksyjt6y4yg.amplifyapp.com

A new `teleport_roles_total` is added to count the number of roles
in the cluster. The metric is intentionally updated infrequently
to avoid putting any additional strain on the backend. Additionally,
the reads performed when calculating the metric are rate limited
to prevent any bursts in reads when the metric timer does fire.

The auth periodic tickers have also been consolidated into a single
multi-interval instead of a ticker per operation. To prevent the
sub-intervals from impacting each other all operations are moved
to their own goroutine once their ticker fires.
@rosstimothy rosstimothy force-pushed the tross/backport-47731/v16 branch from 28070d1 to acf74ef Compare October 22, 2024 15:31
@rosstimothy rosstimothy marked this pull request as ready for review October 22, 2024 15:53
@rosstimothy rosstimothy added this pull request to the merge queue Oct 22, 2024
Merged via the queue into branch/v16 with commit d86ebe9 Oct 22, 2024
@rosstimothy rosstimothy deleted the tross/backport-47731/v16 branch October 22, 2024 17:46
This was referenced Oct 22, 2024
@fheinecke fheinecke mentioned this pull request Apr 9, 2025
@fheinecke fheinecke mentioned this pull request Jan 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants