Use NetworkingType instead of EnablePrivateEnvironment in composer provider#16742
Conversation
46a7ffb to
e55a387
Compare
|
Googlers: For automatic test runs see go/terraform-auto-test-runs. @ScottSuarez, a repository maintainer, has been assigned to review your changes. If you have not received review feedback within 2 business days, please leave a comment on this PR asking them to take a look. You can help make sure that review is quick by doing a self-review and by running impacted tests locally. |
|
@HappyMary16 , can you open a PR only with the api version changes? Thanks. |
|
I moved the api version upgrade to a separate PR #16752 |
ScottSuarez
left a comment
There was a problem hiding this comment.
waiting for api upgrade to be merged
…Environment-by-networkingType-field
|
PR #16752 with API version upgrade is merged, I merged changed from main to my branch |
|
Hi there, I'm the Modular magician. I've detected the following information about your changes: Diff reportYour PR generated some diffs in downstreams - here they are.
|
Tests analyticsTotal tests: 40 Click here to see the affected service packages
Action takenFound 24 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
|
|
🟢 Tests passed during RECORDING mode: 🟢 No issues found for passed tests after REPLAYING rerun. 🔴 Tests failed during RECORDING mode: 🔴 Errors occurred during RECORDING mode. Please fix them to complete your PR. |
|
/gcbrun |
The providers implementation use the new networking_type field in Composer API instead of the old enable_private_environment boolean field.
PR do not introduce any change in the user-visible Terraform surface. The enable_private_environment boolean field in Composer Terraform resource remains the only interface for selecting private/public environment in Terraform.
Fixes: hashicorp/terraform-provider-google#26395
Release Note Template for Downstream PRs (will be copied)
See Write release notes for guidance.