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

GKE node pool upgrade settings are not configurable #3343

Closed
chajath opened this issue Dec 4, 2024 · 3 comments
Closed

GKE node pool upgrade settings are not configurable #3343

chajath opened this issue Dec 4, 2024 · 3 comments
Assignees
Labels
bug Something isn't working

Comments

@chajath
Copy link

chajath commented Dec 4, 2024

Describe the bug

In toolkit, GKE node pool upgrade settings are hardcoded:

upgrade_settings {
strategy = "SURGE"
max_surge = 0
max_unavailable = 1
}

This stops us from efficiently upgrading nodes in-place. Without change, each node upgrade can take up 9+ minutes, which makes maintaining big node pools unrealistic.

Steps to reproduce

Steps to reproduce the behavior:

  1. Trigger in-place node pool upgrade

Expected behavior

You have an option to make sure multiple nodes are made unavailable time to minimize the downtime.

Actual behavior

You don't have any option to upgrade more than one node at a time.

@chajath chajath added the bug Something isn't working label Dec 4, 2024
@ankitkinra
Copy link
Contributor

Quick question, you are okay with SURGE strategy , but want to configure more than one node unavailable at a time ?

@chajath
Copy link
Author

chajath commented Dec 4, 2024

yes. In our particular case, being able to set higher value for max_unavailable while keeping everything else as is, would be enough.

@chajath
Copy link
Author

chajath commented Dec 17, 2024

Could you cherrypick #3359 into experimental branch so we can use with GKE provision?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants