Bug 2240188

Summary: [CNV 4.13.12] CDIStorageProfilesIncomplete alerts for noobaa and rgw storage class along with other SC.
Product: Container Native Virtualization (CNV) Reporter: Ulhas Surse <usurse>
Component: StorageAssignee: Arnon Gilboa <agilboa>
Status: CLOSED NOTABUG QA Contact: Jenia Peimer <jpeimer>
Severity: medium Docs Contact:
Priority: high    
Version: 4.13.5CC: dafrank, ycui
Target Milestone: ---Keywords: Regression
Target Release: 4.13.7   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2023-12-13 14:08:37 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Ulhas Surse 2023-09-22 11:06:19 UTC
Description of problem:

- The CDI doesn't support object/bucket storage. However, a storage profile is created for these storage classes by the CDI.

~~~
oc get storageprofile openshift-storage.noobaa.io -o yaml |yq -y '.spec,.status'
{}
---
provisioner: openshift-storage.noobaa.io/obc
storageClass: openshift-storage.noobaa.io
~~~

- And we have the alert "CDIStorageProfilesIncomplete  Incomplete StorageProfiles exist, accessMode/volumeMode cannot be inferred by CDI for PVC population request" for these StorageProfiles.


- Due to this, the Openshift Virtualization status shows Degraded. 


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

OpenShift Virtualization   4.13.12

How reproducible:

100%

Steps to Reproduce:

1. In an OpenShift cluster which is having openshift-storage.noobaa.io and ocs-storagecluster-ceph-rgw storage class, there will be alerts on incomplete storage profiles.

Actual results:

CDIStorageProfilesIncomplete alerts for noobaa and rgw storage class along with other storage class. 

Expected results:

Suppress the alerts for these StorageClass since they are not supported

Additional info:

1] After patching the storage classes with required fields, the alert clears and Operator becomes ready (green). 

  # oc  patch storageprofile <storage-profile-name> --type=merge -p '{"spec": {"claimPropertySets": [{"accessModes": ["ReadWriteOnce"], "volumeMode": "Filesystem"}]}}'

2] It was reported earlier but seems regression:

  https://bugzilla.redhat.com/show_bug.cgi?id=2169686

Comment 2 Arnon Gilboa 2023-11-22 16:28:57 UTC
@usurse can you please paste here the output of `oc get StorageProfile -A -o yaml`?

Comment 5 Arnon Gilboa 2023-12-13 14:08:37 UTC
@usurse as discussed, please reproduce it on >= v4.13.0.rhel9-898 or close it if can't reproduce.