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

Detect and integrate with tfenv #305

Open
radeksimko opened this issue Nov 17, 2020 · 0 comments
Open

Detect and integrate with tfenv #305

radeksimko opened this issue Nov 17, 2020 · 0 comments
Labels
enhancement New feature or request

Comments

@radeksimko
Copy link
Member

radeksimko commented Nov 17, 2020

Current Version

0.9.0

Use-cases

Users with tfenv can face challenges due to the way tfenv works when executing terraform commands.

e.g. as demonstrated in #151 (comment) tfenv will automatically install Terraform of a particular version per .terraform-version in PWD and the output doesn't match the output of typical terraform version, so terraform-exec is unable to parse that successfully.

Proposal

  • Understand UX and user expectations about how the integration should work, e.g.
    • Should tfenv be installing Terraform as part of initialization?
    • Should that happen silently, or would you expect a popup and confirm the installation?
    • Should LS be always passing ENV var which prevents tfenv from automatically installing anything and should it be left to the user to manage outside of LS?

Note: The work may also involve terraform-exec as that's the library LS uses for executing Terraform.


Related: hashicorp/terraform-exec#6

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

No branches or pull requests

1 participant