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

data azuredevops_group export attribute ID field having group descriptor value #1077

Open
Sharmanand opened this issue Jun 17, 2024 · 5 comments

Comments

@Sharmanand
Copy link

Community Note

  • Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
  • Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
  • If you are interested in working on this issue or have submitted a pull request, please leave a comment

Terraform (and Azure DevOps Provider) Version

Affected Resource(s)

  • azuredevops_group

Terraform Configuration Files

# Copy-paste your Terraform configurations here - for large Terraform configs,
# please use a service like Dropbox and share a link to the ZIP file. For
# security, you can also encrypt the files using our GPG public key: https://keybase.io/hashicorp

data "azuredevops_project" "example" {
  name = "Example Project"
}

Debug Output

Panic Output

id - The ID for this resource is the group descriptor. See below.

Expected Behavior

it should export guid value of group id same as data source: azuredevops_groups

Actual Behavior

currently field returns id - The ID for this resource is the group descriptor

Steps to Reproduce

  1. terraform apply

Important Factoids

References

  • #0000
@xuzhang3
Copy link
Collaborator

@Sharmanand the ID in azuredevops_group is a randomly generated UUId, it has no meaning, it's just an ID. The descriptor can be changed to UUID but need another API call.
https://learn.microsoft.com/en-us/rest/api/azure/devops/graph/storage-keys/get?view=azure-devops-rest-7.1&tabs=HTTP

@Sharmanand
Copy link
Author

@xuzhang3 it do help in lots of use case e.g. azuredevops_branch_policy_auto_reviewers for assigning auto_reviewer_ids

moreover if you check Data Source: azuredevops_groups it have correct export id field. however you check Data Source: azuredevops_group it has duplicate value in both id & descriptor descriptor.

we need to have consistency in Data Source: azuredevops_groups & Data Source: azuredevops_group for id field

@xuzhang3
Copy link
Collaborator

Adding the group ID will add a new property group_id to reduce the impact to existing resources, force changing the ID values will cause the resource recreated.

@Sharmanand
Copy link
Author

hi @xuzhang3, is this new field available in upcoming release?

@xuzhang3
Copy link
Collaborator

@Sharmanand yes, a new property will be added.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants