-
Notifications
You must be signed in to change notification settings - Fork 775
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 log extension methods for LoggerProviderBuilder #4596
Add otlp log extension methods for LoggerProviderBuilder #4596
Conversation
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #4596 +/- ##
==========================================
+ Coverage 83.61% 85.24% +1.63%
==========================================
Files 296 314 +18
Lines 12484 12861 +377
==========================================
+ Hits 10438 10963 +525
+ Misses 2046 1898 -148
Flags with carried forward coverage won't be shown. Click here to find out more.
|
…/github.com/vishweshbankwar/opentelemetry-dotnet into vibankwa/add-loggerprovider-extensions-otlp
This PR was marked stale due to lack of activity and will be closed in 7 days. Commenting or Pushing will instruct the bot to automatically remove the label. This bot runs once per day. |
FYI I just pushed a commit to disable IHttpClientFactory integration for OtlpLogExporter. There is a circular reference issue I haven't been able to figure out a way to make it work reliably. I figured best to just not support it for now. |
This PR was marked stale due to lack of activity and will be closed in 7 days. Commenting or Pushing will instruct the bot to automatically remove the label. This bot runs once per day. |
This PR was marked stale due to lack of activity and will be closed in 7 days. Commenting or Pushing will instruct the bot to automatically remove the label. This bot runs once per day. |
Closed as inactive. Feel free to reopen if this PR is still being worked on. |
This PR was marked stale due to lack of activity and will be closed in 7 days. Commenting or Pushing will instruct the bot to automatically remove the label. This bot runs once per day. |
Closed as inactive. Feel free to reopen if this PR is still being worked on. |
Towards #4454
Fixes #4388
Changes
Adds the otlp log exporter extension methods LoggerProviderBuilder. Also, includes some fixes by @CodeBlanch for the issues surfaced during testing for LoggerProvider.
Usage Examples
TODO:
Add changelog and Readme
Merge requirement checklist
CHANGELOG.md
files updated for non-trivial changes