-
Notifications
You must be signed in to change notification settings - Fork 127
Update Terraform Upgrade Guides for 1.13 and 1.14 #1553
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
Update Terraform Upgrade Guides for 1.13 and 1.14 #1553
Conversation
Vercel Previews Deployed
|
Broken Link CheckerNo broken links found! 🎉 |
trujillo-adam
left a comment
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.
Thanks for fixing this. I added a few suggestions and questions. Approving so you can move forward after addressing them.
|
|
||
| ## Terraform Test | ||
|
|
||
| - External variables referenced within test files should now be accompanied by a `variable` definition block within the test file. |
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.
| - External variables referenced within test files should now be accompanied by a `variable` definition block within the test file. | |
| - External variables referenced within test files should be accompanied by a `variable` definition block within the test file. |
Should or must?
| - The parallelism of Terraform operations within container runtimes may be reduced depending on the CPU bandwidth limit setting. | ||
| - Building Terraform 1.14 requires macOS Monterey or later (due to being built on Go 1.25 which imposes these requirements) |
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.
| - The parallelism of Terraform operations within container runtimes may be reduced depending on the CPU bandwidth limit setting. | |
| - Building Terraform 1.14 requires macOS Monterey or later (due to being built on Go 1.25 which imposes these requirements) | |
| - The CPU bandwidth limit setting affects Terraform's ability to perform parallel operations within container runtimes. Setting the limit too low may reduce parallelism. | |
| - Building Terraform 1.14 requires macOS Monterey or later. This is because Terraform is built on Go 1.25, which imposes these requirements. |
Can we recommend a CPU bandwidth limit?
What
This PR updates upgrade guides for 1.13 and 1.14 to reflect the upgrade considerations in those releases. It also make a minor syntax update to the description for version 1.15
Why
Terraform upgrade guides seems to have stopped updating after the switch to a folder-based structure. Terraform versions 1.13 and 1.14 both reflected version 1.12's upgrade guide notes
Merge Checklist
If items do not apply to your changes, add (N/A) and mark them as complete.
Pull Request
Content
content/terraform-docs-common/redirects.jsoncfor moved, renamed, or deleted pages across all affected versions. Refer to Redirects for examples and guidance.Reviews