Skip to content

feat: [WIP] Implement datadog v0.7 traces api #249

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

RoyShulman
Copy link

Fixes

I've added an option to export traces using the v0.7 api endpoint to datadog. I needed this because as opposed to the v0.5 endpoint, the new one is able to export span links and events.
I've tested my code by using the test agent and added a link in the code as well so that others might find it useful. Also tested against a real datadog instance and everything seems to work correctly.

Design discussion issue (if applicable)

Since the v0.7 endpoint uses message pack encoding with string keys, it's possible to make the code nicer and probably safer by using rmp_serde. I wasn't sure if you wanted to pull in another dependency, so for now I implemented the serialization without it. Tell me if you want to change it :)

Changes

All changes are contained in the opentelemetry-datadog crate, mostly in the opentelemetry-datadog/src/exporter/model/v07.rs file.

Merge requirement checklist

  • CONTRIBUTING guidelines followed
  • Unit tests added/updated (if applicable)
  • Appropriate CHANGELOG.md files updated for non-trivial, user-facing changes
  • Changes in public API reviewed (if applicable)

@RoyShulman RoyShulman requested a review from a team as a code owner April 18, 2025 18:34
Copy link

linux-foundation-easycla bot commented Apr 18, 2025

CLA Not Signed

@hdost hdost force-pushed the add-v07-datadog-traces-export branch from cd97ee1 to 3c32242 Compare May 8, 2025 10:18
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.

2 participants