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.

Sign up for GitHub

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

Jump to bottom

Jupyter spawner UI should provide a warning if pod isn't backed by durable storage #2156

Closed
jlewi opened this issue Dec 21, 2018 · 10 comments
Closed

Jupyter spawner UI should provide a warning if pod isn't backed by durable storage #2156

jlewi opened this issue Dec 21, 2018 · 10 comments
Assignees
@kimwnasptd @ioandr
Labels
area/jupyter Issues related to Jupyter cuj/build-train-deploy priority/p1

Comments

@jlewi
Copy link
Contributor

jlewi commented Dec 21, 2018

Context: see #2155

Using PVC is a barrier to kicking the tires on Kubeflow. So we want to make it possible to deploy Kubeflow without using durable storage backing the Jupyter pod.

The spawner UI should provide a clear warning so that users don't get surprised if their pod gets preempted and their data is lost.
@jlewi jlewi added priority/p1 area/jupyter Issues related to Jupyter labels Dec 21, 2018
@jlewi
Copy link
Contributor Author

jlewi commented Feb 4, 2019

@ioandr @vkoukis @kimwnasptd would you be willing to pick this up as part of your work on the Jupyter UI?

@kimwnasptd
Copy link
Member

@jlewi yes we are working on it, here's a draft of what we where thinking. We will add it to the PR
none-pvc

@vkoukis
Copy link
Member

vkoukis commented Feb 4, 2019

/assign kimwnasptd
/assign ioandr

@k8s-ci-robot
Copy link
Contributor

@vkoukis: GitHub didn't allow me to assign the following users: kimwnasptd.

Note that only kubeflow members and repo collaborators can be assigned and that issues/PRs can only have 10 assignees at the same time.
For more information please see the contributor guide

In response to this:

/assign kimwnasptd
/assign ioandr

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@kimwnasptd kimwnasptd mentioned this issue Feb 5, 2019
Merged
@vkoukis
Copy link
Member

vkoukis commented Feb 6, 2019

/assign kimwnasptd

@k8s-ci-robot
Copy link
Contributor

@vkoukis: GitHub didn't allow me to assign the following users: kimwnasptd.

Note that only kubeflow members and repo collaborators can be assigned and that issues/PRs can only have 10 assignees at the same time.
For more information please see the contributor guide

In response to this:

/assign kimwnasptd

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@ioandr
Copy link
Contributor

ioandr commented Feb 6, 2019

/assign kimwnasptd

@ioandr
Copy link
Contributor

ioandr commented Feb 20, 2019

@jlewi this issue is also fixed by #2357

@ioandr
Copy link
Contributor

ioandr commented Feb 21, 2019

The above warning is shown to users when no Volumes are provided in the Notebook form.

We should consider whether the app should provide this warning after checking the storageClass of the K8s cluster, which is not the case right now.

@jlewi
Copy link
Contributor Author

jlewi commented Mar 4, 2019

Marking this as fixed.

There is a follow on issue #2157 to disable the New PVC option if there is no default storage class defined.

@jlewi jlewi closed this as completed Mar 4, 2019
surajkota pushed a commit to surajkota/kubeflow that referenced this issue Jun 13, 2022
@kimwnasptd
Sync kubeflow katib manifests v0.13.0 (kubeflow#2156)
2263c65 
* hack: Update katib script to update the README

Signed-off-by: Kimonas Sotirchos <kimwnasptd@arrikto.com>

* Update kubeflow/katib manifests from v0.13.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kimwnasptd kimwnasptd

@ioandr ioandr

Labels
area/jupyter Issues related to Jupyter cuj/build-train-deploy priority/p1
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@jlewi @vkoukis @kimwnasptd @ioandr @k8s-ci-robot