dns_zone: Compute resourceId from the parser, add state migration, switch dns package to generated IDs

[favoretti]

Fixes #10778

[favoretti]

Complete output is not included, due to DEBUG logging, but complete suite passed.

PASS
ok  	github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/services/dns	687.482s

[tombuildsstuff]

hey @favoretti

Thanks for pushing those changes - taking a look through on the whole this looks pretty good, but we should be able to remove the redundant Get call since we're not using that result as we're creating the ID's here - but if we can fix those up this should otherwise be good to merge 👍

Thanks!

[favoretti]

hey @favoretti

Thanks for pushing those changes - taking a look through on the whole this looks pretty good, but we should be able to remove the redundant Get call since we're not using that result as we're creating the ID's here - but if we can fix those up this should otherwise be good to merge 👍

Thanks!

Ah, I thought that one served as additional creation verification :)

[tombuildsstuff]

LGTM 👍

[ghost]

This has been released in version 2.50.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.50.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!