-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
Cloud Formation: [POC] Update Graph with Template Traversal & Intrinsic Function Resolution #12378
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
Cloud Formation: [POC] Update Graph with Template Traversal & Intrinsic Function Resolution #12378
Conversation
LocalStack Community integration with Pro 2 files ± 0 2 suites ±0 20m 27s ⏱️ - 1h 31m 32s Results for commit 06308e5. ± Comparison against base commit 257af99. This pull request removes 3703 tests.
♻️ This comment has been updated with latest results. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a nice set of changes! I look forward to pushing this work forwards
Motivation
With the recent changes in #12355, we introduced a new design pattern for representing CloudFormation templates. However, that design did not account for deeper intra-template relationships, preventing accurate detection of indirect resource changes driven by dependencies or intrinsic functions. This PR addresses those limitations by introducing logic to traverse and resolve template elements and intrinsic functions, ensuring that updates and dependencies are correctly captured. It also resolves issues comparing null or empty template versions.
Changes
PROVIDER_OVERRIDE_CLOUDFORMATION=engine-v2
is supplied) with create and describe change setTesting