-
Notifications
You must be signed in to change notification settings - Fork 221
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
New otelcol.exporter.debug
component
#1043
Conversation
c7e15cd
to
738866f
Compare
This comment was marked as outdated.
This comment was marked as outdated.
c611f39
to
46afb9c
Compare
docs/sources/reference/components/otelcol/otelcol.exporter.logging.md
Outdated
Show resolved
Hide resolved
docs/sources/reference/components/otelcol/otelcol.exporter.logging.md
Outdated
Show resolved
Hide resolved
8b1c89d
to
620d846
Compare
Do you think that we need this component now that we have the live debugging feature which supports all otel receivers and processors? |
@wildum I agree that in the long term we should recommend that people use Live Debugging. However, I think there will always be a need for us to have this exporter:
In the long term, I think we should keep this component but we can have a banner in the docs that recommends Live Debugging instead. |
Docs look OK to me now. :-) |
5c33fac
to
bf63ea6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, just one nit
PR Description
The
debug
exporter is a replacement for thelogging
exporter. This exporter was also added to the Agent.PR Checklist