feat: refactor conditions; support a records

*main.tf: remove suffix; update conditions; add a records
*outputs.tf: update conditions
*variables.tf: add new variables

Author: oleh_mykolaishyn

main.tf

@@ -1,29 +1,30 @@
-locals {
-  prefix                     = length(var.prefix) == 0 ? "" : var.prefix
-  private_dns_zone_vnet_link = var.custom_dns_zone_vnet_link_name == null ? "link-${local.prefix}" : "${local.prefix}-${var.custom_dns_zone_vnet_link_name}"
-}
-
 resource "azurerm_private_dns_zone" "this" {
-  count = length(var.dns_zone_name) == 0 ? 0 : 1
+  count = var.create_private_dns_zone ? 1 : 0
 
   name                = var.dns_zone_name
   resource_group_name = var.resource_group
   tags                = var.tags
 }
 
+resource "azurerm_private_dns_a_record" "example" {
+  for_each = {
+    for object in var.private_dns_a_records :
+    object.name => object
+  }
+
+  name                = each.value.name
+  zone_name           = try(azurerm_private_dns_zone.this[0].name, var.dns_zone_name)
+  resource_group_name = try(azurerm_private_dns_zone.this[0].resource_group_name, var.resource_group)
+  ttl                 = each.value.ttl
+  records             = each.value.records
+}
+
 resource "azurerm_private_dns_zone_virtual_network_link" "this" {
   for_each = var.vnet_map
 
-  name                  = "${local.private_dns_zone_vnet_link}-${each.key}"
-  private_dns_zone_name = length(var.dns_zone_name) == 0 ? var.external_dns_zone_name : azurerm_private_dns_zone.this[0].name
-  resource_group_name   = var.resource_group
+  name                  = "vnet-link-${each.key}"
+  private_dns_zone_name = try(azurerm_private_dns_zone.this[0].name, var.dns_zone_name)
+  resource_group_name   = try(azurerm_private_dns_zone.this[0].resource_group_name, var.resource_group)
   virtual_network_id    = each.value
   tags                  = var.tags
-
-  lifecycle {
-    precondition {
-      condition     = alltrue([length(var.dns_zone_name) == 0, length(var.external_dns_zone_name) == 0]) ? false : true
-      error_message = "Provide either 'dns_zone_name' value to create new Private DNS Zone or 'external_dns_zone_name' value to create link with already existing Private DNS Zone"
-    }
-  }
 }

outputs.tf

@@ -1,19 +1,22 @@
 output "id" {
-  value       = length(var.dns_zone_name) == 0 ? "" : azurerm_private_dns_zone.this[0].id
+  value       = try(azurerm_private_dns_zone.this[0].id, null)
   description = "Private DNS Zone Id"
 }
 
 output "name" {
-  value       = length(var.dns_zone_name) == 0 ? "" : azurerm_private_dns_zone.this[0].name
+  value       = try(azurerm_private_dns_zone.this[0].name, null)
   description = "Private DNS Zone Name"
 }
 
 output "resource_group" {
-  value       = length(var.dns_zone_name) == 0 ? "" : azurerm_private_dns_zone.this[0].resource_group_name
+  value       = try(azurerm_private_dns_zone.this[0].resource_group_name, null)
   description = "Private DNS Zone Resource Group"
 }
 
-output "link_id" {
-  value       = length(var.vnet_map) == 0 ? [] : [for vnet_link in azurerm_private_dns_zone_virtual_network_link.this : vnet_link.id]
-  description = "List of Virtual Network Link Ids"
+output "vnet_link_name_to_id_map" {
+  value = {
+    for k, v in var.vnet_map :
+    azurerm_private_dns_zone_virtual_network_link.this[k].name => azurerm_private_dns_zone_virtual_network_link.this[k].id
+  }
+  description = "Map of Virtual Network Link names to it's ids"
 }

variables.tf

@@ -1,40 +1,37 @@
-variable "resource_group" {
+variable "dns_zone_name" {
   type        = string
-  description = "Azure location"
+  description = "The name of the Private DNS Zone that needs to be created or linked to the virtual network"
 }
 
-variable "custom_dns_zone_vnet_link_name" {
+variable "resource_group" {
   type        = string
-  description = "The name of the Private DNS Zone Virtual Network Link"
-  default     = null
+  description = "Resource group where Private DNS zone would be created or it is already exists"
 }
 
-variable "tags" {
-  type        = map(string)
-  description = "Resource tags"
-  default     = {}
+variable "create_private_dns_zone" {
+  type        = bool
+  description = "Boolean flag that determines whether Private DNS Zones is created by this module"
+  default     = true
 }
 
-variable "prefix" {
-  type        = string
-  description = "Custom prefix to add to resource name"
-  default     = ""
+variable "private_dns_a_records" {
+  type = list(object({
+    name    = string
+    ttl     = optional(number, 300)
+    records = list(string)
+  }))
+  description = "List of objects with parameters to create A Record in Private DNS Zone"
+  default     = []
 }
 
-variable "dns_zone_name" {
-  type        = string
-  description = "Name of Private DNS Zone"
-  default     = ""
+variable "tags" {
+  type        = map(string)
+  description = "Resource tags"
+  default     = {}
 }
 
 variable "vnet_map" {
   type        = map(string)
   description = "Map of Virtual Network Name to Id, used to create VNet Link to Private DNS"
   default     = {}
 }
-
-variable "external_dns_zone_name" {
-  type        = string
-  description = "Name of Imported Private DNS Zone. Provide value in case creation of new Private DNS Zone is disabled"
-  default     = ""
-}