Skip to content
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

Fixes #207, supplying AWS_DEFAULT_REGION fixes v0.12 tf validate #208

Closed

Conversation

Gowiem
Copy link
Member

@Gowiem Gowiem commented Apr 24, 2020

what

  • Updates the terraform/validate target to actually run terraform validate instead of logging "unsupported" message.

why

  • This is to provide actual validation on v0.12
  • Seems the workaround was not known when the "unsupported" log was in place. Now we know we can get around the 'validate' region requirement by supplying AWS_DEFAULT_REGION.

references

modules/terraform/Makefile Outdated Show resolved Hide resolved
osterman pushed a commit to cloudposse/test-harness that referenced this pull request May 7, 2020
* Adds prerequisites to README

* Fixes validate.bats issue with "region" error for modules. 

This is the same issue as 
cloudposse/build-harness#208. Without a default 
region, `terraform validate` fails when running against independent 
modules as they do not have a provider/region supplied.

* Fixes validate.bats unsetting the wrong environment var

I believe this was just a mistake but I may be wrong. setup() sets 
TF_CLI_ARGS_init, but teardown unsets TF_CLI_ARGS_plan. That seemed like 
a small typo to me, so I figured I would include this.
@Gowiem Gowiem requested a review from a team June 18, 2020 18:04
@Gowiem Gowiem requested a review from osterman June 27, 2020 07:04
@Gowiem
Copy link
Member Author

Gowiem commented Aug 14, 2020

@osterman ping on this one.

@Gowiem Gowiem closed this Nov 16, 2020
@Gowiem Gowiem reopened this Nov 16, 2020
@Gowiem Gowiem requested a review from aknysh November 18, 2020 23:52
@Gowiem
Copy link
Member Author

Gowiem commented Nov 18, 2020

@osterman any interest in this? Totally fine to close out if not needed. It's been a while and I personally don't need it any longer.

@Gowiem
Copy link
Member Author

Gowiem commented Nov 24, 2020

Closing out due to age and not being needed.

@Gowiem Gowiem closed this Nov 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants