azurerm_eventhub_consumer_group - allow the name property to be set to $Default

[favoretti]

Fixes #7720

[favoretti]

@jackofallops would something like this work? Had to split validation for datasource and resource, to make sure people won't mangle $Default in resource definition.

[jackofallops]

Hi @favoretti
Thanks for this PR, I've had a quick look over and left one comment to look at initially since the new validation method regex would inadvertently allow invalid strings and not match $Default as intended.

Thanks!

[neil-yechenwei]

Address #7720

Please update the comment from "Address #7720" to "fixes #7720"

[favoretti]

@neil-yechenwei @jackofallops Feedback addressed.

[katbyte]

Thanks @favoretti - LGTM now! 🚀

[ghost]

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