-
-
Notifications
You must be signed in to change notification settings - Fork 956
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
Add Terragrunt integration test for latest Terraform. #18
Conversation
This is a simple terraform template on which we’ll apply Terragrunt.
…ate latest changes.
Once a pR is created for the module-ci change that fixes terraform being in PATH and is merged, this commit will pass CI.
- configure-environment-for-gruntwork-module --terraform-version NONE --packer-version NONE --go-src-path . | ||
- curl -Ls https://raw.githubusercontent.com/gruntwork-io/gruntwork-installer/master/bootstrap-gruntwork-installer.sh | bash /dev/stdin --version 0.0.13 | ||
- gruntwork-install --module-name "gruntwork-module-circleci-helpers" --repo "https://github.com/gruntwork-io/module-ci" --tag "v0.0.18" | ||
- configure-environment-for-gruntwork-module --packer-version NONE --go-src-path . |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Won't this install terragrunt now? Shouldn't the test use its local version instead?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The current test runs Terragrunt by directly invoking the CLI app, but the point is well-taken. To avoid any confusion, I'll disable terragrunt explicitly.
Changes lgtm. Just some minor comments. |
5a21331
to
0d584ea
Compare
…ize) considerably.
Now that https://github.com/gruntwork-io/module-ci/pull/6 is merged, tests are passing, so merging. |
This PR adds a simple integration test to validate that Terragrunt works with the latest version of Terraform. It executes a trivial Terraform template that just renders a
data.template_file
resource.Do NOT merge this PR until https://github.com/gruntwork-io/module-ci/pull/6 is merged and a new release is tagged in https://github.com/gruntwork-io/module-ci since running this integration test successfully depended on an update to
configure-environment-for-gruntwork-module
in github.com/gruntwork-io/module-ci.