diff --git a/modules/container-definition/main.tf b/modules/container-definition/main.tf index 2d8c765..cf73471 100644 --- a/modules/container-definition/main.tf +++ b/modules/container-definition/main.tf @@ -28,10 +28,10 @@ locals { # 2. We then merge in the `initProcessEnabled` attribute based on whether `enable_execute_command` is true or false # This also means we will always have something in `linuxParameters` (it will never be `null` or `{}`) # Terraform doesn't allow us to set `initProcessEnabled` to `true` on one side only of the conditional, so we have to merge it in on both sides - # However, in the `true` case, we set it last to ensure `initProcessEnabled` is always `true` when `enable_execute_command` is true - # and the "psuedo-default" is `false` when `enable_execute_command` is false (but can still be overridden by the user) + # The default is `true` when `enable_execute_command` is true but can be overridden by the user + # and the "pseudo-default" is `false` when `enable_execute_command` is false (but can still be overridden by the user) # tflint-ignore: terraform_naming_convention - linuxParameters = var.enable_execute_command ? merge(local.trimedLinuxParameters, { "initProcessEnabled" : true }) : merge({ "initProcessEnabled" : false }, local.trimedLinuxParameters) + linuxParameters = var.enable_execute_command ? merge({ "initProcessEnabled" : true }, local.trimedLinuxParameters) : merge({ "initProcessEnabled" : false }, local.trimedLinuxParameters) # tflint-ignore: terraform_naming_convention trimmedRestartPolicy = { for k, v in var.restartPolicy : k => v if v != null }