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
Update
command can update multiple nodegroups
#3914
Conversation
9368748
to
fcbf47e
Compare
5383016
to
5d68af6
Compare
5d68af6
to
06ce9f0
Compare
Does it make sense to have the integration test update 2 ngs? |
@Callisto13 I don't know 🤔 I was looking at other commands that take multiple nodegroups such as Edit: tested it locally though with multiple nodegroups that have |
yeh I think you are right, no need 👍 |
if err != nil { | ||
if managed.IsNotFound(err) { | ||
return fmt.Errorf("could not find managed nodegroup with name %q", ng.Name) | ||
for _, ng := range m.cfg.ManagedNodeGroups { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, just one small nit which is more of a personal preference:
can we move the contents of this for
into another private func? simply because the process is quite long so it is easy to lose sight that this is all in a loop 🤷♀️
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good idea! I split the contents into updateNodegroup
and the strictly updateConfig
bit into updateUpdateConfig
(I know, cringe)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😂 excellent
* UpdateConfig is never nil - it will have maxUnavailable=1 by default. See: eksctl-io#3855 (comment)
67dd7da
to
ef534c7
Compare
Description
Closes #3857
A config file containing multiple managed nodegroups will iterate over them and update all of them.
Checklist
README.md
, or theuserdocs
directory)area/nodegroup
) and kind (e.g.kind/improvement
)BONUS POINTS checklist: complete for good vibes and maybe prizes?! 🤯