Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The version field should be under configmanagement instead of under oci #6749

Conversation

modular-magician
Copy link
Collaborator

Fixes hashicorp/terraform-provider-google#16684

Release Note Template for Downstream PRs (will be copied)

gkehub: added field `version` under `configmanagement` instead of a child field `oci` in `google_gke_hub_feature` resource

Derived from GoogleCloudPlatform/magic-modules#9587

…ci (#9587)

* Adding Terraform resources for Tenancy APIs in GKEHub

* Segregating MembershipBinding and MembershipRBACRoleBinding to keep things simpler in the review

* Fixing the docu URIs

* Adding TF support for Tenancy API for Membership Binding

* Adding dependent membership binding to the same commit chain

* Making Scope un-updatable and replacing hard coded project number with the one from test env

* Making Scope RRBAC updatable

* Making Namespace immutable

* Adding update test cases

* Removing all memberships field from Scope since it is no longer supported

* Removing all_memberships field for Scope from all test cases

* Making naming in examples consistent across Tenancy APIs documentation

* Update mmv1/templates/terraform/examples/gkehub_membership_binding_basic.tf.erb

Co-authored-by: Shuya Ma <[email protected]>

* Update mmv1/templates/terraform/examples/gkehub_membership_binding_basic.tf.erb

Co-authored-by: Shuya Ma <[email protected]>

* Update mmv1/templates/terraform/examples/gkehub_membership_rbac_role_binding_basic.tf.erb

Co-authored-by: Shuya Ma <[email protected]>

* Fixing typo in the resource name

* Adding state migrations for Membership

* Updating the feature_membership documentation with the current resource state. Adding membership_location field to the doc

* Fixing the version field to be directly under configmanagement instead of under the oci field

* Adding tests for the field version

* Changing version fields value to test upgradation

---------

Co-authored-by: Shuya Ma <[email protected]>
[upstream:de7d39b50c5d9894e25770a457eb9ab2c2f96eaf]

Signed-off-by: Modular Magician <[email protected]>
@modular-magician modular-magician merged commit 16d58fc into hashicorp:main Dec 12, 2023
2 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.

The version field should be under configmanagement for google_gke_hub_feature resource
1 participant