azurerm_healthcare_service: support for customer managed key

[b4rd]

This PR adds CMK support for azurerm_healthcare_service.

Reference document: https://docs.microsoft.com/en-in/azure/healthcare-apis/fhir/customer-managed-key

Fixes: #10632

Test Results

TF_ACC=1 go test -v ./azurerm/internal/services/healthcare  -timeout 60m -ldflags="-X=github.com/terraform-providers/terraform-provider-azurerm/version.ProviderVersion=acc"
2021/04/27 12:58:43 [DEBUG] not using binary driver name, it's no longer needed
2021/04/27 12:58:43 [DEBUG] not using binary driver name, it's no longer needed
=== RUN   TestAccHealthCareServiceDataSource_basic
=== PAUSE TestAccHealthCareServiceDataSource_basic
=== RUN   TestAccHealthCareServiceDataSource_complete
=== PAUSE TestAccHealthCareServiceDataSource_complete
=== RUN   TestAccHealthCareService_basic
=== PAUSE TestAccHealthCareService_basic
=== RUN   TestAccHealthCareService_requiresImport
=== PAUSE TestAccHealthCareService_requiresImport
=== RUN   TestAccHealthCareService_complete
=== PAUSE TestAccHealthCareService_complete
=== CONT  TestAccHealthCareServiceDataSource_basic
=== CONT  TestAccHealthCareService_requiresImport
=== CONT  TestAccHealthCareService_complete
=== CONT  TestAccHealthCareService_basic
=== CONT  TestAccHealthCareServiceDataSource_complete
--- PASS: TestAccHealthCareServiceDataSource_basic (1112.86s)
--- PASS: TestAccHealthCareService_basic (1115.46s)
--- PASS: TestAccHealthCareService_requiresImport (1189.59s)
--- PASS: TestAccHealthCareServiceDataSource_complete (1219.64s)
--- PASS: TestAccHealthCareService_complete (1223.75s)
PASS
ok      github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/services/healthcare  1224.166s

[manicminer]

Hi @b4rd, thanks for the PR! This is off to a great start, the main issue with the proposed implementation is that it removes an existing property which is considered a breaking change. Whilst we can look to adjust the schema in such a way in a major version bump, such changes can't be included in a regular (minor) release. I've made a suggestion inline, if you can restore the original property and add the new one alongside it then I'll be happy to review again.

[manicminer]

@b4rd Thanks for making the changes! This is looking great, I have made a couple of suggestions inline below and committed them. Pending test results, this should be good to merge 👍

[manicminer]

Test results

[ghost]

This has been released in version 2.57.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.57.0"
}
# ... other configuration ...

[github-actions]

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.