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

website: Document TF_PLUGIN_CACHE_MAY_BREAK_DEPENDENCY_LOCK_FILE env var #32765

Merged
merged 1 commit into from
Mar 7, 2023

Conversation

apparentlymart
Copy link
Contributor

This is an alternative way to set the CLI configuration setting plugin_cache_may_break_dependency_lock_file to activate the transitional compatibility behavior that prefers to break the dependency lock file if that would create an additional opportunity to use the plugin cache.

This has been supported since #32726 but I forgot to update this particular doc section when I implemented it, despite @alisdair's attempt to prompt me which apparently I was too absent-minded to think about that day. 😖

While here I also fixed a minor bug from #32494 (where this doc section originated) where the heading was at the wrong nesting level and so would've messed up the navigation tree as previously defined. The heading is now nested under "Provider Installation" as originally intended.

@apparentlymart apparentlymart added documentation 1.4-backport If you add this label to a PR before merging, backport-assistant will open a new PR once merged labels Mar 1, 2023
@apparentlymart apparentlymart added this to the v1.4.0 milestone Mar 1, 2023
@apparentlymart apparentlymart requested a review from a team March 1, 2023 17:19
@apparentlymart apparentlymart self-assigned this Mar 1, 2023
@apparentlymart
Copy link
Contributor Author

I guess the website platform now requires all links to be root-relative, rather than page-relative. I suspect we're going to come to regret that one day when we next move the docs to another location, but for now I'm going to just do what it's telling me so that this will pass the check and then I can merge it.

This is an alternative way to set the CLI configuration setting
plugin_cache_may_break_dependency_lock_file to activate the transitional
compatibility behavior that prefers to break the dependency lock file if
that would create an additional opportunity to use the plugin cache.
@github-actions
Copy link
Contributor

github-actions bot commented Mar 7, 2023

Reminder for the merging maintainer: if this is a user-visible change, please update the changelog on the appropriate release branch.

@github-actions
Copy link
Contributor

github-actions bot commented Apr 6, 2023

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.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 6, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
1.4-backport If you add this label to a PR before merging, backport-assistant will open a new PR once merged documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants