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
Make log statements easier to read #22971
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @yulng and thanks for the PR
Printing the entire structure of the identity in the log makes the log hard to read ,so branch printing
As far as I can see the patch is cosmetic only and introduce no functional change. If correct then I don't understand the commit message, what is "branch printing"?
It is printed line by line, unified style, easy to view |
e4067c4
to
4925f1b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since this change is purely cosmetic, can you reword the commit to
Make log statements easier to read
This is a cosmetic change to break up long log lines into smaller lines.
Push again and update the commit |
932d436
to
1e31296
Compare
This is a cosmetic change to break up long log lines into smaller lines Signed-off-by: yulng <wei.yang@daocloud.io> Signed-off-by: Chris Tarazi <chris@isovalent.com>
1e31296
to
b86e74f
Compare
/test |
/test-runtime |
Printing the entire structure of the identity in the log makes the log hard to read ,so branch printing
Signed-off-by: yulng wei.yang@daocloud.io