• False
    • Hide


    • False
    • OCPSTRAT-16OpenShift - Kubernetes and Core Platform
    • 0% To Do, 0% In Progress, 100% Done
    • 0
    • 0

      Feature Goal*

      What is our purpose in implementing this?  What new capability will be available to customers?

      The goal of this feature is to provide a consistent, predictable and deterministic approach on how the default storage class(es) is managed.

      Why is this important? (mandatory)

      The current default storage class implementation has corner cases which can result in PVs staying in pending because there is either no default storage class OR multiple storage classes are defined

      Scenarios (mandatory) 

      Provide details for user scenarios including actions to be performed, platform specifications, and user personas.  


      No default storage class

      In some cases there is no default SC defined, this can happen during OCP deployment where components such as the registry request a PV whereas the SC are not been defined yet. This can also happen during a change in default SC, there won't be any between the admin unset the current one and set the new on.


      1. The admin marks the current default SC1 as non-default.

      Another user creates PVC requesting a default SC, by leaving pvc.spec.storageClassName=nil. The default SC does not exist at this point, therefore the admission plugin leaves the PVC untouched with pvc.spec.storageClassName=nil.
      The admin marks SC2 as default.
      PV controller, when reconciling the PVC, updates pvc.spec.storageClassName=nil to the new SC2.
      PV controller uses the new SC2 when binding / provisioning the PVC.

      1. The installer creates PVC for the image registry first, requesting the default storage class by leaving pvc.spec.storageClassName=nil.

      The installer creates a default SC.
      PV controller, when reconciling the PVC, updates pvc.spec.storageClassName=nil to the new default SC.
      PV controller uses the new default SC when binding / provisioning the PVC.

      Multiple Storage Classes

      In some cases there are multiple default SC, this can be an admin mistake (forget to unset the old one) or during the period where a new default SC is created but the old one is still present.

      New behavior:

      1. Create a default storage class A
      2. Create a default storage class B
      3. Create PVC with pvc.spec.storageCLassName = nil

      -> the PVC will get the default storage class with the newest CreationTimestamp (i.e. B) and no error should show.

      -> admin will get an alert that there are multiple default storage classes and they should do something about it.


      CSI that are shipped as part of OCP

      The CSI drivers we ship as part of OCP are deployed and managed by RH operators. These operators automatically create a default storage class. Some customers don't like this approach and prefer to:


      1. Create their own default storage class
      2. Have no default storage class in order to disable dynamic provisioning

      Dependencies (internal and external) (mandatory)

      What items must be delivered by other teams/groups to enable delivery of this epic. 

      No external dependencies.

      Contributing Teams(and contacts) (mandatory) 

      Our expectation is that teams would modify the list below to fit the epic. Some epics may not need all the default groups but what is included here should accurately reflect who will be involved in delivering the epic.

      • Development - STOR
      • Documentation - STOR
      • QE - STOR
      • PX - 
      • Others -

      Acceptance Criteria (optional)

      Provide some (testable) examples of how we will know if we have achieved the epic goal.  

      Drawbacks or Risk (optional)

      Can bring confusion to customer as there is a change in the default behavior customer are used to. This needs to be carefully documented.

      Done - Checklist (mandatory)

      The following points apply to all epics and are what the OpenShift team believes are the minimum set of criteria that epics should meet for us to consider them potentially shippable. We request that epic owners modify this list to reflect the work to be completed in order to produce something that is potentially shippable.

      • CI Testing -  Basic e2e automationTests are merged and completing successfully
      • Documentation - Content development is complete.
      • QE - Test scenarios are written and executed successfully.
      • Technical Enablement - Slides are complete (if requested by PLM)
      • Engineering Stories Merged
      • All associated work items with the Epic are closed
      • Epic status should be “Release Pending” 

            rh-gs-gcharot Gregory Charot
            rh-gs-gcharot Gregory Charot
            Rohit Patil Rohit Patil
            Lisa Pettyjohn Lisa Pettyjohn
            Jan Safranek Jan Safranek
            Eric Rich Eric Rich
            0 Vote for this issue
            5 Start watching this issue