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_cloud_configuration_rule] Implemented resource to manage cloud_configuration rules #1677

Merged
merged 2 commits into from
Jan 3, 2023

Conversation

symphony-elias
Copy link
Contributor

Implemented new resource datadog_cloud_configuration_rule to manage security monitoring rules of type cloud_configuration.
Jira card: https://datadoghq.atlassian.net/browse/SEC-6093

@symphony-elias symphony-elias changed the title Implemented resource datadog_cloud_configuration_rule [datadog_cloud_configuration_rule] Implemented resource to manage cloud_configuration rules Dec 20, 2022
@symphony-elias symphony-elias marked this pull request as ready for review December 20, 2022 15:09
@symphony-elias symphony-elias requested a review from a team as a code owner December 20, 2022 15:09
estherk15
estherk15 previously approved these changes Dec 20, 2022
estherk15
estherk15 previously approved these changes Dec 21, 2022
@symphony-elias symphony-elias requested a review from a team as a code owner January 2, 2023 10:55
@symphony-elias symphony-elias force-pushed the elias.croze/sec-6093 branch 2 times, most recently from 4bfc195 to 3d8dbf5 Compare January 2, 2023 14:18

Verified

This commit was signed with the committer’s verified signature.
christophetd Christophe Tafani-Dereeper
Bit-Doctor
Bit-Doctor previously approved these changes Jan 2, 2023
Optional: true,
Description: "Related resource types to be checked by the rule. Defaults to empty list.",
MinItems: 0,
MaxItems: 9,
Copy link
Contributor

Choose a reason for hiding this comment

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

Did we decided to go with max 9 in the end? I thought it was 5 last time I followed the conversation.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Verified

This commit was signed with the committer’s verified signature.
christophetd Christophe Tafani-Dereeper
Co-authored-by: Jonathan Machado <jonathan.machado@outlook.com>
@skarimo skarimo merged commit 9d63606 into master Jan 3, 2023
@skarimo skarimo deleted the elias.croze/sec-6093 branch January 3, 2023 18:23
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.

None yet

5 participants