Skip to content

Commit efc8074

Browse files
committed
Explicitly keep previous default for s3 transition lifecycle property
AWS changed the default for this property from varies_by_storage_class to all_storage_classes_128K. For now we are setting this field explicitly so we keep the old behavior. - The AWS provider has an issue describing this change here: hashicorp/terraform-provider-aws#41126 - AWS docs here: https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-properties-s3-bucket-lifecycleconfiguration.html - Slack thread discussing the change here: https://gsa-tts.slack.com/archives/C0ENP71UG/p1739218411084849
1 parent b8be3ff commit efc8074

9 files changed

Lines changed: 10 additions & 0 deletions

File tree

‎terraform/modules/external_domain_broker/s3.tf‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,7 @@ resource "aws_s3_bucket_lifecycle_configuration" "cloudfront_log_bucket_lifecycl
3838
days = 930
3939
}
4040
}
41+
transition_default_minimum_object_size = "varies_by_storage_class"
4142
}
4243

4344
resource "aws_s3_bucket_acl" "cloudfront_log_bucket_acl" {

‎terraform/modules/log_bucket/log_bucket.tf‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,7 @@ resource "aws_s3_bucket_lifecycle_configuration" "log_bucket_lifecycle" {
5252
days = 180
5353
}
5454
}
55+
transition_default_minimum_object_size = "varies_by_storage_class"
5556
}
5657
resource "aws_s3_bucket_policy" "log_bucket_policy" {
5758
bucket = aws_s3_bucket.log_bucket.id

‎terraform/modules/log_bucket_v2/log_bucket.tf‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,7 @@ resource "aws_s3_bucket_lifecycle_configuration" "log_bucket_lifecycle" {
5050
days = 180
5151
}
5252
}
53+
transition_default_minimum_object_size = "varies_by_storage_class"
5354
}
5455
resource "aws_s3_bucket_policy" "log_bucket_policy" {
5556
bucket = aws_s3_bucket.log_bucket.id

‎terraform/modules/logsearch/platform_log_bucket.tf‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@ resource "aws_s3_bucket_lifecycle_configuration" "log_bucket_lifecycle" {
1919
days = 930 # 31 days * 30 months = 930 days
2020
}
2121
}
22+
transition_default_minimum_object_size = "varies_by_storage_class"
2223
}
2324

2425

‎terraform/modules/s3_bucket/encrypted_bucket/encrypted_bucket.tf‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,7 @@ resource "aws_s3_bucket_lifecycle_configuration" "encrypted_bucket_lifecycle" {
4343
}
4444
}
4545
}
46+
transition_default_minimum_object_size = "varies_by_storage_class"
4647
}
4748

4849
resource "aws_s3_bucket_policy" "encrypted_bucket_policy" {

‎terraform/modules/s3_bucket/encrypted_bucket_v2/encrypted_bucket.tf‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,7 @@ resource "aws_s3_bucket_lifecycle_configuration" "encrypted_bucket_lifecycle" {
4747
}
4848
}
4949
}
50+
transition_default_minimum_object_size = "varies_by_storage_class"
5051
}
5152

5253
resource "aws_s3_bucket_policy" "encrypted_bucket_policy" {

‎terraform/modules/s3_bucket/kms_encrypted_bucket/bucket.tf‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -77,4 +77,5 @@ resource "aws_s3_bucket_lifecycle_configuration" "kms_encrypted_bucket_lifecycle
7777
}
7878
}
7979
}
80+
transition_default_minimum_object_size = "varies_by_storage_class"
8081
}

‎terraform/modules/s3_bucket/log_encrypted_bucket/log_encrypted_bucket.tf‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -67,6 +67,7 @@ resource "aws_s3_bucket_lifecycle_configuration" "log_encrypted_bucket_lifecycle
6767
}
6868
}
6969
}
70+
transition_default_minimum_object_size = "varies_by_storage_class"
7071
}
7172

7273
resource "aws_s3_bucket_logging" "log_encrypted_bucket_access_logging" {

‎terraform/stacks/tooling/s3-logs.tf‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@ resource "aws_s3_bucket_lifecycle_configuration" "s3_audit_logs_trail_bucket_lif
2424
days = 930 # ~30 months for M-21-31 compliance
2525
}
2626
}
27+
transition_default_minimum_object_size = "varies_by_storage_class"
2728
}
2829

2930
resource "aws_s3_bucket_logging" "cg-s3-cloudtrail-bucket-logging" {
@@ -134,4 +135,5 @@ resource "aws_s3_bucket_lifecycle_configuration" "s3_access_logs_trail_bucket_li
134135
days = 930 # ~30 months for M-21-31 compliance
135136
}
136137
}
138+
transition_default_minimum_object_size = "varies_by_storage_class"
137139
}

0 commit comments

Comments
 (0)