-
Notifications
You must be signed in to change notification settings - Fork 9.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
aws_cloud_watch_logs resource appends wildcard to ARN in output #14410
Comments
Hi @GoldFlsh 👋 Thank you for submitting this and its indeed frustrating behavior. Prior versions of the Terraform AWS Provider just passthrough the ARN value from the CloudWatch Logs API, which includes the |
Great to hear, thank you for fixing this @bflad ! |
This has been released in version 3.0.0 of the Terraform AWS provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template for triage. Thanks! |
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thanks! |
Community Note
Terraform CLI and Terraform AWS Provider Version
Terraform v0.12.24
Affected Resource(s)
Terraform Configuration Files
Debug Output
Panic Output
Expected Behavior
Expect output to be:
arn:aws:logs:{region}:xxxxxxxxxxxx:log-group:my-log-group
Actual Behavior
arn:aws:logs:{region}:xxxxxxxxxxxx:log-group:my-log-group:*
Steps to Reproduce
terraform apply
Important Factoids
I would expect it to leave off the
:*
so it can be used in other resources, if a:*
is needed for something, then the dev can know to add it instead of having to strip it off when it's in the way.For example, I am trying to use it in an IAM role where
:*
is not the appropriate format.https://docs.amazonaws.cn/en_us/IAM/latest/UserGuide/list_amazoncloudwatchlogs.html
To ensure the fix is backwards compatible, adding a new output with the strict arn would work.
References
The text was updated successfully, but these errors were encountered: