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 provider - rebased #5251

Merged
merged 6 commits into from
Feb 22, 2016
Merged

DataDog provider - rebased #5251

merged 6 commits into from
Feb 22, 2016

Conversation

jen20
Copy link
Contributor

@jen20 jen20 commented Feb 22, 2016

This is a rebased version of the DataDog provider from #3611. Tests are passing.

vincepri and others added 6 commits February 22, 2016 15:02
- Don't drop wildcard if it's the only one.
- Remove monitor resource, it's been replaced by metric_alert,
  outlier_alert and service_check
- Refactor to be closer to the API; each resource creates exactly *one*
  resource, not 2, this removes much unneeded complexity. A warning
  threshold is now supported by the API.
- Remove fuzzy resources like graph, and resources that used them for
  dashboard and screenboards. I'd welcome these resources, but the
  current state of Terraform and the Datadog API does not allow these to
  be implemented in a clean way.
- Support multiple thresholds for metric alerts, remove notify argument.
`query` is used when it is specified by the user, if not
`metric`/`tags`/`keys`/`time_aggr`/`window` is used instead.
@jen20 jen20 mentioned this pull request Feb 22, 2016
@jen20 jen20 added this to the Terraform 0.6.12 milestone Feb 22, 2016
@jen20
Copy link
Contributor Author

jen20 commented Feb 22, 2016

Test run:

$ make testacc TEST=./builtin/providers/datadog
==> Checking that code complies with gofmt requirements...
/Users/James/Code/go/bin/stringer
GO15VENDOREXPERIMENT=1 go generate $(GO15VENDOREXPERIMENT=1 go list ./... | grep -v /vendor/)
TF_ACC=1 GO15VENDOREXPERIMENT=1 go test ./builtin/providers/datadog -v  -timeout 120m
=== RUN   TestProvider
--- PASS: TestProvider (0.00s)
=== RUN   TestProvider_impl
--- PASS: TestProvider_impl (0.00s)
=== RUN   TestAccDatadogMonitor_Basic
--- PASS: TestAccDatadogMonitor_Basic (81.18s)
=== RUN   TestAccDatadogMonitor_Updated
--- PASS: TestAccDatadogMonitor_Updated (75.18s)
PASS
ok      github.com/hashicorp/terraform/builtin/providers/datadog    156.364s

@jen20 jen20 merged commit c8bd02a into master Feb 22, 2016
@jen20 jen20 deleted the atlassian-datadog_provider branch February 22, 2016 21:49
@ghost
Copy link

ghost commented Apr 28, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@ghost ghost locked and limited conversation to collaborators Apr 28, 2020
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants