Skip to content

Handle latest version restore for Lambda with capacity provider#13495

Merged
joe4dev merged 1 commit intomainfrom
lmi/persistence-testing
Dec 12, 2025
Merged

Handle latest version restore for Lambda with capacity provider#13495
joe4dev merged 1 commit intomainfrom
lmi/persistence-testing

Conversation

@joe4dev
Copy link
Member

@joe4dev joe4dev commented Dec 10, 2025

Motivation

Persistence restore is not taking the special case of Lambda Managed Instances into consideration for the qualifier $LATEST, which is not invokable and should therefore not transition into Pending state nor launch a Lambda version in the Lambda service.

Changes

  • Handle special case of $LATEST when restoring persistence for Lambda

Tests

See localstack-pro companion PR

Related

Closes DRG-41

@joe4dev @anisaoshafi
Handle latest version restore for Lambda with capacity provider
2cadb02 
Co-authored-by: Anisa Oshafi <anisaoshafi@gmail.com>
@joe4dev joe4dev added this to the 4.13 milestone Dec 10, 2025
@joe4dev joe4dev added semver: patch Non-breaking changes which can be included in patch releases docs: skip Pull request does not require documentation changes notes: skip Pull request does not have to be mentioned in the release notes labels Dec 10, 2025
@joe4dev joe4dev requested a review from anisaoshafi December 10, 2025 15:39
@github-actions
Copy link

github-actions bot commented Dec 10, 2025

Test Results - Preflight, Unit

23 000 tests  ±0   21 158 ✅ ±0   6m 26s ⏱️ +3s
     1 suites ±0    1 842 💤 ±0 
     1 files   ±0        0 ❌ ±0 

Results for commit 2cadb02. ± Comparison against base commit c92c8e2.

@github-actions
Copy link

github-actions bot commented Dec 10, 2025

Test Results (amd64) - Acceptance

7 tests  ±0   5 ✅ ±0   3m 8s ⏱️ +12s
1 suites ±0   2 💤 ±0 
1 files   ±0   0 ❌ ±0 

Results for commit 2cadb02. ± Comparison against base commit c92c8e2.

anisaoshafi
anisaoshafi approved these changes Dec 10, 2025
View reviewed changes
Copy link
Contributor

@anisaoshafi anisaoshafi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for meticulously handling this special case 🚀

@github-actions
Copy link

github-actions bot commented Dec 10, 2025

Test Results (amd64) - Integration, Bootstrap

    5 files      5 suites   1h 59m 51s ⏱️
3 757 tests 3 474 ✅ 283 💤 0 ❌
3 763 runs  3 474 ✅ 289 💤 0 ❌

Results for commit 2cadb02.

@github-actions
Copy link

github-actions bot commented Dec 10, 2025

LocalStack Community integration with Pro

    2 files  ±    0      2 suites  ±0   1h 29m 15s ⏱️ - 28m 13s
3 733 tests  - 1 400  3 446 ✅  - 1 292  287 💤  - 108  0 ❌ ±0 
3 735 runs   - 1 400  3 446 ✅  - 1 292  289 💤  - 108  0 ❌ ±0 

Results for commit 2cadb02. ± Comparison against base commit c92c8e2.

This pull request removes 1400 tests. 
tests.aws.scenario.bookstore.test_bookstore.TestBookstoreApplication ‑ test_lambda_dynamodb
tests.aws.scenario.bookstore.test_bookstore.TestBookstoreApplication ‑ test_opensearch_crud
tests.aws.scenario.bookstore.test_bookstore.TestBookstoreApplication ‑ test_search_books
tests.aws.scenario.bookstore.test_bookstore.TestBookstoreApplication ‑ test_setup
tests.aws.scenario.kinesis_firehose.test_kinesis_firehose.TestKinesisFirehoseScenario ‑ test_kinesis_firehose_s3
tests.aws.scenario.lambda_destination.test_lambda_destination_scenario.TestLambdaDestinationScenario ‑ test_destination_sns
tests.aws.scenario.lambda_destination.test_lambda_destination_scenario.TestLambdaDestinationScenario ‑ test_infra
tests.aws.scenario.loan_broker.test_loan_broker.TestLoanBrokerScenario ‑ test_prefill_dynamodb_table
tests.aws.scenario.loan_broker.test_loan_broker.TestLoanBrokerScenario ‑ test_stepfunctions_input_recipient_list[step_function_input0-SUCCEEDED]
tests.aws.scenario.loan_broker.test_loan_broker.TestLoanBrokerScenario ‑ test_stepfunctions_input_recipient_list[step_function_input1-SUCCEEDED]
…

@joe4dev joe4dev marked this pull request as ready for review December 11, 2025 09:42
@joe4dev joe4dev merged commit d6882f9 into main Dec 12, 2025
55 checks passed
@joe4dev joe4dev deleted the lmi/persistence-testing branch December 12, 2025 09:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@anisaoshafi anisaoshafi anisaoshafi approved these changes

@dominikschubert dominikschubert Awaiting requested review from dominikschubert dominikschubert is a code owner

@dfangl dfangl Awaiting requested review from dfangl dfangl is a code owner

@gregfurman gregfurman Awaiting requested review from gregfurman gregfurman is a code owner

Assignees

No one assigned

Labels

docs: skip Pull request does not require documentation changes notes: skip Pull request does not have to be mentioned in the release notes semver: patch Non-breaking changes which can be included in patch releases

Projects

None yet

Milestone

4.13

Development

Successfully merging this pull request may close these issues.

2 participants

@joe4dev @anisaoshafi