Website: Add requirement of dynamic volume provisioner?

[sirkkalap]

Later at https://www.kubeflow.org/docs/started/k8s/kfctl-k8s-istio/ it is stated that "This Kubeflow deployment requires a default StorageClass with a dynamic volume provisioner."

I wonder if this should also be stated in the requirements page: https://www.kubeflow.org/docs/started/k8s/overview/#minimum-system-requirements

Or is this just specific to the Kubeflow Deployment with kfctl_k8s_istio ?

[issue-label-bot[bot]]

Issue-Label Bot is automatically applying the labels:

| Label | Probability |
| ------------- | ------------- |
| kind/feature | 0.60 |
| area/docs | 0.91 |

Please mark this comment with :thumbsup: or :thumbsdown: to give our bot feedback!
Links: app homepage, dashboard and code for this bot.

[sirkkalap]

I am a little bit confused at the end of the https://www.kubeflow.org/docs/started/k8s/kfctl-k8s-istio/ page.

It seems to me that dynamic volume provisioner is after all optional. There seem to be instructions to manually manage PVs instead of dynamic volume provisioner.

Maybe the document could be clarified a bit to make it obvious if DVP is actually a requirement?

If the PersistentVolumeClaims (PVCs) are in Pending state after deployment and they are not bound to PersistentVolumes (PVs), you may have to either manually create PVs for each PVC in your Kubernetes Cluster or an alternative is to set up dynamic volume provisioning to create PVs on demand and redeploy Kubeflow after deleting existing PVCs.

[jbottum]

@sirkkalap Hi Petri - DVP is valuable for several workflows i.e. Do you expect to use Persistent Volumes for your data (in notebooks). Where are you deploying i.e. GCP, AWS, other ?

[8bitmp3]

/assign @Bobgy

(I think @jlewi has previously committed to the doc)

It probably makes sense to add to the minimum requirements/prerequisites that the KF deployment needs a default StorageClass with a dynamic volume provisioner (DVP) - at least in the beginning of docs/started/k8s/kfctl-k8s-istio/ (and possibly to https://www.kubeflow.org/docs/started/k8s/overview/#minimum-system-requirements) as @sirkkalap said.

WDYT?

[jlewi]

I think there are a couple applications that will dynamically provision a volume. For example the jupyter web app will dynamically provision a PD to use as storage for your notebook server.

The jupyter web app supports the case where you don't have a dynamic provisioner.

[jlewi]

@8bitmp3 I would suggest routing this issue to the OWNERs of kfctl-k8s-istio deployment. Is the OWNERs file up to date?

[8bitmp3]

@jlewi The OWNERS should be good for getting started guides (docs/started/) - we've recently refreshed it. I don't think the kfctl-k8s-istio guide (docs/started/k8s/kfctl-k8s-istio/) contributors - @swiftdiaries, you and @sarahmaddox - are listed there though. Shall we ask @swiftdiaries to look into this issue?

[jlewi]

https://github.com/kubeflow/website/tree/master/content/en/docs/started/k8s

Is missing an OWNERs file. We should figure out an appropriate OWNERs file and then route this issue to it.
I'm not an OWNER of this deployment platform.

I think OWNERs are @yanniszark @swiftdiaries

[8bitmp3]

@jlewi OWNERS added.

@yanniszark @swiftdiaries can you take a look at this issue please? Much obliged

[8bitmp3]

/assign @yanniszark @swiftdiaries

[8bitmp3]

/unassign @Bobgy

[stale[bot]]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[8bitmp3]

@Bobgy How do you think we should proceed here? The issue is getting stale. Cheers

[Bobgy]

/assign @yanniszark @swiftdiaries
Friendly ping again.

@8bitmp3 don't worry, you don't need to make sure everything is fixed. The owners should follow up with these issues at their own schedule