Skip to content

[tracing] Assert that OTLP spans have full 128-bit trace IDs#6973

Draft
zacharycmontoya wants to merge 2 commits into
mainfrom
zach.montoya/otlp-trace-id-test
Draft

[tracing] Assert that OTLP spans have full 128-bit trace IDs#6973
zacharycmontoya wants to merge 2 commits into
mainfrom
zach.montoya/otlp-trace-id-test

Conversation

@zacharycmontoya
Copy link
Copy Markdown
Contributor

Motivation

End-to-end testing identified that traces exported as OTLP by dd-trace-py had incorrect trace IDs. The HTTP server span had a 128-bit trace ID while its child spans had that same trace ID except the upper 64 bits were zeroed out. This test will ensure that we track and remediate this behavior across all languages.

Changes

Add test/otel_test_tracing_otlp.py::Test_Otel_Tracing_OTLP::test_128bit_trace_id_consistent_across_spans to validate that implementations export all OTLP spans with the full 128-bit trace ID. This was identified as an issue in the libdatadog implementation

Workflow

  1. ⚠️ Create your PR as draft ⚠️
  2. Work on you PR until the CI passes
  3. Mark it as ready for review
    • Test logic is modified? -> Get a review from RFC owner.
    • Framework is modified, or non obvious usage of it -> get a review from R&P team

🚀 Once your PR is reviewed and the CI green, you can merge it!

🛟 #apm-shared-testing 🛟

Reviewer checklist

  • Anything but tests/ or manifests/ is modified ? I have the approval from R&P team
  • A docker base image is modified?
    • the relevant build-XXX-image label is present
  • A scenario is added, removed or renamed?

…it_trace_id_consistent_across_spans to validate that implementations export all OTLP spans with the full 128-bit trace ID. This was identified as an issue in the libdatadog implementation
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 20, 2026

CODEOWNERS have been resolved as:

manifests/nodejs.yml                                                    @DataDog/dd-trace-js
manifests/python.yml                                                    @DataDog/apm-python @DataDog/asm-python
tests/otel/test_tracing_otlp.py                                         @DataDog/system-tests-core
utils/interfaces/_open_telemetry.py                                     @DataDog/system-tests-core

@datadog-prod-us1-3
Copy link
Copy Markdown
Contributor

datadog-prod-us1-3 Bot commented May 20, 2026

Pipelines  Tests

Fix all issues with BitsAI

⚠️ Warnings

🚦 181 Pipeline jobs failed

DataDog/system-tests | Ubuntu_20_amd64.CO1B: [test-app-php-container-83]   View in Datadog   GitLab

🔧 Fix in code (Fix with Cursor). Assertion error during virtual machine provisioning steps. Previous errors found in logs: Ubuntu_20_amd64.log.

DataDog/system-tests | Ubuntu_20_amd64.CO1B: [test-app-python-alpine]   View in Datadog   GitLab

🔧 Fix in code (Fix with Cursor). AssertionError: There are previous errors in the virtual machine provisioning steps. Check this file in the logs: Ubuntu_20_amd64.log

DataDog/system-tests | Ubuntu_20_amd64.CO1B: [test-app-python-container]   View in Datadog   GitLab

🔧 Fix in code (Fix with Cursor). AssertionError: There are previous errors in the virtual machine provisioning steps. Check this file in the logs scenario folder: Ubuntu_20_amd64.log at tests/auto_inject/utils.py:23

View all 181 failed jobs.

ℹ️ Info

No other issues found (see more)

🧪 All tests passed
❄️ No new flaky tests detected

Useful? React with 👍 / 👎

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: e5ae38d | Docs | Datadog PR Page | Give us feedback!

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.

1 participant