Bug 1956304 - KubevirtHyperconvergedClusterOperatorCRModification alert doesn't work as expected
Summary: KubevirtHyperconvergedClusterOperatorCRModification alert doesn't work as exp...
Status: ON_QA
Alias: None
Product: Container Native Virtualization (CNV)
Classification: Red Hat
Component: Installation
Version: 4.8.0
Hardware: Unspecified
OS: Unspecified
Target Milestone: ---
: 4.8.0
Assignee: Erkan Erol
QA Contact: Inbar Rose
Depends On:
TreeView+ depends on / blocked
Reported: 2021-05-03 12:47 UTC by Erkan Erol
Modified: 2021-05-04 11:42 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Last Closed:
Target Upstream Version:

Attachments (Terms of Use)

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

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


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