Azure / enterprise-azure-policy-as-code

Enterprise-ready Azure Policy-as-Code (PaC) solution (includes Az DevOps pipeline)
https://azure.github.io/enterprise-azure-policy-as-code/
MIT License
422 stars 231 forks source link

Policy Exemptions handling - Enhancement request #620

Closed arrerezai closed 2 months ago

arrerezai commented 5 months ago

Describe the solution you'd like

anwather commented 4 months ago

@arrerezai for the exemption are you running with desired state strategy set to full?

anwather commented 4 months ago

For point 2 - desired state strategy set to full - EPAC is not deleting exemptions and I can reproduce - @apybar - I'll create a new issue from this and fix.

anwather commented 4 months ago

@apybar - fixed point 2 in #636

apybar commented 3 months ago

@arrerezai - what still from the list above is persisting? Or am I good to close this issue?

arrerezai commented 3 months ago

@apybar, while #1 and #2 were the most important ones and are solved, #3 and #4 are related to each other and are still to be overseen. Having the displayName merged with subId and assId is still a thing, which I think is totally unnecessary and like earlier mentioned, limits the freedom of having an own (more) meaningful name. Also, there should exist some logic to strip all parameters that exceed the total number of allowed chars imo, but I guess you need to have your say in that?

apybar commented 2 months ago

@arrerezai - I updated the documentation for point #4 and will push that out soon. Point #3 I will discuss internally and respond asap.

apybar commented 2 months ago

As for the merging of "displayName", this applies when using "PolicyDefintionID" and "Policy DisplayName". This is needed to distinguish the difference between the two when creating the exemption. Therefore, this is actually required to operate correctly and create the exemption.

It is recommended to use the Justification box for any description or additional text.