-
Notifications
You must be signed in to change notification settings - Fork 313
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
exit of vscode remote containter leaves processes with high cpu in container #6543
Comments
I got this too. |
Also when reloading the window. @alexdima This seems to be a variation of the broken stdio we looked at: |
Looking into using |
Fixed in Remote-Containers as #6547. It might still make sense to investigate on the server's side. #6547 (comment) mentions something similar with Remote-SSH. (Reproduced only 1 out of 3 times I tried, so maybe it is actually a different issue.) /cc @tanhakabir |
@chrmarti the linked issue looks like an issue with connecting. The error logs comes from OpenSSH and we don't actually get to the VS Code Server yet. But I'll try to reproduce and investigate if something similar to this issue is happening on my end. |
@tanhakabir Some guessing on my side: It might be caused by the VS Code Server consuming the CPU when this issue hits. I have seen it consume 100% CPU (but only once) after triggering reconnection by terminating the SSH client. I was still able to connect, but that might depend on how many CPUs the SSH server has available. |
@chrmarti what are the repro steps you're using to repro this issue? Also do you have a link to track and learn more about this issue? #6543 (comment) I'm also not familiar with |
@chrmarti What are the repro steps? Just reloading the window again? |
@alexdima With Remote-Containers 0.231.2 a window reload triggers the problem. With Remote-SSH I think once saw it in a reconnection scenario (killing the SSH client locally to trigger reconnection). A window reload did not trigger it. @tanhakabir |
The Remote SSH problem is happening to me every day. It just started after one of the recent VS Code updates. It happens consistently after a period of inactivity (like stepping away from my mac for a bit). My fix right now is to cmd+shift+p and 'Developer: Reload Window'. Let me know if there's any more information that would be helpful that I can gather beyond what I already posted in #6547 (comment) |
…expected error handler (which has async endless loop protection) instead of using the console in the SIGPIPE handler Commit: ed63059a4752d031535182b1d6eb37719483fbbd
I don't hit this on Insiders, but also didn't reproduce on stable. Are there additional verification steps needed for this? |
I think this only reproduces with Remote-Containers 0.231.2 and is visible after doing |
… error handler (which has async endless loop protection) instead of using the console in the SIGPIPE handler
@alexdima Reloading the window with 0.231.2 makes the server-main.js process go to 100% CPU:
This does not happen with 0.231.6, presumably due to the workaround I added in 0.231.3. This is with:
|
I'll close this one given we have put in place a workaround in the extension. @chrmarti Please reopen if we should continue looking into this. |
Repro steps: microsoft/vscode#168194 (comment) |
Verification steps from @connor4312:
it shouldn’t fail to connect. |
Steps to Reproduce:
Does this issue occur when you try this locally?: No
Does this issue occur when you try this locally and all extensions are disabled?: No
Full cmd line of high cpu process:
There are 2 such processes, each using one core of CPU to its full. One with 2M/s memory increase, another with 0.2M/s memory usage increase. I only noticed this after today 1.66.0 vscode update. Not sure if it occurred before.
The text was updated successfully, but these errors were encountered: