Azure / terraform-azurerm-avm-res-desktopvirtualization-hostpool

AVM for Desktop Virtualization Hostpools
https://registry.terraform.io/modules/Azure/avm-res-desktopvirtualization-hostpool/azurerm/latest
MIT License
5 stars 3 forks source link

[AVM Module Issue]: Lowercase Letters Validation - Host Pool Name #67

Closed birdnathan closed 1 month ago

birdnathan commented 1 month ago

Check for previous/existing GitHub issues

Issue Type?

Bug

(Optional) Module Version

0.1.5

(Optional) Correlation Id

No response

Description

Validation on virtual_desktop_host_pool_name variable needs amended. Lowercase letters only should not be a limitation for hostPools - https://learn.microsoft.com/en-us/azure/azure-resource-manager/management/resource-name-rules#microsoftdesktopvirtualization

`variable "virtual_desktop_host_pool_name" { type = string description = "(Required) The name of the Virtual Desktop Host Pool. Changing this forces a new resource to be created." nullable = false

validation { _condition = can(regex("^[a-z0-9-]{3,64}$", var.virtual_desktop_host_poolname)) error_message = "The name must be between 3 and 64 characters long and can only contain lowercase letters, numbers and dashes." } }`