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

Run terraform validate in examples/complete #37

Merged
merged 1 commit into from
Nov 29, 2021

Conversation

nitrocode
Copy link
Member

@nitrocode nitrocode commented Nov 25, 2021

what

  • Run terraform validate in examples/complete

why

  • Allow test/bats to pass in terraform module that require aliased providers
  • The providers are configured correctly in the complete example so the validation should run in the correct directory instead of the module repo root

references

Copy link
Member

@korenyoni korenyoni left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed the underlying problem — makes sense. LGTM.

@nitrocode nitrocode merged commit 4bb088d into master Nov 29, 2021
@nitrocode nitrocode deleted the validate-in-complete branch November 29, 2021 16:39
@Nuru Nuru added the invalid This doesn't seem right label Nov 29, 2021
@Nuru
Copy link
Contributor

Nuru commented Nov 29, 2021

@aknysh @korenyoni This PR was wrong and should not have been approved:

  1. The command change itself is wrong. terraform validate -chdir=examples/complete does not work in any version of Terraform.
  2. The bats tests are already being run in examples/complete via the standard make target. See example.
  3. Probably this should have been released as a patch.

@aknysh
Copy link
Member

aknysh commented Nov 29, 2021

yes, sorry.
This was wrong from the beginning https://github.com/cloudposse/test-harness/blob/master/test/terraform/validate.bats

Nuru added a commit that referenced this pull request Nov 29, 2021
@Nuru Nuru mentioned this pull request Nov 29, 2021
Nuru added a commit that referenced this pull request Nov 29, 2021
* Revert "validate in examples/complete (#37)"

This reverts commit 4bb088d.

* Fix typo in comment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invalid This doesn't seem right
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants