New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Add test for setting SQS DLQ MessageRetentionPeriod #1772

Open

filipsnastins wants to merge 1 commit into kalaspuff:master from filipsnastins:feature/add-test-for-sns-sqs-dlq-retention-period

+19 −0

Contributor

filipsnastins commented Jun 26, 2023

I was investigating why some of our SQS DLQs have the retention period set to 4 days.

Turned out that we just needed to update tomodachi to 0.24.1 where this PR introduced the config for retention period and recreate the DLQs

Wrote a test during the investigation - submitting the PR so that it doesn't go to waste 😄


          add test for setting SQS DLQ MessageRetentionPeriod

354b0c7

# for free to join this conversation on GitHub. Already have an account? # to comment

Labels

None yet