-
Notifications
You must be signed in to change notification settings - Fork 825
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
feat(exporter-logs-otlp-http): implements otlp-http exporters for logs #3755
feat(exporter-logs-otlp-http): implements otlp-http exporters for logs #3755
Conversation
This reverts commit fdd06c6.
…oved from peerDependencies
Co-authored-by: Martin Kuba <[email protected]>
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #3755 +/- ##
==========================================
- Coverage 93.39% 91.13% -2.27%
==========================================
Files 290 82 -208
Lines 8863 1794 -7069
Branches 1838 362 -1476
==========================================
- Hits 8278 1635 -6643
+ Misses 585 159 -426 |
…n docs. Co-authored-by: Daniel Dyla <[email protected]>
Thank you for your contribution, but I'm sorry to say that the development of OTLP-HTTP-Exporter has already been completed in 3442. However, due to the large amount of code, it was split into 2 PRs. The SDK part was merged a few days ago, and I haven't had a chance to submit a PR for the OTLP-HTTP-Exporter part yet, I will do it tomorrow. |
I have submitted pr #3764, and you are very welcome to participate in CR. |
ok. I will contribute when the merge for the #3764 PR proceeds. |
Which problem is this PR solving?
Extend to http for #3712
I needed work to ensure browser sdk usage for the above PR!
For logs in grafana
exporter-logs-otlp-http
Short description of the changes
exporter-logs-otlp-http
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist: