Skip to content

🐛 fix(infra): upgrade LocalStack to 4.14.0 for CFN deletion fix#262

Merged
sodre merged 1 commit into
mainfrom
fix/81-localstack-cfn-stack-deletion-template-validation
Feb 28, 2026
Merged

🐛 fix(infra): upgrade LocalStack to 4.14.0 for CFN deletion fix#262
sodre merged 1 commit into
mainfrom
fix/81-localstack-cfn-stack-deletion-template-validation

Conversation

@sodre

@sodre sodre commented Jan 30, 2026

Copy link
Copy Markdown
Member

Summary

  • Upgrade LocalStack from 4 to 4.14.0 to pick up the fix for CloudFormation v2 engine deletion bug with conditional resources

Context

LocalStack's v2 CloudFormation engine incorrectly tries to resolve !GetAtt references during stack deletion, even for resources that were never created due to their Condition evaluating to false.

Upstream issue: localstack/localstack#13609

Minimal reproduction: Posted at localstack/localstack#13609 (comment)

Status

🚧 WIP - Waiting for LocalStack 4.14.0 to be released with the fix.

Test plan

  • Wait for LocalStack 4.14.0 release
  • Verify fix with minimal reproduction script
  • Run full test suite with new version
  • Remove WIP status and mark ready for review

Fixes #81


🤖 Generated with Claude Code

@codecov

codecov Bot commented Jan 30, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.21%. Comparing base (49a2c53) to head (7819dd1).
⚠️ Report is 2 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #262   +/-   ##
=======================================
  Coverage   92.21%   92.21%           
=======================================
  Files          36       36           
  Lines        7831     7831           
=======================================
  Hits         7221     7221           
  Misses        610      610           
Flag Coverage Δ
doctest 29.03% <0.00%> (ø)
e2e 42.37% <100.00%> (+0.01%) ⬆️
integration 50.59% <100.00%> (ø)
unit 92.00% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@sodre sodre added this to the v1.0.0 milestone Feb 11, 2026
@sodre sodre force-pushed the fix/81-localstack-cfn-stack-deletion-template-validation branch 2 times, most recently from 463a898 to 8c60b9b Compare February 28, 2026 04:24
@sodre sodre changed the title 🚧 fix(infra): upgrade LocalStack to 4.14.0 for CFN deletion fix 🐛 fix(infra): upgrade LocalStack to 4.14.0 for CFN deletion fix Feb 28, 2026
@sodre sodre marked this pull request as ready for review February 28, 2026 04:40
Pin LocalStack image from :4 (floating) to :4.14 (minor-locked) across
CLI, docker-compose.yml, CI workflows, and docs. This resolves issue #81
where LocalStack confused CloudFormation templates during stack deletion.

With the fix in place, the defensive try/except around delete_stack() in
test teardown is no longer needed and has been removed from all 7 test
files. Also fixed the docstring sample output in the `local status`
command to reflect the pinned image version.

Closes #81

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@sodre sodre force-pushed the fix/81-localstack-cfn-stack-deletion-template-validation branch from 8c60b9b to 7819dd1 Compare February 28, 2026 04:57
@sodre sodre merged commit 57c9c03 into main Feb 28, 2026
16 checks passed
@sodre sodre deleted the fix/81-localstack-cfn-stack-deletion-template-validation branch February 28, 2026 05:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🐛 LocalStack: CloudFormation stack deletion fails with template validation error

1 participant