Skip to content

[release/8.0-staging] Fix HttpHandlerDiagnosticListenerTests.TestW3CHeadersTraceStateAndCorrelationContext #112881

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Feb 24, 2025

Backport of #112753 to release/8.0-staging

/cc @antonfirsov

Customer Impact

None. Test-only change to avoid CI noise.

Regression

  • Yes
  • No

[If yes, specify when the regression was introduced. Provide the PR or commit if known.]

Testing

CI outerloop test improvement.

Risk

N/A. Test-only change

Copy link
Contributor

Tagging subscribers to this area: @tarekgh, @tommcdon, @pjanotti
See info in area-owners.md if you want to be subscribed.

@antonfirsov antonfirsov requested a review from tarekgh February 24, 2025 20:32
@antonfirsov
Copy link
Member

/azp run runtime-libraries-coreclr outerloop

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Member

@tarekgh tarekgh left a comment

Choose a reason for hiding this comment

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

Thanks @antonfirsov

@antonfirsov
Copy link
Member

test-only change => tell mode

@antonfirsov
Copy link
Member

/azp run runtime-libraries-coreclr outerloop

@carlossanlop
Copy link
Contributor

@antonfirsov @karelz Today is code complete for the April Release. If you want this fix included in this release, please merge it before 4pm PT.

By the way, you don't have to keep clicking on Update branch. If you're confident the CI is good, add the /ba-g message to unblock the Build Analysis leg. If you still can't merge for any other reason, feel free to ping me and I can bypass the merge restriction and merge it for you.

@antonfirsov
Copy link
Member

(doing one last rerun attempt before I /ba-g this...)

@antonfirsov
Copy link
Member

/ba-g CI failures are unrelated

@antonfirsov antonfirsov added Servicing-approved Approved for servicing release and removed Servicing-approved Approved for servicing release labels Mar 12, 2025
@antonfirsov antonfirsov merged commit c8cdafd into release/8.0-staging Mar 12, 2025
107 of 114 checks passed
@jkotas jkotas deleted the backport/pr-112753-to-release/8.0-staging branch March 16, 2025 05:27
@github-actions github-actions bot locked and limited conversation to collaborators Apr 15, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants