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

Allow default exporter with additional Quarkiverse exporters #36500

Open
brunobat opened this issue Oct 16, 2023 · 3 comments
Open

Allow default exporter with additional Quarkiverse exporters #36500

brunobat opened this issue Oct 16, 2023 · 3 comments
Assignees
Labels
area/tracing good first issue Good for newcomers kind/enhancement New feature or request

Comments

@brunobat
Copy link
Contributor

brunobat commented Oct 16, 2023

Description

Currently, when an exporter is present the default exporter in not instantiated.
A list of exporters through CDI is possible with Quarkiverse exporters because users can activate and deactivate each one individually, however, we need a new property to activate the default exporter, even if other ones are present.
This way we can have many different exporters and the default, working together with CDI.

@azthec
Copy link
Contributor

azthec commented Oct 18, 2023

@brunobat I can take a stab at this issue if you want

@brunobat
Copy link
Contributor Author

brunobat commented Oct 18, 2023

Ok @azthec. Go for it!

@azthec
Copy link
Contributor

azthec commented Oct 18, 2023

@brunobat I've opened a PR, I think these are the changes you mean. I tried not to do any breaking changes but honestly I don't have to scope on the project to see what is affected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/tracing good first issue Good for newcomers kind/enhancement New feature or request
Projects
Status: In Progress
Development

No branches or pull requests

2 participants