[Tracker]: Default-less Storage Class DS Project

[andrewballantyne]

Description

When we do not have a default (annotation storageclass.kubernetes.io/is-default-class: 'true') value on a StorageClass in the cluster, DS Projects just fail, no way around it.

We have a desire to support full set of storage classes, admin UI and the ability to override an existing default on the cluster and prefer a different one for ODH. That's tracked under:

• Ability to configure available storage classes #1270

This effort should only deal with a gap today, where your cluster has no default.

Target Branch

f/storage-class

Requirements

• List all storage classes (all users should have access RBAC wise)
• Look for a default storage class
  • if one is found, do nothing (continue with the flow today)
  • If one is not found, read the spec.notebookController.storageClassName value
    • if set & in the list, use it on the name on the PVC
    • if not set, do nothing (continue with the flow today)
    • if set & not in the list, print an error to the dev console console.error(...) and do nothing else (continue with the flow today)

Itemized UX Issues

N/A -- Entirely under the hood.

Itemized Dev Issues

• [Feature Request]: Support a default-less StorageClass cluster in DS Projects #1701
• [Bug]: Default-less Storage Class Value is not Specific #1921

Related artifacts

No response

[shalberd]

@andrewballantyne this particular tracker can be considered closed. What is described in requirements works. While #1270 is the more all-encompassing tracker.

[andrewballantyne]

@shalberd I was going to keep this open as an option for us to take this to RHODS for self-managed deployments that may want their own state of storage classes. I'd close it when we moved to writing the larger Storage Class feature.

[shalberd]

sounds good, thank you for the heads-up