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

[Enhancement]: For existing alerts, only alert with all events matching the condition are closed #5594

Open
e40pud opened this issue Jul 23, 2024 · 0 comments

Comments

@e40pud
Copy link
Contributor

e40pud commented Jul 23, 2024

Description

We received a bug report on Elastic Stack Community channel claiming that during creating the rule exception the Close all alerts that match this exception and were generated by this rule checkbox does not close existing alerts that match the exception conditions.

After some investigation, I found that it is a current expected behaviour and we need to update our docs to reflect that.

When it comes to alerts which are built based off the group of source events - building blocks (EQL, threshold rules, suppressions etc.):

  • During the rule execution, exceptions applied to all source events and that is why we would not generate an alert if one of the events matches the exceptions conditions.
  • On the other hand, during the add/edit rule exception flow, when user selects to "Close all alerts that match this exception and were generated by this rule" exceptions applied to the generated alerts which will contain source data only if the field present and same in all building blocks - source events.

We need to clarify this behaviour in our docs and make it clear.

cc @yctercero

Related links / assets

Please include each of the following, if applicable:
Doc URL: https://www.elastic.co/guide/en/security/current/add-exceptions.html

Which documentation set needs improvement?

ESS and serverless

Software version

We should add a known behaviour note since the exceptions were introduced. From what I see it is 7.10+

Collaborators

Developer: @e40pud

Timeline / deliverables

If time permits, we can add this into 8.15. Otherwise, next release should be fine as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants