hashicorp / terraform-provider-googleworkspace

Terraform Provider for Google Workspace
https://registry.terraform.io/providers/hashicorp/googleworkspace
Mozilla Public License 2.0
129 stars 56 forks source link

`include_derived_membership` option in `data.googleworkspace_group_members` causing issues #470

Open tcaddy opened 8 months ago

tcaddy commented 8 months ago

The include_derived_membership option in data.googleworkspace_group_members is causing issues.

It is not detecting newly added members to a group if include_derived_membership is set to true.

If I remove the include_derived_membership option, newly added members appear.

Terraform Version

v1.5.6

Affected Resource(s)

Please list the resources as a list, for example:

Terraform Configuration Files

data "googleworkspace_group_members" "good" {
  group_id = "some_group"
}

data "googleworkspace_group_members" "bad" {
  group_id = "some_group"
  include_derived_membership = true
}

output "good" {
  value = data.googleworkspace_group_members.good.members # includes newly added members
}

output "bad" {
  value = data.googleworkspace_group_members.bad.members # does NOT include newly added members
}

Debug Output

n/a

Panic Output

n/a

Expected Behavior

Newly added members should show up in https://registry.terraform.io/providers/hashicorp/googleworkspace/latest/docs/resources/group_members#members regardless of whether include_derived_membership is enabled or not.

Actual Behavior

Newly added members only show up in https://registry.terraform.io/providers/hashicorp/googleworkspace/latest/docs/resources/group_members#members if include_derived_membership is not enabled.

Steps to Reproduce

  1. get an existing workspace setup for existing group members
  2. set the include_derived_membership to true
  3. make sure you've plan and applied and things look green and that you have a state file
  4. add a new member to the group with "ClickOps"
  5. run a TF plan and see no changes (this violates expectations, this is the problem right here)
  6. remove the option for include_derived_membership
  7. run a TF plan and see the expected changes

Important Factoids

n/a

References

n/a