-
Notifications
You must be signed in to change notification settings - Fork 393
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] Add cost alert monitors #2769
Conversation
a5a1379
to
ef8089a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM... I'm not sure this is needed but adding a test for this might be good here datadog/tests/resource_datadog_monitor_test.go
?
fe1454e
ef8089a
to
fe1454e
Compare
/merge |
Devflow running:
|
Adding terraform support for the cost alert monitors
I did have an example in https://github.com/DataDog/terraform-provider-datadog/blob/6fe11d82a1ca55992e511236236577c9f31af879/docs/guides/monitors.md but it was removed when I ran
make docs
-- let me know if I should add it back