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 release - request for cherry-picks #2431

Closed
losipiuk opened this issue Oct 8, 2019 · 11 comments
Closed

Cluster Autoscaler patch release - request for cherry-picks #2431

losipiuk opened this issue Oct 8, 2019 · 11 comments

Comments

@losipiuk
Copy link
Contributor

losipiuk commented Oct 8, 2019

We plan to prepare a set of Cluster Autoscaler patch releases beginning next week. Please issue cherry-picks if you want something in. Just a friendly reminder that as a rule of thumb only bugfixes are cherry-pickable.

cc: @feiskyer, @Jeffwan, @ringtail, @hello2mao, @tghartland

@feiskyer
Copy link
Member

feiskyer commented Oct 8, 2019

@losipiuk thanks for reminding. No cherry-picks from me.

@Jeffwan
Copy link
Member

Jeffwan commented Oct 8, 2019

I will finish cherry-pick by this Thursday.
Candidates:
#2428 to all branches with IAM pods support
#2401 to v1.12+
#2385 to versions with Launch Template support

@ringtail
Copy link
Contributor

ringtail commented Oct 9, 2019

@losipiuk thanks for reminding. No cherry-picks from Alibaba Cloud.

@alfredkrohmer
Copy link
Contributor

Would it be possible to include the AWS SDK updates that enable IAM roles for service accounts? There is already a PR to merge the update into 1.14: #2358 I can confirm that this works and together with my PR #2428 also without the workaround.

@losipiuk
Copy link
Contributor Author

losipiuk commented Oct 9, 2019

Would it be possible to include the AWS SDK updates that enable IAM roles for service accounts? There is already a PR to merge the update into 1.14: #2358 I can confirm that this works and together with my PR #2428 also without the workaround.

@Jeffwan WDYT? It is kinda massive change. Larger than we typically allow for patch releases - and not strictly a bugfix. Yet I think we can merge that if you approve. If so I think we should also cherry pick #2428

@Jeffwan
Copy link
Member

Jeffwan commented Oct 10, 2019

@losipiuk Totally understand the policy, this one is all about godeps changes just for aws SDK.
We will appreciate if this can be merged to 1.14. otherwise, our pod IAM feature can not run on any EKS cluster (AWS supports 1.11-1.14)

#2428 is in the cherry-pick list, I will do that. Thanks a lot!

BTW, I check comments in #2356
Will we update vendor for release branches later? Sounds like there's risk to overwrite the changes.

@Jeffwan
Copy link
Member

Jeffwan commented Oct 10, 2019

Here's the cherry-pick changes. Thanks! @losipiuk

#2443 #2442 #2441 #2440

@Jeffwan
Copy link
Member

Jeffwan commented Oct 10, 2019

Looks like there're two related to priority expander.
#2435 #1920
Do we have plans to backport to older version?

@Jeffwan
Copy link
Member

Jeffwan commented Oct 11, 2019

All AWS changes are done. Thanks! @losipiuk

@losipiuk
Copy link
Contributor Author

Just heads up that I am kinda overwhelmed with other stuff so the patch release my slip a bit. Still I want to do that this week.

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

No branches or pull requests

5 participants