Skip to content
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

'ks delete ${KF_ENV} -c kubeflow-core' doesn't take down user notebook pods #968

Closed
amygdala opened this issue Jun 11, 2018 · 5 comments
Closed
Labels
area/jupyter Issues related to Jupyter kind/question

Comments

@amygdala
Copy link
Contributor

Running: ks delete ${KF_ENV} -c kubeflow-core apparently does not delete the user notebook pod(s) (though tf-hub-0 itself is deleted properly).

I couldn't find any existing issues mentioning this, but maybe I didn't use the right search keywords... as I assume this is not intended? Is it a ksonnet-related thing?

I am using version v0.1.3.

@jlewi jlewi added the area/jupyter Issues related to Jupyter label Jun 11, 2018
@jlewi
Copy link
Contributor

jlewi commented Jun 11, 2018

I think this is working as intended.

The Jupyter pods aren't created by the ksonnet component they are created by the JupyterHub pod spawner.

So if we wanted to delete them when JupyterHub is deleted then the spawner should probably set the owner reference on the notebooks pods. We could file an issue against the spawner.

However, I'm not sure that behavior is desired. Deleting notebook pods could potentially result in loss of work so I think it should be a very explicit action.

If you wanted to delete everything including the pods you could do so by deleting the namespace.

What do others think

/cc @pdmack

@amygdala
Copy link
Contributor Author

a n00b question: could one get to the notebook in a browser after the tf-hub & ambassador stuff is torn down? It might more be that you would instead just need to ssh in to the pod container and recover files?

@pdmack
Copy link
Member

pdmack commented Jun 11, 2018

@jlewi is correct. The ksonnet app doesn't know about the arbitrary Jupyter notebook that was launched from JH, so it won't be cleaned up by ks.

@amygdala Yes, you could possibly exec to the pod but I think if you are concerned about preserving NB content you should enable the jupyterNotebookPVCMount parameter for /home/jovyan instead.

@amygdala
Copy link
Contributor Author

@pdmack That makes sense; I was more asking from the viewpoint of one of our users than specific concerns of my own.
That is, assuming the notebook pods stay up for some indefinite amount of time (the amount of time I guess is up to the cluster admin...) -- how do we best document in the user guide what a given user does to get to their stuff under those circs?

(A full user guide is a ways away, but it's good to keep this kind of thing in mind for when we get to the point where usability moves more to the forefront).

@jlewi
Copy link
Contributor

jlewi commented Jun 11, 2018

@amygdala They should be able to access their pods via JupyterHub which will be accessible via the central UI once #810 is fixed. If the user already launched a notebook for themselves than the JupyterHub UI will show this and allow users to either login into it or stop it.

@jlewi jlewi closed this as completed Aug 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/jupyter Issues related to Jupyter kind/question
Projects
None yet
Development

No branches or pull requests

3 participants