-
Notifications
You must be signed in to change notification settings - Fork 900
feat: Add support for VS Code and JetBrains Gateway via SSH #956
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
Conversation
This enables a "kubernetes_pod" to attach multiple agents that could be for multiple services. Each agent is required to have a unique name, so SSH syntax is: `coder ssh <workspace>.<agent>` A resource can have zero agents too, they aren't required.
This enables a "kubernetes_pod" to attach multiple agents that could be for multiple services. Each agent is required to have a unique name, so SSH syntax is: `coder ssh <workspace>.<agent>` A resource can have zero agents too, they aren't required.
This command was `true` by default, which causes a confusing user experience.
Codecov Report
@@ Coverage Diff @@
## main #956 +/- ##
==========================================
+ Coverage 66.55% 66.63% +0.07%
==========================================
Files 241 241
Lines 14582 14577 -5
Branches 115 115
==========================================
+ Hits 9705 9713 +8
+ Misses 3887 3881 -6
+ Partials 990 983 -7
Continue to review full report at Codecov.
|
This fixes various bugs that made this not work: - Incorrect max message size in `peer` - Incorrect reader buffer size in `peer` - Lack of SFTP support in `agent` - Lack of direct-tcpip support in `agent` - Misuse of command from session. It should always use the shell - Blocking on SSH session, only allowing one at a time Fixes #833 too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit on go.mod is all I'd change, looks good otherwise
421dd0e
to
683f87e
Compare
This fixes various bugs that made this not work:
peer
peer
agent
agent
Fixes #833 too.