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

[collector] add back the logging exporter temporarily #906

Closed
TylerHelmuth opened this issue Oct 3, 2023 · 0 comments · Fixed by #907
Closed

[collector] add back the logging exporter temporarily #906

TylerHelmuth opened this issue Oct 3, 2023 · 0 comments · Fixed by #907
Labels
bug Something isn't working chart:collector Issue related to opentelemetry-collector helm chart

Comments

@TylerHelmuth
Copy link
Member

I realized that when we removed the logging exporter that we broke anyone who was relying on the default configuration when they included it in an exporters list. It is extremely common to add an exporter when using the chart - when people configure the exporters list with the new exporter AND the logging exporter, an upgrade to the latest chart would fail.

I think we should re-add a logging exporter configuration for now so that either can be used. We should also update the UPGRADING doc. In a future, telegraphed release we can remove the default logging export config.

@TylerHelmuth TylerHelmuth added bug Something isn't working chart:collector Issue related to opentelemetry-collector helm chart labels Oct 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working chart:collector Issue related to opentelemetry-collector helm chart
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant