[runtime-diagnostics] Workaround for publish test results regression#124859
Merged
max-charlamb merged 4 commits intodotnet:mainfrom Feb 27, 2026
Merged
[runtime-diagnostics] Workaround for publish test results regression#124859max-charlamb merged 4 commits intodotnet:mainfrom
max-charlamb merged 4 commits intodotnet:mainfrom
Conversation
Contributor
|
Tagging subscribers to this area: @steveisok, @tommcdon, @dotnet/dotnet-diag |
Contributor
There was a problem hiding this comment.
Pull request overview
This PR adds a workaround for a regression in Azure Pipelines' PublishTestResults@2 task version 2.270.0 (tracked in microsoft/azure-pipelines-tasks#21770). The regression causes the task to incorrectly split xUnit test names on all dots, including those within parenthesized test parameters, resulting in garbled test names in the published results.
Changes:
- Added PowerShell script steps to preprocess xUnit XML files before publishing test results
- The script uses regex to replace dots with underscores inside parenthesized sections of test names
- Applied the workaround to both the cDAC and DAC jobs in the runtime-diagnostics pipeline
max-charlamb
force-pushed
the
runtime-diagnostics-publish-fix
branch
from
2a4892c to
25eb1ee
Compare
This was referenced Feb 25, 2026
Open
max-charlamb
force-pushed
the
runtime-diagnostics-publish-fix
branch
from
25eb1ee to
17c2407
Compare
Override AllowPtrToDetectTestRunRetryFiles to false at the job level for the cDAC and DAC jobs, as recommended by the Azure Pipelines team in microsoft/azure-pipelines-tasks#21871 (comment). The variable is set to true by the shared Arcade job template (eng/common/core-templates/job/job.yml). Setting it to false via jobParameters.variables overrides that default and restores v2.262.0 behavior where parameterized test names containing dots (e.g. 'projectk.sdk.prebuilt.cdac.10.0.2') are not incorrectly split into hierarchy fragments. Co-authored-by: Copilot <[email protected]>
max-charlamb
force-pushed
the
runtime-diagnostics-publish-fix
branch
from
17c2407 to
56447f1
Compare
max-charlamb
requested review from
barosiak,
hoyosjs,
mdh1418,
rcj1 and
steveisok
mdh1418
approved these changes
Feb 27, 2026
Member
Author
|
/ba-g runtime-diagnostics pipeline change only (previous successful CI run shows workaround success) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Regression issue: microsoft/azure-pipelines-tasks#21871
Before:
After:
