-
Notifications
You must be signed in to change notification settings - Fork 447
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
min-required and fallback #353
Comments
Yes, this is quite strange :
I was thinking that TFENV_TERRAFORM_VERSION would be used ONLY when I don't specify a version in command line. |
JeanFred
added a commit
to JeanFred/tfenv
that referenced
this issue
Oct 25, 2022
Even when setting `TFENV_TERRAFORM_VERSION` to `min-required`, we might prefer to fallback on the normal inference method rather than a hard fail. Resolves tfutils#353
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I have now tried the
min-required
feature that recently shipped with #335 (after testing the one in #305).It does not seem to work for the use case I described at #305 (comment) and my mixed-version code base.
TFENV_TERRAFORM_VERSION=min-required
, then that firstterraform --version
call fails with [1]#305 ended up implementing a graceful fallback on the next detection method. Would something similar be possible here?
(As said in that comment, perhaps my use-case is somewhat specific ; and I can see how automatic fallbacks can be confusing).
[1] Logs
[2] Logs
The text was updated successfully, but these errors were encountered: