Add information about SSL certificates in the default log level. #459
Conversation
k8s-ci-robot
added
the
cncf-cla: yes
|
This change is |
aledbf
force-pushed
the
log-ssl-information
branch
from
b384a06
to
6fee691
Compare
aledbf
force-pushed
the
log-ssl-information
branch
from
6fee691
to
18492c1
Compare
| s, err = ssl.AddOrUpdateCertAndKey(nsSecName, cert, key, ca) | ||
| } else if ca != nil { | ||
| glog.V(3).Infof("found only ca.crt, configuring %v as an Certificate Authentication secret", secretName) | ||
| glog.Infof("found only ca.crt, configuring %v as an Certificate Authentication secret", secretName) | ||
| s, err = ssl.AddCertAuth(nsSecName, ca) |
There was a problem hiding this comment.
@aledbf these lines above log every 10 seconds. Don't know if the whole func shouldn't be running or if the logs shouldn't be as default logs. What about reinclude V() until a correct fix is provided?
There was a problem hiding this comment.
Don't know if the whole func shouldn't be running or if the logs shouldn't be as default logs
We need to know if there is an issue with the certs. Let me see if I can change this to show only errors in the default level.
There was a problem hiding this comment.
@aledbf Our logs are a bit noisy and I think there is nothing we can do outside Ingress core. Let me know if I can provide some help or if there is a fix in a couple of days. Otherwise no problem! I make a temporary fork and wait the final solution.
No description provided.