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

Cluster Autoscaler patch releases #4251

Closed
towca opened this issue Aug 10, 2021 · 20 comments
Closed

Cluster Autoscaler patch releases #4251

towca opened this issue Aug 10, 2021 · 20 comments

Comments

@towca
Copy link
Collaborator

towca commented Aug 10, 2021

Hi! I want to cut new Cluster Autoscaler patch releases for 1.19, 1.20 and 1.21 (if needed) early next week. If there are any bugfix cherry-picks you want included, please prepare the PRs and let me know in the comments.

Tagging cloud provider owners for visibility:
@ringtail @Jeffwan @jaypipes @gjtempleton @feiskyer @nilo19 @marwanad @hello2mao @davidjumani @enxebre @elmiko @hardikdr @detiber @andrewsykim @MorrisLaw @kevin-wangzefeng @RainbowMango @avorima @ellistarn @tghartland @d-mo @detiber @deitch @displague @gianarb

@gjtempleton
Copy link
Member

Thanks, there are a few on the AWS side that I'll get raised this week.

@feiskyer
Copy link
Member

@marwanad could you cherry-pick the fix #4130 to these branches?

@tghartland
Copy link
Contributor

Hello, is it possible to make a release for 1.22.1 this soon after the .0 release?
I enabled scale to zero for magnum in 1.22, but I missed a requirement on using a newer magnum API microversion.
It fails only for scaling to 0, any other scaling still works so it isn't incredibly urgent.

PR for the fix on master branch: #4261

@gjtempleton
Copy link
Member

gjtempleton commented Aug 16, 2021

One thing this has thrown up is that we should also cherry-pick the Github actions switch back to the 1.19 and 1.20 release branches.

As you can see from the above cherry-pick PRs from me, if the actions aren't in the branches being merged to, they won't be triggered.

I've now raised #4269 and #4271 to address this.

@detiber
Copy link
Member

detiber commented Aug 20, 2021

If the releases haven't been cut yet, then it would be nice to get cherry-picks for #4274 in as well.

/cc @ipochi

@michael-careplanner
Copy link

If possible it would also be great to get #4089 in too.

I think this would only need bringing in to the 1.21 release as it fixes an issue introduced in 9831623, after the 1.20 release.

@towca
Copy link
Collaborator Author

towca commented Aug 24, 2021

@gjtempleton Thanks for the Github actions PRs, I responded on one of them. Have you merged everything you wanted to?
@feiskyer Should I wait for the #4130 cherry-picks?
@tghartland Sure, could you prepare the PR?
@detiber Could you prepare the PRs?
@michael-careplanner I'll look into #4089, thanks for bringing it up!

@tghartland
Copy link
Contributor

My cherry pick PR #4283.

Thanks.

@ipochi
Copy link
Contributor

ipochi commented Aug 27, 2021

My Cherry pick PRs for #4274

#4291 - 1.19
#4290 - 1.20
#4288 - 1.21
#4289 - 1.22

/cc @detiber

@gjtempleton
Copy link
Member

@towca Sorry, I've not had a chance to spend any more time looking into the Github action PRs and why they're failing this wekk, however all the AWS related ones I wanted to charry-pick back have been merged, so happy to go ahead with cutting releases from that side.

@ipochi
Copy link
Contributor

ipochi commented Sep 2, 2021

My Cherry pick PRs for #4274

#4291 - 1.19
#4290 - 1.20
#4288 - 1.21
#4289 - 1.22

/cc @detiber

@towca @detiber

can you please check if I've done everything correctly and nothing is pending from my end ?

@tulsluper
Copy link

Is it possible to get cherry-picks for #3924 ?

@matthias50
Copy link

@tulsluper, regarding #3924 I created #4319 for 1.20. I attempted to create one for 1.19 but it had merge conflicts.

@rbaybutt
Copy link

@towca If you don't have time to get to it I can create the cherry pick PR (just needed in v1.21) for #4089
I'm seeing issues with this in CA v1.21 so I'm happy to to create the PR but don't want want to duplicate effort if you're already on it.

@jcrowthe
Copy link

@feiskyer Can you drive a cherry-pick for #4130 back to at least 1.20? We have strong evidence that this is causing outages in production.

@feiskyer
Copy link
Member

@jcrowthe cherry-pick for #4130 (#4356 and #4355) have been merged now.

@towca
Copy link
Collaborator Author

towca commented Sep 30, 2021

@gjtempleton FYI I managed to backport Github Actions to 1.19 and 1.20 (#4366 , #4367). I think the issue was with the Go version - 1.20 should use go1.15, and 1.19 should use go1.13.
@rbaybutt #4089 has been cherry-picked to 1.21 in #4311
@tulsluper @matthias50 #3924 will only apply cleanly on 1.20. 1.19 doesn't have previous changes to the same logic. Thanks for the cherry-pick, merged!
@feiskyer @marwanad The cherry-picks for #4130 broke the build on 1.19 and 1.20 (CI was not running on these branches). Details: #4368. Please fix ASAP so we can proceed with cutting the release. Also - was not backporting #4130 to 1.21 intentional?

@gjtempleton
Copy link
Member

@towca Thanks for that! Seems I'd not stumbled on the correct go versions somehow.

@marwanad
Copy link
Member

marwanad commented Oct 1, 2021

@towca I missed the 1.21 cherry-pick. I'll do that now (edit: #4372).
The unit tests have also been fixed.

@towca towca closed this as completed Oct 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests