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

Adopt Terraform v0.12 templatefile function #159

Merged

Conversation

dghubble
Copy link
Member

  • Adopt Terrform v0.12 type and templatefile function features to replace the use of terraform-provider-template's template_dir
  • Use of for_each to write local assets requires that consumers use Terraform v0.12.6+ (action required)
  • Continue use of template_file as its quite common. In future, we may replace it as well.
  • Remove outputs id and content_hash (no longer used)

Background:

  • template_dir was added to terraform-provider-template to add support for template directory rendering in CoreOS Tectonic Kubernetes distribution (~2017)
  • Terraform v0.12 introduced a native templatefile function and v0.12.6 introduced native for_each support (July 2019) that makes it possible to replace template_dir usage

* Adopt Terrform v0.12 type and templatefile function
features to replace the use of terraform-provider-template's
`template_dir`
* Use of `for_each` to write local assets requires
that consumers use Terraform v0.12.6+ (action required)
* Continue use of `template_file` as its quite common. In
future, we may replace it as well.
* Remove outputs `id` and `content_hash` (no longer used)

Background:

* `template_dir` was added to `terraform-provider-template`
to add support for template directory rendering in CoreOS
Tectonic Kubernetes distribution (~2017)
* Terraform v0.12 introduced a native `templatefile` function
and v0.12.6 introduced native `for_each` support (July 2019)
that makes it possible to replace `template_dir` usage
@dghubble dghubble merged commit 1bba891 into master Nov 14, 2019
@dghubble dghubble deleted the adopt-terraform-templatefile-instead-of-template_dir branch November 14, 2019 00:26
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.

1 participant