Skip to content

[java] Enable Test_Config_ClientIPHeader_Precedence - #4237

Closed
smola wants to merge 1 commit into
mainfrom
smola/test_ip_headers_precedence
Closed

[java] Enable Test_Config_ClientIPHeader_Precedence#4237
smola wants to merge 1 commit into
mainfrom
smola/test_ip_headers_precedence

Conversation

@smola

@smola smola commented Mar 3, 2025

Copy link
Copy Markdown
Member

Motivation

Fixed by DataDog/dd-trace-java#7946

Changes

Workflow

  1. ⚠️ Create your PR as draft ⚠️
  2. Work on you PR until the CI passes (if something not related to your task is failing, you can ignore it)
  3. Mark it as ready for review
    • Test logic is modified? -> Get a review from RFC owner. We're working on refining the codeowners file quickly.
    • Framework is modified, or non obvious usage of it -> get a review from R&P team

🚀 Once your PR is reviewed, you can merge it!

🛟 #apm-shared-testing 🛟

Reviewer checklist

  • 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)?
@smola smola closed this Nov 27, 2025
@cbeauchesne
cbeauchesne deleted the smola/test_ip_headers_precedence branch December 12, 2025 09:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant