Skip to content
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

CFn: Improve stack set support #10765

Draft
wants to merge 8 commits into
base: master
Choose a base branch
from
Draft

CFn: Improve stack set support #10765

wants to merge 8 commits into from

Conversation

simonrw
Copy link
Contributor

@simonrw simonrw commented May 3, 2024

Motivation

I updated our copilot-local repo to the latest upstream commit, and it did not work against LocalStack any more. I am not 100% sure the current commit works either.

The initial failure reason was our StackSet support in CloudFormation.

Changes

  • Update get_template_body to allow use of the previous stack (UsePreviousStack)
  • Expand on the StackSet entity mostly allowing access to the template body for use by StackInstances
  • Look up stack sets when retrieving template summaries
  • Merge StackInstance parameter overrides with StackSet parameter definitions
  • Introduce convert_in_place_at_jsonpath for deep nested structure in place type conversion
  • Expand on test_create_stack_set_with_stack_instances test to support parameter overrides

@simonrw simonrw added the semver: patch Non-breaking changes which can be included in patch releases label May 3, 2024
@simonrw simonrw self-assigned this May 3, 2024
Copy link

github-actions bot commented May 3, 2024

LocalStack Community integration with Pro

    2 files  ±0      2 suites  ±0   1h 36m 33s ⏱️ +7s
2 932 tests ±0  2 638 ✅ ±0  294 💤 ±0  0 ❌ ±0 
2 934 runs  ±0  2 638 ✅ ±0  296 💤 ±0  0 ❌ ±0 

Results for commit 91d3dea. ± Comparison against base commit 7917b4a.

♻️ This comment has been updated with latest results.

@simonrw simonrw changed the title Improve stack set support CFn: Improve stack set support May 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver: patch Non-breaking changes which can be included in patch releases
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant