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

[datadog_security_monitoring_rule] Add group_signals_by and case actions #2895

Merged
merged 4 commits into from
Mar 11, 2025

Conversation

Moutix
Copy link
Member

@Moutix Moutix commented Mar 10, 2025

Add support for group_signals_by and case actions in security_monitoring rules resource, specially useful for ASM detection rules

@Moutix Moutix requested review from a team as code owners March 10, 2025 16:13
@Moutix Moutix changed the title Add support for application security detection rules Add support for group_signals_by and case actions in security_monitoring rules Mar 10, 2025
@Moutix Moutix requested a review from a team as a code owner March 10, 2025 16:42
@Moutix Moutix changed the title Add support for group_signals_by and case actions in security_monitoring rules [datadog_security_monitoring_rule] Add group_signals_by and case actions Mar 10, 2025
nizox
nizox previously approved these changes Mar 10, 2025
@Moutix Moutix requested a review from nizox March 11, 2025 09:27
@Moutix
Copy link
Member Author

Moutix commented Mar 11, 2025

/merge

@dd-devflow
Copy link

dd-devflow bot commented Mar 11, 2025

View all feedbacks in Devflow UI.
2025-03-11 13:24:07 UTC ℹ️ Start processing command /merge


2025-03-11 13:24:12 UTC ℹ️ MergeQueue: pull request added to the queue

The expected merge time in master is approximately 22m (p90).


2025-03-11 13:44:43 UTC ℹ️ MergeQueue: This merge request was merged

@dd-mergequeue dd-mergequeue bot merged commit 51e58b3 into master Mar 11, 2025
17 checks passed
@dd-mergequeue dd-mergequeue bot deleted the selim.menouar/terraform-appsec-signals branch March 11, 2025 13:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants