Comments (6)
@squat You are right, this check is absolutely right. I will silent this check for resource="imagestreamimports" in alertmanager. Thanks for your fast response.
from cluster-monitoring-operator.
I'm also interested in overwrite default configuration, because of openshift/origin#21508
from cluster-monitoring-operator.
Hi @wenj1104, the resources created by the Cluster Monitoring Operator are entirely managed by the operator. That means that all changes made to them will always be overwritten within a few minutes once the controller re-syncs. This reconciliation is central to the operator pattern and ensures that the resources are always in a known good state and that we can automatically update them at a latter time without worrying about merge-conflicts.
If you want to change an existing rule, as it seems you do, then we'll need to make the change upstream and ship the change with the Cluster Monitoring Operator. Note that many of the rules shipped by this operator are developed outside of this repo and pulled in from different projects. In the case of this particular rule, I'd suggest filing an issue against the Kubernetes-Monitoring project [0], which is where the rule is defined. Chances are others may agree that the rule needs changing and that we'll all benefit from the patch :)
Alternatively, if you want to define an alert at a lower threshold, as you are doing here, then a workaround could be to add a new PrometheusRule objects that specifies this expression. In this case you'd receive an alert both at >90 pods and >100 pods. (please ignore this if that was just an example and not the change you actually needed)
from cluster-monitoring-operator.
@Reamer the same advice applies regarding any changes to the default shipped alerting rules. In the case of your particular alert, the source is also defined in Kubernetes-Monitoring [0]. Please feel free to open a new issue there if you think the alerting rule should be changed for the community and in default OpenShift. From the information in openshift/origin#21508 it looks like the rule is alerting correctly and that the API server latency is indeed too high, so it may not be a desirable change. In any case, let's discuss more over at Kuberentes-Monitoring. If you simply want to ignore that alerting rule, another option would be to silence the alert in Alertmanager.
from cluster-monitoring-operator.
@squat Okay,I get it.Your explanation is quite clear.I decided not to change the default rule.Thanks for your fast response.
from cluster-monitoring-operator.
@wenj1104 ok sounds good. I’ll close this issue for now but please file a new one if you have any more problems :)
from cluster-monitoring-operator.
Related Issues (20)
- Support `Probe` resources HOT 3
- {Request} Ability to configure Prometheus Exporters HOT 5
- Invalid externalURL since ad2d747f2ef037647ebc9f7d5b22118d0dab4bd2 HOT 4
- Release 4.12 Checklist HOT 3
- Missing Metrics in Prometheus on openshift 3.11 HOT 4
- User workload prometheus scraps targets managed by monitoring prometheus instance HOT 4
- Release 4.13 Checklist HOT 5
- crio metrics via http is not safe HOT 7
- Allow kube-state-metrics configuration HOT 3
- Reject invalid PrometheusRule objects beforehand rather than failing the reconciliation HOT 5
- Alerts generated by user-workload-monitoring are not included in remoteWrite HOT 4
- typo in 4.14 CHANGELOG.md and CONTRIBUTING.md HOT 3
- OpenTelemetry Support HOT 16
- kubelet metrics not working in after upgrade to OKD 4.13 HOT 12
- wrong annotation for ThanosRulerConfig.Resources HOT 1
- I can't increase Prometheus startupProbe failureThreshold value HOT 6
- user-workload-monitoring doesn't support honoring labels HOT 4
- Additional Scrape Configuration is not supported HOT 8
- How to change default servicemonitors interval HOT 5
- Support enableRemoteWriteReceiver flag in prometheus{,K8s} config HOT 4
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from cluster-monitoring-operator.