Get Existing AzureChinaCloud Exemptions not Returning Non-Management Group Scoped Exemptions

[ahsulli]

Describe the bug
When getting existing exemptions within an AzureChinaCloud tenant, only management group level exemptions are included in the final returned list of existing exemptions.

To Reproduce

1. Run an Export-AzPolicyResources or Build-DeploymentPlans using a pacSelector whose type is AzureChinaCloud.

Expected behavior
All exemptions scoped to management groups, subscriptions, resource groups, and individual resources within the specified pacSelector configuration should be included in the list of existing exemptions that are factored into the final output.

Screenshots
N/A

EPAC Version
10.3.5

[anwather]

Closing as issue is defined in #692

[ahsulli]

@anwather, just to further clarify, you can also reproduce this issue by running Build-DeploymentPlans as well. However, this version of the repro steps produces an erroneous output that is less obvious to see but is still flawed for the same reason.

[ahsulli]

@anwather, issue is still present. Please fix. Cannot run deployment plans against China tenants as a result.

[anwather]

OK you mentioned you had code to fix this can you please submit a PR. I don't have access to a China tenant to run this against. Get Outlook for Android<https://aka.ms/AAb9ysg>

…

________________________________ From: ahsulli ***@***.***> Sent: Wednesday, July 10, 2024 4:29:59 AM To: Azure/enterprise-azure-policy-as-code ***@***.***> Cc: Mention ***@***.***>; Comment ***@***.***>; Assign ***@***.***>; State change ***@***.***>; Subscribed ***@***.***> Subject: Re: [Azure/enterprise-azure-policy-as-code] Get Existing AzureChinaCloud Exemptions not Returning Non-Management Group Scoped Exemptions (Issue #691) @anwather<https://github.com/anwather>, issue is still present. Please fix. Cannot run deployment plans against China tenants as a result. — Reply to this email directly, view it on GitHub<#691 (comment)> or unsubscribe<https://github.com/notifications/unsubscribe-auth/ACWCJVTCXOIN6EEU5MCZFW3ZLQT2RBFKMF2HI4TJMJ2XIZLTSWBKK5TBNR2WLJDUOJ2WLJDOMFWWLO3UNBZGKYLEL5YGC4TUNFRWS4DBNZ2F6YLDORUXM2LUPGBKK5TBNR2WLJDUOJ2WLJDOMFWWLLTXMF2GG2C7MFRXI2LWNF2HTAVFOZQWY5LFUVUXG43VMWSG4YLNMWVXI2DSMVQWIX3UPFYGLAVFOZQWY5LFVIZDOMRVHE4DSNBXHCSG4YLNMWUWQYLTL5WGCYTFNSBKK5TBNR2WLKRSG4ZDKOJYHE2DQMVENZQW2ZNJNBQXGX3MMFRGK3FMON2WE2TFMN2F65DZOBS2YSLTON2WKQ3PNVWWK3TUUZ2G64DJMNZZJAVEOR4XAZNKOJSXA33TNF2G64TZUV3GC3DVMWUTGMZXGE3DEOBTHCBKI5DZOBS2K2LTON2WLJLWMFWHKZNKGIZTOMZXGA2DONJUQKSHI6LQMWSWYYLCMVWKK5TBNR2WLKRSG4ZDKOJYHE2DOOECUR2HS4DFUVWGCYTFNSSXMYLMOVS2UMRXGI2TSOBZGQ4DFJ3UOJUWOZ3FOKTGG4TFMF2GK>. You are receiving this email because you were mentioned. Triage notifications on the go with GitHub Mobile for iOS<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.

[ahsulli]

@anwather, can someone please grant me access to create a branch and PR? I have the code already prepared on my local. The code has been successfully tested numerous times against my organization's China and Commercial tenants. Hopefully that helps too.

[anwather]

Just fork the repo, create a branch on the fork, make changes and create a pull request please. Get Outlook for Android<https://aka.ms/AAb9ysg>

…

________________________________ From: ahsulli ***@***.***> Sent: Friday, July 12, 2024 2:24:41 AM To: Azure/enterprise-azure-policy-as-code ***@***.***> Cc: Mention ***@***.***>; Comment ***@***.***>; Assign ***@***.***>; State change ***@***.***>; Subscribed ***@***.***> Subject: Re: [Azure/enterprise-azure-policy-as-code] Get Existing AzureChinaCloud Exemptions not Returning Non-Management Group Scoped Exemptions (Issue #691) @anwather<https://github.com/anwather>, can someone please grant me access to create a branch and PR? I have the code already prepared on my local. The code has been successfully tested numerous times against my organization's China and Commercial tenants. Hopefully that helps too. — Reply to this email directly, view it on GitHub<#691 (comment)> or unsubscribe<https://github.com/notifications/unsubscribe-auth/ACWCJVQGKYJNQZHP25XC2DTZL2WUTBFKMF2HI4TJMJ2XIZLTSWBKK5TBNR2WLJDUOJ2WLJDOMFWWLO3UNBZGKYLEL5YGC4TUNFRWS4DBNZ2F6YLDORUXM2LUPGBKK5TBNR2WLJDUOJ2WLJDOMFWWLLTXMF2GG2C7MFRXI2LWNF2HTAVFOZQWY5LFUVUXG43VMWSG4YLNMWVXI2DSMVQWIX3UPFYGLAVFOZQWY5LFVIZDOMRVHE4DSNBXHCSG4YLNMWUWQYLTL5WGCYTFNSBKK5TBNR2WLKRSG4ZDKOJYHE2DQMVENZQW2ZNJNBQXGX3MMFRGK3FMON2WE2TFMN2F65DZOBS2YSLTON2WKQ3PNVWWK3TUUZ2G64DJMNZZJAVEOR4XAZNKOJSXA33TNF2G64TZUV3GC3DVMWUTGMZXGE3DEOBTHCBKI5DZOBS2K2LTON2WLJLWMFWHKZNKGIZTOMZXGA2DONJUQKSHI6LQMWSWYYLCMVWKK5TBNR2WLKRSG4ZDKOJYHE2DOOECUR2HS4DFUVWGCYTFNSSXMYLMOVS2UMRXGI2TSOBZGQ4DFJ3UOJUWOZ3FOKTGG4TFMF2GK>. You are receiving this email because you were mentioned. Triage notifications on the go with GitHub Mobile for iOS<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.

[ahsulli]

@anwather, I have created PR: #702