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

Ability to customise version of Terraform installed #41

Closed
chris-crone opened this issue Jul 23, 2020 · 1 comment · Fixed by #42
Closed

Ability to customise version of Terraform installed #41

chris-crone opened this issue Jul 23, 2020 · 1 comment · Fixed by #42
Labels
enhancement New feature or request

Comments

@chris-crone
Copy link
Contributor

Playing with this mixin on Kubernetes, I found one area of friction is storing the Terraform state. Either one would need to copy the state file between CNAB actions or use some remote backend.

With Terraform 0.13, currently there's an RC out, a Kubernetes secret backend will be added which makes this a lot less painful.

As the version of Terraform is currently hardcoded in the mixin, it's not easy to use this feature as one would have to rebuild the mixin. I think that wanting to specify versions of tools will actually be quite common, would it make sense to add a mixin config that allows this?

@vdice
Copy link
Member

vdice commented Jul 23, 2020

For prior art, see https://github.com/deislabs/porter-helm#mixin-configuration (and PR that implemented the feature: getporter/helm2-mixin#64)

Allowing in the porter.yaml and entry like:

mixins:
  - terraform:
      clientVersion: 0.12.29

(where the terraform version 0.12.29 is downloaded/used in the resulting bundle invocation image.)

@vdice vdice added the enhancement New feature or request label Jul 23, 2020
@vdice vdice closed this as completed in #42 Jul 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants