Skip to content

Don't fail extension activation when the server fails to start. - #23386

Merged
ChayimFriedman2 merged 1 commit into
rust-lang:masterfrom
osdyne:push-lpktqwoxnytr
Sep 24, 2026
Merged

ChayimFriedman2 merged 1 commit into
rust-lang:masterfrom
osdyne:push-lpktqwoxnytr

Conversation

@michaelgruenewald

@michaelgruenewald michaelgruenewald commented Sep 17, 2026 •

Copy link
Copy Markdown
Contributor

Instead of failing extension activation when the server fails to start initially, set the server health to stopped and continue. This results in the same behavior as if the server crashed later.

Tested that changing the config, etc. still correctly restarts the server and that manually restarting it also works.

fixes #23371

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Sep 17, 2026

@ChayimFriedman2 ChayimFriedman2 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.

@ChayimFriedman2
ChayimFriedman2 added this pull request to the merge queue Sep 24, 2026
Merged via the queue into rust-lang:master with commit d56fdf3 Sep 24, 2026
21 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Sep 24, 2026
BenjaminBrienen added a commit to BenjaminBrienen/wgsl-analyzer that referenced this pull request Sep 29, 2026
github-merge-queue Bot pushed a commit to wgsl-analyzer/wgsl-analyzer that referenced this pull request Sep 29, 2026
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.

A failed server start makes the whole extension fail to activate, which can strand an extension that configured it

3 participants