-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[supervisor] Replace dropbear with openssh #6348
Conversation
Codecov Report
@@ Coverage Diff @@
## main #6348 +/- ##
===========================================
+ Coverage 19.04% 38.14% +19.10%
===========================================
Files 2 18 +16
Lines 168 4375 +4207
===========================================
+ Hits 32 1669 +1637
- Misses 134 2576 +2442
- Partials 2 130 +128
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
4b83a30
to
911754b
Compare
68001cc
to
cb6b6a4
Compare
Why do we do it? Please test that it does not introduce regressions in VS Code Desktop. Especially for things which we had to patch in dropbear. |
/werft run 👍 started the job as gitpod-build-aledbf-ssh2.17 |
Good points - I've updated the description of the PR to answer those questions. W.r.t. to regression, once we find that this approach actually provides the benefits we hope it does, we'll need some support from @gitpod-io/engineering-ide to ensure we're not missing some case. |
mark |
cf4cd34
to
50de4e5
Compare
0033510
to
6a62a69
Compare
e109037
to
a6e7ab0
Compare
7fa2bd3
to
a36e3f2
Compare
a36e3f2
to
1f467a3
Compare
@akosyakov both issues should be fixed now |
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.
/lgtm
everything works, that's awesome guys ❤️
LGTM label has been added. Git tree hash: c83b334f9296f82c8633d23453f808b4d116ccab
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: akosyakov Associated issue: #4848 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Description
This PR switches to OpenSSH instead of dropbear, and starts
sshd
only when a connection to the SSH service is made.This should drastically improve workspace startup time, and subsequently image build and prebuild stability.
Related Issue(s)
Fixes #4848
How to test
sshd: gitpod [priv]
process running in the workspace after a couple of secondsRelease Notes
Documentation