Skip to content
This repository has been archived by the owner on Feb 11, 2022. It is now read-only.

Spot Instances #514

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

mkubenka
Copy link

Follow up of #430

With additional enhancements:

  • spot instance is started with same options as on-demand instance, so it's possible to setup device mapping, IAM instance profile and so on
  • user interruption is correctly handled and after spot fulfilment instance is terminated
  • if spot_max_price is not defined average price is computed

Fix #32, #338, https://github.com/KariusDx/vagrant-aws/pull/2

@jwiegley
Copy link

@mitchellh Is it time to merge this?

@luisggpina
Copy link

@mitchellh Having this merged would make my life (and my student's life) so much easier.

@mkubenka
Copy link
Author

@luisggpina As a temporary solution you can use my Gem vagrant-aws-mkubenka with spot instances support.

Installation instructions:

vagrant plugin uninstall vagrant-aws
vagrant plugin install vagrant-aws-mkubenka --plugin-version "0.7.2.pre.22"

@spuppi
Copy link

spuppi commented Mar 11, 2020

Very good, but my instance does not get ip public even adding aws.associate_public_ip = true, any tips?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support Spot Instances
4 participants