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
component-base: test and fix JSON output for KObj #104877
Conversation
Sad that we missed this, I think I remember noticing this at some point, but when final implementation came we didn't have tests. We should definetly fix it for 1.23, @pohly can you file a issue to address this. We should track it in the Structured Logging KEP. |
/hold Let's use this PR to include a fix, if there is one. |
/assign @serathius |
/lgtm |
/kind bug |
/assign @liggitt |
The new releases fix logging of KObj in JSON output: klog implements the new logr.Marshaler interface and zapr uses it instead of Stringer when logging the ObjectRef created by KObj.
Thanks to the new klog, logr and zapr releases, ObjectRef is now logged as struct in JSON output.
/approve |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dims, liggitt, pohly, serathius 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 |
component-base: test and fix JSON output for KObj
What type of PR is this?
/kind bug
What this PR does / why we need it:
KObj wasn't get logged as struct when using JSON output.
Does this PR introduce a user-facing change?
/cc @serathius @thockin