Skip to content

Fix Hierarchical tracing test broken by ForceDefaultIdFormat#3023

Merged
MihaZupan merged 1 commit intodotnet:mainfrom
MihaZupan:test-activity-fix
Apr 28, 2026
Merged

Fix Hierarchical tracing test broken by ForceDefaultIdFormat#3023
MihaZupan merged 1 commit intodotnet:mainfrom
MihaZupan:test-activity-fix

Conversation

@MihaZupan
Copy link
Copy Markdown
Member

Fixes #3020

Copilot's description:

Microsoft.Testing.Extensions.Telemetry (part of the test runner) brings
in Microsoft.ApplicationInsights/2.23.0, which sets
Activity.ForceDefaultIdFormat = true globally.  This forces all new
Activities to use W3C format regardless of the parent activity's format,
causing the Hierarchical test case to emit traceparent/tracestate
headers instead of the expected Request-Id header.

Introduced in #2940

@MihaZupan MihaZupan added this to the YARP 3.0 milestone Apr 28, 2026
@MihaZupan MihaZupan added the Type: Infrastructure Infrastructure work that does not affect the actual shipping code label Apr 28, 2026
@MihaZupan MihaZupan enabled auto-merge (squash) April 28, 2026 13:05
@MihaZupan MihaZupan merged commit 51c57c2 into dotnet:main Apr 28, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type: Infrastructure Infrastructure work that does not affect the actual shipping code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Yarp.ReverseProxy.DistributedTracingTests.DistributedTracing_Works test case fails on branch main

2 participants