[FEAT] Remove use of derecated api_server_access_profile. vnet_integration_enabled

[mloskot]

Community Note

• Please vote on this issue by adding a :thumbsup: 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

Warning: Argument is deprecated
│
│   with module.stack_aks.module.aks.azurerm_kubernetes_cluster.aks,
│   on .terraform/modules/stack_aks.aks/r-aks.tf line 28, in resource "azurerm_kubernetes_cluster" "aks":
│   28:     vnet_integration_enabled = var.vnet_integration.enabled
│
│ This property is not available in the stable API and will be removed in v4.0 of the Azure Provider. Please see https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/guides/4.0-upgrade-guide#aks-migration-to-stable-api for
│ more details

That is due to the following features being deprecated:

api_server_access_profile.subnet_id is deprecated and removed in 4.0 api_server_access_profile. vnet_integration_enabled is deprecated and removed in 4.0

To be honest, I'm not quite sure what the following should be replaced with, if anything:

https://github.com/claranet/terraform-azurerm-aks-light/blob/03dfee7daa17218fa3325a3dc56e2ce30e90ade1/r-aks.tf#L26-L30

New or Affected Resource(s)/Data Source(s)

azurerm_kubernetes_cluster

Potential Terraform Configuration

No response

References

• https://github.com/hashicorp/terraform-provider-azurerm/pull/26863

[Shr3ps]

And now v4 is out, we will have lots of work to rework all our modules 😓 yeah

[mloskot]

As a user of your modules, I'll appreciate that and I'll also try to contribute to the overhaul.