new resource 'azurerm_private_dns_srv_record'

[phires]

New Resource 'azurerm_private_dns_srv_record' for SRV DNS records in Private DNS Zones.
Fixes issue #4739

[katbyte]

Thanks for the PR @phires,

Over all this looks great! i've left a couple minor comments inline that need to be addressed before merge.

[tombuildsstuff]

hey @phires

Thanks for pushing those changes - aside from a couple of minor issues in the documentation this otherwise LGTM 👍

Thanks!

[tombuildsstuff]

hey @phires

Thanks for pushing those changes - taking a look through this now LGTM 👍

Thanks!

dismissing since changes have been pushed

[tombuildsstuff]

running the tests it appears due to the API bug outlined below that the name is also lower-cased in the API response. Since DNS addresses themselves need to be lower-cased - I hope you don't mind but I'm going to push a commit to add some validation to this resource (and the other Private DNS resources) to ensure that the name field is lower-cased

[tombuildsstuff]

With the validation/test changes the tests now pass:

[ghost]

This has been released in version 1.37.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 = "~> 1.37.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 👉 [email protected]. Thanks!