Skip to content

make terraform/validate does not to run on Terraform v0.12.X #207

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

Closed
Gowiem opened this issue Apr 24, 2020 · 1 comment
Closed

make terraform/validate does not to run on Terraform v0.12.X #207

Gowiem opened this issue Apr 24, 2020 · 1 comment
Labels
bug Something isn't working wontfix This will not be worked on

Comments

@Gowiem
Copy link
Member

Gowiem commented Apr 24, 2020

Describe the Bug

Running make terraform/validate on Terraform v0.12.X prints the following message:

Terraform 0.12 does not support validate without skipping variables

Expected Behavior

I would expect the terraform/validate target to not care about this skipping variables flag as it doesn't seem needed, but maybe I am missing something.

Steps to Reproduce

  1. git clone https://github.com/cloudposse/terraform-aws-ssm-parameter-store
  2. make
  3. make terraform/install TERRAFORM_VERSION=0.12.24
  4. make terraform/validate
  5. Notice that Terraform validate is not run and the target prints "Terraform 0.12 does not support validate without skipping variables".

Environment (please complete the following information):

  • Build Harness Version: master
  • OS: MacOSX
  • Version: 10.15.4

Additional Context

  • When I run tf validate locally without the Makefile / Build-harness -- I get the expected, useful output:
Error: Missing required argument

The argument "region" is required, but was not set.

image

@Gowiem Gowiem added the bug Something isn't working label Apr 24, 2020
@Nuru Nuru added the wontfix This will not be worked on label Mar 22, 2021
@Nuru
Copy link
Contributor

Nuru commented Mar 22, 2021

Terraform v0.12 support is no longer a priority. Even if it were, simply supplying region is not enough to get all our modules to pass, and we require that they pass before releasing them.

@Nuru Nuru closed this as completed Mar 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

2 participants