terraform-azurerm-caf-enterprise-scale icon indicating copy to clipboard operation
terraform-azurerm-caf-enterprise-scale copied to clipboard

User Assigned identity for Policy

Open shaneholder opened this issue 2 years ago • 5 comments

Has https://github.com/Azure/terraform-azurerm-caf-enterprise-scale/issues/304 been addressed? I ran into this issue today and I've not seen an update that appears to address it. I did note that the version of azurerm being used 3.18.0 does now support user assigned identity.

Thanks, Shane

shaneholder avatar Oct 31 '22 19:10 shaneholder

Not yet, but we are currently prioritizing our backlog so I will attach this to the related user story as customer evidence to help prioritize this 👍🏻

krowlandson avatar Nov 02 '22 09:11 krowlandson

Coming back to this hoping for an update, but re-thinking my position on this. I perceive a need for this change because it seems that each policy that I apply that has remediation is being assigned to the contributor RBAC role and it just seems to pollute the RBAC assignment page. My goal for this request is to have as few identities as possible and that most remediations would use a single principal. Is that unreasonable?

shaneholder avatar Jan 17 '23 20:01 shaneholder

Linking to AB#25191

However this is not our recommendation due least priv

matt-FFFFFF avatar Mar 03 '23 15:03 matt-FFFFFF

Understood on the least privileged comment, however when you have several policies that are doing the same thing it potentially makes sense. For example policies for tag inheritance like as illustrated: https://github.com/Azure/azure-policy/blob/master/samples/Tags/inherit-resourcegroup-tag-if-missing/azurepolicy.json

If there was a way to have a single policy that trickled all necessary tags from a subscription->RG->Resource then I would have less need of using a user defined identity. But I've yet to find a way to do that.

shaneholder avatar Mar 21 '23 20:03 shaneholder

This is something we will put in the backlog for the next release.

matt-FFFFFF avatar Mar 22 '23 19:03 matt-FFFFFF