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.
Any updates on getting more arm support in the future or blockers for this work?