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.

Sign up for GitHub

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

Jump to bottom

prevent anonymous auth and allow all #38706

Merged
merged 1 commit into from
Jan 17, 2017
Merged

prevent anonymous auth and allow all #38706

merged 1 commit into from
Jan 17, 2017

Conversation

deads2k
Copy link
Contributor

@deads2k deads2k commented Dec 13, 2016 •
edited by liggitt

#38696 for master

@kubernetes/sig-auth

Anonymous authentication is now automatically disabled if the API server is started with the AlwaysAllow authorizer.

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Dec 13, 2016
@k8s-reviewable
Copy link

This change is Reviewable

@deads2k deads2k mentioned this pull request Dec 13, 2016
Closed
@k8s-github-robot k8s-github-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. release-note-label-needed labels Dec 13, 2016
@k8s-ci-robot
Copy link
Contributor

Jenkins GKE smoke e2e failed for commit 3bf5f0c54a87f13bfbcf27818d9faa27fe0c6ee2. Full PR test history.

The magic incantation to run this job again is @k8s-bot cvm gke e2e test this. Please help us cut down flakes by linking to an open flake issue when you hit one in your PR.

@k8s-github-robot k8s-github-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Dec 17, 2016
@liggitt liggitt mentioned this pull request Dec 19, 2016
Merged
@saad-ali saad-ali added this to the v1.6 milestone Dec 19, 2016
@deads2k deads2k mentioned this pull request Jan 4, 2017
Merged
@lavalamp
Copy link
Member

lavalamp commented Jan 7, 2017

LGTM if you can get the test green.

@k8s-github-robot k8s-github-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jan 17, 2017
@liggitt
Copy link
Member

liggitt commented Jan 17, 2017

LGTM

@liggitt liggitt added 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. and removed release-note-label-needed labels Jan 17, 2017
@k8s-github-robot
Copy link

@k8s-bot test this [submit-queue is verifying that this PR is safe to merge]

@k8s-github-robot
Copy link

Automatic merge from submit-queue (batch tested with PRs 34763, 38706, 39939, 40020)

@k8s-github-robot k8s-github-robot merged commit 27d486c into kubernetes:master Jan 17, 2017
@liggitt liggitt mentioned this pull request Jan 17, 2017
Closed
@deads2k deads2k deleted the auth-12-stomp-anonymous branch February 1, 2017 17:32
@MHBauer MHBauer mentioned this pull request Feb 6, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@lavalamp lavalamp

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. release-note Denotes a PR that will be considered when it comes time to generate release notes. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Milestone
v1.6
Development

Successfully merging this pull request may close these issues.

None yet
7 participants
@deads2k @k8s-reviewable @k8s-ci-robot @lavalamp @liggitt @k8s-github-robot @saad-ali