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 #222

Merged
merged 1 commit into from
Oct 17, 2020

Conversation

dghubble
Copy link
Member

  • Originally, 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.
  • 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.bootstrap.assets_dist
  filename = "some-assets/${each.key}"
  content = each.value
}

Related:

* Originally, 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.
* 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.bootstrap.assets_dist
  filename = "some-assets/${each.key}"
  content = each.value
}
```

Related:

* poseidon/typhoon#595
* poseidon/typhoon#678
@maikelvl
Copy link
Contributor

Have to say I always appreciate your detailed descriptions. ❤️

@dghubble dghubble merged commit 9037d73 into master Oct 17, 2020
@dghubble dghubble deleted the remove-asset-dir-variable branch October 17, 2020 23:14
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.

2 participants