Bug 1956304 - KubevirtHyperconvergedClusterOperatorCRModification alert doesn't work as expected
Summary: KubevirtHyperconvergedClusterOperatorCRModification alert doesn't work as exp...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Container Native Virtualization (CNV)
Classification: Red Hat
Component: Installation
Version: 4.8.0
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: ---
: 4.8.0
Assignee: Erkan Erol
QA Contact: Satyajit Bulage
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2021-05-03 12:47 UTC by Erkan Erol
Modified: 2021-07-27 14:32 UTC (History)
2 users (show)

Fixed In Version: hco-bundle-registry-container-v4.8.0-350
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2021-07-27 14:31:30 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
Screenshot is showing actual results. (32.87 KB, image/png)
2021-05-31 10:40 UTC, Satyajit Bulage
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Github kubevirt hyperconverged-cluster-operator pull 1309/ 0 None None None 2021-05-03 15:22:00 UTC
Red Hat Product Errata RHSA-2021:2920 0 None None None 2021-07-27 14:32:08 UTC

Description Erkan Erol 2021-05-03 12:47:07 UTC
Description of problem:

KubevirtHyperconvergedClusterOperatorCRModification doesn't behave as expected.

It is not fired when the resource below are overwritten:
- configmap/kubevirt-storage-class-defaults
- configmap/v2v-vmware
- role/hco.kubevirt.io:config-reader
- rolebinding/hco.kubevirt.io:config-reader


Also, it is not fired when the counter is increased from zero to one. 


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


How reproducible:


Steps to Reproduce:
1. Get a new 4.8 setup
2. Edit labels of configmap/v2v-vmware
3. Observe that the alert is not fired
4. Edit labels of kubevirt 
5. Observe that the alert is not fired
6. Edit labels of kubevirt  again
7. Observe that the alert is fired


Additionally, the alert have component_name label and it does only contain the name of operand. This makes troubleshooting harder since name of some operands don't have any clue about their type (e.g. networkaddonsconfigs/cluster) and there are some operands that have the same name but different types (role/hco.kubevirt.io:config-reader and rolebinding/hco.kubevirt.io:config-reader)

Comment 1 Erkan Erol 2021-05-03 12:50:14 UTC
The PR below
- fixes the behavior for configmaps
- fixes the first change issue

https://github.com/kubevirt/hyperconverged-cluster-operator/pull/1309/

Comment 2 Erkan Erol 2021-05-24 13:53:03 UTC
Note that hco.kubevirt.io:config-reader role/rolebinding are still not strictly reconciled. We have another bug for those objects: https://bugzilla.redhat.com/show_bug.cgi?id=1963963#c0

We think those objects are not critical for the product and we can remove them from the scope of this bug.

Comment 3 Inbar Rose 2021-05-31 05:55:01 UTC
missing fixed in version, please add.

Comment 4 Satyajit Bulage 2021-05-31 10:37:48 UTC
After modifying below components:

- configmap/kubevirt-storage-class-defaults
- configmap/v2v-vmware

Able to see the Alert "KubevirtHyperconvergedClusterOperatorCRModification" is generated.

Comment 5 Satyajit Bulage 2021-05-31 10:40:14 UTC
Created attachment 1788210 [details]
Screenshot is showing actual results.

Comment 8 errata-xmlrpc 2021-07-27 14:31:30 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.8.0 Images), 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-2021:2920


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