chore(providers): update google and google beta version#25
Conversation
|
/code pr |
saidsef
left a comment
There was a problem hiding this comment.
🤖
Review
Changes Summary: Updating Google provider version constraints from >= 4 to >= 5 across three configuration files.
Observations
✓ Consistent updates across all three files (root, complete example, remote example)
-
Breaking change – Incrementing minimum version from 4 to 5 will require users to upgrade. Ensure this aligns with your deprecation policy and consider documenting migration guidance.
-
Upper bound inconsistency – The root
versions.tfmaintains< 8whilst examples already had this constraint; this is now harmonised, which is good. -
Testing – Confirm that existing infrastructure and resource configurations remain compatible with v5+ provider changes, particularly around deprecated resources or altered arguments.
-
Documentation – Consider adding release notes or migration guide if this represents a significant version bump for users.
Recommendation: Acceptable if compatibility has been verified across your managed resources. Ensure this change is communicated clearly to users.
> claude-haiku-4-5 [pr | 2]
There was a problem hiding this comment.
Pull request overview
This PR updates the minimum required version of the Google and Google Beta Terraform providers from version 4 to version 5, while extending the upper bound from version 7 to version 8. This ensures the module supports a wider range of newer provider versions while dropping support for older version 4.x releases.
- Updated version constraints from
>= 4, < 8(main) and>= 4, < 5(examples) to>= 5, < 8across all configuration files - Ensures consistency between the main module and example configurations
- Aligns all version specifications to support Google provider versions 5.x, 6.x, and 7.x
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| versions.tf | Updated main module provider version constraints from >= 4, < 8 to >= 5, < 8 |
| examples/remote/versions.tf | Updated remote example provider version constraints from >= 4, < 5 to >= 5, < 8 |
| examples/complete/versions.tf | Updated complete example provider version constraints from >= 4, < 5 to >= 5, < 8 |
Note: The changes look good and are consistently applied across all three versions.tf files. However, I noticed that the TERRAFORM.md documentation file (which appears to be auto-generated) still shows outdated version constraints (>= 4, < 7) and will need to be regenerated to reflect these updates. Since that file is not part of this PR, it should be updated in a follow-up commit or as part of the CI/CD process.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This pull request updates the required versions for the
googleandgoogle-betaTerraform providers across multiple configuration files. The minimum supported version is now5, ensuring compatibility with newer provider features and APIs.Provider version requirements update:
googleandgoogle-betaprovider version constraints from>= 4to>= 5inversions.tf,examples/complete/versions.tf, andexamples/remote/versions.tfto require at least version 5 and support up to but not including version 8.