Skip to content

[datadog_security_monitoring_rule] Add instantaneous baseline option to NewValue#3400

Merged
dd-mergequeue[bot] merged 4 commits intomasterfrom
mhouertani/instantaneous_baseline
Jan 7, 2026
Merged

[datadog_security_monitoring_rule] Add instantaneous baseline option to NewValue#3400
dd-mergequeue[bot] merged 4 commits intomasterfrom
mhouertani/instantaneous_baseline

Conversation

@hachem-dd
Copy link
Copy Markdown
Contributor

  • Add instantaneous_baseline option for security monitoring new value rules.

Jira

@hachem-dd hachem-dd force-pushed the mhouertani/instantaneous_baseline branch from 0ecaa7d to 1a34803 Compare January 6, 2026 15:53
@hachem-dd hachem-dd force-pushed the mhouertani/instantaneous_baseline branch from a1c4265 to 5db4435 Compare January 6, 2026 17:08
@hachem-dd hachem-dd marked this pull request as ready for review January 6, 2026 17:17
@hachem-dd hachem-dd requested a review from a team as a code owner January 6, 2026 17:17
@hachem-dd hachem-dd requested a review from a team January 6, 2026 17:17
@hachem-dd hachem-dd requested a review from a team as a code owner January 6, 2026 17:17
},
"instantaneous_baseline": {
Type: schema.TypeBool,
Required: true,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Should this be required? Maybe it should be optional and default to false if not included? Otherwise users of our terraform provider will have to add instantaneous_baseline = false to all of their existing rules

Copy link
Copy Markdown
Contributor Author

@hachem-dd hachem-dd Jan 7, 2026

Choose a reason for hiding this comment

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

You're right. Oversight from my part 🫡 47d1bd5

@hachem-dd hachem-dd force-pushed the mhouertani/instantaneous_baseline branch from ab199aa to 47d1bd5 Compare January 7, 2026 09:04
@hachem-dd hachem-dd changed the title Add instantaneous baseline option to NewValue [datadog_security_monitoring_rule_test.go] Add instantaneous baseline option to NewValue Jan 7, 2026
@hachem-dd hachem-dd changed the title [datadog_security_monitoring_rule_test.go] Add instantaneous baseline option to NewValue [datadog_security_monitoring_rule] Add instantaneous baseline option to NewValue Jan 7, 2026
@hachem-dd
Copy link
Copy Markdown
Contributor Author

/merge

@dd-devflow-routing-codex
Copy link
Copy Markdown

dd-devflow-routing-codex Bot commented Jan 7, 2026

View all feedbacks in Devflow UI.

2026-01-07 16:41:43 UTC ℹ️ Start processing command /merge


2026-01-07 16:41:48 UTC ℹ️ MergeQueue: pull request added to the queue

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


2026-01-07 17:13:20 UTC ℹ️ MergeQueue: This merge request was merged

@dd-mergequeue dd-mergequeue Bot merged commit 4dd6a0b into master Jan 7, 2026
13 of 15 checks passed
@dd-mergequeue dd-mergequeue Bot deleted the mhouertani/instantaneous_baseline branch January 7, 2026 17:13
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.

4 participants