Add an optional health check to the container definition (#15)

main.tf

@@ -93,7 +93,6 @@ resource "aws_security_group_rule" "egress_service" {
 # LB Target group
 # ------------------------------------------------------------------------------
 resource "aws_lb_target_group" "task" {
-  name        = "${var.name_prefix}-${var.task_container_port}"
   count       = var.lb_arn == "" ? 0 : 1
   vpc_id      = var.vpc_id
   protocol    = var.task_container_protocol
@@ -157,12 +156,14 @@ locals {
     "environment"      = local.task_container_environment
     "environmentFiles" = var.task_container_environment_file
     "MountPoints"      = local.task_container_mount_points
+    "ulimits"          = var.task_container_ulimits
     "logConfiguration" = {
       "logDriver" = "awslogs"
       "options"   = local.log_configuration_options
     }
     "privileged" : var.privileged
     "readonlyRootFilesystem" : var.readonlyRootFilesystem
+    "healthCheck" : var.container_health_check
   }, local.task_container_secrets, local.repository_credentials)
 }
 

outputs.tf

@@ -11,6 +11,11 @@ output "target_group_arn" {
   value       = var.lb_arn == "" ? null : aws_lb_target_group.task[0].arn
 }
 
+output "target_group_arn_suffix" {
+  description = "The ARN suffix for use with CloudWatch Metrics."
+  value       = var.lb_arn == "" ? null : aws_lb_target_group.task[0].arn_suffix
+}
+
 output "target_group_name" {
   description = "The Name of the Target Group."
   value       = var.lb_arn == "" ? null : aws_lb_target_group.task[0].name

variables.tf

@@ -314,3 +314,27 @@ variable "extra_target_groups" {
   }))
   default = []
 }
+
+variable "container_health_check" {
+  description = "An ECS TaskDefinition HealthCheck object to set in each container"
+  default     = null
+  type = object(
+    {
+      command     = list(string)
+      interval    = number
+      retries     = number
+      startPeriod = number
+      timeout     = number
+    }
+  )
+}
+
+variable "task_container_ulimits" {
+  type = list(object({
+    name      = string
+    hardLimit = number
+    softLimit = number
+  }))
+  description = "(Optional) Container ulimit settings. This is a list of maps, where each map should contain \"name\", \"hardLimit\" and \"softLimit\""
+  default     = null
+}