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 prometheus/client_golang to 1.17.0 #6192

Merged
merged 1 commit into from
Oct 3, 2023
Merged

Conversation

bboreham
Copy link
Contributor

@bboreham bboreham commented Oct 2, 2023

What this PR does

Update prometheus/client_golang to 1.17.0, particularly to bring in a performance fix prometheus/client_golang#1296.

Most changes are fixing spelling or punctuation in spelling, but there is also the addition of creation timestamps from prometheus/client_golang#1313.

Checklist

  • NA Tests updated
  • NA Documentation added
  • NA CHANGELOG.md updated - not user-visible.

@bboreham bboreham requested review from grafanabot and a team as code owners October 2, 2023 17:04
@aknuds1
Copy link
Contributor

aknuds1 commented Oct 3, 2023

I restarted the failing test, to see if it's a flake.

@krajorama
Copy link
Contributor

duplicate of #6200

@aknuds1
Copy link
Contributor

aknuds1 commented Oct 3, 2023

Seems to keep segfaulting in TestBucketStore_ManyParts_e2e/inmem/0,streamingBatchSize=0. Same thing happens in the dependabot PR, guess it needs fixing.

@aknuds1
Copy link
Contributor

aknuds1 commented Oct 3, 2023

I've fixed the failing test in #6203.

@aknuds1
Copy link
Contributor

aknuds1 commented Oct 3, 2023

Rebase the PR and it should be good @bboreham :)

Particularly to bring in a performance fix prometheus/client_golang#1296.

Most changes are fixing spelling or punctuation in spelling, but there
is also the addition of creation timestamps from prometheus/client_golang#1313.
Copy link
Contributor

@aknuds1 aknuds1 left a comment

Choose a reason for hiding this comment

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

LGTM!

@bboreham bboreham enabled auto-merge (squash) October 3, 2023 13:56
@bboreham bboreham merged commit 2f3309f into main Oct 3, 2023
28 checks passed
@bboreham bboreham deleted the prom-client-1-17 branch October 3, 2023 14:25
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.

3 participants