Depend explicitly on the challenge to prevent some errors

like ``` "registry.terraform.io/cloudflare/cloudflare" produced an invalid new value for .ownership_challenge: was null, but now cty.StringVal("[redacted]..[redacted]") ``` As recommended by @jacobbednarz in cloudflare#2752 (comment) Workaround for issues like cloudflare#2794 cloudflare#2752 cloudflare#3001
gdubicki · Apr 22, 2024 · ae0ebb8 · ae0ebb8
1 parent 4e0a09e
commit ae0ebb8
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/docs/resources/logpush_job.md b/docs/resources/logpush_job.md
@@ -74,6 +74,9 @@ data "aws_s3_bucket_object" "challenge_file" {
 }
 
 resource "cloudflare_logpush_job" "example_job" {
+  depends_on = [
+    cloudflare_logpush_ownership_challenge.ownership_challenge, data.aws_s3_bucket_object.challenge_file
+  ]
   enabled             = true
   zone_id             = "0da42c8d2132a9ddaf714f9e7c920711"
   name                = "My-logpush-job"

diff --git a/examples/resources/cloudflare_logpush_job/resource.tf b/examples/resources/cloudflare_logpush_job/resource.tf
@@ -45,6 +45,9 @@ data "aws_s3_bucket_object" "challenge_file" {
 }
 
 resource "cloudflare_logpush_job" "example_job" {
+  depends_on = [
+    cloudflare_logpush_ownership_challenge.ownership_challenge, data.aws_s3_bucket_object.challenge_file
+  ]
   enabled             = true
   zone_id             = "0da42c8d2132a9ddaf714f9e7c920711"
   name                = "My-logpush-job"