Skip to content

fix: Reverted the removal of the lifecycle hook completion #999

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Conversation

stevehipwell
Copy link
Contributor

Issue #, if available:

Description of changes:
This PR reverts the removal of the lifecycle hook completions for ASG lifecycle events.

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

Copy link
Contributor

@LikithaVemulapalli LikithaVemulapalli left a comment

Choose a reason for hiding this comment

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

Hello @stevehipwell, I appreciate the efforts for these changes. I left a comment, could you please take a look at it today so that I can merge this and do an NTH release soon. Thanks :)

Signed-off-by: Steve Hipwell <steve.hipwell@gmail.com>
@stevehipwell stevehipwell force-pushed the revert-lifecycle-hook-completion-removal branch from cc15557 to d67783c Compare May 10, 2024 06:19
Copy link
Contributor

@LikithaVemulapalli LikithaVemulapalli left a comment

Choose a reason for hiding this comment

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

/lgtm, thanks for the quick fix. We are planning to do the release on Monday. Will update the original issue with release version 👍

@LikithaVemulapalli LikithaVemulapalli merged commit 486831b into aws:main May 10, 2024
13 checks passed
@stevehipwell stevehipwell deleted the revert-lifecycle-hook-completion-removal branch May 10, 2024 19:41
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

NTH queue handler behaviour broken by PR #940
2 participants