Merge pull request #774 from NHSDigital/fix-insights-variable

fix: add enable_alerting variable to infra

Author: Harriethw

infrastructure/modules/container-apps/variables.tf

@@ -156,7 +156,7 @@ variable "use_apex_domain" {
 }
 
 variable "enable_alerting" {
-  description = "Whether monitoring and alerting is enabled for the PostgreSQL Flexible Server."
+  description = "Whether monitoring and alerting is enabled."
   type        = bool
 }
 

infrastructure/modules/infra/main.tf

@@ -79,7 +79,7 @@ module "app_insights_audit" {
 
   log_analytics_workspace_id = module.log_analytics_workspace_audit.id
 
-  enable_alerting = var.enable_alerting
+  enable_alerting     = var.enable_alerting
 }
 
 module "private_link_scoped_service_law" {

infrastructure/modules/infra/variables.tf

@@ -48,6 +48,11 @@ variable "cae_zone_redundancy_enabled" {
   type        = bool
 }
 
+variable "enable_alerting" {
+  description = "Whether monitoring and alerting is enabled."
+  type        = bool
+}
+
 locals {
   hub_vnet_rg_name = "rg-hub-${var.hub}-uks-hub-networking"
 }

infrastructure/terraform/main.tf

@@ -18,6 +18,7 @@ module "infra" {
   protect_keyvault            = var.protect_keyvault
   vnet_address_space          = var.vnet_address_space
   cae_zone_redundancy_enabled = var.cae_zone_redundancy_enabled
+  enable_alerting             = var.enable_alerting
 }
 
 module "shared_config" {

infrastructure/terraform/variables.tf

@@ -157,7 +157,7 @@ variable "alert_window_size" {
 }
 
 variable "enable_alerting" {
-  description = "Whether monitoring and alerting is enabled for the PostgreSQL Flexible Server."
+  description = "Whether monitoring and alerting is enabled."
   type        = bool
   default     = false
 }