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_synthetics_test does not output an id compatible with composite monitors #205

Closed
hmklabs-dclow opened this issue May 16, 2019 · 5 comments

Comments

@hmklabs-dclow
Copy link

Terraform Version

Terraform v0.11.13

  • provider.datadog v1.9.0

Affected Resource(s)

  • datadog_synthetics_test

Expected Behavior

resource "datadog_monitor" "service_health_check" {
  name  = "Service health check failed"
  type  = "composite"

  message = "A service has failed a health check"

  query = "${join(" || ", datadog_synthetics_test.service_health.*.id)}"
}

Actual Behavior

Output ids are in the format b59-spd-ua6 but it looks like the composite monitor expects a seven digit number.

@nmuesch
Copy link
Contributor

nmuesch commented May 29, 2019

Hey @hmklabs-dclow Thanks for the issue report and terraform config. I was able to reproduce this and can confirm that this isn't possible today. That being said it's something we're looking into. I'll reach back out if there are any updates here.

@platinummonkey
Copy link
Contributor

.monitor_id is the correct correlation for synthetics resource mapping to it's synthetic monitor

@nmuesch
Copy link
Contributor

nmuesch commented May 30, 2019

The monitor_id field isn't currently returned through the go api client, but it looks like thats currently being worked on.

@alksl
Copy link

alksl commented Jun 10, 2019

Datadog is now also returning the monitor_id and it's available in the API client .
zorkian/go-datadog-api#243

@bkabrda
Copy link
Contributor

bkabrda commented Jun 10, 2019

This was actually implemented in https://github.com/terraform-providers/terraform-provider-datadog/pull/222 (added monitor_id attribute of the synthetics tests). It will be part of next release of this provider. Closing this issue.

@bkabrda bkabrda closed this as completed Jun 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants