Skip to content

Reduce apiserver metric cardinality#2531

Merged
simonpasquier merged 2 commits intoprometheus-operator:mainfrom
SuperQ:reduce_apiserver_metrics
Oct 24, 2024
Merged

Reduce apiserver metric cardinality#2531
simonpasquier merged 2 commits intoprometheus-operator:mainfrom
SuperQ:reduce_apiserver_metrics

Conversation

@SuperQ
Copy link
Copy Markdown
Contributor

@SuperQ SuperQ commented Oct 7, 2024

Description

Current releases of Kubernetes added new metrics that also have the same excessive bucket cardinality. Remove the extra buckets from these metrics.

  • apiserver_request_sli_duration_seconds_bucket
  • etcd_request_duration_seconds_bucket

Also reduce the cardinality of apiserver byte histogram metrics.

  • apiserver_request_bdy_size_bytes_bucket

This reduces API server metrics in our configuration by about 25%.

Type of change

  • CHANGE (fix or feature that would cause existing functionality to not work as expected)
  • FEATURE (non-breaking change which adds functionality)
  • BUGFIX (non-breaking change which fixes an issue)
  • ENHANCEMENT (non-breaking change which improves existing functionality)
  • NONE (if none of the other choices apply. Example, tooling, build system, CI, docs, etc.)

Changelog entry

- Reduce apiserver histogram metric cardinality by removing excess buckets.

Current releases of Kubernetes added new metrics that also have the same
excessive bucket cardinality. Remove the extra buckets from these
metrics.
* `apiserver_request_sli_duration_seconds_bucket`
* `etcd_request_duration_seconds_bucket`

Also reduce the cardinality of apiserver byte histogram metrics.
* `apiserver_request_bdy_size_bytes_bucket`

This reduces API server metrics in our configuration by about 25%.

Signed-off-by: SuperQ <superq@gmail.com>
@SuperQ SuperQ requested a review from a team as a code owner October 7, 2024 14:51
Copy link
Copy Markdown
Member

@metalmatze metalmatze left a comment

Choose a reason for hiding this comment

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

Nice! I like that a lot.

Copy link
Copy Markdown
Contributor

@simonpasquier simonpasquier left a comment

Choose a reason for hiding this comment

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

thanks!

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.

3 participants