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_monitor_config_policy] Add monitor config policies resource and data source #1750

Merged
merged 15 commits into from
Feb 3, 2023

Conversation

carlmartensen
Copy link
Contributor

@carlmartensen carlmartensen commented Feb 1, 2023

Adds resource and data source for monitor config policies. Monitor policies allow users to enforce the presence of certain tags and tag values on your Datadog monitors. This feature was GA'd and released in the public API on Jan 27, 2023.

@carlmartensen carlmartensen changed the title Add monitor config policies resource and data source [datadog_monitor_config_policy] Add monitor config policies resource and data source Feb 1, 2023
@carlmartensen carlmartensen force-pushed the carl/add-monitor-config-policy branch from 0ad44bb to 55b2d29 Compare February 1, 2023 23:17
@carlmartensen carlmartensen marked this pull request as ready for review February 2, 2023 22:01
@carlmartensen carlmartensen requested review from a team as code owners February 2, 2023 22:01
Copy link
Contributor

@phillip-dd phillip-dd left a comment

Choose a reason for hiding this comment

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

lgtm, just one question for error handling

Copy link
Contributor

@jtappa jtappa left a comment

Choose a reason for hiding this comment

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

👍🏻 from docs team

@nkzou nkzou merged commit b0dc0bf into master Feb 3, 2023
@nkzou nkzou deleted the carl/add-monitor-config-policy branch February 3, 2023 20:40
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.

5 participants