Skip to content

Fix "WSLCCreateContainer" telemetry event#41007

Open
craigloewen-msft wants to merge 2 commits into
masterfrom
user/crloewen/fix-container-telemetry
Open

Fix "WSLCCreateContainer" telemetry event#41007
craigloewen-msft wants to merge 2 commits into
masterfrom
user/crloewen/fix-container-telemetry

Conversation

@craigloewen-msft

Copy link
Copy Markdown
Member

"WSLCCreateContainer" event isn't coming through properly. Looking at the code it seems like it is missing a required line when compared to the "CreateSession" and "ExecCritical" events.

This PR adds it so that it can report out the telemetry.

@craigloewen-msft craigloewen-msft requested a review from a team as a code owner July 6, 2026 18:20
Copilot AI review requested due to automatic review settings July 6, 2026 18:20

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 aims to fix missing telemetry emission metadata for the WSLCCreateContainer event so it matches the pattern used by other critical telemetry events (e.g., WSLCCreateSession, ExecCritical) and is ingested correctly.

Changes:

  • Adds an explicit TraceLoggingLevel(WINEVENT_LEVEL_INFO) argument to the WSLCCreateContainer WSL_LOG call in WSLCSession::CreateContainer.

Comment thread src/windows/wslcsession/WSLCSession.cpp Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings July 6, 2026 19:01

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@craigloewen-msft craigloewen-msft enabled auto-merge (squash) July 6, 2026 19:06
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.

2 participants