Skip to content

Commit 9243aeb

Browse files
authored
VED-1005: Fix S3 replication config. (#1116)
* VED-1005: Add required metrics block to replication config. * VED-1005: Add required delete marker config. * VED-1005: Avoid changing the storage class when replicating.
1 parent 6c94988 commit 9243aeb

1 file changed

Lines changed: 24 additions & 8 deletions

File tree

infrastructure/instance/s3_config.tf

Lines changed: 24 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -145,20 +145,28 @@ resource "aws_s3_bucket_replication_configuration" "replication" {
145145
priority = strcontains(var.sub_environment, "blue") ? 0 : 1
146146
status = "Enabled"
147147

148-
filter {
149-
prefix = ""
148+
filter {}
149+
150+
delete_marker_replication {
151+
status = "Disabled"
150152
}
151153

152154
destination {
153-
bucket = aws_s3_bucket.batch_data_source_bucket.arn
154-
storage_class = "STANDARD"
155+
bucket = aws_s3_bucket.batch_data_source_bucket.arn
155156

156157
replication_time {
157158
status = "Enabled"
158159
time {
159160
minutes = 15
160161
}
161162
}
163+
164+
metrics {
165+
status = "Enabled"
166+
event_threshold {
167+
minutes = 15
168+
}
169+
}
162170
}
163171
}
164172

@@ -167,20 +175,28 @@ resource "aws_s3_bucket_replication_configuration" "replication" {
167175
priority = strcontains(var.sub_environment, "blue") ? 1 : 0
168176
status = "Disabled"
169177

170-
filter {
171-
prefix = ""
178+
filter {}
179+
180+
delete_marker_replication {
181+
status = "Disabled"
172182
}
173183

174184
destination {
175-
bucket = strcontains(aws_s3_bucket.batch_data_source_bucket.arn, "blue") ? replace(aws_s3_bucket.batch_data_source_bucket.arn, "blue", "green") : replace(aws_s3_bucket.batch_data_source_bucket.arn, "green", "blue")
176-
storage_class = "STANDARD"
185+
bucket = strcontains(aws_s3_bucket.batch_data_source_bucket.arn, "blue") ? replace(aws_s3_bucket.batch_data_source_bucket.arn, "blue", "green") : replace(aws_s3_bucket.batch_data_source_bucket.arn, "green", "blue")
177186

178187
replication_time {
179188
status = "Enabled"
180189
time {
181190
minutes = 15
182191
}
183192
}
193+
194+
metrics {
195+
status = "Enabled"
196+
event_threshold {
197+
minutes = 15
198+
}
199+
}
184200
}
185201
}
186202
}

0 commit comments

Comments
 (0)