Uploaded image for project: 'OpenShift Virtualization'
  1. OpenShift Virtualization
  2. CNV-21844

Support immediate binding annotation with WFFC storage that uses snapshot

XMLWordPrintable

    • Icon: Story Story
    • Resolution: Duplicate
    • Icon: Undefined Undefined
    • None
    • CNV v4.11.0
    • Storage Platform
    • False
    • Hide

      None

      Show
      None
    • False
    • Hide
      • Adding the annotation to WFFC storage that supports snapshot clone causes the clone to happen instead of the PVC remaining pending
      • Unit tests
      • Functional tests (this is tricky because our default storage for the test lanes does not have WFFC with snapshot support)
      Show
      Adding the annotation to WFFC storage that supports snapshot clone causes the clone to happen instead of the PVC remaining pending Unit tests Functional tests (this is tricky because our default storage for the test lanes does not have WFFC with snapshot support)
    • None

      We have added an annotation cdi.kubevirt.io/storage.bind.immediate.requested: "true" to tell CDI we don't care about WaitForFirstConsumer storage and to just put the data where it ends up.

      This works well for most cases, including import/upload and host assisted clone. However if we have WFFC storage that supports snapshot clone it doesn't work at all (CSI clone might also be affected). It works for host assisted clone because that form of cloning will create an upload server pod and a source pod and those pods will trigger the binding of the volume when they start. So all we have to do is check for the annotation and proceed normally if found.

      However in the snapshot case, we don't check for the annotation at all, and basically ignore it. This causes confusion when people add the annotation and they are expecting it to just bind somewhere.

      In order to honor the annotation we will have to create a separate trigger pod that just uses the volume that acts as the first consumer.

              akalenyu Alex Kalenyuk
              rhn-support-awels Alexander Wels
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Created:
                Updated:
                Resolved: