-
Notifications
You must be signed in to change notification settings - Fork 257
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[release-v1.58] Suppress alerts to reduce noise of dependent ones #3181
[release-v1.58] Suppress alerts to reduce noise of dependent ones #3181
Conversation
Manual backport of kubevirt#3129 This is a follow-up to kubevirt#2998 introducing the following changes to alert rules: - CDIDefaultStorageClassDegraded - do not fire when no default SC (either k8s or virt). - CDIDataImportCronOutdated - do not fire when no default SC (either k8s or virt), as DIC import DVs use default SC. - CDINoDefaultStorageClass - fire not only when there is a pending DV (and PVC) but also when DV has an empty status (waiting for default SC etc.) Signed-off-by: Arnon Gilboa <agilboa@redhat.com>
Manual backport of kubevirt#3128 It's time to get the storage capabilities for all no-provisioner storage classes based on the existing PVs, and not only for those labeled with `local.storage.openshift.io/owner-name`, as done so far. It will also silence the unnecessary CDIStorageProfilesIncomplete alert for local SC in the CI. Signed-off-by: Arnon Gilboa <agilboa@redhat.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: akalenyu The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Somehow dropped from the manual backport due to monitoring refactoring. Missed by CI since it does not run the destructive lane where the CDIDefaultStorageClassDegraded alert is tested [test_id:10720]. The kubevirt_cdi_dataimportcron_outdated utest does not check the actual CDIDataImportCronOutdated alert but only the metric and labels, as decided in order to save CI time. Signed-off-by: Arnon Gilboa <agilboa@redhat.com>
Somehow dropped from the manual backport due to monitoring refactoring. Missed by CI since it does not run the destructive lane where the CDIDefaultStorageClassDegraded alert is tested [test_id:10720]. The kubevirt_cdi_dataimportcron_outdated utest does not check the actual CDIDataImportCronOutdated alert but only the metric and labels, as decided in order to save CI time. Signed-off-by: Arnon Gilboa <agilboa@redhat.com>
What this PR does / why we need it:
Manual backport of #3129 and #3128
Which issue(s) this PR fixes:
Fixes # https://issues.redhat.com/browse/CNV-38918
Special notes for your reviewer:
Release note: