Skip to content

Publish each test assembly's results as its own CI run instead of merging - #20574

Open
T-Gro wants to merge 4 commits into
mainfrom
t-gro-investigate-testresultsconverter-warning
Open

T-Gro wants to merge 4 commits into
mainfrom
t-gro-investigate-testresultsconverter-warning

Conversation

@T-Gro

@T-Gro T-Gro commented Sep 18, 2026

Copy link
Copy Markdown
Member

mergeTestResults: true merges the disjoint per-assembly XUnit files from one job into a single Azure DevOps test run. The publisher then emits a TestResultConverter.ConvertResults : No matching test case result found on server warning for every test in every file after the first — 63k+ warnings in a single build — and intermittently aborts with TestCaseTitle must be specified, which silently drops a whole assembly's results. Publishing each file as its own run removes both.

Publishing multiple disjoint per-assembly XUnit result files into a single
ADO test run (mergeTestResults: true) makes the Publish Test Results task
emit a 'TestResultConverter.ConvertResults : No matching test case result
found on server' warning for every test in every file after the first --
tens of thousands of warnings per build (63k+ observed) -- and can abort
with 'TestCaseTitle must be specified', silently dropping a whole
assembly's results.

Publish each assembly's XUnit file as its own run instead. Applied to the
shared batched-test-steps template and the three explicit PublishTestResults
tasks in azure-pipelines-PR.yml. The official azure-pipelines.yml already
omits mergeTestResults (defaults to false).

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: ee9b5f2a-94a7-4b63-ad52-6602d2a0bda6
@T-Gro
T-Gro requested a review from a team as a code owner September 18, 2026 17:32
@github-actions github-actions Bot added the AI-Tooling-Check-Bypassed Tooling check: non-fork PR, not diff-analyzed label Sep 18, 2026
Copilot and others added 3 commits September 21, 2026 12:06
Split Linux CoreCLR tests into the existing two batches to reduce test-process memory pressure after the exit-137 termination. Keep complete coverage and give each batch distinct test-run and artifact names.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…tresultsconverter-warning

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

# Conflicts:
#	azure-pipelines-PR.yml
# found on server" warning for every test in every file after the first (tens of thousands
# of warnings per build), and can abort with "TestCaseTitle must be specified", silently
# dropping a whole assembly's results.
mergeTestResults: false

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This still drops assembly results in build 1605407's Linux Batch1 run: the publisher sends 117 results to run 44355152, then tries to publish the 6,175 component results to that same run and aborts with TestCaseTitle must be specified…. Setting mergeTestResults: false hasn't isolated the runs here. We need to verify that each XML actually produces a separate, complete run before calling this fixed.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI-Tooling-Check-Bypassed Tooling check: non-fork PR, not diff-analyzed

Projects

Status: New

Development

Successfully merging this pull request may close these issues.

2 participants