-
Notifications
You must be signed in to change notification settings - Fork 66
feat: Add internal logs receiver #1663
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
base: main
Are you sure you want to change the base?
Conversation
…ernal telemetry receiver.
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #1663 +/- ##
==========================================
- Coverage 84.06% 84.04% -0.03%
==========================================
Files 469 471 +2
Lines 137651 138042 +391
==========================================
+ Hits 115715 116014 +299
- Misses 21402 21494 +92
Partials 534 534
🚀 New features to boost your workflow:
|
|
Thank you @andborja. Comparing and contrasting with the draft I'm working on, #1672, your approach configures the OTel SDK with its various processor and exporter options from the configuration. I agree that we want an option to configure the OTel SDK in all of the ways implied by the declarative configuration model, and at the same time I'm worried about relying on the SDK's data representation in this code base. In this code base, I think it makes more sense to focus on the OTLP bytes approach outlined in #1672 which is to optimize for the self-hosted internal telemetry code path at the expense of making the use |
Add internal logs sdk exporter sending telemetry to an internal logs receiver.
Opening as draft initially to discuss the general model and decide on the proper objects to send through the channel.
This PR presents the bridge to receive the internal telemetry. It also demonstrate how to implement the receiver to be included in a pipeline.