Skip to content

CloudFormation v2 Engine: Base Support for AWS::NoValue and Migration to Nothing Types #12668

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
Open
wants to merge 4 commits into
base: MEP-CFN-fn_transform
Choose a base branch
from
Open

CloudFormation v2 Engine: Base Support for AWS::NoValue and Migration to Nothing Types #12668

wants to merge 4 commits into from

Conversation

MEPalma
Copy link
Contributor

@MEPalma MEPalma commented May 27, 2025

* Extends #12662

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 offers no support for AWS::NoValue pseudo parameter.

Changes

  • Add base support for AWS::NoValue
  • Migrate type system in v2 engine to use Maybe and Nothing to signal missing values
  • Improve and fix the computation of ChangeType upon change set model construction
  • Unblock AWS::NoValue v1 tests

Future Work

Add further boundary tests for the use of AWS::NoValue during update sequences.

@MEPalma MEPalma added this to the 4.5 milestone May 27, 2025
@MEPalma MEPalma self-assigned this May 27, 2025
@MEPalma MEPalma added the semver: minor Non-breaking changes which can be included in minor releases, but not in patch releases label May 27, 2025
@github-actions GitHub Actions
Copy link

github-actions bot commented May 27, 2025 •
edited
Loading

Test Results - Preflight, Unit

21 601 tests  +22   19 955 ✅ +28   6m 11s ⏱️ +12s
     1 suites ± 0    1 646 💤  -  6 
     1 files   ± 0        0 ❌ ± 0 

Results for commit 52008ee. ± Comparison against base commit 75d12d6.

♻️ This comment has been updated with latest results.

@github-actions GitHub Actions
Copy link

github-actions bot commented May 27, 2025 •
edited
Loading

Test Results (amd64) - Acceptance

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

Results for commit 52008ee. ± Comparison against base commit 75d12d6.

♻️ This comment has been updated with latest results.

@github-actions GitHub Actions
Copy link

github-actions bot commented May 27, 2025 •
edited
Loading

Test Results - Alternative Providers

971 tests  ±0   539 ✅ +8   21m 12s ⏱️ +12s
  4 suites ±0   432 💤  - 8 
  4 files   ±0     0 ❌ ±0 

Results for commit 52008ee. ± Comparison against base commit 75d12d6.

♻️ This comment has been updated with latest results.

@github-actions GitHub Actions
Copy link

github-actions bot commented May 27, 2025 •
edited
Loading

LocalStack Community integration with Pro

  2 files  ±    0    2 suites  ±0   21m 21s ⏱️ - 1h 21m 14s
869 tests  - 3 970  319 ✅  - 3 761  550 💤  - 209  0 ❌ ±0 
871 runs   - 3 970  319 ✅  - 3 761  552 💤  - 209  0 ❌ ±0 

Results for commit 52008ee. ± Comparison against base commit 75d12d6.

♻️ This comment has been updated with latest results.

@github-actions GitHub Actions
Copy link

github-actions bot commented May 27, 2025 •
edited
Loading

Test Results (amd64) - Integration, Bootstrap

    5 files  ±0      5 suites  ±0   2h 27m 9s ⏱️ + 3m 52s
5 192 tests  - 2  4 280 ✅  - 2  912 💤 ±0  0 ❌ ±0 
5 198 runs   - 2  4 280 ✅  - 2  918 💤 ±0  0 ❌ ±0 

Results for commit 52008ee. ± Comparison against base commit 75d12d6.

♻️ This comment has been updated with latest results.

@MEPalma MEPalma changed the base branch from master to transcribe-flaky-2 May 28, 2025 09:29
@MEPalma MEPalma changed the base branch from transcribe-flaky-2 to MEP-CFN-fn_transform May 28, 2025 09:34
@MEPalma MEPalma mentioned this pull request May 28, 2025
Open
simonrw
simonrw approved these changes May 29, 2025
View reviewed changes
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.

5 more passing tests, nice one

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@simonrw simonrw simonrw approved these changes

Assignees

@MEPalma MEPalma

Labels
semver: minor Non-breaking changes which can be included in minor releases, but not in patch releases
Projects
None yet
Milestone
4.5
Development

Successfully merging this pull request may close these issues.
2 participants
@MEPalma @simonrw