Skip to content

Emit assemblyName payload for FallbackAssemblyLoad EventSource events - #14014

Merged
AR-May merged 2 commits into
dotnet:mainfrom
AR-May:fix/fallback-assembly-load-eventsource-payload
Aug 3, 2026
Merged

Emit assemblyName payload for FallbackAssemblyLoad EventSource events#14014
AR-May merged 2 commits into
dotnet:mainfrom
AR-May:fix/fallback-assembly-load-eventsource-payload

Conversation

@AR-May

@AR-May AR-May commented Jun 10, 2026

Copy link
Copy Markdown
Member

Context

FallbackAssemblyLoadStart and FallbackAssemblyLoadStop events declare an assemblyName parameter but call WriteEvent without it. The payload is dropped before reaching ETW, so every trace consumer sees an empty payload.

Testing

Existing unit tests

FallbackAssemblyLoadStart/Stop (events 97/98) declared an assemblyName
parameter but called WriteEvent(97)/WriteEvent(98) without passing it,
so the payload was silently dropped in ETW/EventPipe traces.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings June 10, 2026 08:58

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

Fixes MSBuild’s Microsoft-Build EventSource so FallbackAssemblyLoadStart/Stop (EventIds 97/98) actually emit their declared assemblyName payload to ETW/EventPipe, instead of dropping it due to calling WriteEvent without arguments.

Changes:

  • Pass assemblyName into WriteEvent(97, ...) for FallbackAssemblyLoadStart.
  • Pass assemblyName into WriteEvent(98, ...) for FallbackAssemblyLoadStop.

Comment thread src/Framework/MSBuildEventSource.cs
Comment thread src/Framework/MSBuildEventSource.cs
@AR-May

AR-May commented Jul 31, 2026

Copy link
Copy Markdown
Member Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).

@AR-May
AR-May enabled auto-merge (squash) August 3, 2026 08:10
@AR-May
AR-May merged commit f0430ed into dotnet:main Aug 3, 2026
12 checks passed
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.

3 participants