Skip to content

Commit e06b38e

Browse files
authored
Merge pull request #331 from alphagov/auth-1316-ecs-sizing
AUTH-1316: Bring ECS sizing in line with PaaS deployment
2 parents 8b7304b + b723952 commit e06b38e

3 files changed

Lines changed: 14 additions & 1 deletion

File tree

ci/terraform/ecs.tf

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,9 @@ resource "aws_ecs_service" "account_management_ecs_service" {
1010
desired_count = var.account_management_ecs_desired_count
1111
launch_type = "FARGATE"
1212

13+
deployment_minimum_healthy_percent = var.deployment_min_healthy_percent
14+
deployment_maximum_percent = var.deployment_max_percent
15+
1316
network_configuration {
1417
security_groups = [
1518
local.allow_egress_security_group_id,

ci/terraform/production.tfvars

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,3 +4,5 @@ your_account_url = "https://www.gov.uk/account/home"
44
redis_node_size = "cache.m4.xlarge"
55
common_state_bucket = "digital-identity-prod-tfstate"
66
public_access = false
7+
8+
account_management_ecs_desired_count = 4

ci/terraform/variables.tf

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -137,4 +137,12 @@ variable "logging_endpoint_enabled" {
137137
variable "public_access" {
138138
type = bool
139139
default = false
140-
}
140+
}
141+
142+
variable "deployment_min_healthy_percent" {
143+
default = 50
144+
}
145+
146+
variable "deployment_max_percent" {
147+
default = 100
148+
}

0 commit comments

Comments
 (0)