Validate arguments before applying them

[antonbabenko]

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

Description

Validate obvious arguments before calling SDK. It is probably better to validate on both sides (SDK and Terraform provider).

This is an example of a failure which took 2 minutes:

* azurerm_kubernetes_cluster.test: Long running operation terminated with status 'Failed': Code="InvalidParameter" Message="The Admin Username specified is not allowed."

New or Affected Resource(s)

• azurerm_kubernetes_cluster

Potential Terraform Configuration

resource "azurerm_kubernetes_cluster" "test" {
  linux_profile {
    admin_username = "user"
    # ...
  }
}

[metacpp]

@antonbabenko Thanks for opening this issue.

I'd like to share some updates here:

1. user is acceptable format by admin_username now.
2. Working on PR to add more validations for K8s fields.

[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!