Terraform-provider-azurerm: AKS cluster creation fails with no error message if VM size is incorrect

Created on 20 Jun 2018  ยท  4Comments  ยท  Source: terraform-providers/terraform-provider-azurerm

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

Terraform Version

Terraform v0.11.6
+ provider.azurerm v1.7.0

Affected Resource(s)

  • azurerm v1.7.0

Debug Output

https://gist.github.com/timwebster9/0b71442e8e04c6fcfc03f586b3df7aad

Expected Behavior

Terraform should have provided the actual error

Actual Behavior

* module.default.azurerm_kubernetes_cluster.aks_cluster: 1 error(s) occurred:

* azurerm_kubernetes_cluster.aks_cluster: containerservice.ManagedClustersClient#CreateOrUpdate: Failure sending request: StatusCode=400 -- Original Error: Code="" Message=""

Steps to Reproduce

You just need to provide an incorrect VM size value. I used D2s_v3 instead of Standard_D2s_v3 by mistake.

bug servickubernetes-cluster

Most helpful comment

๐Ÿ‘‹ We've upgraded to SDKv17 in #1418 - as such this should be fixed in the next release (and I'm going to close this for the moment)

All 4 comments

hey @timwebster9

Thanks for opening this issue :)

All communication with from Terraform to the Azure API's is handled by the Azure SDK for Go, in this case it appears the error is available in the HTTP Response but isn't being parsed out correctly - as such I've opened an issue on the Azure SDK for Go repository about this.

Thanks!

As mentioned in https://github.com/Azure/azure-sdk-for-go/issues/2085 it requires an upgrade of the sdk v16 -> v17), would be awesome if that will be picked up soon.

๐Ÿ‘‹ We've upgraded to SDKv17 in #1418 - as such this should be fixed in the next release (and I'm going to close this for the moment)

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!

Was this page helpful?
0 / 5 - 0 ratings