-
Notifications
You must be signed in to change notification settings - Fork 523
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
feat: Add aws spot instance request #2178
feat: Add aws spot instance request #2178
Conversation
Thanks for PR this @lucastercas, I'll take a peek at this when I get a moment in the next few days. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@lucastercas this looks great, thanks. However, we just need to change the way we test the output price. A golden file test won't work unfortunately. Thanks
...roviders/terraform/aws/testdata/spot_instance_request_test/spot_instance_request_test.golden
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems like the tests are failing as you just need to add in a check for the t3_large
spot resource you've got in your tests
...roviders/terraform/aws/testdata/spot_instance_request_test/spot_instance_request_test.golden
Outdated
Show resolved
Hide resolved
...al/providers/terraform/aws/testdata/spot_instance_request_test/spot_instance_request_test.tf
Outdated
Show resolved
Hide resolved
...iders/terraform/aws/testdata/spot_instance_request_test/spot_instance_request_test.usage.yml
Outdated
Show resolved
Hide resolved
Any update on this? |
Add support for
aws_spot_instance_request
Fixes: #1223