-
Notifications
You must be signed in to change notification settings - Fork 1.7k
[cluster-autoscaler]: Add AWS Support #1292
Conversation
Fixing migconfig test
Hi @iterion, we are both working on the same problem. :) Would you be interested in joining efforts? I took a slightly different approach with extracting the interface for the providers to implement, and I hid the I was about to implement the AWS provider when I saw your PR. Here's my current progress. Are you interested in merging the approaches at all? |
Great! Thank you for this contribution. We definitely want to have it merged fast. However due to tight schedule of 1.3 we took a little bit of technical debt here which we would like to pay back before we onboard other cloud providers. |
Hi @mwielgus, I would be interested in having a video conference to talk about this. I am compiling some questions. I'll try to ask them here or in a GH issue first, if that helps. |
@pbitty Cool, any preference regarding date/time? |
@pbitty definitely interested in cooperating on this, no sense in duplicating work! I think your approach is a lot cleaner. I basically just hacked it until it worked, aiming to do the bare minimum refactor :) @mwielgus I'd also like to join in on the conference call and am happy to help out with tech debt. As for late next week, my schedule is pretty open. I'm almost always available from 11am - 5pm CST. Please let me know if there is any way I can help out with the tech debt effort? |
I am generally available between 11am and 5pm EST. Is there a day this week that works best for everyone? |
Umbrella issue: #1311 Ok, then lets meet on Thursday, 2016/07/07, 12:00 EDT / 11:00 CDT / 18:00 CEST. Does it work for everyone? |
That time works for me. |
That time also works for me, see you all then. I'm also going to close this PR as we now have #1311 to track the related work. |
Thanks for the video call @mwielgus @iterion, @andrewsykim. @iterion, feel free to ping myself and @andrewsykim to pair on this. I'll be away next week but @andrewsykim is around. |
@iterion I've been taking a look into the AWS implementation for this interface https://github.com/kubernetes/contrib/blob/master/cluster-autoscaler/cloudprovider/cloud_provider.go. Is there a fork you have that I can follow so we don't duplicate the work? If you haven't started that implementation I wouldn't mind submitting a PR against your fork. |
@andrewsykim thanks for checking in. I haven't gotten around to implementing the new interface yet. Feel free submitting a PR against my fork, I can definitely be available to review and/or help out. I'm also @iterion on the Kubernetes slack channel, if you want more real time communication for questions. |
Adding support for scaling AWS AutoScalingGroups. I've tested this in a test cluster on AWS (with a few more changes needed to allow this to work on Kubernetes v1.2) and everything seemed to function correctly.
It's currently a bit messy to use AWS. You must specify
--cloud-provider=aws
and provide the ASG name and zone in a GCE formatted url:https://content.googleapis.com/compute/v1/projects/dummy/zones/us-east-1a/autoscalingGroupName
PTAL and let me know how I can improve this to get it merged. I haven't yet tested this against a v1.3 cluster or against a GCE cluster.