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

Remove asset_dir variable and optional asset writes #854

Merged
merged 1 commit into from
Oct 17, 2020

Conversation

dghubble
Copy link
Member

  • Originally, poseidon/terraform-render-bootstrap generated TLS certificates, manifests, and cluster "assets" written
    to local disk (asset_dir) during terraform apply cluster bootstrap
  • Typhoon v1.17.0 introduced bootstrapping using only Terraform state to store cluster assets, to avoid ever writing sensitive
    materials to disk and improve automated use-cases. asset_dir was changed to optional and defaulted to "" (no writes)
  • Typhoon v1.18.0 deprecated the asset_dir variable, removed docs, and announced it would be deleted in future.
  • Add Terraform output assets_dir map
  • Remove the asset_dir variable

Cluster assets are now stored in Terraform state only. For those who wish to write those assets to local files, this is possible doing so explicitly.

resource local_file "assets" {
  for_each = module.yavin.assets_dist
  filename = "some-assets/${each.key}"
  content = each.value
}

Related:

* Originally, poseidon/terraform-render-bootstrap generated
TLS certificates, manifests, and cluster "assets" written
to local disk (`asset_dir`) during terraform apply cluster
bootstrap
* Typhoon v1.17.0 introduced bootstrapping using only Terraform
state to store cluster assets, to avoid ever writing sensitive
materials to disk and improve automated use-cases. `asset_dir`
was changed to optional and defaulted to "" (no writes)
* Typhoon v1.18.0 deprecated the `asset_dir` variable, removed
docs, and announced it would be deleted in future.
* Add Terraform output `assets_dir` map
* Remove the `asset_dir` variable

Cluster assets are now stored in Terraform state only. For those
who wish to write those assets to local files, this is possible
doing so explicitly.

```
resource local_file "assets" {
  for_each = module.yavin.assets_dist
  filename = "some-assets/${each.key}"
  content = each.value
}
```

Related:

* #595
* #678
@dghubble dghubble merged commit afac46e into master Oct 17, 2020
@dghubble dghubble deleted the remove-asset-dir branch October 17, 2020 23:15
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