Skip to content

CloudFormation V2 Engine: Support for Fn::Sub #12650

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

MEPalma
Copy link
Contributor

@MEPalma MEPalma commented May 20, 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 support for Fn::Sub intrinsic function.

Future efforts should add validation checks and cyclic reference detections.

Changes

  • Add support for Fn::Sub (modeling, describing, and executing)
  • Add base ChangeSet lifecycle test

@MEPalma MEPalma added this to the 4.5 milestone May 20, 2025
@MEPalma MEPalma self-assigned this May 20, 2025
@MEPalma MEPalma requested a review from dominikschubert as a code owner May 20, 2025 16:11
@MEPalma MEPalma added the semver: minor Non-breaking changes which can be included in minor releases, but not in patch releases label May 20, 2025
@MEPalma MEPalma requested review from pinzon and simonrw as code owners May 20, 2025 16:11
Copy link

github-actions bot commented May 20, 2025

Test Results - Preflight, Unit

21 601 tests  ±0   19 955 ✅ ±0   6m 31s ⏱️ +5s
     1 suites ±0    1 646 💤 ±0 
     1 files   ±0        0 ❌ ±0 

Results for commit 648fcf1. ± Comparison against base commit 20c40d1.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented May 20, 2025

LocalStack Community integration with Pro

  2 files    2 suites   20m 42s ⏱️
504 tests 315 ✅ 189 💤 0 ❌
506 runs  315 ✅ 191 💤 0 ❌

Results for commit 648fcf1.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented May 20, 2025

Test Results (amd64) - Acceptance

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

Results for commit 648fcf1. ± Comparison against base commit 20c40d1.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented May 20, 2025

Test Results - Alternative Providers

606 tests   428 ✅  15m 17s ⏱️
  4 suites  178 💤
  4 files      0 ❌

Results for commit 648fcf1.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented May 20, 2025

Test Results (amd64) - Integration, Bootstrap

    5 files      5 suites   2h 20m 36s ⏱️
4 827 tests 4 276 ✅ 551 💤 0 ❌
4 833 runs  4 276 ✅ 557 💤 0 ❌

Results for commit 648fcf1.

♻️ This comment has been updated with latest results.

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.

1 participant