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

Improve lock contention affecting read and write latencies during TSDB head compaction (cherry-pick Prometheus PR 15242) #9822

Merged
merged 3 commits into from
Nov 4, 2024

Conversation

pracucci
Copy link
Collaborator

@pracucci pracucci commented Nov 4, 2024

What this PR does

Update mimir-prometheus (from main) to get the cherry-pick of prometheus/prometheus#15242

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.

Signed-off-by: Marco Pracucci <marco@pracucci.com>
@pracucci pracucci requested review from stevesg, grafanabot and a team as code owners November 4, 2024 09:10
Copy link
Contributor

@colega colega left a comment

Choose a reason for hiding this comment

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

Change LGTM, so approving, but I think this deserves a changelog entry (especially since it's very likely to be backported to 2.14)

Signed-off-by: Marco Pracucci <marco@pracucci.com>
Signed-off-by: Marco Pracucci <marco@pracucci.com>
@pracucci
Copy link
Collaborator Author

pracucci commented Nov 4, 2024

Change LGTM, so approving, but I think this deserves a changelog entry (especially since it's very likely to be backported to 2.14)

Absolutely! Done.

@pracucci pracucci changed the title Cherry-pick Prometheus PR 15242 Improve lock contention affecting read and write latencies during TSDB head compaction (cherry-pick Prometheus PR 15242) Nov 4, 2024
@pracucci pracucci enabled auto-merge (squash) November 4, 2024 09:25
@pracucci pracucci merged commit 03f2f06 into main Nov 4, 2024
29 checks passed
@pracucci pracucci deleted the main-with-prometheus-pr-15242 branch November 4, 2024 09:33
aknuds1 added a commit that referenced this pull request Nov 4, 2024
* [release-2.14] fix(deps): update github.com/thanos-io/objstore digest to f90c89a (main) (#9625)

* Update github.com/thanos-io/objstore digest to f90c89a (#9534)

Signed-off-by: Arve Knudsen <arve.knudsen@gmail.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
(cherry picked from commit 3c97a61)

* update changelog

Signed-off-by: Vladimir Varankin <vladimir.varankin@grafana.com>

---------

Signed-off-by: Vladimir Varankin <vladimir.varankin@grafana.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Vladimir Varankin <vladimir.varankin@grafana.com>

* 2.14.1 Prepare patch release (#9796)

* bump patch version

Signed-off-by: Vladimir Varankin <vladimir.varankin@grafana.com>

* rebuild assets

Signed-off-by: Vladimir Varankin <vladimir.varankin@grafana.com>

---------

Signed-off-by: Vladimir Varankin <vladimir.varankin@grafana.com>

* MQE: Fix handling of string results (#9803)

This would previously panic after the query was closed

* chore(deps): update grafana/mimirtool docker tag to v2.14.1 (#9806)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

* Revert upgrade to google.golang.org/grpc v1.66.2 (#9811)

* Revert "Don't free buffers after reading query stream (#9721)"

This reverts commit f7b6017.

* Revert: Upgrade to google.golang.org/grpc v1.66.2 / modify certain protobuf messages to retain their unmarshaling buffer (#9401)

Signed-off-by: Yuri Nikolic <durica.nikolic@grafana.com>

* Revert "Distributor.queryIngesterStream: Free gRPC buffers upon error (#9756)"

This reverts commit eda1a4b.

---------

Signed-off-by: Yuri Nikolic <durica.nikolic@grafana.com>

* Improve lock contention affecting read and write latencies during TSDB head compaction (cherry-pick Prometheus PR 15242) (#9822)

* Cherry-pick Prometheus PR 15242

Signed-off-by: Marco Pracucci <marco@pracucci.com>

* Added CHANGELOG entry

Signed-off-by: Marco Pracucci <marco@pracucci.com>

* Updated CHANGELOG

Signed-off-by: Marco Pracucci <marco@pracucci.com>

---------

Signed-off-by: Marco Pracucci <marco@pracucci.com>

---------

Signed-off-by: Vladimir Varankin <vladimir.varankin@grafana.com>
Signed-off-by: Yuri Nikolic <durica.nikolic@grafana.com>
Signed-off-by: Marco Pracucci <marco@pracucci.com>
Co-authored-by: Grot (@grafanabot) <43478413+grafanabot@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Vladimir Varankin <vladimir.varankin@grafana.com>
Co-authored-by: Joshua Hesketh <joshua.hesketh@grafana.com>
Co-authored-by: Đurica Yuri Nikolić <durica.nikolic@grafana.com>
Co-authored-by: Marco Pracucci <marco@pracucci.com>
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