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

Handle provider lock file when fetching dependency outputs #2568

Conversation

geekofalltrades
Copy link
Contributor

@geekofalltrades geekofalltrades commented May 15, 2023

Fixes errors when using the plugin cache caused by new cache behavior in Terraform 1.4+.

Description

Fixes #2542.

TODOs

Read the Gruntwork contribution guidelines.

  • Update the docs.
  • Run the relevant tests successfully, including pre-commit checks.
  • Include release notes. If this PR is backward incompatible, include a migration guide.

Release Notes (draft)

Fixed provider lock file handling when retrieving outputs from dependencies via remote state.

Migration Guide

Fixes errors when using the plugin cache caused by new cache behavior in
Terraform 1.4+.
@levkohimins
Copy link
Contributor

@geekofalltrades, thank you for your contribution!

I have optimized the code a bit. Could you please merge this bug/remote-state-dependency-lockfile-2542 branch with yours ?

@geekofalltrades
Copy link
Contributor Author

@levkoburburas done.

@levkohimins levkohimins self-requested a review May 17, 2023 14:49
Copy link
Contributor

@levkohimins levkohimins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@levkohimins levkohimins merged commit 5c29736 into gruntwork-io:master May 17, 2023
szesch pushed a commit to szesch/terragrunt that referenced this pull request May 23, 2023
…-io#2568)

* Handle provider lock file when fetching dependency outputs

Fixes errors when using the plugin cache caused by new cache behavior in
Terraform 1.4+.

* code optimization

---------

Co-authored-by: Levko Burburas <levko.burburas@biptec.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Init concurrency/parallelism issues
2 participants