-
Notifications
You must be signed in to change notification settings - Fork 14.5k
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
AppArmor documentation #1147
AppArmor documentation #1147
Conversation
Review status: 0 of 4 files reviewed at latest revision, 1 unresolved discussion. docs/admin/apparmor/index.md, line 7 [r1] (raw file):
For clarity, I'd recommend changing to "reduce the potential attack surface of an Application and provide greater defense in depth." Comments from Reviewable |
Overall excellent. One very minor issue, but Docs LGTM. |
Thanks! Done. Review status: 0 of 4 files reviewed at latest revision, 1 unresolved discussion. docs/admin/apparmor/index.md, line 7 [r1] (raw file):
|
LGTM |
aec885a
to
05b2829
Compare
Squashed commits. |
Looks good, merging to release-1.4 line. |
Adds full documentation for AppArmor.
Original proposal: https://github.com/kubernetes/kubernetes/blob/master/docs/proposals/apparmor.md
Feature issue: kubernetes/enhancements#24
Note: 2 features referenced in this document have not yet been merged:
/cc @thockin @jfrazelle @pwittrock @dchen1107 @matchstick @kubernetes/docs
This change is