-
Notifications
You must be signed in to change notification settings - Fork 60
add PID namespace #452
Comments
We can consider it. The downside is that container set up is considerably more complicated with a PID namespace. We'd need to add a second code path, basically. Do you have links to relevant documentation from RStudio and tini? What is tini? Google thinks you mean the Argentinian singer, which is probably not right. ;) |
I think this project is meant: |
Ah, I'm sorry. I don't have particular documentation, but here are some bugs exploring that: jupyterhub/the-littlest-jupyterhub#18, jupyterhub/jupyter-rsession-proxy#44 and jupyterhub/jupyter-rsession-proxy#33. Daemonizing seems to be reasonably common among old style applications that fork twice, and kill the parent - making themselves be children of pid 1... |
Thanks for that specific link, @olifre! That is very useful. Now this makes me wonder what happens now with zombie processes and signal forwarding when using ch-run... |
Without having a separate PID namespace, the system's |
Hmm, right. I just tested it out, and So when running inside something like JupyterHub, that spawns and supervises multiple calls to ch-run, I'll have to make sure that my application reaps any processes that ch-run's exec leaves behind. I guess I can also possibly run tini as the process that ch-run starts, which would probably also solve this issue. Either way, I'm currently surprised by the behavior of zombie processes from inside ch-run, regardless of PID namespace use. |
Hello @yuvipanda; sorry for dropping this. Can you provide reproduction steps for the zombie processes? |
Hello @yuvipanda; friendly reminder that I'd love to see steps to reproduce. Thanks! |
Heya! It would be great to add a commandline flag that allows you to set up a PID namespace to isolate users from seeing other commands elsewhere.
Here are some of my use cases:
In general, especially for interactive use cases, this gives you the option to provide a more isolated view of the system than otherwise.
The text was updated successfully, but these errors were encountered: