Skip to content

Commit

Permalink
Remove CreateLogGroup permission from service role
Browse files Browse the repository at this point in the history
This permission is not needed because we create the log group with Terraform so the VPC Flow Logs service doesn’t need to do it. On the other hand having this permission causes a bug where, on terraform destroy the log group will be destroyed, but then if there is still a few messages in a VPC Flow Logs queue the managed service will see that the log group does not exist and create it again using.

You’ll then have the log group lingering after the tf destroy, which can cause trouble if you try to terraform apply again with the same name: the log group will be already existing and your apply will fail. Not having the permission prevents that as the managed service will not be able to recreate the log group after the tf destroy.
  • Loading branch information
navaati authored Jan 6, 2021
1 parent af51566 commit 9a0d70e
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion vpc-flow-logs.tf
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,6 @@ data "aws_iam_policy_document" "vpc_flow_log_cloudwatch" {
effect = "Allow"

actions = [
"logs:CreateLogGroup",
"logs:CreateLogStream",
"logs:PutLogEvents",
"logs:DescribeLogGroups",
Expand Down

0 comments on commit 9a0d70e

Please sign in to comment.