Thanks to visit codestin.com
Credit goes to github.com

Skip to content

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

Merged
merged 18 commits into from
Apr 12, 2022
Merged

Conversation

kylecarbs
Copy link
Member

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.

kylecarbs added 14 commits April 8, 2022 14:12
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
Copy link

codecov bot commented Apr 11, 2022

Codecov Report

Merging #956 (81f3849) into main (fb9dc4f) will increase coverage by 0.07%.
The diff coverage is 73.33%.

@@            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     
Flag Coverage Δ
unittest-go-macos-latest 53.52% <73.33%> (-0.18%) ⬇️
unittest-go-postgres- 66.39% <73.33%> (-0.03%) ⬇️
unittest-go-ubuntu-latest 56.29% <73.33%> (-0.02%) ⬇️
unittest-go-windows-2022 53.03% <73.33%> (+0.04%) ⬆️
unittest-js 61.56% <ø> (ø)
Impacted Files Coverage Δ
agent/agent.go 64.59% <70.37%> (-3.47%) ⬇️
peer/channel.go 83.23% <100.00%> (+0.19%) ⬆️
peerbroker/proxy.go 57.55% <0.00%> (-4.07%) ⬇️
coderd/workspaceagents.go 60.00% <0.00%> (-1.78%) ⬇️
provisioner/terraform/provision.go 68.11% <0.00%> (-0.44%) ⬇️
provisionerd/provisionerd.go 80.29% <0.00%> (+0.29%) ⬆️
coderd/provisionerdaemons.go 63.51% <0.00%> (+0.50%) ⬆️
peer/conn.go 80.45% <0.00%> (+2.79%) ⬆️
peerbroker/listen.go 87.50% <0.00%> (+3.33%) ⬆️
... and 2 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fb9dc4f...81f3849. Read the comment docs.

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.
Copy link
Contributor

@f0ssel f0ssel left a 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

@kylecarbs kylecarbs force-pushed the clihelp branch 4 times, most recently from 421dd0e to 683f87e Compare April 11, 2022 23:44
Base automatically changed from clihelp to main April 11, 2022 23:54
@kylecarbs kylecarbs requested a review from a team as a code owner April 11, 2022 23:54
@kylecarbs kylecarbs enabled auto-merge (squash) April 12, 2022 00:07
@kylecarbs kylecarbs merged commit e8b1a57 into main Apr 12, 2022
@kylecarbs kylecarbs deleted the ides branch April 12, 2022 00:17
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.

3 participants