Skip to content

fix: suppress certain prometheus alerts#82

Merged
reshnm merged 5 commits into
mainfrom
fix/noisy-cr-alerts
May 13, 2026
Merged

fix: suppress certain prometheus alerts#82
reshnm merged 5 commits into
mainfrom
fix/noisy-cr-alerts

Conversation

@reshnm
Copy link
Copy Markdown
Contributor

@reshnm reshnm commented May 13, 2026

What this PR does / why we need it:

Certain controller-runtime and onboarding cluster rules will produce too many alerts.
Therefore these rules will still be present in the prometheus dashboard, but will no longer generate an alert via the alert manager.

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:

Release note:

Suppress alerts for certain prometheus rules

reshnm added 5 commits May 5, 2026 15:59
On-behalf-of: @SAP rene.schuenemann@sap.com
Signed-off-by: René Schünemann (SAP) <rene.schuenemann@sap.com>
On-behalf-of: @SAP rene.schuenemann@sap.com
Signed-off-by: René Schünemann (SAP) <rene.schuenemann@sap.com>
On-behalf-of: @SAP rene.schuenemann@sap.com
Signed-off-by: René Schünemann (SAP) <rene.schuenemann@sap.com>
On-behalf-of: @SAP rene.schuenemann@sap.com
Signed-off-by: René Schünemann (SAP) <rene.schuenemann@sap.com>
On-behalf-of: @SAP rene.schuenemann@sap.com
Signed-off-by: René Schünemann (SAP) <rene.schuenemann@sap.com>
christophrj
christophrj previously approved these changes May 13, 2026
@reshnm reshnm dismissed christophrj’s stale review May 13, 2026 10:16

The merge-base changed after approval.

@reshnm reshnm merged commit 39fab87 into main May 13, 2026
9 checks passed
@reshnm reshnm deleted the fix/noisy-cr-alerts branch May 13, 2026 10:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants