Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert "Update aks-pod-identity-assignment.tf" #249

Merged
merged 1 commit into from
Aug 26, 2021
Merged

Conversation

hieumoscow
Copy link
Contributor

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.

@hieumoscow hieumoscow self-assigned this Aug 26, 2021
@arnaudlh arnaudlh changed the base branch from patch.5.4.2 to master August 26, 2021 10:11
@arnaudlh arnaudlh added the bug Something isn't working label Aug 26, 2021
@arnaudlh arnaudlh added this to the 2108 milestone Aug 26, 2021
@arnaudlh arnaudlh merged commit 8995322 into master Aug 26, 2021
@arnaudlh arnaudlh deleted the revert-246-patch-1 branch August 26, 2021 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants