-
Notifications
You must be signed in to change notification settings - Fork 476
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
Update mimir-prometheus, and remove support for selecting chunk mapper. #2762
Update mimir-prometheus, and remove support for selecting chunk mapper. #2762
Conversation
bb6f9f9
to
f498cb7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! But see my comment about default config, please 😉
# (advanced) The size of the write queue used by the head chunks mapper. Lower | ||
# values reduce memory utilisation at the cost of potentially higher ingest | ||
# latency. Value of 0 switches chunks mapper to implementation without a | ||
# queue. | ||
# CLI flag: -blocks-storage.tsdb.head-chunks-write-queue-size | ||
[head_chunks_write_queue_size: <int> | default = 0] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would change the default to 1M. It's what we use at Grafana Labs in prod, and we should upstream for the rest of the world too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done in the latest commit c96cca3
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for addressing my feedback. Just a tiny edit to CHANGELOG and 🚀 !
Signed-off-by: Peter Štibraný <pstibrany@gmail.com>
…r we have. Signed-off-by: Peter Štibraný <pstibrany@gmail.com>
Signed-off-by: Peter Štibraný <pstibrany@gmail.com>
…size to 1000000. Signed-off-by: Peter Štibraný <pstibrany@gmail.com>
3193dc1
to
6bcfcd9
Compare
Signed-off-by: Peter Štibraný <pstibrany@gmail.com>
What this PR does
This PR updates mimir-prometheus, which removes support for running with old chunk mapper. This support is also removed from Mimir.
Using new chunk mapper is no longer considered experimental. Option to enable async queue is now "advanced" instead. (Queue is disabled by default, maybe we want to change that?)
Checklist
CHANGELOG.md
updated - the order of entries should be[CHANGE]
,[FEATURE]
,[ENHANCEMENT]
,[BUGFIX]