azurerm_kubernetes_cluster: Add ingress_application_gateway_identity export for add-on ingress_application_gateway

[aristosvo]

Fixes #11564

Acceptance tests

• New functionality tested and passing

  ❯ make acctests SERVICE='containers' TESTARGS='-run=TestAccKubernetesCluster_addonProfileIngressApplicationGateway_appGatewayId'
  ==> Checking that code complies with gofmt requirements...
  ==> Checking that Custom Timeouts are used...
  ==> Checking that acceptance test packages are used...
  TF_ACC=1 go test -v ./azurerm/internal/services/containers -run=TestAccKubernetesCluster_addonProfileIngressApplicationGateway_appGatewayId -timeout 180m -ldflags="-X=github.com/terraform-providers/terraform-provider-azurerm/version.ProviderVersion=acc"
  2021/05/07 18:15:43 [DEBUG] not using binary driver name, it's no longer needed
  2021/05/07 18:15:44 [DEBUG] not using binary driver name, it's no longer needed
  === RUN   TestAccKubernetesCluster_addonProfileIngressApplicationGateway_appGatewayId
  === PAUSE TestAccKubernetesCluster_addonProfileIngressApplicationGateway_appGatewayId
  === CONT  TestAccKubernetesCluster_addonProfileIngressApplicationGateway_appGatewayId
  --- PASS: TestAccKubernetesCluster_addonProfileIngressApplicationGateway_appGatewayId (1302.15s)
  PASS
  ok      github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/services/containers    1306.024s
  

  ❯ make acctests SERVICE='containers' TESTARGS='-run=TestAccDataSourceKubernetesCluster_addOnProfileIngressApplicationGatewayAppGateway'
  ==> Checking that code complies with gofmt requirements...
  ==> Checking that Custom Timeouts are used...
  ==> Checking that acceptance test packages are used...
  TF_ACC=1 go test -v ./azurerm/internal/services/containers -run=TestAccDataSourceKubernetesCluster_addOnProfileIngressApplicationGatewayAppGateway -timeout 180m -ldflags="-X=github.com/terraform-providers/terraform-provider-azurerm/version.ProviderVersion=acc"
  2021/05/07 20:59:29 [DEBUG] not using binary driver name, it's no longer needed
  2021/05/07 20:59:30 [DEBUG] not using binary driver name, it's no longer needed
  === RUN   TestAccDataSourceKubernetesCluster_addOnProfileIngressApplicationGatewayAppGateway
  === PAUSE TestAccDataSourceKubernetesCluster_addOnProfileIngressApplicationGatewayAppGateway
  === CONT  TestAccDataSourceKubernetesCluster_addOnProfileIngressApplicationGatewayAppGateway
  --- PASS: TestAccDataSourceKubernetesCluster_addOnProfileIngressApplicationGatewayAppGateway (1341.64s)
  PASS
  ok      github.com/terraform-providers/terraform-provider-azurerm/azurerm/internal/services/containers  1345.299s
  

[mbfrahry]

Hey @aristosvo! This looks mostly good minus a couple things I called out below! Thanks for taking the time to get this PR out

[aristosvo]

Thanks @mbfrahry! Sorry to bother you with correcting my Friday night code 😅

I think this should do!

[mbfrahry]

No worries at all and LGTM! Thanks for taking care of that!

[ghost]

This has been released in version 2.59.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.59.0"
}
# ... other configuration ...

[github-actions]

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.