Log Configuration Options should be options

[fszymanski-blvd]

Found a bug? Maybe our Slack Community can help.

Describe the Bug

According to the log configuration documentation, the options section is not required. Our use case for this is a docker container with log configuration embedded in the dockerfile to allow for multiple outputs when using ECS Fargate.

If you try to create a container definition with the following setup

log_configuration = {
  logDriver = "json-file"
}

You will get the following error:

Error: Invalid function argument

  on .terraform/modules/fargate.container_definition/main.tf line 45, in locals:
  45:     options   = tomap(lookup(var.log_configuration, "options"))
    |----------------
    | var.log_configuration is object with 1 attribute "logDriver"

Invalid value for "inputMap" parameter: the given object has no attribute
"options"

Expected Behavior

The container definition should be created with empty options.

[nitrocode]

Unfortunately every key needs to be defined

See this in our examples

terraform-aws-ecs-container-definition/examples/complete/fixtures.us-east-2.tfvars

Lines 49 to 56 in ec4c4a6

	log_configuration = {
	logDriver = "json-file"
	options = {
	"max-size" = "10m"
	"max-file" = "3"
	}
	secretOptions = null
	}

For every key that is unneeded, try using null

[obataku]

#154 should resolve