Skip to content

[golang] Activate CSS Test_Client_Stats_With_Client_Obfuscation#7282

Draft
ichinaski wants to merge 1 commit into
mainfrom
inigo/css-activate-go-client-obfuscation
Draft

[golang] Activate CSS Test_Client_Stats_With_Client_Obfuscation#7282
ichinaski wants to merge 1 commit into
mainfrom
inigo/css-activate-go-client-obfuscation

Conversation

@ichinaski

Copy link
Copy Markdown
Contributor

Motivation

APMSP-3668. dd-trace-go performs end-to-end client-side stats obfuscation negotiation (version-negotiated, no dedicated flag). Test_Client_Stats_With_Client_Obfuscation runs under the base TRACE_STATS_COMPUTATION scenario and already xpasses on the released tracer (observed on v2.9.1 in CI) while marked missing_feature.

Changes

Version-gate tests/stats/test_stats.py::Test_Client_Stats_With_Client_Obfuscation at v2.9.1 in manifests/golang.yml (was an unconditional missing_feature). The _With_Client_Obfuscation_Disabled variant stays missing_feature (APMSP-3672) — Go, like dotnet, exposes no runtime switch to disable client obfuscation, so that scenario is not applicable.

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?

…v2.9.1

dd-trace-go performs end-to-end client-side stats obfuscation negotiation;
the test xpasses on the released tracer (v2.9.1) in the base
TRACE_STATS_COMPUTATION scenario. Gate it at v2.9.1 instead of missing_feature.
The _Disabled variant stays missing_feature (no client-obfuscation off switch).

APMSP-3668
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

CODEOWNERS have been resolved as:

manifests/golang.yml                                                    @DataDog/dd-trace-go-guild

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