Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

chore: update prom-client to v15.1.0 #6230

Merged
merged 1 commit into from
Dec 22, 2023

Conversation

nflaig
Copy link
Member

@nflaig nflaig commented Dec 22, 2023

Motivation

prom-client v15 includes some promising performance improvements

I also want to make sure v15 types are compatible with refactoring done in #6201.

Description

Update prom-client to v15.1.0

@nflaig nflaig requested a review from a team as a code owner December 22, 2023 09:47
@nflaig nflaig changed the base branch from unstable to nflaig/type-safe-metric-labels December 22, 2023 09:51
@nflaig nflaig force-pushed the nflaig/update-prom-client branch from ff29a2f to a4f0e5d Compare December 22, 2023 09:53
@nflaig nflaig merged commit 6324605 into nflaig/type-safe-metric-labels Dec 22, 2023
4 of 5 checks passed
@nflaig nflaig deleted the nflaig/update-prom-client branch December 22, 2023 09:55
wemeetagain pushed a commit that referenced this pull request Jan 2, 2024
* refactor: type safe metric labels

* Update metrics after merging unstable

* Remove collect method from GaugeExtra

* Remove startTimer method from Gauge interface

* Default to NoLabels to fix type issues

* Allow to partially set labels in startTimer

* Fix type compatibility with prom-client Histogram

* Sort metric types

* chore: update prom-client to v15.1.0 (#6230)

* Add metric type tests
@nflaig nflaig mentioned this pull request Jan 12, 2024
ensi321 pushed a commit to ensi321/lodestar that referenced this pull request Jan 22, 2024
* refactor: type safe metric labels

* Update metrics after merging unstable

* Remove collect method from GaugeExtra

* Remove startTimer method from Gauge interface

* Default to NoLabels to fix type issues

* Allow to partially set labels in startTimer

* Fix type compatibility with prom-client Histogram

* Sort metric types

* chore: update prom-client to v15.1.0 (ChainSafe#6230)

* Add metric type tests
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant