Update aks-pod-identity-assignment.tf

[nashgongchan]

Remove Boolean value

Issue-id

PR Checklist

---

• I have updated the documentation accordingly.
• I have added tests to cover my changes.
• All new and existing tests passed.
• My code follows the code style of this project.
• I ran lint checks locally prior to submission.
• Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Description

Does this introduce a breaking change

• YES
• NO

Testing

[LaurentLesle]

LGTM - clearer. thanks

[hieumoscow]

Hi @nashgongchan, I made the change before (It used to be like your code) because var.vnets[var.aks_cluster_vnet_key].subnet_keys was a list.
Thus if you supply the above value then we would have the below error.
Reverting this for now as this is breaking one of my customer.

 line 29, in resource "azurerm_role_assignment" "kubelet_subnets_networkcontrib":
│   29:   for_each = lookup(var.vnets[var.aks_cluster_vnet_key], "subnet_keys", { vnet = true })
│     ├────────────────
│     │ var.aks_cluster_vnet_key is "vnet_gitops"
│     │ var.vnets is object with 1 attribute "vnet_gitops"
│ 
│ The given "for_each" argument value is unsuitable: the "for_each" argument
│ must be a map, or set of strings, and you have provided a value of type
│ tuple.
╵