Merge pull request #10 from Flaconi/plt-894

PLT-894 - Only run data for the zone when cert needs to created
Flaconi · Jul 26, 2024 · 66a70a6 · 66a70a6
2 parents f2d5808 + ee7904b
commit 66a70a6
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/data.tf b/data.tf
@@ -1,3 +1,4 @@
 data "cloudflare_zone" "this" {
-  name = var.zone_name
+  count = var.create_certificate ? 1 : 0
+  name  = var.zone_name
 }
diff --git a/main.tf b/main.tf
@@ -19,7 +19,7 @@ resource "aws_acm_certificate" "this" {
 resource "cloudflare_record" "validation" {
   count = var.create_certificate && var.validate_certificate ? length(local.distinct_domain_names) : 0
 
-  zone_id = data.cloudflare_zone.this.id
+  zone_id = data.cloudflare_zone.this[0].id
   name    = element(local.validation_domains, count.index)["resource_record_name"]
   type    = element(local.validation_domains, count.index)["resource_record_type"]
   value   = replace(element(local.validation_domains, count.index)["resource_record_value"], "/.$/", "")