aws-node-termination-handler icon indicating copy to clipboard operation
aws-node-termination-handler copied to clipboard

Published 20 hours ago • verified publisher icon aws

imds asg target lifecycle hook

Open pete911 opened this issue 1 year ago • 5 comments

Describe the feature Target lifecycle state is available through instance metadata since March 2022 - https://docs.aws.amazon.com/autoscaling/ec2/userguide/retrieving-target-lifecycle-state-through-imds.html It would be nice to enable this on aws-node-termination-handler when deployed in imds mode.

Is the feature request related to a problem? Even though ASG Termination lifecycle hook is available in Queue processor mode, for some cases this adds additional infrastructure complexity (SQS, private link to SQS, permissions etc.), it would be nice to add this in IMDS mode as well, since it is available.

Initial pull request (tested in EKS cluster as well) is here https://github.com/aws/aws-node-termination-handler/pull/825

pete911 avatar Jun 08 '23 09:06 pete911

@cjerad is this something you would like to merge? I removed staled label from the PR twice, but it has now been closed.

pete911 avatar Jul 17 '23 12:07 pete911

Hi @jillmon, @cjerad any news on this?

pete911 avatar Oct 27 '23 09:10 pete911

Hello @pete911, apologies for late response. I saw your PR and we feel including the Lifecycle hooks in IMDS mode will be very useful feature. If possible could you work on the e2e tests as this is a major feature request we would also like some e2e tests to ensure the work flow is not broken and is working as expected. The setup on how we will include LifeCycle Hooks in IMDS in README section will be very helpful for others who want to use this feature. Please let us know if you are still interested working on this feature. Thanks :)

LikithaVemulapalli avatar Aug 05 '24 21:08 LikithaVemulapalli

Hi @LikithaVemulapalli, PR has been opened for more than a year (that was the time we wanted to use node termination handler), I asked multiple times @jillmon and @cjerad that were assigned to this PR, but I got no response from them. I currently do not work with EKS/kubernetes and do not use node termination handler. PR should contain everything needed, it has been tested at that time in EKS cluster, but unfortunately I won't have time to add e2e test.

pete911 avatar Aug 06 '24 05:08 pete911

Hello @pete911, thanks for the response, we will take forward from here. Thanks for your contribution, appreciate it :)

LikithaVemulapalli avatar Aug 06 '24 16:08 LikithaVemulapalli