azuread_service_principal: Add Tag Property

[midacts]

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

Description

azurerm_azuread_service_principal: Add option for tag

New or Affected Resource(s)

• azurerm_azuread_service_principal

Potential Terraform Configuration

resource "azurerm_azuread_service_principal" "test" {
  application_id = "${azurerm_azuread_application.test.application_id}"
  principal_tags = {
    WindowsAzureActiveDirectoryIntegratedApp = true
  }
}

References

• New-AzureADServicePrincipal -AppId $ADApplication.AppID -Tags @("WindowsAzureActiveDirectoryIntegratedApp")
• https://docs.microsoft.com/en-us/powershell/module/azuread/new-azureadserviceprincipal?view=azureadps-2.0
• There seems to be quite a bit missing from the AzureAD side of things. I haven't gotten around to trying to do them all through Terraform yet. : [

[tombuildsstuff]

hi @midacts

Thanks for opening this issue :)

Given there's quite a few of them and they're different to the other resources we're looking to split the Azure AD resources out of this Provider and into their own azuread provider - more information can be found in #2322 - and as such rather than implementing this in the existing resources in this provider I think it'd make sense to implement support for in the new provider; as such I'm going to block this on #2322 for the moment.

Thanks!

[midacts]

Thanks for the reply.
I think it is a great idea. Thanks for the link. : )

[katbyte]

Hi @midacts,

As in 2.0 we are deprecating all Azure AD resources and data sources in the Azure RM provider in favour of this new provider I have moved the issue here.

[midacts]

Thanks for the heads up and for getting this issue transferred over to the terraform-provider-azuread repo.

[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. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 [email protected]. Thanks!