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

Verify PVC for /home/jovyan works #854

Closed
jlewi opened this issue May 23, 2018 · 5 comments
Closed

Verify PVC for /home/jovyan works #854

jlewi opened this issue May 23, 2018 · 5 comments
Labels
area/jupyter Issues related to Jupyter priority/p1

Comments

@jlewi
Copy link
Contributor

jlewi commented May 23, 2018

In #561 we made the changes to support mounting all of /home/jovyan on PVC.

We should verify that is working in dev.kubeflow.org.

To do that we will need newly built Jupyter notebook images.

The cron job to rebuild notebook images (#666) should be enabled soon.

@pdmack
Copy link
Member

pdmack commented May 23, 2018

I would work on this if someone could guide me through the test infra. Like, from local to e2e.

@jlewi
Copy link
Contributor Author

jlewi commented May 23, 2018

We're using dev.kubeflow.org as a way of seeing Kubeflow @ HEAD to test things like UI changes that we don't have E2E tests for.

It looks like the cron job was enabled and images were newly built
https://console.cloud.google.com/gcr/images/kubeflow-images-public/GLOBAL/tensorflow-1.8.0-notebook-cpu?gcrImageListsize=50

You should have access to via your @redhat account to dev.kubeflow.org so you should be able to launch a notebook using the new image and just confirm its working.

@pdmack
Copy link
Member

pdmack commented May 23, 2018

The non-PVC default...
screen shot 2018-05-23 at 10 58 56 am

screen shot 2018-05-23 at 11 17 17 am

@jlewi
Copy link
Contributor Author

jlewi commented May 23, 2018

@pdmack Can you explain the last comment? Is PVC working as expected? I think I made a mistake we probably need to update dev.kubeflow.org to pick up the ksonnet changes that set the mount point.

There are instructions here for how to update dev.kubeflow.org.

I think we will need to modify recreate_app.sh to set the location to mount the PVC
https://github.com/kubeflow/testing/blob/master/dev-kubeflow-org/recreate_app.sh#L82

@pdmack
Copy link
Member

pdmack commented May 23, 2018

The screen caps are from the NB launch in d.k.o without a PVC. @jlewi are you saying we want to now have all d.k.o launches use a PVC? If so, I'll have a look at those updates.

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 priority/p1
Projects
None yet
Development

No branches or pull requests

2 participants