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

child management groups (if any), are part of the exemptions #43

Merged
merged 1 commit into from
Apr 15, 2022

Conversation

wetwicky
Copy link
Contributor

The value of the $child.Type is splitted and doesn't contain the prefix "/providers/". This cause the management Group to be added in the $subscriptionIds. Therefore added to the exemption list of the assignment.

The value of the $child.Type is splitted and doesn't contain the prefix "/providers". This cause the management Group to be added in the $subscriptionIds. Therefore added to the exemption list of the assignment.
Copy link
Contributor

@techlake techlake left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just found this myself independently (today). I'll have a fix with additional error checking later. In the meantime, your fix will work correctly and I'm therefore merging the change in.

@techlake techlake merged commit 4bcfdb1 into Azure:main Apr 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants