You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Aug 28, 2024. It is now read-only.
There are some issues with the current unit test logging used by the repo:
Some information is sent to the log file, while other information is sent to console
a. This makes getting all the required problem info from the Jenkins/autotest log difficult
b. Since you must get information from 2 separate places (console & log file) and also have access to the local machine to check the log (and runs happen on several different target machines based on version)
Both print statements and logging statements are used in the code (and somewhat inconsistently) when this is unnecessary
a. Logging statements can just be directed toward multiple "handlers" (file and stdout)
b. print statements used throughout the code with if Configuration.DEBUG == True: print(...)
Suggest we:
Short term: add logger handler to also direct logger calls to stdout (I have PR for this just need issue to tie to) - this is a higher priority since it is needed to debug autotest issues
Longer term: replace:
a. prints with logger.info
b. if DEBUG : print with logger.debug
The text was updated successfully, but these errors were encountered:
There are some issues with the current unit test logging used by the repo:
a. This makes getting all the required problem info from the Jenkins/autotest log difficult
b. Since you must get information from 2 separate places (console & log file) and also have access to the local machine to check the log (and runs happen on several different target machines based on version)
a. Logging statements can just be directed toward multiple "handlers" (file and stdout)
b. print statements used throughout the code with
if Configuration.DEBUG == True: print(...)
Suggest we:
a. prints with logger.info
b. if DEBUG : print with logger.debug
The text was updated successfully, but these errors were encountered: