-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add SFTP service to basehub #20
Comments
I think there's a lot of demand for sftp in particular, so I think we should definitely do this! I also would say we should turn on sftp by default (but not ssh), as it just uses openssh and is fairly secure. |
@yuvipanda I think its reasonable to expose by default long term, but that we for the sake of stability across hubs let it be piloted a while in a few hubs until we have ensured that the dependency is sufficiently mature. I'm thinking for example if this service would be enbled in a hub where security is critical, and we end up using a quite old build of a docker image that may have outdated dependencies with known vulnerabilities, for example in OpenSSH. |
Yep makes sense! Let's not turn it on by default to start with. |
Hi @yuvipanda and @consideRatio. Bumping SSH capabilities in the cloud. We have been getting a number of use cases for SSH capabilities in CryoCloud. Here are some of the use cases laid out to include the one mentioned in CryoInTheCloud/hub-image/issues/54:
|
This service, which ships with the yuvipanda/jupyterhub-ssh helm chart, is used to bring data in and out of home directories and works without involvement from a user server by having an SFTP server mount the user storage's directly.
The
jupyterhub-ssh
chart also provides another kind of service - to start or access already started user servers viassh
. Setup of that is not part of this issue - only thesftp
server.Action point
sftp
to help us not make mistakes.See technical notes for an example of what this can involve, as examplified with configuration from the JMTE deployment I've managed.
Technical notes
This ought to be an opt-in feature not enabled by default initially.
Below are snippets of config from configuring jupyterhub-ssh (both ssh part and sftp part) for hub.jupytearth.org also referred to as the JMTE project, as seen in changes in 2i2c-org/infrastructure#436. There, jupyterhub-ssh was added as a dependency to the daskhub helm chart.
From config/clusters/jmte/common.values.yaml
From
config/clusters/jmte/prod.values.yaml
:From helm-charts/daskhub/Chart.yaml:
From helm-charts/daskhub/values.schema.yaml:
The text was updated successfully, but these errors were encountered: