Skip to content

chore(providers): update google and google beta version#25

Merged
saidsef merged 1 commit intomainfrom
providers-version-bump
Dec 12, 2025
Merged

chore(providers): update google and google beta version#25
saidsef merged 1 commit intomainfrom
providers-version-bump

Conversation

@saidsef
Copy link
Copy Markdown
Owner

@saidsef saidsef commented Dec 12, 2025

This pull request updates the required versions for the google and google-beta Terraform providers across multiple configuration files. The minimum supported version is now 5, ensuring compatibility with newer provider features and APIs.

Provider version requirements update:

  • Updated the google and google-beta provider version constraints from >= 4 to >= 5 in versions.tf, examples/complete/versions.tf, and examples/remote/versions.tf to require at least version 5 and support up to but not including version 8.

Copilot AI review requested due to automatic review settings December 12, 2025 22:36
@saidsef saidsef added the enhancement New feature or request label Dec 12, 2025
@saidsef saidsef self-assigned this Dec 12, 2025
@saidsef saidsef added the enhancement New feature or request label Dec 12, 2025
@saidsef
Copy link
Copy Markdown
Owner Author

saidsef commented Dec 12, 2025

/code pr

Copy link
Copy Markdown
Owner Author

@saidsef saidsef left a comment

Choose a reason for hiding this comment

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

🤖

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)

⚠️ Considerations:

  1. 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.

  2. Upper bound inconsistency – The root versions.tf maintains < 8 whilst examples already had this constraint; this is now harmonised, which is good.

  3. Testing – Confirm that existing infrastructure and resource configurations remain compatible with v5+ provider changes, particularly around deprecated resources or altered arguments.

  4. 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]

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

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, < 8 across 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.

@saidsef saidsef merged commit 4694b9d into main Dec 12, 2025
33 of 34 checks passed
@saidsef saidsef deleted the providers-version-bump branch December 12, 2025 22:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants