Skip to content

Remove ineffective blanket integration-test retry - #36994

Merged
kubaflo merged 2 commits into
mainfrom
copilot/remove-integration-test-retry
Aug 1, 2026
Merged

Remove ineffective blanket integration-test retry#36994
kubaflo merged 2 commits into
mainfrom
copilot/remove-integration-test-retry

Conversation

Copilot AI commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Note

Are you waiting for the changes in this PR to be merged?
It would be very helpful if you could test the resulting artifacts from this PR and let us know in a comment if this change resolves your issue. Thank you!

Analysis of 744 maui-pr builds in July 2026 showed that 98.2% of integration-test retries reproduced identical failures, with only a 0.56% task-recovery rate, while consuming ~211 machine-hours and adding ~53 hours of wall-clock pipeline delay.

Changes

  • eng/pipelines/arcade/stage-integration-tests.yml: Remove retryCountOnTaskFailure: 1 from the Run Integration Tests task.

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
1 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

Copilot AI linked an issue Jul 31, 2026 that may be closed by this pull request
Co-authored-by: mmitche <8725170+mmitche@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove ineffective blanket integration-test retry Remove ineffective blanket integration-test retry Jul 31, 2026
Copilot AI requested a review from mmitche July 31, 2026 19:01
@mmitche

mmitche commented Jul 31, 2026

Copy link
Copy Markdown
Member

Part of EngThrive P80 reduction. This accounts for a fairly large slice of Maui's P80 contribution.

@mmitche
mmitche marked this pull request as ready for review July 31, 2026 19:04
Copilot AI review requested due to automatic review settings July 31, 2026 19:04
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
There may be pipelines that require an authorized user to comment /azp run to run.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR removes the blanket Azure Pipelines task-level retry for integration tests, reducing wasted reruns of deterministic failures and reclaiming CI time in the maui-pr integration stage.

Changes:

  • Removed retryCountOnTaskFailure: 1 from the integration test run-dotnet-preview.yml invocation.
  • Leaves all test filtering, environment setup, and result publishing behavior unchanged.

@github-actions

Copy link
Copy Markdown
Contributor

🚀 Dogfood this PR with:

⚠️ WARNING: Do not do this without first carefully reviewing the code of this PR to satisfy yourself it is safe.

curl -fsSL https://raw.githubusercontent.com/dotnet/maui/main/eng/scripts/get-maui-pr.sh | bash -s -- 36994

Or

  • Run remotely in PowerShell:
iex "& { $(irm https://raw.githubusercontent.com/dotnet/maui/main/eng/scripts/get-maui-pr.ps1) } 36994"

@mmitche

mmitche commented Jul 31, 2026

Copy link
Copy Markdown
Member

Thanks @Redth Merge when ready.

@kubaflo
kubaflo merged commit 53bc365 into main Aug 1, 2026
29 of 32 checks passed
@kubaflo
kubaflo deleted the copilot/remove-integration-test-retry branch August 1, 2026 12:28
@github-actions github-actions Bot added this to the .NET 10 SR10 milestone Aug 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] Remove ineffective blanket integration-test retry

5 participants