set users for data_iam_group

[kl4w]

Community Note

• Please vote on this pull request by adding a 👍 reaction to the original pull request comment to help the community and maintainers prioritize this request
• Please do not leave "+1" comments, they generate extra noise for pull request followers and do not help prioritize the request

Closes #1154

Release note for CHANGELOG:

Add `users` property to iam_group data source.

Output from acceptance testing:

$ make testacc TEST=./aws TESTARGS='-run=TestAccAWSDataSourceIAMGroup_'
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./aws -v -count 1 -parallel 20 -run=TestAccAWSDataSourceIAMGroup_ -timeout 120m
go: finding golang.org/x/net latest
=== RUN   TestAccAWSDataSourceIAMGroup_basic
=== PAUSE TestAccAWSDataSourceIAMGroup_basic
=== RUN   TestAccAWSDataSourceIAMGroup_users
=== PAUSE TestAccAWSDataSourceIAMGroup_users
=== CONT  TestAccAWSDataSourceIAMGroup_basic
=== CONT  TestAccAWSDataSourceIAMGroup_users
--- PASS: TestAccAWSDataSourceIAMGroup_basic (15.43s)
--- PASS: TestAccAWSDataSourceIAMGroup_users (16.68s)
PASS
ok  	github.com/terraform-providers/terraform-provider-aws/aws	16.741s

[bflad]

Hi @kl4w 👋 Thank you for submitting this. It appears there were two separate feature request issues for this with the other being #7076 and its associated pull request #7132 which was recently merged and released in version 2.36.0 of the Terraform AWS Provider. Since this functionality seems to be covered by the earlier pull request, I'm going to close this one and close the earlier feature request as well. 👍

[ghost]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thanks!