Skip to content

Commit 451c886

Browse files
added variable message retention sec
1 parent c4418fb commit 451c886

File tree

5 files changed

+48
-38
lines changed

5 files changed

+48
-38
lines changed

autotagging.tf

Lines changed: 10 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,16 @@
11
# This is the optional Autotagging feature.
22

33
resource "aws_lambda_function" "auto_tagging" {
4-
count = var.enable_auto_tagging == true ? 1 : 0
5-
architectures = var.architectures
6-
description = var.lambda_description
7-
s3_key = var.auto_tagging_s3_key
8-
s3_bucket = var.auto_tagging_s3_bucket
9-
function_name = "${var.lambda_function_name}-auto_tagging"
10-
role = aws_iam_role.auto_tagging_lambda[0].arn
11-
handler = "provided"
12-
runtime = "provided.al2023"
13-
memory_size = var.lambda_memory_size
14-
# lets set 2 minutes
4+
count = var.enable_auto_tagging == true ? 1 : 0
5+
architectures = var.architectures
6+
description = var.lambda_description
7+
s3_key = var.auto_tagging_s3_key
8+
s3_bucket = var.auto_tagging_s3_bucket
9+
function_name = "${var.lambda_function_name}-auto_tagging"
10+
role = aws_iam_role.auto_tagging_lambda[0].arn
11+
handler = "provided"
12+
runtime = "provided.al2023"
13+
memory_size = var.lambda_memory_size
1514
timeout = var.lambda_timeout
1615
reserved_concurrent_executions = var.lambda_reserved_concurrent_executions
1716

glue_create.tf

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -73,8 +73,8 @@ data "aws_iam_policy_document" "glue_create_sqs_dl" {
7373
}
7474

7575
resource "aws_sqs_queue" "glue_create" {
76-
count = var.enable_glue_create ? 1 : 0
77-
76+
count = var.enable_glue_create ? 1 : 0
77+
message_retention_seconds = var.message_retention_seconds
7878
name = var.glue_create_config.sqs_queue_name
7979
policy = data.aws_iam_policy_document.glue_create_sqs[0].json
8080
visibility_timeout_seconds = var.sqs_visibility_timeout_seconds
@@ -87,11 +87,11 @@ resource "aws_sqs_queue" "glue_create" {
8787
}
8888

8989
resource "aws_sqs_queue" "glue_create_dl" {
90-
count = var.enable_glue_create ? 1 : 0
91-
92-
name = var.glue_create_config.sqs_queue_name_dl
93-
policy = data.aws_iam_policy_document.glue_create_sqs_dl[0].json
94-
tags = var.tags
90+
count = var.enable_glue_create ? 1 : 0
91+
message_retention_seconds = var.message_retention_seconds
92+
name = var.glue_create_config.sqs_queue_name_dl
93+
policy = data.aws_iam_policy_document.glue_create_sqs_dl[0].json
94+
tags = var.tags
9595
}
9696

9797
resource "aws_sqs_queue_redrive_allow_policy" "terraform_queue_redrive_allow_policy" {

glue_sync.tf

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -37,8 +37,8 @@ data "aws_iam_policy_document" "glue_sync_sqs_dl" {
3737
}
3838

3939
resource "aws_sqs_queue" "glue_sync" {
40-
count = var.enable_glue_sync ? 1 : 0
41-
40+
count = var.enable_glue_sync ? 1 : 0
41+
message_retention_seconds = var.message_retention_seconds
4242
name = var.glue_sync_config.sqs_queue_name
4343
policy = data.aws_iam_policy_document.glue_sync_sqs[0].json
4444
visibility_timeout_seconds = var.sqs_visibility_timeout_seconds
@@ -51,11 +51,11 @@ resource "aws_sqs_queue" "glue_sync" {
5151
}
5252

5353
resource "aws_sqs_queue" "glue_sync_dl" {
54-
count = var.enable_glue_sync ? 1 : 0
55-
56-
name = var.glue_sync_config.sqs_queue_name_dl
57-
policy = data.aws_iam_policy_document.glue_sync_sqs_dl[0].json
58-
tags = var.tags
54+
count = var.enable_glue_sync ? 1 : 0
55+
message_retention_seconds = var.message_retention_seconds
56+
name = var.glue_sync_config.sqs_queue_name_dl
57+
policy = data.aws_iam_policy_document.glue_sync_sqs_dl[0].json
58+
tags = var.tags
5959
}
6060

6161
resource "aws_sqs_queue_redrive_allow_policy" "glue_syncredrive_allow_policy" {

main.tf

Lines changed: 19 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -141,6 +141,7 @@ resource "aws_sqs_queue" "oxbow_lambda_fifo_sqs" {
141141
policy = data.aws_iam_policy_document.oxbow_lambda_fifo_sqs[0].json
142142
visibility_timeout_seconds = var.sqs_visibility_timeout_seconds
143143
delay_seconds = var.sqs_delay_seconds
144+
message_retention_seconds = var.message_retention_seconds
144145
content_based_deduplication = true
145146
fifo_queue = true
146147
tags = var.tags
@@ -151,11 +152,12 @@ resource "aws_sqs_queue" "oxbow_lambda_fifo_sqs" {
151152
}
152153

153154
resource "aws_sqs_queue" "oxbow_lambda_fifo_sqs_dlq" {
154-
count = local.enable_group_events ? 1 : 0
155-
name = "${var.sqs_fifo_DL_queue_name}.fifo"
156-
policy = data.aws_iam_policy_document.oxbow_lambda_fifo_sqs_dlq[0].json
157-
fifo_queue = true
158-
tags = var.tags
155+
count = local.enable_group_events ? 1 : 0
156+
name = "${var.sqs_fifo_DL_queue_name}.fifo"
157+
message_retention_seconds = var.message_retention_seconds
158+
policy = data.aws_iam_policy_document.oxbow_lambda_fifo_sqs_dlq[0].json
159+
fifo_queue = true
160+
tags = var.tags
159161
}
160162

161163
resource "aws_lambda_event_source_mapping" "group_events_lambda_sqs_trigger" {
@@ -210,6 +212,7 @@ data "aws_iam_policy_document" "group_event_lambda_sqs_dlq" {
210212
resource "aws_sqs_queue" "group_events_lambda_sqs" {
211213
count = local.enable_group_events ? 1 : 0
212214
name = var.sqs_group_queue_name
215+
message_retention_seconds = var.message_retention_seconds
213216
policy = var.sns_topic_arn == "" ? data.aws_iam_policy_document.group_event_lambda_sqs[0].json : data.aws_iam_policy_document.this_sns_to_sqs[0].json
214217
visibility_timeout_seconds = var.sqs_visibility_timeout_seconds
215218
delay_seconds = var.sqs_delay_seconds
@@ -221,10 +224,11 @@ resource "aws_sqs_queue" "group_events_lambda_sqs" {
221224
}
222225

223226
resource "aws_sqs_queue" "group_events_lambda_sqs_dlq" {
224-
count = local.enable_group_events ? 1 : 0
225-
policy = data.aws_iam_policy_document.group_event_lambda_sqs_dlq[0].json
226-
name = var.sqs_group_DL_queue_name
227-
tags = var.tags
227+
count = local.enable_group_events ? 1 : 0
228+
message_retention_seconds = var.message_retention_seconds
229+
policy = data.aws_iam_policy_document.group_event_lambda_sqs_dlq[0].json
230+
name = var.sqs_group_DL_queue_name
231+
tags = var.tags
228232
}
229233

230234

@@ -239,6 +243,7 @@ resource "aws_lambda_event_source_mapping" "this_lambda_events" {
239243
resource "aws_sqs_queue" "this_sqs" {
240244
count = local.enable_group_events ? 0 : 1
241245
name = var.sqs_queue_name
246+
message_retention_seconds = var.message_retention_seconds
242247
policy = var.sns_topic_arn == "" ? data.aws_iam_policy_document.this_sqs_queue_policy_data.json : data.aws_iam_policy_document.this_sns_to_sqs[0].json
243248
visibility_timeout_seconds = var.sqs_visibility_timeout_seconds
244249
delay_seconds = var.sqs_delay_seconds
@@ -250,10 +255,11 @@ resource "aws_sqs_queue" "this_sqs" {
250255
}
251256

252257
resource "aws_sqs_queue" "this_DL" {
253-
count = local.enable_group_events ? 0 : 1
254-
name = var.sqs_queue_name_dl
255-
policy = data.aws_iam_policy_document.this_dead_letter_queue_policy.json
256-
tags = var.tags
258+
count = local.enable_group_events ? 0 : 1
259+
message_retention_seconds = var.message_retention_seconds
260+
name = var.sqs_queue_name_dl
261+
policy = data.aws_iam_policy_document.this_dead_letter_queue_policy.json
262+
tags = var.tags
257263
}
258264

259265
resource "aws_sns_topic_subscription" "this_sns_sub" {

variables.tf

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -343,3 +343,8 @@ variable "architectures" {
343343
type = list(string)
344344
default = ["x86_64"]
345345
}
346+
347+
variable "message_retention_seconds" {
348+
type = number
349+
default = 1209600
350+
}

0 commit comments

Comments
 (0)