-
Notifications
You must be signed in to change notification settings - Fork 403
Improve logging #18016
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
Improve logging #18016
Conversation
✅ Deploy Preview for kyma-project-docs-preview ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Skipping CI for Draft Pull Request. |
👷 Deploy request for kyma-project pending review.Visit the deploys page to approve it
|
211c6e3
to
16333fd
Compare
Example req: GET http://localhost:8080/httpbin/raw/status/500
Accept: application/json response:
logs: {"level":"debug","ts":1692635703.967423,"caller":"proxy/reverseproxy.go:133","msg":"GET localhost:3000 /status/500 HTTP/1.1","requestID":null,"method":"GET","host":"localhost:3000","url":"/status/500","proto":"HTTP/1.1","code":500,"contentLength":0,"kind":"response"}
{"level":"debug","ts":1692635703.967733,"caller":"httptools/logging.go:71","msg":"GET localhost:8080 status/500 HTTP/1.1","label":"Internal handler: ","duration":80,"code":502,"requestID":"bc5c5573-7012-455a-98b2-3f5d9372d24b","method":"GET","host":"localhost:8080","url":"status/500","proto":"HTTP/1.1","kind":"request"} There is a problem with |
16333fd
to
cd20612
Compare
cd20612
to
63e669d
Compare
Co-authored-by: Grzegorz Karaluch <[email protected]>
@VOID404: The following test failed, say
Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
Description
We already had logging of
A
andC
(in some form).Logging of
B
andD
would be helpful.Changes proposed in this pull request:
B
Related issue(s)
#17905
#17265