ISSUE Fixed: azurerm_virtual_desktop_application_group name allowed length is 3-31, should be 1-260 #8980

[ritesh-modi]

Acceptance tests results

TF_PROVIDER_SPLIT_COMBINED_TESTS=1 TF_ACC=1 go test -v ./azurerm/internal/services/containers/tests/ -timeout=60m -run=TestAccAzureRMKubernetesCluster_sameSize
TF_ACC=1 go test -v ./azurerm/internal/services/desktopvirtualization -timeout=60m -run=TestAccAzureRMVirtualDesktopApplicationGroup_basic
=== RUN TestAccAzureRMVirtualDesktopApplicationGroup_basic
=== PAUSE TestAccAzureRMVirtualDesktopApplicationGroup_basic
=== CONT TestAccAzureRMVirtualDesktopApplicationGroup_basic
--- PASS: TestAccAzureRMVirtualDesktopApplicationGroup_basic (185.70s)
PASS
ok github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/services/desktopvirtualization 185.749s

TF_ACC=1 go test -v ./azurerm/internal/services/desktopvirtualization -timeout=60m -run=TestAccAzureRMVirtualDesktopApplicationGroup_complete
=== RUN TestAccAzureRMVirtualDesktopApplicationGroup_complete
=== PAUSE TestAccAzureRMVirtualDesktopApplicationGroup_complete
=== CONT TestAccAzureRMVirtualDesktopApplicationGroup_complete
--- PASS: TestAccAzureRMVirtualDesktopApplicationGroup_complete (177.54s)
PASS
ok github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/services/desktopvirtualization 177.591s

TF_ACC=1 go test -v ./azurerm/internal/services/desktopvirtualization -timeout=60m -run=TestAccAzureRMVirtualDesktopApplicationGroup_update

=== RUN TestAccAzureRMVirtualDesktopApplicationGroup_update
=== PAUSE TestAccAzureRMVirtualDesktopApplicationGroup_update
=== CONT TestAccAzureRMVirtualDesktopApplicationGroup_update
--- PASS: TestAccAzureRMVirtualDesktopApplicationGroup_update (314.00s)
PASS
ok github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/services/desktopvirtualization 314.068s

TF_ACC=1 go test -v ./azurerm/internal/services/desktopvirtualization -timeout=60m -run=TestAccAzureRMVirtualDesktopApplicationGroup_requiresImport
=== RUN TestAccAzureRMVirtualDesktopApplicationGroup_requiresImport
=== PAUSE TestAccAzureRMVirtualDesktopApplicationGroup_requiresImport
=== CONT TestAccAzureRMVirtualDesktopApplicationGroup_requiresImport
--- PASS: TestAccAzureRMVirtualDesktopApplicationGroup_requiresImport (193.34s)
PASS
ok github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/services/desktopvirtualization 193.410s

(fixes #8980)

[mbfrahry]

Hey @ritesh-modi, this looks mostly good but it looks like a kubernetes commit snuck in. Do you mind removing that so this PR is only focused on azurerm_virtual_desktop_application_group?

[katbyte]

Thanks @ritesh-modi, LGTM 👍

[ghost]

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