Skip to content

add support for guava instrumentation with 1.X.X#979

Merged
dhoard merged 2 commits intoprometheus:mainfrom
pheyken:prometheus-metrics-instrumentation-guava-keep-history
Aug 29, 2024
Merged

add support for guava instrumentation with 1.X.X#979
dhoard merged 2 commits intoprometheus:mainfrom
pheyken:prometheus-metrics-instrumentation-guava-keep-history

Conversation

@pheyken
Copy link
Contributor

@pheyken pheyken commented Aug 9, 2024

Hi all, this is my attempt on migrating one of the simpleclient collectors over to the new Prometheus 1.X Client. This is supposed to behave exactly the same as simpleclient_guava, thus keeping the exact same metrics and the same naming. Let me know if you see any issue with this and I'm happy to adopt the Collector

pheyken added 2 commits August 9, 2024 14:45
Signed-off-by: Petar Heyken <p.heyken@rebuy.com>
Signed-off-by: Petar Heyken <p.heyken@rebuy.com>
@pheyken pheyken force-pushed the prometheus-metrics-instrumentation-guava-keep-history branch from 267c1e6 to f7c4bca Compare August 9, 2024 12:45
@pheyken
Copy link
Contributor Author

pheyken commented Aug 22, 2024

Hey @fstab & @dhoard, if you have a few minutes to spare, a review would be highly appreciated

@dhoard dhoard merged commit c615b0a into prometheus:main Aug 29, 2024
@dhoard
Copy link
Collaborator

dhoard commented Aug 29, 2024

@pheyken merged. Thanks for the PR!

@pheyken pheyken deleted the prometheus-metrics-instrumentation-guava-keep-history branch August 29, 2024 05:52
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