Skip to content

Lambda functions created will only send logs to the cloudwatch group created by the cloudwatch_log_group module if var.function_name is lowercase. #95

Open

Description

Describe the Bug

The cloudwatch log group created with the cloudwatch_log_group module will only match the name of lambda function if var.function_name is lowercase. For example, if var.function_name is Prod-FooBar then the name of the cloudwatch log group created will be /aws/lambda/prod-foobar.

AWS automatically creates cloudwatch log groups based on the name of the lambda function, so once Prod-FooBar is invoked for the first time yet another (non-terraform managed) cloudwatch log group is created called /aws/lambda/Prod-Foobar with default settings.

Expected Behavior

The cloudwatch log group created with the cloudwatch_log_group module follows the same casing conventions as the lambda function name, so if var.function_name is 'Prod-FooBar' then the aws_cloudwatch_log_group created should have the name /aws/lambda/Prod-FooBar

Steps to Reproduce

Set function_name to a string that contains a mix of capital and lowercase letters.

Screenshots

No response

Environment

Module version: 0.6.1
Terraform: 1.11.3

Additional Context

If this is the intended behavior of this module, and users are meant to either pass in a lowercase string for function_name or edit the cloudposse/terraform-null-label/variables.tf, then the inputs documentation should be updated to reflect this.

If not, then the cloudwatch log group created by this module should ALWAYS follow the /aws/lambda/<function name> convention or the log_group should be defined in a logging_config block.

Since the aws_lambda_function block defined in main.tf already depends_on the cloudwatch_log_group module block, setting the function_name of that block to cloudwatch_log_group.outputs.log_group_name would guarantee that the name of the lambda and the log group match, but that would require anyone wanting to use something other than the default lowercase conventions to copy and edit the contents of cloudposse/terraform-null-label/variables.tf.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    bug🐛 An issue with the system

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions