It is possible to use this module without specifying an aws_account_id

[jim80net]

This breaks when enable_aws_integration is not true.

[houqp]

https://github.com/scribd/terraform-aws-datadog/blob/master/examples/cloudwatch_log_sync/main.tf is already using it without account id isn't it?

[jim80net]

I misrepresented the conditions for error. The module accepts invalid parameters:

• enable_aws_integration is true, or the default value (true)
• aws_account_id is null

I would want the module to reject this misconfiguration, rather than what it currently does, which is create a datadog aws integration with no aws_account value.

[houqp]

I see, yeah, that's a valid bug. alternatively, we can pull default aws account id withiin the datadog module if account_id is not provided but aws_integration is enabled.