Skip to content

@WI-24614336: Enable tracing with MRT distributed tracing flag - #4073

Merged
shethj merged 5 commits into
developfrom
feat/mrt-distributed-tracing
Oct 23, 2026
Merged

shethj merged 5 commits into
developfrom
feat/mrt-distributed-tracing

Conversation

@shethj

@shethj shethj commented Oct 22, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Treat MRT_DISTRIBUTED_TRACING_ENABLED=true as an alternative to the existing OTEL_TRACING_ENABLED=true switch.
  • Keep getOTELConfig().enabled as the shared gate, so distributed tracing, the MRT exporter, and serverTiming follow the same OR condition.
  • Document both flags and record the compatibility behavior.

Work item: W-24614336

Validation

  • git diff --check
  • Prettier check for changed source, test, documentation, and plan files
  • Tests not run (per request)

@shethj
shethj requested a review from a team as a code owner October 22, 2026 20:40
@git2gus

git2gus Bot commented Oct 22, 2026

Copy link
Copy Markdown

Git2Gus App is installed but the .git2gus/config.json doesn't have right values. You should add the required configuration.

@cc-prodsec

cc-prodsec commented Oct 22, 2026 •

Copy link
Copy Markdown
Collaborator

✅ Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
✅ Open Source Security 0 0 0 0 0 issues
✅ Licenses 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@shethj
shethj merged commit 8137759 into develop Oct 23, 2026
41 checks passed
@shethj
shethj deleted the feat/mrt-distributed-tracing branch October 23, 2026 17:17
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.

4 participants