The default owner of azuread_service_principal is not the owner who runs terraform apply in v1.1.0

[moonape1226]

Hi, I encounter some default owner problem in azuread_service_principal today.

Terraform (and AzureAD Provider) Version

Terraform v0.13.5
+ provider registry.terraform.io/hashicorp/azuread v1.1.0
+ provider registry.terraform.io/hashicorp/azurerm v2.36.0

Affected Resource(s)

azuread_application
azuread_service_principal

Terraform Configuration Files

resource "azuread_application" "test-application" {
  name = "test-application"
}

resource "azuread_service_principal" "test-sp" {
  application_id = azuread_application.test-application.application_id
}

Debug Output

Error: patching Application with ID "xxx": graphrbac.ApplicationsClient#Patch: Failure responding to request: StatusCode=403 -- Original Error: autorest/azure: Service returned an error. Status=403 Code="Unknown" Message="Unknown service error" Details=[{"odata.error":{"code":"Authorization_RequestDenied","date":"2020-11-26T03:56:19","message":{"lang":"en","value":"Insufficient privileges to complete the operation."},"requestId":"xxx"}}]

Expected Behavior

In v1.0.0, the application registry created by terraform is owned by the user that ran terraform apply by default.

Actual Behavior

However, in v1.1.0, the application registry created by terraform is no longer owned by the user that ran terraform apply. So I am not able to access the application registry I created in other resources.

Steps to Reproduce

1. terraform apply

[manicminer]

Hi @moonape1226, thanks for reporting. We've done a lot of testing with application owners (along with some other complex properties) and agree that the default owner assigned by Azure AD should be retained. We'll be issuing a fix for this shortly.

[ghost]

This has been released in version 1.1.1 of the provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. As an example:

provider "azuread" {
    version = "~> 1.1.1"
}
# ... other configuration ...

[moonape1226]

Thanks for the prompt reply and update!

[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 👉 hashibot-feedback@hashicorp.com. Thanks!