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

fix: adapt terraform download due to new folder zip structure #4478

Closed

Conversation

rkferreira
Copy link

what

Short-term fix for #4471. With this change Atlantis will download terraform version 1.8.2 or any new one.

why

Terraform packaging was changed in release 1.8.2 (see https://github.com/hashicorp/terraform/releases/tag/v1.8.2) and Atlantis throws an error for every default and custom workflow when trying to download it. Proper, long-term fix is described here - #4472 (comment).

tests

No test cases added.
It was tested running atlantis testdrive and reproducing the scenario.

references

See #4471 and #4472 for more info about issue and various workarounds.

@rkferreira rkferreira requested review from a team as code owners April 25, 2024 00:36
@rkferreira rkferreira requested review from GenPage, lukemassa and nitrocode and removed request for a team April 25, 2024 00:36
@github-actions github-actions bot added the go Pull requests that update Go code label Apr 25, 2024
@rkferreira rkferreira changed the title adapt terraform download due to new folder zip structure fix: adapt terraform download due to new folder zip structure Apr 25, 2024
@lukemassa
Copy link
Contributor

My opinion here is this isn't a big enough step forward from our current workaround (#4474) to warrant another change just yet. We should consider #4472 (comment) from Core TF to see if we can reach a long term stable solution.

If that turns out to be impossible, or take too long, such that users need some feature of 2.8.2 or beyond, I'd suggest we'd revisit this as an alternative.

@jamengual jamengual added the wont-do Unfortunately due to scope or project direction we won't build this in a reasonable amount of time label Apr 30, 2024
@rkferreira rkferreira force-pushed the workaround_latet_tf_folder branch from 9df775d to 24f338e Compare April 30, 2024 20:54
@james0209
Copy link
Contributor

@rkferreira I think you can close this now that the hc-install PR has been merged and released: #4494

@rkferreira
Copy link
Author

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
go Pull requests that update Go code wont-do Unfortunately due to scope or project direction we won't build this in a reasonable amount of time
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants