-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
POC: role management policy resource #20496
POC: role management policy resource #20496
Conversation
90fc00a
to
4c1762d
Compare
4c1762d
to
374d9f3
Compare
Hi @josh-barker Thanks again. |
Hey @jackofallops , Thanks for your explanation. The way I was thinking about it was keeping it in draft until Tom comes back with some feedback, as this PR requires more thought and changes in the sdk generator, etc. I'm keen to get feedback on this. Thanks! |
Hello @jackofallops and @josh-barker, really looking forward using this new resource, it is the missing piece to deploy a proper PIM configuration using Terraform. Many thanks for handling this subject. |
Hi @josh-barker / @jackofallops, Are there any plans to re-open this PR? Like other commenters, we have an interest in seeing role management policy resources so we can use Azure PIM in our environment. |
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions. |
This PR is a POC to create a resource for managing role management policy.
It's worth noting that the Azure API requires an
ID
andDescription
for user and group approvers.The Azure API does not validate the description, so you could enter
foo bar
that will be presented in the Azure Portal.If an empty string is set, nothing is shown in the Azure Portal.
Currently, the Vendor Dependencies check failure is expected as I needed to modify the vendored copy of the authorization API to get this functionality to work.
See:
Related issues with the SDK: