Skip to content

[Java] enable tracecontext test for java tracer v1.35#2527

Merged
am312 merged 3 commits into
mainfrom
andrew.munn/add-java-135-w3c
Jun 9, 2024
Merged

[Java] enable tracecontext test for java tracer v1.35#2527
am312 merged 3 commits into
mainfrom
andrew.munn/add-java-135-w3c

Conversation

@am312

@am312 am312 commented Jun 6, 2024

Copy link
Copy Markdown
Contributor

Motivation

Java tracer v1.35 adds w3c phase 2 support (PR).

Changes

From class Test_Headers_Tracecontext should now run:

  • test_tracestate_w3c_p_extract
  • test_tracestate_w3c_p_inject

Reviewer checklist

  • Relevant label (run-parametric-scenario, run-profiling-scenario...) are presents
  • If PR title starts with [<language>], double-check that only <language> is impacted by the change
  • No system-tests internal is modified. Otherwise, I have the approval from R&P team
  • CI is green, or failing jobs are not related to this change (and you are 100% sure about this statement)
  • A docker base image is modified?
    • the relevant build-XXX-image label is present
  • A scenario is added (or removed)?

@cbeauchesne cbeauchesne left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@am312 am312 requested a review from cbeauchesne June 7, 2024 19:45
@am312 am312 merged commit 4315b80 into main Jun 9, 2024
@am312 am312 deleted the andrew.munn/add-java-135-w3c branch June 9, 2024 18:34
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