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

Vendor update mimir-prometheus at aa96f2e80ba9 #10168

Merged
merged 6 commits into from
Dec 9, 2024

Conversation

jhesketh
Copy link
Contributor

@jhesketh jhesketh commented Dec 9, 2024

Which is upstream Prometheus at b9dd95f499aa2912d9cfbcfddfbdd6bc0a5207a5

grafana/mimir-prometheus#786 has the relevant changes

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.

@jhesketh jhesketh marked this pull request as ready for review December 9, 2024 04:33
@jhesketh jhesketh requested review from stevesg, grafanabot and a team as code owners December 9, 2024 04:33
@krajorama krajorama self-requested a review December 9, 2024 07:37
Signed-off-by: György Krajcsovits <[email protected]>
Copy link
Contributor

@krajorama krajorama left a comment

Choose a reason for hiding this comment

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

LGTM

@jhesketh jhesketh merged commit 76ab843 into grafana:main Dec 9, 2024
29 checks passed
@jhesketh jhesketh deleted the jhesketh/update-mimir-prometheus branch December 10, 2024 04:27
krajorama added a commit that referenced this pull request Dec 11, 2024
Looked through changes from #10102, #10123, #10136, #10168, #10188

Signed-off-by: György Krajcsovits <[email protected]>
krajorama added a commit that referenced this pull request Dec 11, 2024
…12d74 (#10188)

* Vendor update mimir-prometheus at 2cb07a115f0ccdd767d07e02e2cb4e29a7a24206

This is upstream at
https://github.com/prometheus/prometheus/tree/af2a1cb10c89de496cd4309ac532624b34112d74

Changes listed in
grafana/mimir-prometheus#787

* make generate-otlp

* Sync upstream tests

* Fix and improve changes/resets functions

* fix(test): time functions ignore native histograms

No need to test query sharding on time functions for native histograms
since prometheus/prometheus#15479 . As those
functions do not return samples on native histograms.

* Update changelog retroactively

Looked through changes from #10102, #10123, #10136, #10168, #10188

Signed-off-by: György Krajcsovits <[email protected]>
Co-authored-by: Joshua Hesketh <[email protected]>
bjorns163 pushed a commit to bjorns163/mimir that referenced this pull request Dec 30, 2024
* Vendor update mimir-prometheus at aa96f2e80ba9

Which is upstream Prometheus at b9dd95f499aa2912d9cfbcfddfbdd6bc0a5207a5

* Do not use new limit in storage.NewMergeSeriesSet

* MQE: Fix our test cases

Mostly just annotation infos.

* MQE: re-sync upstream tests

* Update otlp

* Update changelog

Signed-off-by: György Krajcsovits <[email protected]>

---------

Signed-off-by: György Krajcsovits <[email protected]>
Co-authored-by: György Krajcsovits <[email protected]>
bjorns163 pushed a commit to bjorns163/mimir that referenced this pull request Dec 30, 2024
…12d74 (grafana#10188)

* Vendor update mimir-prometheus at 2cb07a115f0ccdd767d07e02e2cb4e29a7a24206

This is upstream at
https://github.com/prometheus/prometheus/tree/af2a1cb10c89de496cd4309ac532624b34112d74

Changes listed in
grafana/mimir-prometheus#787

* make generate-otlp

* Sync upstream tests

* Fix and improve changes/resets functions

* fix(test): time functions ignore native histograms

No need to test query sharding on time functions for native histograms
since prometheus/prometheus#15479 . As those
functions do not return samples on native histograms.

* Update changelog retroactively

Looked through changes from grafana#10102, grafana#10123, grafana#10136, grafana#10168, grafana#10188

Signed-off-by: György Krajcsovits <[email protected]>
Co-authored-by: Joshua Hesketh <[email protected]>
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