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

GCI/Trusty: Support ABAC authorization #24950

Merged
merged 1 commit into from
May 6, 2016
Merged

GCI/Trusty: Support ABAC authorization #24950

merged 1 commit into from
May 6, 2016

Conversation

andyzheng0831
Copy link

This PR is for supporting the ABAC authorization added in PR #24210. It is tested on GCI and Ubuntu Trusty with two modes, i.e., entire cluster on GCI/Trusty and hybrid mode with only master on GCI/Trusty.

@k8s-github-robot k8s-github-robot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. release-note-label-needed labels Apr 28, 2016
@andyzheng0831
Copy link
Author

andyzheng0831 commented Apr 28, 2016

@roberthbailey @dchen1107 @cjcullen please review it.

cc/ @fabioy @zmerlynn @wonderfly FYI

@cjcullen cjcullen assigned cjcullen and unassigned brendandburns Apr 28, 2016
@cjcullen
Copy link
Member

LGTM. I'll let it simmer for a bit in case anybody else has comments.

@andyzheng0831
Copy link
Author

@k8s-bot test this issue #24422

@andyzheng0831
Copy link
Author

@k8s-bot test this issue #24642

@danwinship
Copy link
Contributor

@andyzheng0831 I think I'm not the person you meant to tag...

@andyzheng0831
Copy link
Author

@danwinship sorry for mis-tagging you.

@andyzheng0831
Copy link
Author

@dchen1107 @roberthbailey do you have any comments for this change? Thanks!

@roberthbailey roberthbailey added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Apr 30, 2016
@k8s-github-robot k8s-github-robot removed the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Apr 30, 2016
@andyzheng0831
Copy link
Author

@cjcullen @roberthbailey please add "release-note" label for it, we can use the PR title for the changelog. Thanks

@roberthbailey roberthbailey added release-note Denotes a PR that will be considered when it comes time to generate release notes. lgtm "Looks good to me", indicates that a PR is ready to be merged. and removed release-note-label-needed labels May 2, 2016
@roberthbailey
Copy link
Contributor

done.

@k8s-bot
Copy link

k8s-bot commented May 6, 2016

GCE e2e build/test passed for commit 27c9ee3.

@roberthbailey roberthbailey merged commit 5f4f59e into kubernetes:master May 6, 2016
@andyzheng0831 andyzheng0831 deleted the apiserver branch May 7, 2016 03:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm "Looks good to me", indicates that a PR is ready to be merged. release-note Denotes a PR that will be considered when it comes time to generate release notes. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants