Feature/private dns zone

[payasam-raghuramakrishna-nttd]

No description provided.

[Copilot]

Pull Request Overview

This PR introduces private DNS zone management functionality to a Terraform module, specifically focusing on Azure Postgres database private endpoints. The changes enable creation and management of private DNS zones with configurable options and provide outputs for integration with other infrastructure components.

Key Changes:

• Added default Postgres private DNS zone configuration with conditional enablement
• Enhanced outputs to provide DNS zone IDs in multiple formats for flexible consumption
• Introduced targeted DNS zone output capability for specific zone retrieval

Reviewed Changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 3 comments.

File	Description
variables.tf	Added default Postgres DNS zone, new variables for zone enablement and targeted output selection
outputs.tf	Added comprehensive DNS zone ID outputs including mapped, set, and Postgres-specific formats
main.tf	Minor formatting cleanup removing empty line in private DNS zones module

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[Copilot]

The variable target_private_dns_zone_name is defined but not used in any of the outputs or module logic shown. This creates confusion about its purpose and may indicate incomplete implementation.

Suggested change

[Copilot]

The variable private_dns_zone_enabled is not used in the module configuration. If this is meant to conditionally control DNS zone creation, it should be integrated with the for_each logic in the private_dns_zones module.

Suggested change

	description = "Enable or disable the creation of the Postgres private DNS zone."
	description = "Enable or disable the creation of the Postgres private DNS zone. This variable should be used in the for_each or count argument of the private_dns_zones resource/module to conditionally control DNS zone creation."

[Copilot]

Having a hardcoded default DNS zone conflicts with the private_dns_zone_enabled variable. Consider making the default an empty list and using the enabled flag to conditionally include the Postgres zone, or remove the enabled flag if the default is always desired.

Suggested change

	]
	default = []