Update azurerm_eventhub_namespace with optional dedicated_cluster_id

[favoretti]

Update azurerm_eventhub_namespace with optional dedicated_cluster_id property.

[favoretti]

@tombuildsstuff @jackofallops here we go again :)
I would really appreciate a couple of pointers on how you want me to implement validation for limits in azurerm_eventhub depending on a property of azurerm_eventhub_namespace.

[tombuildsstuff]

hey @favoretti

Thanks for splitting this back out :)

I've taken a look through and left some comments inline, but this is mostly looking good so far. To reply to your question:

I would really appreciate a couple of pointers on how you want me to implement validation for limits in azurerm_eventhub depending on a property of azurerm_eventhub_namespace.

In this instance since these fields can both be set, albeit to different maximum values - unfortunately we don't support conditional validation yet so we'll be best to increase the maximum value of the validation to the maximum (e.g. where Dedicated is higher) and then add a yellow "information" box (~> **Note:** Lorem Ipsum..) to call that out in the documentation; potentially making a check inside the Create/Update function to check that those are within the expected bounds when not using a a Dedicated Cluster. Hopefully that makes sense?

Thanks!

[tombuildsstuff]

LGTM - thanks for pushing those changes @favoretti

[ghost]

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