-
Notifications
You must be signed in to change notification settings - Fork 9.5k
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
Detailing -chdir
behavior with terraform.d/plugin
folders
#31460
Detailing -chdir
behavior with terraform.d/plugin
folders
#31460
Conversation
Thanks for these documentation changes. I'll run these past the docs team for review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @fnickels - thanks for adding this! I'm making a suggested edit to help with clarity and flow. Please let me know if I have misunderstood your intended meaning. Do you feel comfortable merging it in?
@crw After this, I would like a quick engineering technical 👍 as well - this is a case I'm not super familiar with and I want to make sure engineering is okay with this. Especially since there has been discussion about -chdir
in other threads. Thank you!
lgtm Co-authored-by: Laura Pacilio <83350965+laurapacilio@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀 @crw - over to eng. for a final 👍
For future reference: I believe this comment confirms the behavior as described in these docs changes. However, I'll queue this up for final review. |
-chdir
behavior with terraform.d/plugin
folders
Reminder for the merging maintainer: if this is a user-visible change, please update the changelog on the appropriate release branch. |
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions. |
Fixes Bug reported in #31442
This PR is a 4th alternate implementation - documentation change to match code behavior.
Use instead if the following PRs are not workable:
-chdir
option before resolving implied provider filesystem mirror directories, and then reset it back #31452-chdir
option before resolving implied provider filesystem mirror directories #31443