-
Notifications
You must be signed in to change notification settings - Fork 3.2k
[receiver/datadogreceiver] Adding logs support to the datadog receiver #43841
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
Conversation
|
Welcome, contributor! Thank you for your contribution to opentelemetry-collector-contrib. Important reminders:
A maintainer will review your pull request soon. Thank you for helping make OpenTelemetry better! |
Co-Authored-By: Robin Law <[email protected]>
…lector-contrib into ddreceiver_logs
atoulme
left a comment
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.
nothing jumps at me, everything LGTM - this is alpha and I haven't tested it myself.
…lector-contrib into ddreceiver_logs
…lector-contrib into ddreceiver_logs
|
Could somebody approve my workflows to run please? |
|
Thank you for your contribution @bvigar! 🎉 We would like to hear from you about your experience contributing to OpenTelemetry by taking a few minutes to fill out this survey. If you are getting started contributing, you can also join the CNCF Slack channel #opentelemetry-new-contributors to ask for guidance and get help. |
open-telemetry#43841) <!--Ex. Fixing a bug - Describe the bug and how this fixes the issue. Ex. Adding a feature - Explain what this achieves.--> #### Description Adding a feature to the datadog receiver to support logs, in addition to the currently supported traces and metrics. This change adds support for the `api/v2/logs` endpoint. Note: There's purposefully no datadog semantics or conforming in this changeset because I plan to make those changes next in the [ddsemanticsprocessor](https://github.com/Sawmills/opentelemetry-collector-contrib/tree/e70d1b2ba66405e10dfc02afc170f8e2462b4a36/processor/datadogsemanticsprocessor). I didn't think they belonged here. <!-- Issue number (e.g. open-telemetry#1234) or full URL to issue, if applicable. --> #### Link to tracking issue Fixes <!--Describe what testing was performed and which tests were added.--> #### Testing - Built a custom collector locally with these changes and verified with the debug exporter that otel logs are being exported properly - Added tests for the log translation layer - Added e2e tests for the receiver for single and multiple logs <!--Describe the documentation added.--> #### Documentation Updated the `receiver/datadogreceiver/README.md` to now show support for logs. <!--Please delete paragraphs that you did not use before submitting.--> --------- Co-authored-by: Robin Law <[email protected]>
Description
Adding a feature to the datadog receiver to support logs, in addition to the currently supported traces and metrics. This change adds support for the
api/v2/logsendpoint.Note: There's purposefully no datadog semantics or conforming in this changeset because I plan to make those changes next in the ddsemanticsprocessor. I didn't think they belonged here.
Link to tracking issue
Fixes
Testing
Documentation
Updated the
receiver/datadogreceiver/README.mdto now show support for logs.