Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove "track_sizes" option from Memcached in jsonnet and helm #10032

Merged
merged 2 commits into from
Nov 27, 2024

Conversation

56quarters
Copy link
Contributor

@56quarters 56quarters commented Nov 27, 2024

What this PR does

The option tracks the size of items in the cache in a histogram. We don't use this information for anything or monitor it in any way so remove it.

Which issue(s) this PR fixes or relates to

N/A

Checklist

  • Tests updated.
  • Documentation added.
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX].
  • about-versioning.md updated with experimental features.

The option tracks the size of items in the cache in a histogram. We don't
use this information for anything or monitor it in any way so remove it.

Signed-off-by: Nick Pillitteri <[email protected]>
Signed-off-by: Nick Pillitteri <[email protected]>
@56quarters 56quarters marked this pull request as ready for review November 27, 2024 14:23
@56quarters 56quarters requested a review from a team as a code owner November 27, 2024 14:23
Copy link
Contributor

@dimitarvdimitrov dimitarvdimitrov left a comment

Choose a reason for hiding this comment

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

Is this adding a lot of extra cardinality?

@56quarters
Copy link
Contributor Author

Is this adding a lot of extra cardinality?

It's not actually exposed as Prometheus metrics, just the stats sizes Memcached command. It's not enabled by default since there's some overhead but the counts are actually lossy because they don't use locking or atomics.

@56quarters 56quarters merged commit 2cedd5f into main Nov 27, 2024
31 checks passed
@56quarters 56quarters deleted the 56quarters/rm-track-sizes branch November 27, 2024 15:26
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.

2 participants