Skip to content
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

Document the unconfined profile for AppArmor #6201

Merged
merged 1 commit into from
Nov 8, 2017

Conversation

tengqm
Copy link
Contributor

@tengqm tengqm commented Nov 6, 2017

This adds documentation about the support to unconfined profile for AppArmor.
xref: kubernetes/kubernetes#52395


This change is Reviewable

@k8s-ci-robot k8s-ci-robot added size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Nov 6, 2017
@tengqm
Copy link
Contributor Author

tengqm commented Nov 6, 2017

@dixudx

@k8sio-netlify-preview-bot
Copy link
Collaborator

k8sio-netlify-preview-bot commented Nov 6, 2017

Deploy preview ready!

Built with commit 07e5c8b

https://deploy-preview-6201--kubernetes-io-vnext-staging.netlify.com

@dixudx
Copy link
Member

dixudx commented Nov 6, 2017

@tengqm Thanks for updating this. Please rebase your commits. Only your last commit is need.

@k8s-ci-robot k8s-ci-robot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. and removed size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Nov 6, 2017
@steveperry-53
Copy link
Contributor

@dixudx Are you ready to give tech approval? Thanks.

@dixudx
Copy link
Member

dixudx commented Nov 8, 2017

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Nov 8, 2017
@tengqm tengqm merged commit c0a712a into kubernetes:release-1.9 Nov 8, 2017
@zacharysarah
Copy link
Contributor

zacharysarah commented Nov 8, 2017

@tengqm Did you intend to base this PR off of the 1.9 release branch instead of master? #5978

EDIT: For clarity, the docs workflow bases release-specific PRs in release-X.x branches; for everything else, we work in master.

@tengqm
Copy link
Contributor Author

tengqm commented Nov 9, 2017

@zacharysarah em... I'm still not clear how the workflow looks like. It would be good if it is documented somewhere.

@zacharysarah
Copy link
Contributor

@tengqm Sorry for the confusion! We added some docs to the 1.9 release page to help make the workflow clearer.

Because this PR isn't part of the 1.9 release, I'm going to revert this PR so we can rebase it onto master. 👍

@tengqm
Copy link
Contributor Author

tengqm commented Nov 11, 2017

Thanks.

@tengqm tengqm deleted the apparmor-unconfined branch November 11, 2017 01:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants