Skip to content

Commit fc0c871

Browse files
committed
fix e2e bucket deletion
1 parent e7d8da8 commit fc0c871

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
@@ -284,11 +284,10 @@ private Stack createStackWithLambda() {
284284
if (!StringUtils.isEmpty(largeMessagesBucket)) {
285285
Bucket offloadBucket = Bucket.Builder
286286
.create(stack, "LargeMessagesOffloadBucket")
287-
// .removalPolicy(RemovalPolicy.DESTROY)
288-
// .autoDeleteObjects(true)
287+
.removalPolicy(RemovalPolicy.RETAIN) // autodelete does not work without cdk deploy
289288
.bucketName(largeMessagesBucket)
290289
.build();
291-
// just in case ...
290+
// instead of autodelete, have a lifecycle rule to delete files after a day
292291
LifecycleRule.builder().expiration(Duration.days(1)).enabled(true).build();
293292
offloadBucket.grantReadWrite(function);
294293
}

0 commit comments

Comments
 (0)