Skip to content

Commit 4db2ada

Browse files
Support connectivity fields for Composer 3 (#9889) (#1993)
* add composer_network_attachment * indicate conflicting configs * commas * no need for bidirectional conflict definition (generates double errors) * protect nit PrivateClusterConfig * for optimizing error messages about conflicts * add 2 step update for composer_network_attachment * make composer_network_attachment available in beta only * add two step update for network and subnetwork * corrections in 2 phase update for network/subnetwork * remove composer3 check(CustomizeDiff will solve this), filter api error, add tests (unsetting netwok/subnetwork not working) * added ForceNewIf fot network/subnetwork, problem with unsetting these fields remains * add docs for composer_network_attachment * add test for network attachment * ignore non empty plan in network attachment test * add networkAttachment update and conflicting fields tests * add ComputedIf for network, change isComposer3 * minor corrections * remove computedIf * filter equivalent values of network/subnetwork in ForceNewIf * simplify ResourceConditionFunc, add beta/ga version conditions * typo * more general comparison of network references * use tpgresource.CompareSelfLinkRelativePaths instead of custom function * modify isComposer3 to avoid merge conflicts later. * removing this since documentation is handled in other PR and to avoid conflicts while merging. * replace ExpectNonEmptyPlan with lifecycle.ignore_changes * add testcase for changing network attachment to network and subnetwork * add third step to TestAccComposerEnvironmentComposer3_updateWithNetworkAndSubnetwork * modify tests to use different network for attachment * remove unused constant * remove ExpectNonEmptyPlan (already replaced with lifecycle.ignore_changes) [upstream:76474ecb686e095b4436de3ce17d739a2a1a4b21] Signed-off-by: Modular Magician <[email protected]>
1 parent 8f5a507 commit 4db2ada

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

go.mod

+1-1
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ require (
1111
github.com/hashicorp/hcl/v2 v2.19.1
1212
github.com/hashicorp/terraform-json v0.18.0
1313
github.com/hashicorp/terraform-plugin-sdk/v2 v2.31.0
14-
github.com/hashicorp/terraform-provider-google-beta v1.20.1-0.20240226204949-f415c35a81aa
14+
github.com/hashicorp/terraform-provider-google-beta v1.20.1-0.20240226215813-dbe862d7b326
1515
github.com/mitchellh/go-homedir v1.1.0 // indirect
1616
github.com/pkg/errors v0.9.1
1717
github.com/stretchr/testify v1.8.4

go.sum

+2-2
Original file line numberDiff line numberDiff line change
@@ -175,8 +175,8 @@ github.com/hashicorp/terraform-plugin-log v0.9.0/go.mod h1:rKL8egZQ/eXSyDqzLUuwU
175175
github.com/hashicorp/terraform-plugin-mux v0.13.0 h1:79U401/3nd8CWwDGtTHc8F3miSCAS9XGtVarxSTDgwA=
176176
github.com/hashicorp/terraform-plugin-sdk/v2 v2.31.0 h1:Bl3e2ei2j/Z3Hc2HIS15Gal2KMKyLAZ2om1HCEvK6es=
177177
github.com/hashicorp/terraform-plugin-sdk/v2 v2.31.0/go.mod h1:i2C41tszDjiWfziPQDL5R/f3Zp0gahXe5No/MIO9rCE=
178-
github.com/hashicorp/terraform-provider-google-beta v1.20.1-0.20240226204949-f415c35a81aa h1:PbM4KpYv8srLhv9rVrVWDTOXBztI7ro4y+jlDTQlO4Q=
179-
github.com/hashicorp/terraform-provider-google-beta v1.20.1-0.20240226204949-f415c35a81aa/go.mod h1:j4utSRUvX6e1CkTVw1EzJIuR2uKXjefCULxJRwDF+08=
178+
github.com/hashicorp/terraform-provider-google-beta v1.20.1-0.20240226215813-dbe862d7b326 h1:5pEq+zGJRvpUlsr0RZ8wMu5pqWkORrdC0cqU3mZMr44=
179+
github.com/hashicorp/terraform-provider-google-beta v1.20.1-0.20240226215813-dbe862d7b326/go.mod h1:j4utSRUvX6e1CkTVw1EzJIuR2uKXjefCULxJRwDF+08=
180180
github.com/hashicorp/terraform-registry-address v0.2.3 h1:2TAiKJ1A3MAkZlH1YI/aTVcLZRu7JseiXNRHbOAyoTI=
181181
github.com/hashicorp/terraform-registry-address v0.2.3/go.mod h1:lFHA76T8jfQteVfT7caREqguFrW3c4MFSPhZB7HHgUM=
182182
github.com/hashicorp/terraform-svchost v0.1.1 h1:EZZimZ1GxdqFRinZ1tpJwVxxt49xc/S52uzrw4x0jKQ=

0 commit comments

Comments
 (0)