-
Notifications
You must be signed in to change notification settings - Fork 121
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
Make it possible to provide additional workspace mount #565
Comments
@bmicklea please comment on urgency and motivation for this issue. |
Hi @eivantsov , yes I have the same issue since my product needs to mount to a NAS server, and after some search I found it's impossible to do that directly from the container and I need to mount the host to the NAS server first then to mount container to host. |
@eivantsov - I am starting to see variations of this come up from different customers. You said specifically that an admin would need to identify the host paths. Maybe it would be better to have two modes: How would a user who is authoring a workspace recipe know about, and be able to use these mounts? Would they just magically work inside the workspace, or would we display something to the user, perhaps at the top of their terminal when it opens? Something like, "# Workspace has added host mounts for x, y, and z" |
@TylerJewell as Codenvy On Prem admin I will not let any user mount any path - that's for sure. It's a huge risk of bringing the entire system down. A user can mount some critical Unix paths and mess up with files. It should be admin who defines mount paths and mode (read only?). Just my 2 cents. |
Yes, absolutely agree -it would be an admin selection only, and it would come with a lot of risk factors. |
Closing this as a duplicate of #499 |
Some customer use cases imply mounting host directories in workspace container. Currently it is impossible both on user and admin level.
It should be possible for an installation admin to provide host paths to be mounted in workspace containers that start on a machine node.
The text was updated successfully, but these errors were encountered: