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

Update vendor of mimir-prometheus to latest commit #3261

Merged
merged 1 commit into from
Oct 20, 2022

Conversation

56quarters
Copy link
Contributor

@56quarters 56quarters commented Oct 19, 2022

What this PR does

Update our vendor of Prometheus to c77b24bcb23f9c2281ee9596be272f884a9c2bd7 which contains the upstream OOO implementation. Most of the other changes are related to changes to the labels builder.

Signed-off-by: Nick Pillitteri nick.pillitteri@grafana.com

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]

@56quarters 56quarters force-pushed the 56quarters/prometheus-update branch 2 times, most recently from a8db480 to 13a19aa Compare October 19, 2022 19:31
@56quarters 56quarters changed the title Update vendor of mimir-prometheus to c77b24bcb23f9c2281ee9596be272f884a9c2bd7 Update vendor of mimir-prometheus to latest commit Oct 19, 2022
Update our vendor of Prometheus to c77b24bcb23f9c2281ee9596be272f884a9c2bd7
which contains the upstream OOO implementation. Most of the other changes
are related to changes to the labels builder.

Signed-off-by: Nick Pillitteri <nick.pillitteri@grafana.com>
@56quarters 56quarters marked this pull request as ready for review October 19, 2022 19:49
@56quarters 56quarters requested review from osg-grafana and a team as code owners October 19, 2022 19:49
@56quarters 56quarters merged commit 31d9b85 into main Oct 20, 2022
@56quarters 56quarters deleted the 56quarters/prometheus-update branch October 20, 2022 14:27
Copy link
Collaborator

@pracucci pracucci left a comment

Choose a reason for hiding this comment

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

I think we need to address #3175 in this PR. Since this PR has been merged, can you look at the issue and eventually follow up, please?

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.

None yet

3 participants