Skip to content

init#6511

Draft
mhlidd wants to merge 2 commits intomainfrom
mhlidd/enable_telemetry_tests
Draft

init#6511
mhlidd wants to merge 2 commits intomainfrom
mhlidd/enable_telemetry_tests

Conversation

@mhlidd
Copy link
Contributor

@mhlidd mhlidd commented Mar 16, 2026

Motivation

Changes

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?

@github-actions
Copy link
Contributor

CODEOWNERS have been resolved as:

manifests/java.yml                                                      @DataDog/asm-java @DataDog/apm-java
tests/parametric/test_telemetry.py                                      @DataDog/system-tests-core @DataDog/apm-sdk-capabilities
utils/telemetry_utils.py                                                @DataDog/system-tests-core

@datadog-datadog-prod-us1-2
Copy link

datadog-datadog-prod-us1-2 bot commented Mar 16, 2026

⚠️ Tests

Fix all issues with BitsAI or with Cursor

⚠️ Warnings

🧪 33 Tests failed

tests.parametric.test_telemetry.Test_Defaults.test_library_settings[library_env0, parametric-java] from system_tests_suite (Datadog) (Fix with Cursor)
AssertionError: No configuration found for any of DD_LOGS_INJECTION with origin 'default'
assert None is not None

self = <tests.parametric.test_telemetry.Test_Defaults object at 0x7f27b72fa120>
test_agent = <utils.docker_fixtures._test_agent.TestAgentAPI object at 0x7f2782b97080>
test_library = <utils.docker_fixtures._test_clients._test_client_parametric.ParametricTestClientApi object at 0x7f27828bac30>

    @pytest.mark.parametrize(
        "library_env",
        [
...
tests.parametric.test_telemetry.Test_Environment.test_library_settings[library_env0, parametric-java] from system_tests_suite (Datadog) (Fix with Cursor)
AssertionError: Missing telemetry config item for any of DD_LOGS_INJECTION
assert None is not None

self = <tests.parametric.test_telemetry.Test_Environment object at 0x7f27b72f8f80>
test_agent = <utils.docker_fixtures._test_agent.TestAgentAPI object at 0x7f2782b957c0>
test_library = <utils.docker_fixtures._test_clients._test_client_parametric.ParametricTestClientApi object at 0x7f2782b94a70>

    @pytest.mark.parametrize(
        "library_env",
        [
...
tests.parametric.test_telemetry.Test_Stable_Configuration_Origin.test_stable_configuration_origin[local_cfg0-library_env0-fleet_cfg0-expected_origins0, parametric-java] from system_tests_suite (Datadog) (Fix with Cursor)
AssertionError: No configuration found for any of DD_LOGS_INJECTION with origin 'fleet_stable_config'
assert None is not None

self = <tests.parametric.test_telemetry.Test_Stable_Configuration_Origin object at 0x7f245b2b9430>
local_cfg = {'DD_DYNAMIC_INSTRUMENTATION_ENABLED': True, 'DD_LOGS_INJECTION': True, 'DD_RUNTIME_METRICS_ENABLED': True}
fleet_cfg = {'DD_LOGS_INJECTION': True}
test_agent = <utils.docker_fixtures._test_agent.TestAgentAPI object at 0x7f245aa35310>
test_library = <utils.docker_fixtures._test_clients._test_client_parametric.ParametricTestClientApi object at 0x7f245ad25c40>
expected_origins = {'dynamic_instrumentation_enabled': 'local_stable_config', 'logs_injection_enabled': 'fleet_stable_config'}

...
View all

ℹ️ Info

No other issues found (see more)

❄️ No new flaky tests detected

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: a8ab8aa | Docs | Datadog PR Page | Was this helpful? React with 👍/👎 or 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