Merge pull request #18 from companieshouse/feature/eric

Fixing terraform
companieshouse · Jan 19, 2024 · 9b17c03 · 9b17c03
2 parents ab96bae + 63627b7
commit 9b17c03
Showing 1 changed file with 1 addition and 7 deletions.
diff --git a/terraform/groups/ecs-service/main.tf b/terraform/groups/ecs-service/main.tf
@@ -42,13 +42,7 @@ module "ecs-service" {
   lb_listener_arn           = data.aws_lb_listener.service_lb_listener.arn
   lb_listener_rule_priority = local.lb_listener_rule_priority
   lb_listener_paths         = local.lb_listener_paths
-  health_check_grace_period_seconds = 240
-  healthcheck_healthy_threshold     = "2"
 
-  # ECS Task container health check
-  use_task_container_healthcheck = true
-  healthcheck_path               = local.healthcheck_path
-  healthcheck_matcher            = local.healthcheck_matcher
 
   # Docker container details
   docker_registry   = var.docker_registry
@@ -67,8 +61,8 @@ module "ecs-service" {
   use_task_container_healthcheck    = true
   healthcheck_path                  = local.healthcheck_path
   healthcheck_matcher               = local.healthcheck_matcher
-  health_check_grace_period_seconds = 300
   healthcheck_healthy_threshold     = "2"
+  health_check_grace_period_seconds = 240
 
   # Service performance and scaling configs
   desired_task_count = var.desired_task_count