Skip to content

Coder SSH lag #9352

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.

Sign up for GitHub

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

Jump to bottom
Closed
smolinari opened this issue Aug 26, 2023 · 5 comments · Fixed by #9410
Closed

Coder SSH lag #9352

smolinari opened this issue Aug 26, 2023 · 5 comments · Fixed by #9410
Assignees
@coadler
Labels
s3 Bugs that confuse, annoy, or are purely cosmetic

Comments

@smolinari
Copy link

Hi,

I started a discussion in Discord but didn't find any takers. I'd now like to give the feedback officially, because I think it is important for Coder usage.

Please read the initial post: https://discord.com/channels/747933592273027093/991429648200245358/1143075523124277248

...and the thread right after.

If you need the full story of the issue added here, please let me know.

Scott
@cdr-bot cdr-bot bot added the bug label Aug 26, 2023
@matifali
Copy link
Member

matifali commented Aug 26, 2023 •
edited
Loading

Hi @smolinari is it related to #9337?

@smolinari
Copy link
Author

@matifali - I'd say no. I'm not seeing overall connectivity issues. I'm only seeing a major lag in reactiveness, when I run a process in a console connected via Coder's SSH and only after about 15 minutes of it running. Once Coder finally responds, i.e. I get an active SSH connection again, everything seems to run as usual. So, I'd say my issue is fairly minor compared to that one.

My issue is more a "usability annoyance". 😁

Scott

@ammario ammario added the s3 Bugs that confuse, annoy, or are purely cosmetic label Aug 26, 2023
@ammario
Copy link
Member

ammario commented Aug 26, 2023

From @coadler:

this is because tailscale removed keepalives, without activity tailscale will make the conn idle and require another handshake. already working on a patch

I'm also experiencing the issue.

@smolinari
Copy link
Author

smolinari commented Aug 27, 2023 •
edited
Loading

The removal of the keepalive is interesting. I wonder what the reasoning is on Tailscale's part. And overall, if It is technical or political in nature. 🤔

Scott

@coadler coadler mentioned this issue Aug 29, 2023
Merged
coadler added a commit that referenced this issue Aug 29, 2023
@coadler
fix(tailnet): readd keepalives
a39004a 
Fixes #9352
@coadler coadler mentioned this issue Aug 29, 2023
Merged
@smolinari
Copy link
Author

I'd like to report back that this is fixed now. Thanks so much! What a nice difference to come back to my terminals and have them react quickly. Much, much better!

Scott

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@coadler coadler

Labels
s3 Bugs that confuse, annoy, or are purely cosmetic
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(tailnet): re-add keepalives coder/coder
4 participants
@smolinari @coadler @ammario @matifali