-
Notifications
You must be signed in to change notification settings - Fork 40k
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
*: remove --insecure-allow-any-token option #49045
*: remove --insecure-allow-any-token option #49045
Conversation
0b02ee7
to
828e964
Compare
/test pull-kubernetes-federation-e2e-gce |
/unassign |
If we're going to remove it, let's remove it. |
828e964
to
1a267db
Compare
1a267db
to
7636491
Compare
e2e and integration tests have been switched over to the tokenfile authenticator instead. ```release-note The --insecure-allow-any-token flag has been removed from kube-apiserver. Users of the flag should use impersonation headers instead for debugging. ```
7636491
to
e2f2ab6
Compare
/test pull-kubernetes-kubemark-e2e-gce |
Fully removed |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: deads2k, ericchiang Associated issue: 49031 The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these OWNERS Files:
You can indicate your approval by writing |
Automatic merge from submit-queue (batch tested with PRs 49058, 49072, 49137, 49182, 49045) |
Since the authenticator is still used in e2e tests, don't removethe actual package. Maybe a follow up?
edit: e2e and integration tests have been switched over to the tokenfile
authenticator instead.
closes #49031
cc @kubernetes/sig-auth-pr-reviews