Bug 2066086 - DataImportCrons do not automatically recover from unconfigured default storage class
Summary: DataImportCrons do not automatically recover from unconfigured default storag...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Container Native Virtualization (CNV)
Classification: Red Hat
Component: Storage
Version: 4.10.0
Hardware: Unspecified
OS: Unspecified
unspecified
urgent
Target Milestone: ---
: 4.10.1
Assignee: Arnon Gilboa
QA Contact: Yan Du
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2022-03-20 15:57 UTC by Arnon Gilboa
Modified: 2022-05-18 20:28 UTC (History)
3 users (show)

Fixed In Version: v4.10.1-6
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2022-05-18 20:27:26 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Github kubevirt containerized-data-importer pull 2196 0 None Merged Reconcile DVs waiting for default SC 2022-03-27 12:44:26 UTC
Github kubevirt containerized-data-importer pull 2206 0 None Merged [release-v1.43] Reconcile DVs waiting for default SC 2022-03-29 09:43:33 UTC
Red Hat Product Errata RHSA-2022:4668 0 None None None 2022-05-18 20:28:56 UTC

Description Arnon Gilboa 2022-03-20 15:57:35 UTC
Description of problem:
DataImportCrons do not automatically recover from unconfigured default storage class

Version-Release number of selected component (if applicable):
CNV 4.10

How reproducible:
Always

Steps to Reproduce:
@alitke installed CNV v4.10.0 on a cluster with OCS where there was no default sc defined.

Actual results:
As expected, auto import process could not complete because PVCs cannot be created without a default storage class.

Even after defining default SC there are still the following messages in the cdi logs and there are no PVCs.

{"level":"error","ts":1647610839.6949024,"logger":"controller.datavolume-controller","msg":"Reconciler error","name":"centos-stream9-8112d4195e45","namespace":"openshift-virtualization-os-images","error":"DataVolume spec is missing accessMode"..}

Expected results:
Resolve the situation automatically and create the PVC when default SC is defined.

Additional info:
This behavior has nothing to do with DataImportCrons, but the regular DV controller flow where DV is not reconciled due to SC update.

In 4.10.0, the way to resolve it manually is to delete the Pending DV/s after setting the default SC. The DIC controller will then immediately re-create the DV using the default SC.

The trivial way to auto-resolve it is by forcing a reconcile in such cases (by RequeueAfter, skipping the repeating event/log after the first one). Another (wiser) option is by watching the SCs for updates.

Comment 1 Adam Litke 2022-03-25 11:59:42 UTC
Retargeting to 4.10.1 as this is a pretty serious usability issue with a relatively simple fix.  We should be able to deliver this.

Comment 2 Yan Du 2022-04-06 12:54:29 UTC
Test on CNV-v4.10.1-50, issue has been fixed.

Comment 8 errata-xmlrpc 2022-05-18 20:27:26 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory (Moderate: OpenShift Virtualization 4.10.1 Images security and bug fix update), and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2022:4668


Note You need to log in before you can comment on or make changes to this bug.