Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Lambda: adjust qualifier validation for integrations#13517

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
anisaoshafi wants to merge1 commit intomain
base:main
Choose a base branch
Loading
fromlmi-integrations

Conversation

@anisaoshafi
Copy link
Contributor

Motivation

API operationCreateFunctionUrlConfig needs some adjustment for the validation to pass in the context of Lambda Managed Instances.

Changes

Adjust qualifier validation for integrations using "$LATEST.PUBLISHED" qualifier.

Tests

Related

@anisaoshafianisaoshafi added aws:lambdaAWS Lambda semver: patchNon-breaking changes which can be included in patch releases docs: skipPull request does not require documentation changes notes: skipPull request does not have to be mentioned in the release notes labelsDec 12, 2025
@github-actions
Copy link

github-actionsbot commentedDec 12, 2025•
edited
Loading

Test Results - Preflight, Unit

23 001 tests  ±0   21 158 ✅ ±0   6m 24s ⏱️ +3s
     1 suites ±0    1 843 💤 ±0 
     1 files   ±0        0 ❌ ±0 

Results for commit0ec07f9. ± Comparison against base commitee8518d.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actionsbot commentedDec 12, 2025•
edited
Loading

Test Results (amd64) - Acceptance

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

Results for commit0ec07f9. ± Comparison against base commitee8518d.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actionsbot commentedDec 12, 2025•
edited
Loading

Test Results (amd64) - Integration, Bootstrap

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

Results for commit0ec07f9.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actionsbot commentedDec 12, 2025•
edited
Loading

LocalStack Community integration with Pro

    2 files  ±    0      2 suites  ±0   1h 30m 38s ⏱️ - 26m 4s
3 727 tests  - 1 414  3 440 ✅  - 1 305  287 💤  - 109  0 ❌ ±0 
3 729 runs   - 1 414  3 440 ✅  - 1 305  289 💤  - 109  0 ❌ ±0 

Results for commit0ec07f9. ± Comparison against base commitee8518d.

This pull requestremoves 1414 tests.
tests.aws.scenario.bookstore.test_bookstore.TestBookstoreApplication ‑ test_lambda_dynamodbtests.aws.scenario.bookstore.test_bookstore.TestBookstoreApplication ‑ test_opensearch_crudtests.aws.scenario.bookstore.test_bookstore.TestBookstoreApplication ‑ test_search_bookstests.aws.scenario.bookstore.test_bookstore.TestBookstoreApplication ‑ test_setuptests.aws.scenario.kinesis_firehose.test_kinesis_firehose.TestKinesisFirehoseScenario ‑ test_kinesis_firehose_s3tests.aws.scenario.lambda_destination.test_lambda_destination_scenario.TestLambdaDestinationScenario ‑ test_destination_snstests.aws.scenario.lambda_destination.test_lambda_destination_scenario.TestLambdaDestinationScenario ‑ test_infratests.aws.scenario.loan_broker.test_loan_broker.TestLoanBrokerScenario ‑ test_prefill_dynamodb_tabletests.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.

Copy link
Member

@joe4devjoe4dev left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@joe4devjoe4devjoe4dev approved these changes

@dominikschubertdominikschubertAwaiting requested review from dominikschubertdominikschubert is a code owner

@dfangldfanglAwaiting requested review from dfangldfangl is a code owner

@gregfurmangregfurmanAwaiting requested review from gregfurmangregfurman is a code owner

Assignees

No one assigned

Labels

aws:lambdaAWS Lambdadocs: skipPull request does not require documentation changesnotes: skipPull request does not have to be mentioned in the release notessemver: patchNon-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.

3 participants

@anisaoshafi@joe4dev
[8]ページ先頭
©2009-2025 Movatter.jp