We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
The runner module hardcoded the usages of spot usags by setting market option to spot see:
terraform-aws-github-runner/modules/runners/variables.tf
Line 54 in b4838d7
Would be better to make it configurable with default spot to support also on demand instances.
See also https://docs.aws.amazon.com/AWSEC2/latest/APIReference/API_InstanceMarketOptionsRequest.html
The text was updated successfully, but these errors were encountered:
@npalm Implemented here: #657
Sorry, something went wrong.
@npalm @dimisjim Thanks for implementing this feature! It will be super useful in my work, and I'm grateful you're providing this public good.
Can be closed now that #657 is merged, I think. I just deployed that patch to my production server! (^-^)/ Thanks again.
Spot can be disabled by setting variable market_options
market_options
No branches or pull requests
The runner module hardcoded the usages of spot usags by setting market option to spot see:
terraform-aws-github-runner/modules/runners/variables.tf
Line 54 in b4838d7
Would be better to make it configurable with default spot to support also on demand instances.
See also https://docs.aws.amazon.com/AWSEC2/latest/APIReference/API_InstanceMarketOptionsRequest.html
The text was updated successfully, but these errors were encountered: