Adam Blackwell
Adam Blackwell
@liamrahav & @dtan4, this change would save us a lot of time with a large import we're currently doing, do you know if there are any blockers to merging it...
I would also love to have this feature back, but am doubtful that I could replace the existing dependency on ec2-price. I use https://ec2instances.info for quick lookups, but it looks...
Any update on implementing this @ankit-arora? We would appreciate the feature.
*It may be worth rebasing this PR and building a sandbox off of the branch before merging.
I should build a sandbox from this branch before merging per https://openedx.atlassian.net/wiki/spaces/SRE/pages/28967861/Ops+Ansible+Testing+Checklist
LGTM, we should run this in --check mode first though.
> @adzuci it this only going to surface codeql issues on the delta change in the PR? I'm not sure about the PR reporting, but I'm looking for this view:...
Looks like https://github.com/hashicorp/terraform-provider-aws/pull/27712 has merged. Any ideas on the LOE of getting support added to the provider?
@katebygrace I'm closing this PR, but think it may be worth exploring more in the future.