Skip to content

[release/11.0.1xx-preview7] Harden MTP artifact post-processing and add an opt-out - #55493

Merged
Evangelink merged 1 commit into
release/11.0.1xx-preview7from
backport/pr-55480-to-release/11.0.1xx-preview7
Jul 29, 2026
Merged

[release/11.0.1xx-preview7] Harden MTP artifact post-processing and add an opt-out#55493
Evangelink merged 1 commit into
release/11.0.1xx-preview7from
backport/pr-55480-to-release/11.0.1xx-preview7

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Backport of #55480 to release/11.0.1xx-preview7

/cc @Evangelink

Artifact post-processing runs after a test run has already completed and
reported its results, so it must never be able to change that outcome.
Two ways it could are fixed:

- Any exception outside a small hand-picked set escaped ExecuteAsync,
  was not caught by the caller, and reached the top-level CLI handler,
  turning a finished run into a crash with a different exit code. Every
  failure is now logged to trace and downgraded to the existing warning.
- Cancelling mid-job kills the post-processing process, and the resulting
  non-zero exit code was then reported as a post-processing failure.
  Both warning sites now route through ReportFailureUnlessCancelled.

Also adds '--no-artifact-post-processing', which skips post-processing so
each test application keeps its own artifacts instead of test
applications being relaunched to merge them.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: e002463b-c925-4c2f-90c3-b56d480faa7d
@github-actions github-actions Bot added the backport PR that has been backported to a servicing branch label Jul 28, 2026
@github-actions
github-actions Bot requested review from a team as code owners July 28, 2026 10:46
@azure-pipelines

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

@Evangelink

Copy link
Copy Markdown
Member

/ba-g #55516 and #55408

@Evangelink
Evangelink merged commit fb1d29c into release/11.0.1xx-preview7 Jul 29, 2026
22 of 24 checks passed
@Evangelink
Evangelink deleted the backport/pr-55480-to-release/11.0.1xx-preview7 branch July 29, 2026 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Approved to merge backport PR that has been backported to a servicing branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants