Skip to content

Conversation

@yuktatechgog
Copy link
Contributor

@yuktatechgog yuktatechgog commented Oct 28, 2025

-BREAKING CHANGE: Renamed the project input variable to project_id across root and interface modules and examples for ADC consistency. An upgrade guide has been added to docs/upgrading_to_v8.0.md.

ADC Metadata:

-Updated metadata.yaml files with corrected version constraints for connections.
-Removed an unsupported values() function from a connection in the interface submodule's metadata.

This change updates the blueprint to be compliant with the Application Design Center.

Key changes:
- Updated metadata.yaml files to include output types.
- Added connection metadata for network, router, and vpn_tunnel inputs.
- Added titles to display metadata to pass linting checks.
- Enabled blueprint metadata validation in the CI pipeline.
This change renames the 'project' input variable to 'project_id' across the root module, submodules, and all examples to align with ADC compliance standards.

An upgrade guide has been added to docs/upgrading_to_v8.0.md.

BREAKING CHANGE: The 'project' variable has been renamed to 'project_id'. Users of this module will need to update their module calls to use the new 'project_id' input when upgrading to version 8.0.0 or later.
@yuktatechgog yuktatechgog requested review from a team and imrannayer as code owners October 28, 2025 20:52
@yuktatechgog
Copy link
Contributor Author

yuktatechgog commented Oct 28, 2025

@bharathkkb @ayushmjain @q2w @imrannayer @krprabhat-eng

Requesting to review this PR.

@bharathkkb bharathkkb changed the title Adc compliance feat!: rename project input variable to project_id for consistency Oct 29, 2025
@bharathkkb
Copy link
Member

/gcbrun

@yuktatechgog yuktatechgog changed the title feat!: rename project input variable to project_id for consistency feat!: rename project input variable to project_id for consistency Oct 29, 2025
@bharathkkb
Copy link
Member

/gcbrun

@ayushmjain ayushmjain merged commit 8862c88 into terraform-google-modules:main Oct 29, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants