Skip to content

stacks: include existing components when deferring nested stacks #36788

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

Conversation

liamcervante
Copy link
Member

Currently, components in nested stacks are just ignored during the plan when the nested stack call is deferred.

This PR updates the stacks runtime so a deferred stack call still triggers deferred updates of any components in the nested stack that already exist.

@liamcervante liamcervante added the no-changelog-needed Add this to your PR if the change does not require a changelog entry label Mar 27, 2025
@liamcervante liamcervante requested a review from a team as a code owner March 27, 2025 11:19
@liamcervante liamcervante force-pushed the liamcervante/stacks/removed-chain-1/remove-embedded-components branch from ffa57f4 to 4881ea2 Compare April 1, 2025 07:12
@liamcervante liamcervante force-pushed the liamcervante/stacks/removed-chain-2/unknown-stack-instances branch from 44c41e1 to c87fbdb Compare April 1, 2025 07:13
@liamcervante liamcervante force-pushed the liamcervante/stacks/removed-chain-1/remove-embedded-components branch from 3780e70 to dd7b3ba Compare April 2, 2025 13:25
@liamcervante liamcervante force-pushed the liamcervante/stacks/removed-chain-2/unknown-stack-instances branch from c87fbdb to 6f07866 Compare April 2, 2025 13:29
dsa0x
dsa0x previously approved these changes Apr 2, 2025
Base automatically changed from liamcervante/stacks/removed-chain-1/remove-embedded-components to main April 3, 2025 08:29
@liamcervante liamcervante dismissed dsa0x’s stale review April 3, 2025 08:29

The base branch was changed.

@liamcervante liamcervante force-pushed the liamcervante/stacks/removed-chain-2/unknown-stack-instances branch from 6f07866 to 6846081 Compare April 3, 2025 08:31
@liamcervante liamcervante merged commit 2b5101f into main Apr 3, 2025
8 checks passed
@liamcervante liamcervante deleted the liamcervante/stacks/removed-chain-2/unknown-stack-instances branch April 3, 2025 08:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-changelog-needed Add this to your PR if the change does not require a changelog entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants