Skip to content

CloudFormation v2 Engine: Batch of Parity Improvements #12589

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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
May 8, 2025

Conversation

MEPalma
Copy link
Contributor

@MEPalma MEPalma commented May 7, 2025

Motivation

The introduction of the CloudFormation v2 engine laid the foundation for a redesigned engine capable of accurately determining update requirements between CloudFormation deployments, while also enabling parallel execution during updates. However, the current implementation still lacks parity with AWS CloudFormation in several key areas. This set of changes addresses a number of those gaps by introducing parity improvements and establishing a structured approach for incrementally porting the existing test suite to the v2 engine.

Changes

  • Added support for sampling runtime resource properties.
  • Resolved issues where conditions could incorrectly be sampled via Ref intrinsic functions;
  • Improved/simplified the logic associated with computing conditions.
  • Fixed issues preventing bindings to empty data structures from being evaluated as Created or Removed during template updates.
  • Added support for the Fn::Join intrinsic function.
  • Addressed other minor parity issues
  • Initiated porting of v1 tests to the v2 package, beginning with resources.apigateway. Added annotations explaining current test failures or limitations with the v2 engine.

@MEPalma MEPalma added this to the Playground milestone May 7, 2025
@MEPalma MEPalma self-assigned this May 7, 2025
@MEPalma MEPalma added the semver: minor Non-breaking changes which can be included in minor releases, but not in patch releases label May 7, 2025
Copy link

github-actions bot commented May 7, 2025

LocalStack Community integration with Pro

  2 files    2 suites   21m 32s ⏱️
489 tests 314 ✅ 175 💤 0 ❌
491 runs  314 ✅ 177 💤 0 ❌

Results for commit 6d1d6c2.

♻️ This comment has been updated with latest results.

@MEPalma MEPalma marked this pull request as ready for review May 7, 2025 13:34
Copy link
Contributor

@simonrw simonrw left a comment

Choose a reason for hiding this comment

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

This PR was pair reviewed with myself, @MEPalma and @dominikschubert

@MEPalma MEPalma merged commit c38fd15 into master May 8, 2025
32 checks passed
@MEPalma MEPalma deleted the MEP-CFN-parity_b0 branch May 8, 2025 13:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver: minor Non-breaking changes which can be included in minor releases, but not in patch releases
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants