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

imds asg target lifecycle hook #837

Closed
pete911 opened this issue Jun 8, 2023 · 5 comments
Closed

imds asg target lifecycle hook #837

pete911 opened this issue Jun 8, 2023 · 5 comments
Assignees
Labels
stalebot-ignore To NOT let the stalebot update or close the Issue / PR Type: Enhancement New feature or request

Comments

@pete911
Copy link
Contributor

pete911 commented Jun 8, 2023

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 #825

@cjerad cjerad added the Type: Enhancement New feature or request label Jun 20, 2023
@pete911
Copy link
Contributor Author

pete911 commented Jul 17, 2023

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

@jillmon jillmon added the stalebot-ignore To NOT let the stalebot update or close the Issue / PR label Jul 18, 2023
@cjerad cjerad self-assigned this Jul 25, 2023
@pete911
Copy link
Contributor Author

pete911 commented Oct 27, 2023

Hi @jillmon, @cjerad any news on this?

@LikithaVemulapalli
Copy link
Contributor

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 :)

@pete911
Copy link
Contributor Author

pete911 commented Aug 6, 2024

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.

@LikithaVemulapalli
Copy link
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stalebot-ignore To NOT let the stalebot update or close the Issue / PR Type: Enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

5 participants