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
kube-apiserver: improve bootstrap token authentication error messages #41985
kube-apiserver: improve bootstrap token authentication error messages #41985
Conversation
// | ||
func tokenErrorf(s *api.Secret, format string, i ...interface{}) { | ||
format = fmt.Sprintf("Bootstrap secret %s/%s matching bearer token ", s.Namespace, s.Name) + format | ||
glog.V(3).Infof(format, i...) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't know if this is the correct verbosity level. It was the existing one taken from isSecretExpired
.
@k8s-bot kops aws e2e test this |
9bc8e47
to
ba31bf7
Compare
ping @ericchiang @jbeda @liggitt |
ba31bf7
to
f5fa115
Compare
rebased for good measure. Anyone want to approve? |
@k8s-bot gce etcd3 e2e test this |
/release-note-none |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: deads2k, ericchiang
Needs approval from an approver in each of these OWNERS Files:
You can indicate your approval by writing |
Thanks @deads2k! |
Automatic merge from submit-queue (batch tested with PRs 41530, 44814, 43620, 41985) |
This was requested by @jbeda as a follow up to #41281.
cc @jbeda @luxas @kubernetes/sig-auth-pr-reviews