Skip to content

Commit fb08355

Browse files
committed
fix e2e bucket deletion
1 parent 45ad7a7 commit fb08355

File tree

2 files changed

+3
-4
lines changed
  • powertools-e2e-tests
    • handlers/largemessage/src/main/java/software/amazon/lambda/powertools/e2e
    • src/test/java/software/amazon/lambda/powertools/testutils

2 files changed

+3
-4
lines changed

powertools-e2e-tests/handlers/largemessage/src/main/java/software/amazon/lambda/powertools/e2e/Function.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ public SQSBatchResponse handleRequest(SQSEvent event, Context context) {
4141
return SQSBatchResponse.builder().build();
4242
}
4343

44-
@LargeMessage(deleteS3Object = false)
44+
@LargeMessage
4545
private void processRawMessage(SQSMessage sqsMessage, Context context) {
4646
String bodyMD5 = md5(sqsMessage.getBody());
4747
if (!sqsMessage.getMd5OfBody().equals(bodyMD5)) {

powertools-e2e-tests/src/test/java/software/amazon/lambda/powertools/testutils/Infrastructure.java

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -360,11 +360,10 @@ private Stack createStackWithLambda() {
360360
if (!StringUtils.isEmpty(largeMessagesBucket)) {
361361
Bucket offloadBucket = Bucket.Builder
362362
.create(stack, "LargeMessagesOffloadBucket")
363-
// .removalPolicy(RemovalPolicy.DESTROY)
364-
// .autoDeleteObjects(true)
363+
.removalPolicy(RemovalPolicy.RETAIN) // autodelete does not work without cdk deploy
365364
.bucketName(largeMessagesBucket)
366365
.build();
367-
// just in case ...
366+
// instead of autodelete, have a lifecycle rule to delete files after a day
368367
LifecycleRule.builder().expiration(Duration.days(1)).enabled(true).build();
369368
offloadBucket.grantReadWrite(function);
370369
}

0 commit comments

Comments
 (0)