azurerm_app_service_managed_certificate - create certificate in service plan resource group

[AdamCoulterOz]

Fixes #9699

FYI - @jackofallops

[jackofallops]

Thanks for the catch @AdamCoulterOz - one minor change requested, but otherwise will be good to merge.

[AdamCoulterOz]

@jackofallops just a reminder this small fix is still waiting... we need this to make 2.40 also in order to make use of all the other work. I already addressed your previous feedback.

[jackofallops]

Thanks @AdamCoulterOz - LGTM now 👍

[jackofallops]

Tests passing:

=== RUN   TestAccAzureRMAppServiceManagedCertificate_basicLinux
=== PAUSE TestAccAzureRMAppServiceManagedCertificate_basicLinux
=== CONT  TestAccAzureRMAppServiceManagedCertificate_basicLinux
--- PASS: TestAccAzureRMAppServiceManagedCertificate_basicLinux (225.23s)
=== RUN   TestAccAzureRMAppServiceManagedCertificate_requiresImport
=== PAUSE TestAccAzureRMAppServiceManagedCertificate_requiresImport
=== CONT  TestAccAzureRMAppServiceManagedCertificate_requiresImport
--- PASS: TestAccAzureRMAppServiceManagedCertificate_requiresImport (229.70s)
=== RUN   TestAccAzureRMAppServiceManagedCertificate_basicWindows
=== PAUSE TestAccAzureRMAppServiceManagedCertificate_basicWindows
=== CONT  TestAccAzureRMAppServiceManagedCertificate_basicWindows
--- PASS: TestAccAzureRMAppServiceManagedCertificate_basicWindows (220.59s)
PASS

[ghost]

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

provider "azurerm" {
    version = "~> 2.40.0"
}
# ... other configuration ...

[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!