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

Add OTLP LogExporter to the example #1168

Merged
merged 4 commits into from
Jul 29, 2023

Conversation

cijothomas
Copy link
Member

#1164 (comment)

Adds Logging to the OTLP examples, along with nit cleanups.

@cijothomas cijothomas requested a review from a team July 27, 2023 16:57

exporters:
logging:
loglevel: debug

jaeger:
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this was cleaned up as the purpose of this example is to show how to use OTLP Exporter. The config will print the telemetry to stdout which is good enough to confirm things working. Even if there is a need to use Jaeger to visualize, Jaeger can natively understand OTLP, so no need to futher export from collector.

service:
extensions: [pprof, zpages, health_check]
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

removed everything unnecessary to keep the example config the the minimal.

@codecov
Copy link

codecov bot commented Jul 27, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (fb14199) 49.6% compared to head (1e009c9) 49.6%.

Additional details and impacted files
@@          Coverage Diff          @@
##            main   #1168   +/-   ##
=====================================
  Coverage   49.6%   49.6%           
=====================================
  Files        165     165           
  Lines      20522   20522           
=====================================
+ Hits       10198   10199    +1     
+ Misses     10324   10323    -1     

see 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Member

@jtescher jtescher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good 👍

@jtescher jtescher merged commit 1f59a15 into open-telemetry:main Jul 29, 2023
11 checks passed
@cijothomas cijothomas deleted the cijothomas/log-otlp-example branch July 29, 2023 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants