Skip to content
This repository was archived by the owner on Mar 23, 2026. It is now read-only.

Lambda: Update CapacityProviderArn for managed instances#13702

Merged
anisaoshafi merged 1 commit intomainfrom
drg-169
Feb 6, 2026
Merged

Lambda: Update CapacityProviderArn for managed instances#13702
anisaoshafi merged 1 commit intomainfrom
drg-169

Conversation

@anisaoshafi
Copy link
Copy Markdown
Contributor

@anisaoshafi anisaoshafi commented Feb 4, 2026 •
edited
Loading

Motivation

Support update of CapacityProviderArn for managed instances for UpdateFunctionConfiguration API.
There is some existing validation upon removing and adding a capacity provider from/to a lambda function.

Changes

Tests

Related

Closes DRG-169

@anisaoshafi anisaoshafi added aws:lambda AWS Lambda 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 Feb 4, 2026
@anisaoshafi anisaoshafi marked this pull request as ready for review February 4, 2026 16:15
@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 4, 2026 •
edited
Loading

Test Results - Preflight, Unit

23 114 tests  ±0   21 255 ✅ ±0   6m 7s ⏱️ -11s
     1 suites ±0    1 859 💤 ±0 
     1 files   ±0        0 ❌ ±0 

Results for commit d0e06ce. ± Comparison against base commit cf9a6fc.

♻️ This comment has been updated with latest results.

@anisaoshafi anisaoshafi removed the request for review from dominikschubert February 4, 2026 16:16
@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 4, 2026 •
edited
Loading

Test Results (amd64) - Acceptance

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

Results for commit d0e06ce. ± Comparison against base commit cf9a6fc.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 4, 2026 •
edited
Loading

Test Results (amd64) - Integration, Bootstrap

    5 files      5 suites   2h 2m 39s ⏱️
3 780 tests 3 542 ✅ 238 💤 0 ❌
3 786 runs  3 542 ✅ 244 💤 0 ❌

Results for commit d0e06ce.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 4, 2026 •
edited
Loading

LocalStack Community integration with Pro

    2 files  ±    0      2 suites  ±0   1h 30m 40s ⏱️ - 28m 32s
3 756 tests  - 1 436  3 514 ✅  - 1 327  242 💤  - 109  0 ❌ ±0 
3 758 runs   - 1 436  3 514 ✅  - 1 327  244 💤  - 109  0 ❌ ±0 

Results for commit d0e06ce. ± Comparison against base commit cf9a6fc.

This pull request removes 1436 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]
…

♻️ This comment has been updated with latest results.

joe4dev
joe4dev approved these changes Feb 5, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@joe4dev joe4dev left a comment

Choose a reason for hiding this comment

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

lgtm 👍

@bentsku bentsku mentioned this pull request Feb 5, 2026
Merged
@anisaoshafi anisaoshafi merged commit 965a755 into main Feb 6, 2026
41 checks passed
@anisaoshafi anisaoshafi deleted the drg-169 branch February 6, 2026 08:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@dfangl dfangl Awaiting requested review from dfangl

@gregfurman gregfurman Awaiting requested review from gregfurman

1 more reviewer

@joe4dev joe4dev joe4dev approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

aws:lambda AWS Lambda 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

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@anisaoshafi @joe4dev