-
Notifications
You must be signed in to change notification settings - Fork 32
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
Add support for terragrunt #36
Comments
@dvaumoron how do you think, will it be difficult? |
@gercograndia Thanks for issue! I think that we'll try to add Terragrunt support in the near future. Do you have any requirement that we need to keep in mind? |
Great!! From a requirements perspective:
In general it should behave exactly the same as for terraform and opentofu. |
Probably not very difficult with the current design, i will take a look |
The harder part will be changing the version detection part (not same kind of behaviour as OpenTofu/Terraform handling), however the last refactoring done have made that easier. There will also be a difference on the download part, as terragrunt asset are directly binaries, not zip archive. |
There will also be subtleties to support tgswitch.toml file, .tgswitchrc file and .terragrunt-version file at the same time. |
For me the .terragrunt-version would be sufficient. But hey, this is just how we are using it of course. But also here I would not necessarily aim for feature-partity, but keep the way of working the same as for the rest of |
Done (not tested yet) |
@dvaumoron wow! that was so quick! |
And quick it was, damn! 😁 |
Seem to work as expected now, i will update doc |
flat files like |
@dvaumoron I think that we can move this part of work to another issue. |
@dvaumoron great job! |
Is your feature request related to a problem? Please describe.
No direct problem.
Describe the solution you'd like
It would be cool if we could use this for managing the terragrunt version as well. We're now using tfenv and tgswitch respectively, both behaving in a slightly different way. Harmonizing this would be great.
The text was updated successfully, but these errors were encountered: