Revert "fix: Adding a health_check generates a new task definition revision on every terraform apply (terraform-aws-modules#149)"

This reverts commit 492e323.

Author: sestrella

examples/complete/main.tf

@@ -69,11 +69,6 @@ module "ecs" {
           memory    = 1024
           essential = true
           image     = "public.ecr.aws/aws-containers/ecsdemo-frontend:776fd50"
-
-          health_check = {
-            command = ["CMD-SHELL", "curl -f http://localhost:${local.container_port}/health || exit 1"]
-          }
-
           port_mappings = [
             {
               name          = local.container_name

modules/container-definition/main.tf

@@ -19,12 +19,6 @@ locals {
 
   linux_parameters = var.enable_execute_command ? merge({ "initProcessEnabled" : true }, var.linux_parameters) : merge({ "initProcessEnabled" : false }, var.linux_parameters)
 
-  health_check = length(var.health_check) > 0 ? merge({
-    interval = 30,
-    retries  = 3,
-    timeout  = 5
-  }, var.health_check) : null
-
   definition = {
     command                = length(var.command) > 0 ? var.command : null
     cpu                    = var.cpu
@@ -40,7 +34,7 @@ locals {
     essential              = var.essential
     extraHosts             = local.is_not_windows && length(var.extra_hosts) > 0 ? var.extra_hosts : null
     firelensConfiguration  = length(var.firelens_configuration) > 0 ? var.firelens_configuration : null
-    healthCheck            = local.health_check
+    healthCheck            = length(var.health_check) > 0 ? var.health_check : null
     hostname               = var.hostname
     image                  = var.image
     interactive            = var.interactive