airbnb / streamalert

StreamAlert is a serverless, realtime data analysis framework which empowers you to ingest, analyze, and alert on data from any environment, using datasources and alerting logic you define.
https://streamalert.io
Apache License 2.0
2.85k stars 331 forks source link

support terraform v0.12.N #947

Closed ryandeivert closed 4 years ago

ryandeivert commented 5 years ago

Background

Terraform 0.12.X requires explicit relative paths (using a ./ prefix). See: poseidon/typhoon#457.

This breaks things, and we should update to support the newest release of terraform.

ryandeivert commented 5 years ago

I was looking to see why this check was not working, and verified locally that module resolution happens before the version check.... wtf.

Test Steps

module "test" { source = "modules/foo_bar_test_modules" // implicitly relative path }


* Ran `terraform init` and got error
* Changed implicitly relative path to explicit via: `./modules/foo_bar_test_modules`
* Ran again and no error
ryandeivert commented 4 years ago

complete with #1035