r/kubernetes_cluster: updating the default for load_balancer_sku to Standard

[tombuildsstuff]

Matches the new default used by the Azure API

Fixes #4465

[jackofallops]

LGTM 👍

[jluk]

@tombuildsstuff apologies for getting back to this issue a bit later, the AKS API defaults haven't swapped yet due to some issues found in VMSS. This is still planned but delayed a few months.

Is this work changing the default Terraform will fall back to if left empty? If that is the case this is fine, but just want to be clear on API behaviors today.

[tombuildsstuff]

@jluk no worries - if this works planned at some point in the future we have a choice between making this change now or waiting for 3.0 (which is quite a way off - we try and do major versions yearly) - as such is there a downside to defaulting this to Standard early, or should we be flipping this back and waiting for 3.0?

[jluk]

I think we're good with this change now, thanks Tom.

[ghost]

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