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

azurerm_application_insights could not target application_type = "MobileCenter" #1815

Closed
guitarrapc opened this issue Aug 23, 2018 · 3 comments

Comments

@guitarrapc
Copy link

guitarrapc commented Aug 23, 2018

hi, I'm trying import existing application insight resource. I faced to when Application Insight is made for MobileCenter, application_type would not match with any value.

Step to reproduce

  1. Create Application insight from AppCenter.
  2. Import resource with Terraform.
  3. run terraform plan will get mis-match for "application_type".

terraform configuration

resource "azurerm_application_insights" "AppCenter" {
  name                = "AppCenter"
  location            = "westus2"
  resource_group_name = "resourceGroupName"
  application_type    = "other"
}

Actual result

application_type:    "MobileCenter" => "Other" (forces new resource)

Expected

If I can set MobileCenter as a application_type, issue will not happen.

resource "azurerm_application_insights" "AppCenter" {
  name                = "AppCenter"
  location            = "westus2"
  resource_group_name = "resourceGroupName"
  application_type    = "MobileCenter"
}

terraform plan will not detect change.

no diff

Question

Am I missing some feature to match application_type?

@tombuildsstuff tombuildsstuff added this to the 1.14.0 milestone Sep 6, 2018
tombuildsstuff added a commit that referenced this issue Sep 6, 2018
```
$ acctests azurerm TestAccAzureRMApplicationInsights_basicMobileCenter
=== RUN   TestAccAzureRMApplicationInsights_basicMobileCenter
--- PASS: TestAccAzureRMApplicationInsights_basicMobileCenter (76.28s)
PASS
ok  	github.com/terraform-providers/terraform-provider-azurerm/azurerm	76.616s
```

Fixes #1815
tombuildsstuff added a commit that referenced this issue Sep 6, 2018
…1878)

```
$ acctests azurerm TestAccAzureRMApplicationInsights_basicMobileCenter
=== RUN   TestAccAzureRMApplicationInsights_basicMobileCenter
--- PASS: TestAccAzureRMApplicationInsights_basicMobileCenter (76.28s)
PASS
ok  	github.com/terraform-providers/terraform-provider-azurerm/azurerm	76.616s
```

Fixes #1815
@tombuildsstuff
Copy link
Contributor

hey @guitarrapc

Just to let you know that this has been released in v1.14.0 of the AzureRM Provider which is now available: https://github.com/terraform-providers/terraform-provider-azurerm/blob/v1.14.0/CHANGELOG.md

Thanks!

@guitarrapc
Copy link
Author

Thank you so much!! I'll take a look and deploy to dev/staging!
Thanks!

@ghost
Copy link

ghost commented Mar 6, 2019

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!

@ghost ghost locked and limited conversation to collaborators Mar 6, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants