Skip to content

Commit

Permalink
Merge pull request #63 from DNXLabs/idle_timeout_var
Browse files Browse the repository at this point in the history
add idle_timeout_var and default 400
  • Loading branch information
lucasmacedot authored Nov 6, 2024
2 parents 94db318 + e771369 commit 4954e04
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 1 deletion.
1 change: 1 addition & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -105,6 +105,7 @@ module "ecs_apps" {
| extra\_certificate\_arns | Extra ACM certificates to add to ALB Listeners | `list(string)` | `[]` | no |
| extra\_task\_policies\_arn | Extra policies to add to the task definition permissions | `list(string)` | `[]` | no |
| fargate\_only | Enable when cluster is only for fargate and does not require ASG/EC2/EFS infrastructure | `bool` | `false` | no |
| idle\_timeout | IDLE time for ALB on seconds. | `number` | `400` | no |
| instance\_types | Instance type for ECS workers | `list(any)` | `[]` | no |
| instance\_volume\_size | Volume size for docker volume (in GB). | `number` | `30` | no |
| instance\_volume\_size\_root | Volume size for root volume (in GB). | `number` | `16` | no |
Expand Down
6 changes: 6 additions & 0 deletions _variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -384,3 +384,9 @@ variable "code_deploy" {
default = true
description = "Enables CodeDeploy role to be used for deployment"
}

variable "idle_timeout" {
type = number
default = 400
description = "IDLE time for ALB on seconds."
}
2 changes: 1 addition & 1 deletion alb.tf
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ resource "aws_lb" "ecs" {
aws_security_group.alb[0].id,
]

idle_timeout = 400
idle_timeout = var.idle_timeout

dynamic "access_logs" {
for_each = compact([var.lb_access_logs_bucket])
Expand Down

0 comments on commit 4954e04

Please sign in to comment.