Skip to content

docs/admin: document RBAC authorizer#643

Merged
johndmulhausen merged 1 commit intokubernetes:release-1.3from
ericchiang:authorization-rbac-alpha-docs-release-notes
Jun 24, 2016
Merged

docs/admin: document RBAC authorizer#643
johndmulhausen merged 1 commit intokubernetes:release-1.3from
ericchiang:authorization-rbac-alpha-docs-release-notes

Conversation

@ericchiang
Copy link
Copy Markdown
Contributor

@ericchiang ericchiang commented Jun 9, 2016

This PR adds documentation for the RBAC authorizer. (this time to the correct branch)

The API examples include fixes from kubernetes/kubernetes#26924 and kubernetes/kubernetes#26984

cc @erictune

Closes #631

Comment thread docs/admin/authorization.md Outdated
Here's an example of a role which grants read access to pods within the
"default" namespace.

```yml
Copy link
Copy Markdown

@johndmulhausen johndmulhausen Jun 9, 2016

Choose a reason for hiding this comment

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

192: ```yaml

@ericchiang ericchiang force-pushed the authorization-rbac-alpha-docs-release-notes branch 2 times, most recently from 044b09a to 0bd045f Compare June 13, 2016 23:47
@erictune
Copy link
Copy Markdown
Contributor

@johndmulhausen are there any Docs reviewers available?

Comment thread docs/admin/authorization.md Outdated
The RBAC API Group declares four top level types which will be covered in this
section. Users can interact with these resources as they would with any other
API resource. Through `kubectl`, direct calls to the API, etc. For instance,
`kubectl create -f { resource }.yml` can be used with any of these examples,
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

How about kubectl create -f (resource).yml

Curly braces conflict with the syntax for Liquid templating language, which the docs use

@ericchiang ericchiang force-pushed the authorization-rbac-alpha-docs-release-notes branch from 0bd045f to c2e4bd9 Compare June 23, 2016 21:52
@ericchiang
Copy link
Copy Markdown
Contributor Author

Updated to address @johndmulhausen's feedback.

@johndmulhausen
Copy link
Copy Markdown

Didn't mean to remove Tech Review LGTM. Thanks for the PR, @ericchiang !

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.

5 participants