Skip to content

[Go] Opentelemetry log support#6039

Open
rachelyangdog wants to merge 12 commits intomainfrom
rachel.yang/go-otel-logs
Open

[Go] Opentelemetry log support#6039
rachelyangdog wants to merge 12 commits intomainfrom
rachel.yang/go-otel-logs

Conversation

@rachelyangdog
Copy link
Copy Markdown
Contributor

@rachelyangdog rachelyangdog commented Jan 15, 2026

Motivation

  • adds support for testing OTel logs in dd-trace-go

Changes

New OTel Logs Endpoints: Added new endpoints to the Go parametric client:

  • /otel/logger/create: Creates a logger with optional scope metadata (name, version, schema_url, attributes)
  • /otel/logger/write: Emits log records using pre-created loggers
  • /log/otel/flush: Forces immediate flush of all log records

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?

@rachelyangdog rachelyangdog requested review from a team as code owners January 15, 2026 21:51
@rachelyangdog rachelyangdog requested review from mhlidd and removed request for a team January 15, 2026 21:51
@rachelyangdog rachelyangdog marked this pull request as draft January 15, 2026 21:51
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jan 15, 2026

CODEOWNERS have been resolved as:

utils/build/docker/golang/parametric/logs.go                            @DataDog/dd-trace-go-guild @DataDog/system-tests-core
manifests/golang.yml                                                    @DataDog/dd-trace-go-guild
tests/parametric/test_otel_logs.py                                      @DataDog/system-tests-core @DataDog/apm-sdk-capabilities
utils/build/docker/golang/go.work.sum                                   @DataDog/dd-trace-go-guild @DataDog/system-tests-core
utils/build/docker/golang/parametric/go.mod                             @DataDog/dd-trace-go-guild @DataDog/system-tests-core
utils/build/docker/golang/parametric/go.sum                             @DataDog/dd-trace-go-guild @DataDog/system-tests-core
utils/build/docker/golang/parametric/helpers.go                         @DataDog/dd-trace-go-guild @DataDog/system-tests-core
utils/build/docker/golang/parametric/main.go                            @DataDog/dd-trace-go-guild @DataDog/system-tests-core

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

datadog-datadog-prod-us1 Bot commented Jan 27, 2026

⚠️ Tests

Fix all issues with BitsAI or with Cursor

⚠️ Warnings

🧪 9 Tests failed

tests.parametric.test_otel_logs.Test_FR03_Resource_Attributes.test_dd_env_vars_override_otel[library_env0, parametric-nodejs] from system_tests_suite (Datadog) (Fix with Cursor)
AssertionError: assert 'otelenv' == 'ddenv'
  - ddenv
  + otelenv

self = <tests.parametric.test_otel_logs.Test_FR03_Resource_Attributes object at 0x7f0a0dfd1f10>
test_agent = <utils.docker_fixtures._test_agent.TestAgentAPI object at 0x7f09d940e630>
test_library = <utils.docker_fixtures._test_clients._test_client_parametric.ParametricTestClientApi object at 0x7f09d940dcd0>

    @pytest.mark.parametrize(
        "library_env",
...
tests.parametric.test_otel_logs.Test_FR03_Resource_Attributes.test_dd_env_vars_override_otel[library_env0, parametric-python] from system_tests_suite (Datadog) (Fix with Cursor)
AssertionError: assert 'otelenv' == 'ddenv'
  - ddenv
  + otelenv

self = <tests.parametric.test_otel_logs.Test_FR03_Resource_Attributes object at 0x7f3956690c80>
test_agent = <utils.docker_fixtures._test_agent.TestAgentAPI object at 0x7f3921dc1370>
test_library = <utils.docker_fixtures._test_clients._test_client_parametric.ParametricTestClientApi object at 0x7f3921cc6180>

    @pytest.mark.parametrize(
        "library_env",
...
tests.parametric.test_otel_logs.Test_FR03_Resource_Attributes.test_dd_env_vars_override_otel[library_env0, parametric-rust] from system_tests_suite (Datadog) (Fix with Cursor)
AssertionError: assert 'otelenv' == 'ddenv'
  - ddenv
  + otelenv

self = <tests.parametric.test_otel_logs.Test_FR03_Resource_Attributes object at 0x7fa2ac9d68a0>
test_agent = <utils.docker_fixtures._test_agent.TestAgentAPI object at 0x7fa27753cb00>
test_library = <utils.docker_fixtures._test_clients._test_client_parametric.ParametricTestClientApi object at 0x7fa27755bb90>

    @pytest.mark.parametrize(
        "library_env",
...
View all

ℹ️ Info

❄️ No new flaky tests detected

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 64b090e | Docs | Datadog PR Page | Was this helpful? React with 👍/👎 or give us feedback!

@rachelyangdog rachelyangdog marked this pull request as ready for review February 2, 2026 18:18
@rachelyangdog rachelyangdog requested a review from a team as a code owner February 2, 2026 18:18
@rachelyangdog rachelyangdog changed the title WIP: [Go] Opentelemetry log support [Go]: Opentelemetry log support Feb 2, 2026
@rachelyangdog rachelyangdog changed the title [Go]: Opentelemetry log support [Go] Opentelemetry log support Feb 2, 2026
@rachelyangdog rachelyangdog requested a review from genesor February 6, 2026 18:51
@genesor
Copy link
Copy Markdown
Member

genesor commented Apr 27, 2026

This PR kinda slipped through the review. What's the state of this @rachelyangdog should it be rebased and reviewed/merged ?

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.

3 participants