-
Notifications
You must be signed in to change notification settings - Fork 504
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
Remove logging exporter references #1040
Comments
@liustanley I believe you'll only run into this issue if you use the image created from |
I was imagining we'd keep the logging export in the helm chart until it is officially removed. |
@TylerHelmuth We're using a fork of opentelemetry-collector-contrib with these helm charts, so this broke our CI workflows when we updated our fork with upstream. |
I think this config will remove it from the confmap:
|
Thanks! That seems to work |
This PR was merged recently which removed the logging exporter from the list of valid exporters: open-telemetry/opentelemetry-collector-contrib#30408. This results in the following errors when using the helm chart:
We should revert the changes made in this PR in order to remove references to the logging exporter: #907
The text was updated successfully, but these errors were encountered: