-
Notifications
You must be signed in to change notification settings - Fork 367
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
Automate rbac documentation 8201 draft #8634
base: master
Are you sure you want to change the base?
Automate rbac documentation 8201 draft #8634
Conversation
…-draft' into automate-rbac-documentation-8201-draft
@tkalir Thank you for this contribution! And you definitely made the right choice of feeling out the water before diving in! I wanted to know, what did you have in mind regarding the doc generation in regards with this approach? |
My general plan is:
As I mentioned here instead of writing the permission factories code manually it's also possible to have the permission node data in the yaml from point 2# and use code generation. Thanks |
Hi @tkalir, |
@arielshaqed ok, thanks, I'm working on it. |
Thanks! I won't get into your workflow, and whatever works for you will be great. The only disadvantage to opening a new PR is that it loses existing comments. But I don't think we have any line-based comments, so I am not worried here. |
This is related to issue #8201
This is a draft, to get feedback on this solution before doing the refactoring for the rest of the actions.
The idea is that every action will have a permissionFactory that returns a permission node (like the ones used in controller.go right now) but the "resource" string will be a template that can be
I chose the copy objects action for the test here to show the need for RenamePlaceholder, which otherwise will both have the base placeholder from ObjectArnTemplate.
Actions names, like "copy_objects" here, will be extracted to a variable.