Skip to content

Cleanup trigger logging#808

Merged
Charles-Gagnon merged 6 commits intorelease/triggerfrom
chgagnon/cleanupTriggerLogging
Apr 18, 2023
Merged

Cleanup trigger logging#808
Charles-Gagnon merged 6 commits intorelease/triggerfrom
chgagnon/cleanupTriggerLogging

Conversation

@Charles-Gagnon
Copy link
Contributor

Part 2 of logging cleanup, same stuff as #802

With this we now have :

13 Debug log messages - Extra information that could be useful for debugging, mostly startup/config info
2 Information messages - Telemetry opt-out and connection re-established messages
6 Warning messages - Non-critical errors
28 Error messages - Critical errors that indicate something is wrong

I plan on doing another pass after this is in to verify that the levels we have stuff at makes sense, but this should be pretty much complete at this point. Normal execution should only have a dozen or so messages per instance on startup with the occasional status update message.

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants