Skip to content

Filter by active stack when executing a cloudformation changeset #11384

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

Closed
wants to merge 4 commits into from

Conversation

migraf
Copy link

@migraf migraf commented Aug 20, 2024

Motivation

This PR fixes the behaviour of the cloudformation api, when deleting and recreating a stack with the same name.
Now stacks which have the status DELETE_COMPLETE are filtered out when a changeset execution is requested for a stack, which closes #11333

Changes

The find_changeset function in stores.py of the cloudformation package now accepts an additional optional parameter active_only which filters the stacks by their status when set. This parameter is set to true only in execute_changeset handler of the cloudformation provider.

Testing

TODO

TODO

What's left to do:

  • Proper testing for this functionality

Copy link
Collaborator

@localstack-bot localstack-bot left a comment

Choose a reason for hiding this comment

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

Welcome to LocalStack! Thanks for raising your first Pull Request and landing in your contributions. Our team will reach out with any reviews or feedbacks that we have shortly. We recommend joining our Slack Community and share your PR on the #community channel to share your contributions with us. Please make sure you are following our contributing guidelines and our Code of Conduct.

@migraf migraf changed the title Filter by active stack when executing a changeset Patch: Filter by active stack when executing a cloudformation changeset Aug 20, 2024
@localstack-bot
Copy link
Collaborator

localstack-bot commented Aug 20, 2024

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@migraf migraf changed the title Patch: Filter by active stack when executing a cloudformation changeset Filter by active stack when executing a cloudformation changeset Aug 20, 2024
@migraf
Copy link
Author

migraf commented Aug 20, 2024

I have read the CLA Document and I hereby sign the CLA

localstack-bot added a commit that referenced this pull request Aug 20, 2024
@thrau thrau added the semver: minor Non-breaking changes which can be included in minor releases, but not in patch releases label Aug 20, 2024
@simonrw
Copy link
Contributor

simonrw commented Oct 3, 2024

Thanks for this contribution. I am going to close this PR in favour of #11632 as this includes a validated test.

@simonrw simonrw closed this Oct 3, 2024
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.

bug: repeated CDK deploy fails with InvalidChangeSetStatus
4 participants