Skip to content
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

[Monitoring]Add Filter in stack monitoring rules #96800

Closed
ravikesarwani opened this issue Apr 12, 2021 · 5 comments · Fixed by #111663
Closed

[Monitoring]Add Filter in stack monitoring rules #96800

ravikesarwani opened this issue Apr 12, 2021 · 5 comments · Fixed by #111663
Assignees
Labels
Feature:Stack Monitoring SM alerting improvements Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services Team:Monitoring Stack Monitoring team v7.16.0

Comments

@ravikesarwani
Copy link
Contributor

ravikesarwani commented Apr 12, 2021

Add “Filter” as an optional field in all the stack monitoring rules.
This can be similar to what we have in Metric threshold rule.
Filter (optional)

This will allows users more control in terms of the scope of alerts.
Users should be able to filter (include or exclude) based on things like cluster id, node id etc. fields.

This combined with allowing users to create additional rules in Rules and connectors will help users to setup different rules for different workloads when monitoring many production clusters to the same monitoring cluster (a typical scenario).

Also this can help with scenarios where a user wants to exclude certain cluster or nodes from a specific alert.
Example1: They do not want alerts to be generated for a Dev or QA cluster.
Example2: They want to exclude certain nodes for a specific alert (do not want disk alert on frozen tier nodes)

@elasticmachine
Copy link
Contributor

Pinging @elastic/stack-monitoring (Team:Monitoring)

@jasonrhodes jasonrhodes removed this from the Stack Monitoring UI 7.14 milestone Jul 7, 2021
@ravikesarwani ravikesarwani changed the title [Monitoring]Add Filter in stack monitoring alerts [Monitoring]Add Filter in stack monitoring rules Aug 3, 2021
@ravikesarwani ravikesarwani added the Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services label Aug 3, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/logs-metrics-ui (Team:logs-metrics-ui)

@ravikesarwani
Copy link
Contributor Author

With 91145 in good shape for 7.15 (thanks team!) I was wondering if we maybe able to look at this one in the near future to see what it would take to implement?

@simianhacker
Copy link
Member

@ravikesarwani For the KQL auto complete fields, do we want to just get the fields from the SM index patterns? OR do we want to use a curated list of fields?

@ravikesarwani
Copy link
Contributor Author

ravikesarwani commented Sep 1, 2021

@simianhacker I think keeping it versatile but simple for the most common use cases maybe the best option here.
In that perspective I was thinking we get the fields from the SM index pattern (versatile) but also provide some way for the user to quickly be able to add include/exclude filter for cluster and node (at least that’s the most common use cases I have seen asks for).
Maybe we provide some quick examples in the text form for key use cases like:

  • Include/Exclude one or more cluster from the rule
  • Include/Exclude one or more node from the rule

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Stack Monitoring SM alerting improvements Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services Team:Monitoring Stack Monitoring team v7.16.0
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants