Skip to content

[Security][9.2 & Serverless]: New advanced setting that controls suppression window behavior #2735

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 36 commits into
base: main
Choose a base branch
from
Open

[Security][9.2 & Serverless]: New advanced setting that controls suppression window behavior #2735

wants to merge 36 commits into from
+61 −39

Conversation

nastasha-solomon
Copy link
Contributor

@nastasha-solomon nastasha-solomon commented Aug 27, 2025 •
edited
Loading

Contributes to #2526 by documenting the new securitySolution:suppressionBehaviorOnAlertClosure advanced setting. (preview)

Also made the following changes:

  • Moved pre-reqs for using alert suppression to the section that has instructions for using it. This placement seems more appropriate. Also made light revisions to wordy steps. (preview)
  • Moved that giant note about suppressing fields with multiple values to a dedicated section with the goal of streamlining the instructions for configuring suppression. (preview)
  • Added a new section that explained the impact of closing alerts that were generated by alert suppression. (preview)
  • Resized a few images that looked disproportionately large.

NOTE: Made the same content improvements to the 8.x suppression docs in TBC

@nastasha-solomon nastasha-solomon self-assigned this Aug 27, 2025
@github-actions GitHub Actions
Copy link

github-actions bot commented Aug 27, 2025 •
edited
Loading

🔍 Preview links for changed docs

@nastasha-solomon nastasha-solomon marked this pull request as ready for review August 28, 2025 03:41
@nastasha-solomon nastasha-solomon requested review from a team as code owners August 28, 2025 03:41
approksiu
approksiu requested changes Aug 28, 2025
View reviewed changes
@nastasha-solomon nastasha-solomon changed the title [Security][9.x & Serverless]: New advanced setting that allows the suppression window [Security][9.2 & Serverless]: New advanced setting that allows the suppression window Aug 29, 2025
approksiu
approksiu approved these changes Aug 29, 2025
View reviewed changes
Copy link
Contributor

@approksiu approksiu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome work!

@nastasha-solomon nastasha-solomon changed the title [Security][9.2 & Serverless]: New advanced setting that allows the suppression window [Security][9.2 & Serverless]: New advanced setting that controls suppression window behavior Sep 1, 2025
denar50
denar50 approved these changes Sep 2, 2025
View reviewed changes
Copy link

@denar50 denar50 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bmorelli25 bmorelli25 bmorelli25 left review comments

@denar50 denar50 denar50 approved these changes

@approksiu approksiu approksiu approved these changes

@mdbirnstiehl mdbirnstiehl mdbirnstiehl approved these changes

Assignees

@nastasha-solomon nastasha-solomon

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@nastasha-solomon @bmorelli25 @denar50 @approksiu @mdbirnstiehl