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

Feature Request: Support for Autoscaling on Spot Fleets #8693

Closed
justintime opened this issue Sep 6, 2016 · 8 comments
Closed

Feature Request: Support for Autoscaling on Spot Fleets #8693

justintime opened this issue Sep 6, 2016 · 8 comments

Comments

@justintime
Copy link

Recent support was added to TF that enabled the ability to spin up AWS Spot Fleets: https://www.terraform.io/docs/providers/aws/r/spot_fleet_request.html

Since that time, AWS recently announced that they now support AutoScaling on Spot Fleet: https://aws.amazon.com/blogs/aws/new-auto-scaling-for-ec2-spot-fleets/

It would be truly awesome to be able to configure autoscaling on our spot fleet requests!

@niclic
Copy link
Contributor

niclic commented Sep 7, 2016

I was just about to push a PR with this feature.

@niclic
Copy link
Contributor

niclic commented Sep 7, 2016

See #8697

@beastie29a
Copy link

Also have a need for this feature along with automated bidding. Thanks

@beastie29a
Copy link

@niclic, I apologize for the confusion. Was just wondering what the status of PR #8697 was and didn't want to add to the noise in that thread.

@niclic
Copy link
Contributor

niclic commented Dec 15, 2016

@beastie29a

It's been a while since I reviewed #8697, but the last time I looked I recommended it be merged as is. It needs a rebase as I explained in this comment, but that should be it.

I'm hoping it can be reviewed and merged soon as it seems some other folks would like to take advantage of this feature too.

Looks like the terraform team are making their way through the backlog of PRs, so hopefully soon?

@stack72
Copy link
Contributor

stack72 commented Feb 2, 2017

Closed via #8697

@stack72 stack72 closed this as completed Feb 2, 2017
@niclic
Copy link
Contributor

niclic commented Feb 13, 2017

With this feature finally merged, you may encounter a backwards compatibility issue. Both the arn and name attributes were removed form the aws_appautoscaling_target resource. Unfortunately, these attributes were not annotated with the Deprecated or Removed fields. Neither arn or name are required by aws_appautoscaling_target but you may have included them in your configurations and may encounter an error when using version v.0.8.6.

See this note for details: #8697 (comment)

@ghost
Copy link

ghost commented Apr 16, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@ghost ghost locked and limited conversation to collaborators Apr 16, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants