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!
I was just about to push a PR with this feature.
Also have a need for this feature along with automated bidding. Thanks
@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.
@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?
Closed via #8697
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: https://github.com/hashicorp/terraform/pull/8697#issuecomment-278797631
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.
Most helpful comment
@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?