Skip to content

CFNV2: fix delete change set #12977

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.

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom
Merged
simonrw merged 1 commit into from
Aug 8, 2025
Merged

CFNV2: fix delete change set #12977

simonrw merged 1 commit into from
Aug 8, 2025

Conversation

@simonrw
Copy link
Contributor

@simonrw simonrw commented Aug 8, 2025

Motivation

We still have one outstanding parity test for delete_change_set. This PR fixes that.

Changes

We don't need to have special handling for finding the change set in the provider since find_change_set_v2 handles this case for us.

@simonrw simonrw added this to the 4.8 milestone Aug 8, 2025
@simonrw simonrw added semver: patch Non-breaking changes which can be included in patch releases review: merge when ready Signals to the reviewer that a PR can be merged if accepted labels Aug 8, 2025
@github-actions
Copy link

github-actions bot commented Aug 8, 2025 •
edited
Loading

Test Results - Preflight, Unit

22 063 tests  ±0   20 329 ✅ ±0   6m 22s ⏱️ -5s
     1 suites ±0    1 734 💤 ±0 
     1 files   ±0        0 ❌ ±0 

Results for commit 34a28a2. ± Comparison against base commit 4464cd2.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Aug 8, 2025 •
edited
Loading

LocalStack Community integration with Pro

  2 files  ±    0    2 suites  ±0   22m 49s ⏱️ - 1h 23m 5s
551 tests  - 4 072  329 ✅  - 3 857  222 💤  - 215  0 ❌ ±0 
553 runs   - 4 072  329 ✅  - 3 857  224 💤  - 215  0 ❌ ±0 

Results for commit 34a28a2. ± Comparison against base commit 4464cd2.

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

@github-actions
Copy link

github-actions bot commented Aug 8, 2025 •
edited
Loading

Test Results (amd64) - Acceptance

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

Results for commit 34a28a2. ± Comparison against base commit 4464cd2.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Aug 8, 2025 •
edited
Loading

Test Results (amd64) - Integration, Bootstrap

  5 files  ±    0    5 suites  ±0   34m 0s ⏱️ - 1h 46m 33s
575 tests  - 4 407  354 ✅  - 4 039  221 💤  - 368  0 ❌ ±0 
581 runs   - 4 407  354 ✅  - 4 039  227 💤  - 368  0 ❌ ±0 

Results for commit 34a28a2. ± Comparison against base commit 4464cd2.

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

@simonrw simonrw force-pushed the cfn/v2/fix-delete-change-sets branch from 12cf636 to 34a28a2 Compare August 8, 2025 11:44
@simonrw simonrw marked this pull request as ready for review August 8, 2025 12:37
@simonrw simonrw merged commit c1d19e2 into main Aug 8, 2025
39 of 40 checks passed
@simonrw simonrw deleted the cfn/v2/fix-delete-change-sets branch August 8, 2025 18:44
@simonrw simonrw mentioned this pull request Sep 5, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pinzon pinzon pinzon approved these changes

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

Assignees

No one assigned

Labels

review: merge when ready Signals to the reviewer that a PR can be merged if accepted semver: patch Non-breaking changes which can be included in patch releases

Projects

None yet

Milestone

4.8

Development

Successfully merging this pull request may close these issues.

3 participants

@simonrw @pinzon