Skip to content

[Java] Disable all tests depending on telemetry key#6486

Merged
gh-worker-dd-mergequeue-cf854d[bot] merged 2 commits intomainfrom
mhlidd/disable_telemetry_tests
Mar 12, 2026
Merged

[Java] Disable all tests depending on telemetry key#6486
gh-worker-dd-mergequeue-cf854d[bot] merged 2 commits intomainfrom
mhlidd/disable_telemetry_tests

Conversation

@mhlidd
Copy link
Contributor

@mhlidd mhlidd commented Mar 12, 2026

Motivation

Followup to #6477, this PR disables all tests that depend on a normalized telemetry key for Java. The tests disabled will be updated w/ new normalized telemetry keys once the change in dd-trace-java is merged.

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?

@mhlidd mhlidd requested review from a team as code owners March 12, 2026 19:24
@mhlidd mhlidd requested review from claponcet, jandro996 and ygree March 12, 2026 19:24
@github-actions
Copy link
Contributor

CODEOWNERS have been resolved as:

manifests/java.yml                                                      @DataDog/asm-java @DataDog/apm-java

Copy link
Contributor

@mabdinur mabdinur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Send it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants