Adding Import support to Search Service

[tombuildsstuff]

Adding Import support to azurerm_search_service

Tests pass:

$ TF_ACC=1 envchain azurerm go test ./azurerm -v -timeout 120m -run TestAccAzureRMSearchService
=== RUN   TestAccAzureRMSearchService_importBasic
--- PASS: TestAccAzureRMSearchService_importBasic (88.71s)
=== RUN   TestAccAzureRMSearchService_basic
--- PASS: TestAccAzureRMSearchService_basic (310.06s)
=== RUN   TestAccAzureRMSearchService_updateReplicaCountAndTags

--- PASS: TestAccAzureRMSearchService_updateReplicaCountAndTags (1997.81s)
PASS
ok  	github.com/terraform-providers/terraform-provider-azurerm/azurerm	2396.616s

[radeksimko]

This looks functionally good to me, but I'm not sure the %+v is a good idea.

I understand the motivation is probably to get all the details about the error (including type, code etc.), so we can then easily catch it if necessary, but I think this is developer's concern, not user's.
User should see a user-friendly error, which is hopefully what %s provides - i.e. the String() or default string representation of that error is user-friendly.

That said I'm not familiar with error handling conventions in Azure, so I might not know the full background.

[radeksimko]

I guess the intention here was to put back ticks just around id?

[tombuildsstuff]

We can do, but we should update all of them to match this, as this was C&P'd from an existing page for consistency (example)

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