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

[Azure] Fixing Azure CI workflow to pin to Terraform 15.1 #882

Merged
merged 10 commits into from
Apr 29, 2021

Conversation

hattan
Copy link
Contributor

@hattan hattan commented Apr 29, 2021

Closes #881 881

This PR includes changes to support Terraform 0.15.1 . As part of the 0.15.x update to Terraform there are changes are sensitive values that require them to be explicitly marked as sensitive. In addition, 0.15.0 has a bug with sensitive outputs that causes a panic and the tests failing.

This changes pins the workflow to Terraform 0.15.1 and also adds the sensitive flag to modules that need it.

Copy link
Contributor

@HadwaAbdelhalem HadwaAbdelhalem left a comment

Choose a reason for hiding this comment

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

LGTM @@gruntwork-io/gruntwork-terratest-terragrunt ready for review

Copy link
Contributor

@yorinasub17 yorinasub17 left a comment

Choose a reason for hiding this comment

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

Updates LGTM! Doing a sanity check build on our pipeline to make sure the precommit hooks pass, and if so, will merge this in!

@yorinasub17
Copy link
Contributor

Ok precommit hooks passed, so going to merge this in!

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.

Fixate terraform version in Azure ci-workflow
6 participants