-
Notifications
You must be signed in to change notification settings - Fork 34k
Closed
Labels
debtCode quality issuesCode quality issuesengineeringVS Code - Build / issue tracking / etc.VS Code - Build / issue tracking / etc.extension-hostExtension host issuesExtension host issuesinsiders-releasedPatch has been released in VS Code InsidersPatch has been released in VS Code InsiderssandboxRunning VSCode in a node-free environmentRunning VSCode in a node-free environment
Milestone
Description
As part of #120431, we have to move the extension host out of the workbench.
Currently, the workbench is the parent of the extension host. When we do a reload, the workbench is recreated. When process reuse is enabled, the workbench PID stays the same throughout the reload, but the node environment is still taken down. Due to this cleanup, the child exit signal from the extension host is not handled, creating a zombie.
By moving the extension host out of the workbench, another longer-lived process can handle the signal instead.
@alexdima do you have any specific ideas as to how the new architecture should look?
Metadata
Metadata
Assignees
Labels
debtCode quality issuesCode quality issuesengineeringVS Code - Build / issue tracking / etc.VS Code - Build / issue tracking / etc.extension-hostExtension host issuesExtension host issuesinsiders-releasedPatch has been released in VS Code InsidersPatch has been released in VS Code InsiderssandboxRunning VSCode in a node-free environmentRunning VSCode in a node-free environment