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

adding option to add extra nodes to each asg #131

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

crosbymichael1
Copy link
Contributor

@crosbymichael1 crosbymichael1 commented Aug 9, 2022

When cluster-autoscaler is disabled during the upgrade process this causes new pods being scheduled onto the cluster to be stuck in Pending until the script finishes running and cluster autoscaler is turned back on. Adding additional nodes to the cluster prior to the upgrade can help circumvent this issue and then when cluster auto-scaler is turned back on it can remove unneeded nodes. This is especially needed in Production Cluster where upgrades can take hours and we still want to account for being able to schedule new Pods on the cluster.

@crosbymichael1
Copy link
Contributor Author

@crhuber Could I get some eyes on this please. Its something I need as a use case where I work.

@crosbymichael1
Copy link
Contributor Author

@chadlwilson are you able to review this please?

@chadlwilson
Copy link
Contributor

I'm not a maintainer here so can't review things. I'm also not actively using eks-rolling-update any longer so not well set up to give useful input and think about how all the options interact with one another.

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.

2 participants