Adam Blackwell

Results 9 comments of 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.