Skip to content

Enable MTP Test Retry in Pipelines#5596

Merged
mikaelweave merged 3 commits into
mainfrom
feature/enable-mtp-retry-linux
Jun 1, 2026
Merged

Enable MTP Test Retry in Pipelines#5596
mikaelweave merged 3 commits into
mainfrom
feature/enable-mtp-retry-linux

Conversation

@mikaelweave
Copy link
Copy Markdown
Contributor

@mikaelweave mikaelweave commented May 29, 2026

Description

Enables the Microsoft Testing Platform Azure retry feature on all test jobs, now that Linux support has been fixed in the upstream Azure Pipelines infrastructure

AB#194340

Testing

You can now see tests as "Passed on Rerun" in Azure DevOps
CleanShot 2026-06-01 at 08 26 05@2x

mikaelweave and others added 3 commits May 4, 2026 14:43
- Add AllowPtrToDetectTestRunRetryFiles: true as a job-level variable
  to all test jobs (SQL, Cosmos, export, and build/unit test jobs)
- Re-enable failTaskOnFailedTests: true in all PublishTestResults@2 tasks
  (removes workaround for microsoft/testfx#7167
  now that cross-platform support is fixed via azure-pipelines-agent#5496
  and azure-pipelines-tasks#21934)
- Add --report-trx to unit test arguments in build.yml (non-coverage path)
  for consistency with coverage path

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
The SQL and Cosmos integration test jobs run tests via DotNetCoreCLI@2
'command: test', which auto-publishes results (publishTestResults
defaults to true). That built-in publish is not retry-aware, so each
MTP retry attempt surfaced as its own test run (e.g. 'R5 SQL
Integration Tests_1' failed and '_2' passed) in addition to the merged
retry-aware run published by the explicit PublishTestResults@2 task.

Set publishTestResults: false on the test tasks so the explicit
retry-aware PublishTestResults@2 task is the sole publisher, removing
the duplicate and failed-attempt test runs.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@mikaelweave mikaelweave requested a review from a team as a code owner May 29, 2026 21:10
@mikaelweave mikaelweave added this to the FY26\Q4\2Wk\2Wk24 milestone May 29, 2026
@mikaelweave mikaelweave added Build Open source This change is only relevant to the OSS code or release. No-ADR ADR not needed No-PaaS-breaking-change labels May 29, 2026
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.30%. Comparing base (545c184) to head (79b0b02).
⚠️ Report is 24 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #5596      +/-   ##
==========================================
- Coverage   77.61%   77.30%   -0.32%     
==========================================
  Files         983      996      +13     
  Lines       36005    36550     +545     
  Branches     5469     5541      +72     
==========================================
+ Hits        27947    28254     +307     
- Misses       6710     6927     +217     
- Partials     1348     1369      +21     

see 61 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@mikaelweave mikaelweave merged commit 1b30c40 into main Jun 1, 2026
55 of 59 checks passed
@mikaelweave mikaelweave deleted the feature/enable-mtp-retry-linux branch June 1, 2026 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Build No-ADR ADR not needed No-PaaS-breaking-change Open source This change is only relevant to the OSS code or release.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants