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

JavaScript (v3): Kinesis - Update CDK/CFN deletion policy to delete stream on stack deletion. #6877

Merged
merged 1 commit into from
Sep 18, 2024

Conversation

cpyle0819
Copy link
Contributor

Kinesis streams were being retained during integ test runs and breaking the suite. This updates the deletion policy to prevent that.


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@cpyle0819 cpyle0819 added the Task A general update to the code base for language clarification, missing actions, tests, etc. label Sep 18, 2024
@cpyle0819 cpyle0819 requested a review from a team September 18, 2024 13:43
@cpyle0819 cpyle0819 self-assigned this Sep 18, 2024
@cpyle0819 cpyle0819 requested review from rlhagerm and removed request for a team September 18, 2024 13:43
@github-actions github-actions bot added the Javascript-v3 This issue relates to the AWS SDK for Javascript V3 label Sep 18, 2024
@cpyle0819 cpyle0819 added the On Call Review needed This work needs an on-call review label Sep 18, 2024
@scmacdon scmacdon added On Call Review complete On call review complete and removed On Call Review needed This work needs an on-call review labels Sep 18, 2024
@scmacdon scmacdon merged commit ed25e93 into awsdocs:main Sep 18, 2024
20 of 23 checks passed
@cpyle0819 cpyle0819 deleted the corepyle/integ-test branch September 30, 2024 18:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Javascript-v3 This issue relates to the AWS SDK for Javascript V3 On Call Review complete On call review complete Task A general update to the code base for language clarification, missing actions, tests, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants