Skip to content

Conversation

simonrw
Copy link
Contributor

@simonrw simonrw commented Aug 29, 2025

Motivation

When starting the new provider we want to make it obvious that it has loaded, as well as provide the user with how to proceed if there is an issue with the new provider.

Changes

Log a message on service startup including:

  • how to fall back to the old provider
  • how to submit a bug report with metadata so we can easily find it again

Testing

Check out this branch and start LocalStack with python -m localstack.dev.run. Then invoke awslocal cloudformation list-stacks and you should see the startup message presented.

@simonrw simonrw added semver: patch Non-breaking changes which can be included in patch releases docs: skip Pull request does not require documentation changes labels Aug 29, 2025
@simonrw simonrw added this to the 4.8 milestone Aug 29, 2025
@simonrw simonrw marked this pull request as ready for review August 29, 2025 15:28
Copy link

github-actions bot commented Aug 29, 2025

Test Results - Preflight, Unit

22 117 tests  ±0   20 379 ✅ ±0   6m 48s ⏱️ +24s
     1 suites ±0    1 738 💤 ±0 
     1 files   ±0        0 ❌ ±0 

Results for commit cd0d28c. ± Comparison against base commit 445c32f.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Aug 29, 2025

LocalStack Community integration with Pro

  2 files  ±    0    2 suites  ±0   21m 9s ⏱️ - 1h 21m 30s
555 tests  - 4 080  328 ✅  - 3 864  227 💤  - 216  0 ❌ ±0 
557 runs   - 4 080  328 ✅  - 3 864  229 💤  - 216  0 ❌ ±0 

Results for commit cd0d28c. ± Comparison against base commit 445c32f.

This pull request removes 4080 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.

Copy link

github-actions bot commented Aug 29, 2025

Test Results (amd64) - Acceptance

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

Results for commit cd0d28c. ± Comparison against base commit 445c32f.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Aug 29, 2025

Test Results (amd64) - Integration, Bootstrap

  5 files    5 suites   33m 43s ⏱️
579 tests 353 ✅ 226 💤 0 ❌
585 runs  353 ✅ 232 💤 0 ❌

Results for commit cd0d28c.

♻️ This comment has been updated with latest results.

@simonrw simonrw force-pushed the cfn/v2/test-with-new-provider branch 2 times, most recently from 025a7bf to 8d46c88 Compare September 3, 2025 12:14
Base automatically changed from cfn/v2/test-with-new-provider to main September 4, 2025 10:03
@simonrw simonrw force-pushed the cfn/v2/startup-message branch from 62807ee to cd0d28c Compare September 4, 2025 12:26
@simonrw simonrw merged commit c426dbb into main Sep 4, 2025
42 checks passed
@simonrw simonrw deleted the cfn/v2/startup-message branch September 4, 2025 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs: skip Pull request does not require documentation changes semver: patch Non-breaking changes which can be included in patch releases
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants