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

Nested stack events #24

Merged
merged 3 commits into from
Apr 12, 2024
Merged

Nested stack events #24

merged 3 commits into from
Apr 12, 2024

Conversation

connec
Copy link
Owner

@connec connec commented Apr 12, 2024

  • 637b47f fix: use debug representation for generic cfn API errors

    It's not clear if something changed upstream or if this never really
    worked, but the display output from CloudFormation errors would often
    just be "service error".

  • df324bd fix: allow ResourceTargetDefinition without name for attribute Properties

    This situation arose when describing the changeset of a nested stack.
    It's not clear if this was triggered by an upstream change or has always
    been an ommission.

  • 8311a89 feat: report nested stack events

    Events from nested stacks are now reported, with a "stack alias" built
    built from the logical IDs of the nested stacks combined with / (e.g.
    child/grantchild/Resource).

connec added 2 commits April 12, 2024 15:32
It's not clear if something changed upstream or if this never really
worked, but the display output from CloudFormation errors would often
just be "service error".
…perties`

This situation arose when describing the changeset of a nested stack.
It's not clear if this was triggered by an upstream change or has always
been an ommission.
Events from nested stacks are now reported, with a "stack alias" built
built from the logical IDs of the nested stacks combined with `/` (e.g.
`child/grantchild/Resource`).
@connec connec merged commit 8f0f3a4 into master Apr 12, 2024
5 checks passed
@connec connec deleted the nested-stacks branch April 12, 2024 14:53
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.

1 participant