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

Introduce a way to completely customize MeterRegistry #34573

Merged
merged 3 commits into from Jul 7, 2023

Conversation

geoand
Copy link
Contributor

@geoand geoand commented Jul 6, 2023

This is done via the new MeterRegistryCustomizer interface
the implementations of which are meant to be registered as
CDI beans.

Closes: #32996

P.S. The customizer pattern is heavily used in Quarkus while the qualifier constraint pattern
is the same idea as used for MeterFilter classes.

@quarkus-bot
Copy link

quarkus-bot bot commented Jul 6, 2023

/cc @brunobat (micrometer), @ebullient (micrometer)

@geoand geoand marked this pull request as draft July 6, 2023 10:50
@geoand
Copy link
Contributor Author

geoand commented Jul 6, 2023

If we think this makes sense, I will update the PR with a docs update

@quarkus-bot

This comment has been minimized.

This is done via the new `MeterRegistryCustomizer` interface
the implementations of which are meant to be registered as
CDI beans.

Closes: quarkusio#32996
@geoand
Copy link
Contributor Author

geoand commented Jul 7, 2023

PR updated with a commit that adds the new stuff to the docs.

@geoand geoand marked this pull request as ready for review July 7, 2023 06:23
@quarkus-bot quarkus-bot bot added this to To do in Quarkus Documentation Jul 7, 2023
@github-actions
Copy link

github-actions bot commented Jul 7, 2023

🙈 The PR is closed and the preview is expired.

@quarkus-bot
Copy link

quarkus-bot bot commented Jul 7, 2023

✔️ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

@geoand geoand merged commit 34e133b into quarkusio:main Jul 7, 2023
40 of 41 checks passed
Quarkus Documentation automation moved this from To do to Done Jul 7, 2023
@geoand geoand deleted the #32996 branch July 7, 2023 07:38
@quarkus-bot quarkus-bot bot added the kind/enhancement New feature or request label Jul 7, 2023
@quarkus-bot quarkus-bot bot added this to the 3.3 - main milestone Jul 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

Provide a way to customize quarkus' micrometer MeterRegistry/CompositeMeterRegistry
2 participants