Skip to content

[main] Source code updates from dotnet/msbuild#5433

Closed
dotnet-maestro[bot] wants to merge 1 commit into
mainfrom
darc-main-8e945ebc-621b-4dc8-9f7e-bdd7c6e8467d
Closed

[main] Source code updates from dotnet/msbuild#5433
dotnet-maestro[bot] wants to merge 1 commit into
mainfrom
darc-main-8e945ebc-621b-4dc8-9f7e-bdd7c6e8467d

Conversation

@dotnet-maestro

@dotnet-maestro dotnet-maestro Bot commented Mar 13, 2026

Copy link
Copy Markdown
Contributor

Note

This is a codeflow update. It may contain both source code changes from
the source repo
as well as dependency updates. Learn more here.

This pull request brings the following source code changes

From https://github.com/dotnet/msbuild

Diff the source with this PR branch 
darc vmr diff --name-only https://github.com/dotnet/msbuild:df3c977425e16a17b54b80ec687f312afea7ba81..https://github.com/dotnet/dotnet:darc-main-8e945ebc-621b-4dc8-9f7e-bdd7c6e8467d

@dotnet-maestro

dotnet-maestro Bot commented Mar 13, 2026

Copy link
Copy Markdown
Contributor Author

Note

PRs from original repository included in this codeflow update:

💡 You may consult the FAQ for more information or tag @dotnet/prodconsvcs for assistance.

@dotnet-policy-service dotnet-policy-service Bot requested a review from a team March 13, 2026 02:08
@steveisok

steveisok commented Mar 13, 2026

Copy link
Copy Markdown
Member

Scenario Test Failures — MSBuild ProjectImports.zip Regression

All scenario tests are failing on this codeflow PR across all platforms (Linux, CentOS, Windows).

Root Cause

dotnet run with /bl: (binary logging) crashes with:

Unhandled exception: Could not find a part of the path
  '.../MSBuildTemp.../MSBuild<PID>-1/run-dotnet-run.ProjectImports.zip'

The app itself succeeds (Hello, World! prints), but MSBuild then tries to write ProjectImports.zip alongside the binlog into a temp directory that has already been cleaned up. This causes an unhandled IOException, making dotnet run exit with code 1.

Failing Tests (9 total, all same root cause)

  • VerifyConsoleTemplate — CSharp, FSharp, VB
  • VerifyConsoleTemplateComplex — CSharp, FSharp, VB
  • VerifyReferenceInConsoleTemplate — CSharp, VB
  • VerifyPreMadeSolution

Evidence

  • Build 1333470 (main ← msbuild): Failed on Ubuntu, CentOS, Windows
  • Build 1332856 (release/10.0.3xx ← msbuild): Same failures on all platforms

MSBuild Commit Range

8cee1da...df3c977 — this range likely introduced a change to temp directory cleanup timing that races with binlog import archival.

cc @dotnet/msbuild-maintainers

@steveisok steveisok mentioned this pull request Mar 13, 2026
Merged
This was referenced Mar 13, 2026
Closed
Merged
@dotnet-maestro

dotnet-maestro Bot commented Mar 14, 2026

Copy link
Copy Markdown
Contributor Author

Important

While this PR was open, the source repository has received code changes from this repository (an opposite codeflow merged).
To avoid complex conflicts, the codeflow cannot continue until this PR is closed or merged.

You can continue with one of the following options:

  • Ignore this and merge this PR as usual without waiting for the new changes.
    Once merged, Maestro will create a new codeflow PR with the new changes.
  • Close this PR and wait for Maestro to open a new one with old and new changes included.
    You will lose any manual changes made in this PR.
    You can also manually trigger the new codeflow right away by running: 
    darc trigger-subscriptions --id d1c6afb2-7bf1-43e4-8f2e-af7e0680b540
  • Force a codeflow into this PR at your own risk if you want the new changes.
    User commits made to this PR might be reverted. 
    darc trigger-subscriptions --id d1c6afb2-7bf1-43e4-8f2e-af7e0680b540 --force

💡 You may consult the FAQ for more information or tag @dotnet/prodconsvcs for assistance.

@ViktorHofer ViktorHofer deleted the darc-main-8e945ebc-621b-4dc8-9f7e-bdd7c6e8467d branch March 19, 2026 12:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@steveisok @ViktorHofer