-
Notifications
You must be signed in to change notification settings - Fork 45
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
Log formatting #437
Labels
Comments
debug-level tracing really should have logs pervasively, though info-level shouldnt appear in library code since library code should be using RC/RSN instead. |
timestamps must not break ZIS messages (it already has its own) |
Many code comments of old conversations here |
zowe/zowe-common-c#167 has some info too. |
ifakhrutdinov
added a commit
to ifakhrutdinov/zowe-common-c
that referenced
this issue
Apr 20, 2022
Fixes: zowe#167 and zowe/zss#437
I've made a POC based on zowe/zowe-common-c#167:
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Some ZSS logs now have message IDs
Yet, many don't
And none of them have timestamps.
This makes troubleshooting more difficult.
We've tried to implement the format in zowe/zac#90 multiple times and PRs have never finished.
Please lay out rules to follow with logs so that PRs on the subject will succeed.
Previous discussions said things like
And some thoughts
The text was updated successfully, but these errors were encountered: