Skip to content

Conversation

k-a-il
Copy link
Contributor

@k-a-il k-a-il commented Sep 2, 2025

Motivation

We plan to update the default value of CFN_IGNORE_UNSUPPORTED_RESOURCE_TYPES in upcoming LocalStack releases in CloudFormation provider v2. Before making this change, we will log a warning message for users deploying CloudFormation stacks with unsupported CloudFormation resource types.

Changes

  • Add a warning about the upcoming change in the default behavior of the CloudFormation V2 provider for unsupported resources

@k-a-il k-a-il self-assigned this Sep 2, 2025
@k-a-il k-a-il added semver: patch Non-breaking changes which can be included in patch releases docs: skip Pull request does not require documentation changes docs: needed Pull request requires documentation updates and removed docs: skip Pull request does not require documentation changes labels Sep 2, 2025
Copy link

github-actions bot commented Sep 2, 2025

Test Results - Preflight, Unit

22 117 tests  ±0   20 379 ✅ ±0   6m 27s ⏱️ -19s
     1 suites ±0    1 738 💤 ±0 
     1 files   ±0        0 ❌ ±0 

Results for commit 37cfea5. ± Comparison against base commit 025a7bf.

@k-a-il k-a-il marked this pull request as ready for review September 2, 2025 14:02
Copy link

github-actions bot commented Sep 2, 2025

Test Results (amd64) - Acceptance

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

Results for commit 37cfea5. ± Comparison against base commit 025a7bf.

Copy link

github-actions bot commented Sep 2, 2025

LocalStack Community integration with Pro

  2 files  ±0    2 suites  ±0   21m 54s ⏱️ -28s
555 tests ±0  328 ✅ ±0  227 💤 ±0  0 ❌ ±0 
557 runs  ±0  328 ✅ ±0  229 💤 ±0  0 ❌ ±0 

Results for commit 37cfea5. ± Comparison against base commit 025a7bf.

Copy link

github-actions bot commented Sep 2, 2025

Test Results (amd64) - Integration, Bootstrap

  5 files  ±0    5 suites  ±0   33m 19s ⏱️ -37s
579 tests ±0  353 ✅ ±0  226 💤 ±0  0 ❌ ±0 
585 runs  ±0  353 ✅ ±0  232 💤 ±0  0 ❌ ±0 

Results for commit 37cfea5. ± Comparison against base commit 025a7bf.

Copy link
Member

@pinzon pinzon left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@k-a-il k-a-il changed the base branch from cfn/v2/test-with-new-provider to main September 3, 2025 10:27
@k-a-il k-a-il force-pushed the cfn/v2/log_update_cfn_ignore_unsupported_resource_types branch from 37cfea5 to a4e96cb Compare September 3, 2025 11:09
@k-a-il k-a-il merged commit f74d79b into main Sep 3, 2025
13 checks passed
@k-a-il k-a-il deleted the cfn/v2/log_update_cfn_ignore_unsupported_resource_types branch September 3, 2025 11:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs: needed Pull request requires documentation updates 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.

3 participants