Skip to content

Commit

Permalink
Align CDK DLQ Naming (#1542)
Browse files Browse the repository at this point in the history
  • Loading branch information
samchungy committed Apr 17, 2024
1 parent aa9a11d commit 6600a20
Show file tree
Hide file tree
Showing 3 changed files with 19 additions and 10 deletions.
5 changes: 5 additions & 0 deletions .changeset/heavy-apples-applaud.md
@@ -0,0 +1,5 @@
---
'skuba': patch
---

template/lambda-sqs-worker-cdk: Align dead letter queue naming with Serverless template
Expand Up @@ -934,7 +934,7 @@ exports[`returns expected CloudFormation stack for dev 1`] = `
"RedrivePolicy": {
"deadLetterTargetArn": {
"Fn::GetAtt": [
"workerqueuedlq42262778",
"workerqueuedeadletters83F3505C",
"Arn",
],
},
Expand Down Expand Up @@ -997,7 +997,7 @@ exports[`returns expected CloudFormation stack for dev 1`] = `
},
"Type": "AWS::SNS::Subscription",
},
"workerqueuedlq42262778": {
"workerqueuedeadletters83F3505C": {
"DeletionPolicy": "Delete",
"Properties": {
"KmsMasterKeyId": {
Expand All @@ -1006,7 +1006,7 @@ exports[`returns expected CloudFormation stack for dev 1`] = `
"Arn",
],
},
"QueueName": "serviceName-dlq",
"QueueName": "serviceName-dead-letters",
},
"Type": "AWS::SQS::Queue",
"UpdateReplacePolicy": "Delete",
Expand Down Expand Up @@ -1976,7 +1976,7 @@ exports[`returns expected CloudFormation stack for prod 1`] = `
"RedrivePolicy": {
"deadLetterTargetArn": {
"Fn::GetAtt": [
"workerqueuedlq42262778",
"workerqueuedeadletters83F3505C",
"Arn",
],
},
Expand Down Expand Up @@ -2039,7 +2039,7 @@ exports[`returns expected CloudFormation stack for prod 1`] = `
},
"Type": "AWS::SNS::Subscription",
},
"workerqueuedlq42262778": {
"workerqueuedeadletters83F3505C": {
"DeletionPolicy": "Delete",
"Properties": {
"KmsMasterKeyId": {
Expand All @@ -2048,7 +2048,7 @@ exports[`returns expected CloudFormation stack for prod 1`] = `
"Arn",
],
},
"QueueName": "serviceName-dlq",
"QueueName": "serviceName-dead-letters",
},
"Type": "AWS::SQS::Queue",
"UpdateReplacePolicy": "Delete",
Expand Down
12 changes: 8 additions & 4 deletions template/lambda-sqs-worker-cdk/infra/appStack.ts
Expand Up @@ -32,10 +32,14 @@ export class AppStack extends Stack {

kmsKey.grantEncrypt(accountPrincipal);

const deadLetterQueue = new aws_sqs.Queue(this, 'worker-queue-dlq', {
queueName: '<%- serviceName %>-dlq',
encryptionMasterKey: kmsKey,
});
const deadLetterQueue = new aws_sqs.Queue(
this,
'worker-queue-dead-letters',
{
queueName: '<%- serviceName %>-dead-letters',
encryptionMasterKey: kmsKey,
},
);

const queue = new aws_sqs.Queue(this, 'worker-queue', {
queueName: '<%- serviceName %>',
Expand Down

0 comments on commit 6600a20

Please sign in to comment.