azurerm_kusto_cluster name check is not correct

[Shadikho]

Is there an existing issue for this?

• I have searched the existing issues

Community Note

• Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
• Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
• If you are interested in working on this issue or have submitted a pull request, please leave a comment and review the contribution guide to help.

Terraform Version

1.5.5

AzureRM Provider Version

3.77.0

Affected Resource(s)/Data Source(s)

azurerm_kusto_cluster

Terraform Configuration Files

import {
  id = "id of the existing resource with "-" inside its name"
  to = azurerm_kusto_cluster.kusto_cluster
}

Debug Output/Panic Output

- Error: "name" must begin with a letter and may only contain alphanumeric characters: "project-nonprod-dx"

Expected Behaviour

The resource code is generated without an error

Actual Behaviour

The resource code is generated (when adding the -generate-config-out=file.tf in the plan command) but the command throws the following error:

Error: "name" must begin with a letter and may only contain alphanumeric characters: "project-nonprod-dx"

Steps to Reproduce

1. Create an Azure data Explorer inside Azure Portal and use hyphen inside its name, like "test-kusto-2"
2. Try to import this already existing resource to terraform using import block and generating the code.
3. The error will happen

Important Factoids

No response

References

No response

[Shadikho]

Hello all,

isn't there anyone who would like to comment on this issue or pay attention to it at all?! I would highly appreciate any information on whether there is a plan to fix this or not :)

[ezeeb]

I have the same problem. The fix should be quite simple:
the regex in https://github.com/liuwuliuyun/terraform-provider-azurerm/blob/ca0d186d65af0de9291b161fb961854a038f3eb9/internal/services/kusto/validate/name.go#L50 simply needs an additional "-"

[Dirrbick]

I had this same issue. Instead of waiting for them to notice, I went ahead and took it upon myself to make this change since it is open. I currently have a PR in place to fix this. Now to wait on the change to be approved. Here is the link to the PR for reference.

[tombuildsstuff]

Fixed via #23790 - thanks @Dirrbick :)

[Shadikho]

Thanks @Dirrbick.

[github-actions]

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 have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.