-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Rules and Alerting][Stack Monitoring] during a deployment change from ESS; ECE; ECK alerts should be muted. #130128
Comments
Pinging @elastic/infra-monitoring-ui (Team:Infra Monitoring UI) |
@ravikesarwani I know you've spent time thinking about this in the past, FYI. |
Not something we have discussed and planned so far but feels like something important that we should think about. |
I think there are a few edge cases that also need to be taken care off. Like |
How does the future Health/Topology APIs play into this? Has there been any thought for how Cloud might "tell" those APIs "I'm changing things under you, chill out for a bit"? |
If the alert API has a mute option, I'm thinking it might make sense for the orchestrator to call it before operations start. We could potentially tag the rules with metadata about which ones are expected to get noisy during certain workflows (like upgrading). |
Moved to Refining on our board. We need to determine and decide whether this is something that we could/would implement or otherwise route this issue to the correct place. |
Closing this. It doesn't look like something @elastic/infra-monitoring-ui would be capable of implementing at this time. |
Kibana has a mute api, maybe we should revisit this issue? |
Describe the feature:
When we run a change e.g. adding nodes, upgrading, .... The built in stack rules go crazy and alert a ton because
node changed
,version mismatch
and so on.Describe a specific use case for the feature:
I would expect that ESS; ECE; ECK "talks" to the stack alerts and mutes them / reconfigures them, so that during a deployment change, no alert is created. Since there is no need for one, as this is expected.
The text was updated successfully, but these errors were encountered: