DTOSS-00000: Fix circular depedency on healthcheck

Fix circular dependency involving the health check target_url. The Container App child module derives the health check URL from external_url, so passing target_url into the module is unnecessary.

Author: mrlockstar

infrastructure/modules/container-apps/main.tf

@@ -63,5 +63,5 @@ module "azurerm_application_insights_standard_web_test" {
   location                = var.region
   action_group_id         = var.action_group_id
   application_insights_id = var.app_insights_id
-  target_url              = var.target_url
+  target_url              = "${local.external_url}healthcheck"
 }

infrastructure/modules/container-apps/variables.tf

@@ -159,11 +159,6 @@ variable "enable_alerting" {
   type        = bool
 }
 
-variable "target_url" {
-  description = "The external url"
-  type        = string
-}
-
 variable "alert_window_size" {
   type     = string
   nullable = false
@@ -223,6 +218,7 @@ locals {
   env_vars_from_yaml = yamldecode(
     file("${path.module}/../../environments/${var.env_config}/variables.yml")
   )
+  external_url = "https://${module.frontdoor_endpoint.custom_domains["${var.environment}-domain"].host_name}/"
   common_env = merge(
     local.env_vars_from_yaml,
     {
@@ -239,9 +235,9 @@ locals {
 
   azure_db_env = {
     AZURE_DB_CLIENT_ID = var.deploy_database_as_container ? null : module.db_connect_identity[0].client_id
-    DATABASE_HOST   = var.deploy_database_as_container ? null : module.postgres[0].host
-    DATABASE_NAME   = var.deploy_database_as_container ? null : module.postgres[0].database_names[0]
-    DATABASE_USER   = var.deploy_database_as_container ? null : module.db_connect_identity[0].name
+    DATABASE_HOST      = var.deploy_database_as_container ? null : module.postgres[0].host
+    DATABASE_NAME      = var.deploy_database_as_container ? null : module.postgres[0].database_names[0]
+    DATABASE_USER      = var.deploy_database_as_container ? null : module.db_connect_identity[0].name
   }
 
   storage_account_name = "st${var.app_short_name}${var.environment}uks"

infrastructure/terraform/main.tf

@@ -78,7 +78,6 @@ module "container-apps" {
   use_apex_domain                       = var.use_apex_domain
   infra_key_vault_name                  = local.infra_key_vault_name
   infra_key_vault_rg                    = local.infra_key_vault_rg
-  target_url                            = var.deploy_container_apps ? "${module.container-apps[0].external_url}healthcheck" : null
   resource_group_name_infra             = local.resource_group_name
   container_memory                      = var.container_memory
   min_replicas                          = var.min_replicas