-
Notifications
You must be signed in to change notification settings - Fork 11
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
Clarify adding users to RBAC group #370
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for pluraldocs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@@ -51,7 +51,7 @@ subjects: | |||
name: [email protected] | |||
``` | |||
|
|||
To add the specific user `[email protected]` identified by that email to the RBAC rules. | |||
To add the specific user `[email protected]` identified by that email to the RBAC rules. For the purposes of this tutorial you will need to manually add the user to the Group created above by navigating to Settings > User Managment > Groups going to https://{you-console-instance}/settings/user-management/groups. Find the Edit Members tab in the group and add your member. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Isn't this already added at line 31? Maybe it should really just be a callout to make it prominent. It also isn't relevant to the ClusterRoleBinding above as that's a user binding and doesn't need the group to be created
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fair enough but there should still be some callout given directing that the user(s) be added to the group in the console. That could go on line 31. Something like "add users to this group ..."
Fixes PROD-3076