forked from nspcc-dev/neofs-node
-
Notifications
You must be signed in to change notification settings - Fork 6
Revise logger levels #41
Copy link
Copy link
Open
Labels
P1HighHighfrostfs-irfrostfs-nodehelp wantedExtra attention is neededExtra attention is neededquestionFurther information is requestedFurther information is requestedtriage
Metadata
Metadata
Assignees
Labels
P1HighHighfrostfs-irfrostfs-nodehelp wantedExtra attention is neededExtra attention is neededquestionFurther information is requestedFurther information is requestedtriage
Type
Fields
Give feedbackNo fields configured for issues without a type.
Suggest general rule: "info" for external events AND for accepting events to share them with others (pushing "new epoch" event to chain); "debug" for internal changes (node gets "new epoch" and logs it via "info" but its internal handling could be logged with "debug").
Also, as it was discussed before, we have an quite rare but really important events that deserve "info"/"warn" level (since we store objects, we can log their removal with "warn" to prevent/investigate errors).
Also, some important events can move from "debug" to "info":
Also, i would expand "debug" with logs about every step inside every main process: before/after every contract info fetching; before/after every network communication; before/after every disk operation.
Suggestions are appreciated.