Merge pull request #129 from DFE-Digital/2134_txt_record_list

[2134] Create DNS zones having multiple values TXT records

dns/records/tfdocs.md

@@ -24,7 +24,6 @@ No modules.
 
 | Name | Description | Type | Default | Required |
 |------|-------------|------|---------|:--------:|
-| <a name="input_azure_credentials"></a> [azure\_credentials](#input\_azure\_credentials) | n/a | `any` | `null` | no |
 | <a name="input_hosted_zone"></a> [hosted\_zone](#input\_hosted\_zone) | n/a | `map(any)` | `{}` | no |
 
 ## Outputs

dns/records/variables.tf

@@ -1,11 +1,4 @@
-
-variable "azure_credentials" { default = null }
-
 variable "hosted_zone" {
   type    = map(any)
   default = {}
 }
-
-locals {
-  azure_credentials = try(jsondecode(var.azure_credentials), null)
-}

dns/zones/resources.tf

@@ -82,20 +82,33 @@ resource "azurerm_dns_caa_record" "caa_record_list" {
 }
 
 # TXT record
-
 locals {
+  # Create a list of maps. Each map contains the zone, the TXT record name and value, and other required fields
   txt_records = flatten([
     for zone_name, zone_cfg in var.hosted_zone : [
-      for record_name, record_cfg in zone_cfg["txt_records"] : {
+      for record_name, record_cfg in try(zone_cfg["txt_records"], []) : {
         record_name         = record_name
         zone_name           = zone_name
         resource_group_name = zone_cfg["resource_group_name"]
         value               = record_cfg["value"]
       }
     ]
   ])
+
+  # Create a list of maps. Each map contains the zone, the TXT record name and list of values, and other required fields
+  txt_record_lists = flatten([
+    for zone_name, zone_cfg in var.hosted_zone : [
+      for record_name, record_cfg in try(zone_cfg["txt_record_lists"], []) : {
+        record_name         = record_name
+        zone_name           = zone_name
+        resource_group_name = zone_cfg["resource_group_name"]
+        txt_record_list     = record_cfg
+      }
+    ]
+  ])
 }
 
+# Create TXT records, each one with a record name a single value
 resource "azurerm_dns_txt_record" "txt_records" {
   for_each = {
     for zone in local.txt_records : "${zone.zone_name}.${zone.record_name}" => zone
@@ -115,3 +128,27 @@ resource "azurerm_dns_txt_record" "txt_records" {
   ]
 
 }
+
+# Create TXT records, each one with a record name and a list of values
+resource "azurerm_dns_txt_record" "txt_record_list" {
+  for_each = {
+    for zone in local.txt_record_lists : "${zone.zone_name}.${zone.record_name}" => zone
+  }
+
+  name                = each.value.record_name
+  zone_name           = each.value.zone_name
+  resource_group_name = each.value.resource_group_name
+  ttl                 = 300
+
+  dynamic "record" {
+    for_each = toset(each.value.txt_record_list)
+    content {
+      value = record.value
+    }
+  }
+
+  depends_on = [
+    azurerm_dns_zone.dns_zone
+  ]
+
+}

dns/zones/tfdocs.md

@@ -18,14 +18,14 @@ No modules.
 |------|------|
 | [azurerm_dns_caa_record.caa_record_list](https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/dns_caa_record) | resource |
 | [azurerm_dns_caa_record.caa_records](https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/dns_caa_record) | resource |
+| [azurerm_dns_txt_record.txt_record_list](https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/dns_txt_record) | resource |
 | [azurerm_dns_txt_record.txt_records](https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/dns_txt_record) | resource |
 | [azurerm_dns_zone.dns_zone](https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/dns_zone) | resource |
 
 ## Inputs
 
 | Name | Description | Type | Default | Required |
 |------|-------------|------|---------|:--------:|
-| <a name="input_azure_credentials"></a> [azure\_credentials](#input\_azure\_credentials) | n/a | `any` | `null` | no |
 | <a name="input_hosted_zone"></a> [hosted\_zone](#input\_hosted\_zone) | n/a | `map(any)` | `{}` | no |
 | <a name="input_tags"></a> [tags](#input\_tags) | n/a | `any` | `null` | no |
 

dns/zones/variables.tf

@@ -1,6 +1,4 @@
 
-variable "azure_credentials" { default = null }
-
 variable "hosted_zone" {
   type    = map(any)
   default = {}
@@ -9,7 +7,3 @@ variable "hosted_zone" {
 variable "tags" {
   default = null
 }
-
-locals {
-  azure_credentials = try(jsondecode(var.azure_credentials), null)
-}