-
Notifications
You must be signed in to change notification settings - Fork 457
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
tfenv install
fails if Terraform is already installed, but Hashicorp's CDN is unavailable
#395
Comments
Zordrak
added a commit
that referenced
this issue
Dec 19, 2023
Do not check the release list if a Terraform is already installed (fixes #395)
Zordrak
added a commit
that referenced
this issue
Dec 19, 2023
Zordrak
added a commit
that referenced
this issue
Dec 19, 2023
Revert "Do not check the release list if a Terraform is already installed (fixes #395)"
bkzhn
pushed a commit
to bkzhn/tfenv
that referenced
this issue
Jan 14, 2024
tfutils#395) Today, tfenv checks the Terraform release list when running `tfenv install`, even if the requested Terraform is already installed. This fixes an issue where a script containing `tfenv install` becomes inoperable even if Terraform is installed, but releases.hashicorp.com is down. See tfutils#395 for more information.
bkzhn
pushed a commit
to bkzhn/tfenv
that referenced
this issue
Jan 14, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The releases.hashicorp.com site was down briefly this morning. We pre-bake needed Terraform versions into our CI/CD executors, but still run a
tfenv install ...
command to make local dev easier.Since Hashicorp releases are immutable, I'd expect that if a given revision of Terraform is already installed, tfenv wouldn't have any need to reach out to their CDN. Instead, it would detect that this version is already installed, and pass.
If releases.hashicorp.com is down, and a requested version of Terraform is already installed, tfenv will fail instead:
Here's a test case with Docker to reproduce:
The text was updated successfully, but these errors were encountered: