Skip to content

Commit

Permalink
Merge pull request #308 from trussworks/remove-lifecycle-ignores
Browse files Browse the repository at this point in the history
removes the lifecycle ignore rules put in place for aws provider v4 upgrade
  • Loading branch information
chtakahashi authored Apr 25, 2023
2 parents d44df91 + 41d8fb4 commit a9f4972
Showing 1 changed file with 0 additions and 30 deletions.
30 changes: 0 additions & 30 deletions main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -378,36 +378,6 @@ resource "aws_s3_bucket" "aws_logs" {
Name = var.s3_bucket_name
}
)
lifecycle {
# These lifecycle ignore_changes rules exist to permit a smooth upgrade
# path from version 3.x of the AWS provider to version 4.x
ignore_changes = [
# While no special usage instructions are documented for needing this
# ignore_changes rule, changes are still detected during the upgrade
# process, so this serves to avoid drift detection since the
# aws_s3_bucket_policy will be used instead.
policy,
# While no special usage instructions are documented for needing this
# ignore_changes rule, this should avoid drift detection if conflicts
# with the aws_s3_bucket_versioning exist.
versioning,
# https://registry.terraform.io/providers/hashicorp%20%20/aws/3.75.1/docs/resources/s3_bucket_acl#usage-notes
acl,
grant,
# https://registry.terraform.io/providers/hashicorp%20%20/aws/3.75.1/docs/resources/s3_bucket_lifecycle_configuration#usage-notes
lifecycle_rule,
# https://registry.terraform.io/providers/hashicorp%20%20/aws/3.75.1/docs/resources/s3_bucket_logging#usage-notes
logging,
# https://registry.terraform.io/providers/hashicorp%20%20/aws/3.75.1/docs/resources/s3_bucket_server_side_encryption_configuration#usage-notes
server_side_encryption_configuration,
]
}
}
resource "aws_s3_bucket_policy" "aws_logs" {
Expand Down

0 comments on commit a9f4972

Please sign in to comment.