Change the default logging level of agent to INFO (bugfix)#2009
Change the default logging level of agent to INFO (bugfix)#2009fernando79513 merged 1 commit intomainfrom
Conversation
This is done only if debug is not requested
Codecov Report❌ Patch coverage is
❌ Your patch status has failed because the patch coverage (25.00%) is below the target coverage (90.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## main #2009 +/- ##
==========================================
- Coverage 50.60% 50.60% -0.01%
==========================================
Files 384 384
Lines 41180 41184 +4
Branches 7642 7643 +1
==========================================
+ Hits 20841 20842 +1
- Misses 19592 19594 +2
- Partials 747 748 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
fernando79513
left a comment
There was a problem hiding this comment.
LGTM.
I think it makes sense to have the default logging level as INFO. I tested it locally and the output of the agent is more informative but still manageable.
Change the default logging level of agent to INFO This is done only if debug is not requested
Change the default logging level of agent to INFO This is done only if debug is not requested
Change the default logging level of agent to INFO This is done only if debug is not requested
Change the default logging level of agent to INFO This is done only if debug is not requested
This is done only if debug is not requested
Description
With INFO level logging, the Checkbox agent outputs a little bit more information about the test run. This is useful because, given that the agent runs as a systemd unit, this information contains some contextual clues as to what the machine is doing, greatly simplifying reading the debug logs.
Resolved issues
Fixes: https://warthogs.atlassian.net/browse/CHECKBOX-1971
Documentation
N/A
Tests
Tested it offline, it works