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
fix: implement MarshalLog for structures in volumemanager for structured-logging. #119829
Conversation
Please note that we're already in Test Freeze for the Fast forwards are scheduled to happen every 6 hours, whereas the most recent run was: Mon Aug 7 22:33:07 UTC 2023. |
/sig storage |
/assign |
Hey, @cvvz can you provide the steps to produce this issue also? |
@nikzayn Just change the kubelet log level to greater than 4 and start kubelet then you can get the log, which didn't print any useful volumes information. https://github.com/kubernetes/kubernetes/blob/master/pkg/kubelet/volumemanager/reconciler/reconstruct_common.go#L184 |
/retest |
/release-note-none |
246b6b6
to
ebea00a
Compare
@cvvz please squash your commits into one, thanks. |
commit d623614de31fe411f1dcb1e784472135f3ca0c5e Merge: 8054af3b303 91344b4 Author: cvvz <ftdchenwz@gmail.com> Date: Mon Aug 28 18:43:49 2023 +0800 Merge branch 'master' of https://github.com/kubernetes/kubernetes into fix-volumemanager-logs commit 8054af3b303e10e7b74b1ba4d3c4035f488cbdad Author: cvvz <ftdchenwz@gmail.com> Date: Fri Aug 25 22:03:08 2023 +0800 fix commit b414972831c4e4030162ee385d8f600e1e0257ac Author: cvvz <ftdchenwz@gmail.com> Date: Fri Aug 25 21:41:36 2023 +0800 fix commit ebea00a8dd50eb3d8859a912b464bbda5548b1d4 Author: cvvz <ftdchenwz@gmail.com> Date: Fri Aug 25 20:54:40 2023 +0800 123 commit 9f6f1dbbe717fa34e1c13fec645f4c474cbf99a0 Author: cvvz <ftdchenwz@gmail.com> Date: Fri Aug 25 20:53:16 2023 +0800 add MarshalLog commit d7d2878409343df937c770d6796f8c125e18ce7a Author: cvvz <ftdchenwz@gmail.com> Date: Tue Aug 8 23:57:47 2023 +0800 fix volumemanager logs
d623614
to
94d03cc
Compare
@bart0sh sure. |
/retest |
/lgtm |
LGTM label has been added. Git tree hash: 5ac672784988f8f335e6a0b5487220b5cec6542e
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cvvz, jsafrane, togettoyou 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 |
What type of PR is this?
/kind bug
What this PR does / why we need it:
Before fix:
After fix:
This is because
klog
rely onencoding/json
lib to encode the structure and print out, so it can't print un-exported fields.Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: