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
Enhance client-go url timing logging with http statistics #105156
Conversation
5572a0c
to
7258999
Compare
7258999
to
af55731
Compare
it interleaves the logging
,so I've added a new commit to use structured logging , but I can't make it to look pretty :/
|
/triage accepted |
structured require more work, and the format is not great, I'll keep using the current format |
af55731
to
7226185
Compare
e3d3dfb
to
800810c
Compare
I think that this is ready, it provides all the necessary info for debugging client-go transport problems, easy to grep and is limited to V9 loglevel. |
800810c
to
6629aa4
Compare
one comment about the trace var locking, lgtm otherwise |
enhance client-go DebugURLTiming roundtripper using golang httptrace to provide timing statistics for: - dns lookup - tcp dialing - tls handshake - time to get a connection from the pool - time to process a request
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: aojea, liggitt The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
client-go httpstats
enhance client-go DebugURLTiming roundtripper using golang httptrace
to provide timing statistics for:
and trace information for DNSLookups and Dials
Example: