Skip to content
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

chore(deps): update terraform cloudposse/cloudwatch-logs/aws to v0.6.8 #214

Merged
merged 1 commit into from
Mar 2, 2024

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Dec 14, 2023

Mend Renovate

This PR contains the following updates:

Package Type Update Change
cloudposse/cloudwatch-logs/aws (source) module patch 0.6.5 -> 0.6.8

Release Notes

cloudposse/terraform-aws-cloudwatch-logs (cloudposse/cloudwatch-logs/aws)

v0.6.8

Compare Source

🚀 Enhancements

Fix mistake in policy. Part2 @​ramses999 (#​39)

what

This is just a continuation of the fix https://github.com/cloudposse/terraform-aws-cloudwatch-logs/pull/38.
Prod environment tested. That's how it works correctly.

v0.6.7

Compare Source

🚀 Enhancements

Fix mistake in policy @​ramses999 (#​38)

what

Fix mistake in policy

why

The policy is created simply by ARN without the ":" construct, which is necessary to create the correct policy for the role.
Without this ":" construct, the policy is created, but it does not work correctly.
This error was discovered when I tried to create a cloudwatch group in the cloudtrail module.
I got the response "Error: Error updating CloudTrail: InvalidCloudWatchLogsLogGroupArnException: Access denied. Verify in IAM that the role has adequate permissions."
After studying the code, I realized that I need to add the construction ":*" in a couple of lines.
My solution looks like this, I need to replace the lines in file :

This line:
join("", aws_cloudwatch_log_group.default..arn),
replaced by
"${join("", aws_cloudwatch_log_group.default.
.arn)}:*"
You need to do this in both identical lines.

Perhaps you can suggest a better solution, I'm new to terraforming.

references

https://github.com/cloudposse/terraform-aws-cloudwatch-logs/issues/37
https://github.com/cloudposse/terraform-aws-cloudwatch-logs/blob/master/iam.tf#L55

v0.6.6

Compare Source

🤖 Automatic Updates

Update Terraform cloudposse/iam-role/aws to v0.16.2 @​renovate (#​33)

This PR contains the following updates:

Package Type Update Change
cloudposse/iam-role/aws (source) module patch 0.16.1 -> 0.16.2


Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR has been generated by Mend Renovate. View repository job log here.

@renovate renovate bot requested a review from a team as a code owner December 14, 2023 18:28
@renovate renovate bot added the auto-update This PR was automatically generated label Dec 14, 2023
@renovate renovate bot requested review from a team as code owners December 14, 2023 18:28
@renovate renovate bot requested review from johncblandii and woz5999 December 14, 2023 18:28
@renovate renovate bot force-pushed the renovate/cloudposse-cloudwatch-logs-aws-0.x branch from bee2a68 to 776f9bc Compare December 21, 2023 17:58
@renovate renovate bot force-pushed the renovate/cloudposse-cloudwatch-logs-aws-0.x branch from 776f9bc to 3845d67 Compare February 22, 2024 18:06
@hans-d
Copy link

hans-d commented Mar 2, 2024

These changes were released in v1.2.3.

@hans-d hans-d enabled auto-merge (squash) March 2, 2024 11:41
@renovate renovate bot force-pushed the renovate/cloudposse-cloudwatch-logs-aws-0.x branch from 3845d67 to fdddbcd Compare March 2, 2024 17:29
@hans-d
Copy link

hans-d commented Mar 2, 2024

/terratest

@renovate renovate bot force-pushed the renovate/cloudposse-cloudwatch-logs-aws-0.x branch from fdddbcd to 96cd375 Compare March 2, 2024 19:44
@hans-d
Copy link

hans-d commented Mar 2, 2024

/terratest

@renovate renovate bot force-pushed the renovate/cloudposse-cloudwatch-logs-aws-0.x branch from 96cd375 to 3eee007 Compare March 2, 2024 21:05
@hans-d
Copy link

hans-d commented Mar 2, 2024

/terratest

@hans-d hans-d merged commit 5005fd0 into main Mar 2, 2024
10 checks passed
@hans-d hans-d deleted the renovate/cloudposse-cloudwatch-logs-aws-0.x branch March 2, 2024 23:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-update This PR was automatically generated
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant