-
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
Update role binding docs #11344
Update role binding docs #11344
Conversation
a276f25
to
c3df292
Compare
Rebase |
c3df292
to
9082c50
Compare
9082c50
to
b65108f
Compare
+1 |
Signed-off-by: Monis Khan <[email protected]>
b65108f
to
731169a
Compare
Rebased. |
Note you only need api review if the doc get worse :) LGTM [merge] |
continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/10356/) (Image: devenv-rhel7_5216) |
Evaluated for origin merge up to 731169a |
[Test]ing while waiting on the merge queue |
Evaluated for origin test up to 731169a |
continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/10356/) (Base Commit: e1d9196) |
Just trying to clarify role binding creation based on all the fun @benjaminapetersen had.
Signed-off-by: Monis Khan [email protected]
cc @openshift/api-review