Update var

ministryofjustice · Oct 29, 2024 · f00b42b · f00b42b
1 parent 7fa8b80
commit f00b42b
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 7 deletions.
diff --git a/terraform/account/region.tf b/terraform/account/region.tf
@@ -45,4 +45,4 @@ module "eu_west_2" {
     aws.management = aws.management_eu_west_2
     aws.shared     = aws.shared
   }
-}
+}
diff --git a/terraform/account/region/pagerduty.tf b/terraform/account/region/pagerduty.tf
@@ -104,7 +104,7 @@ resource "aws_sns_topic_subscription" "cloudwatch_sns_subscription" {
 resource "pagerduty_service_integration" "cloudwatch_application_insights" {
   count   = var.account.cloudwatch_application_insights_enabled ? 1 : 0
   name    = "Use an LPA ${data.aws_region.current.name} Cloudwatch Application Insights Ops Item Alarm"
-  service = var.pagerduty_service_id
+  service = var.account.pagerduty_service_id
   vendor  = data.pagerduty_vendor.cloudwatch.id
 }
 

diff --git a/terraform/account/region/variables.tf b/terraform/account/region/variables.tf
@@ -42,11 +42,6 @@ variable "lambda_container_version" {
   type        = string
 }
 
-variable "pagerduty_service_id" {
-  description = "The ID of the PagerDuty service to use."
-  type        = string
-}
-
 variable "vpc_flow_logs_iam_role" {
   description = "The IAM role for VPC flow logs"
   type = object({
-Original file line number
+Diff line change
@@ Expand Up / @@ -45,4 +45,4 @@ module "eu_west_2" { @@
         aws.management = aws.management_eu_west_2
         aws.shared     = aws.shared
       }
-    }
+    }