Skip to content

Ask for user consent if wsl is active during store updates.#40786

Open
chemwolf6922 wants to merge 2 commits into
masterfrom
user/chemwolf6922/defer-auto-update-if-wsl-is-active
Open

Ask for user consent if wsl is active during store updates.#40786
chemwolf6922 wants to merge 2 commits into
masterfrom
user/chemwolf6922/defer-auto-update-if-wsl-is-active

Conversation

@chemwolf6922

@chemwolf6922 chemwolf6922 commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Summary of the Pull Request

Check if WSL is active before installing the msi during a msix installation. If so, ask the user if they would like to shutdown wsl and apply the update. If declined or there is no active user session, the updated will be deferred to the next wslinstaller service start, likely next boot.

PR Checklist

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

Add headless test without the user prompt:
InstallerTests::MsixUpgradeDefer

User prompt manually tested:
image

@chemwolf6922 chemwolf6922 requested a review from a team as a code owner June 12, 2026 05:10
Copilot AI review requested due to automatic review settings June 12, 2026 05:10

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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

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 13 out of 13 changed files in this pull request and generated 3 comments.

Comment thread src/windows/wslinstaller/exe/WslInstaller.cpp
Comment thread localization/strings/en-US/Resources.resw
Comment thread test/windows/InstallerTests.cpp
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