scout icon indicating copy to clipboard operation
scout copied to clipboard
verified publisher icon trynthink

Better log outputs for ecm_prep.py

Open aspeake opened this issue 1 year ago • 2 comments

To help with debugging, update the following behavior for log messages:

  • write the ecm_prep log file (added here: https://github.com/trynthink/scout/pull/402) to CI artifacts after the integration test is run.
  • Enable log messages in ecm_prep.py to be optionally output to the console with the --verbose argument

aspeake avatar Aug 25 '24 18:08 aspeake

@aspeake We also discussed exposing the log file contents in the STDOUT so that it would be immediately visible in the runner logs, possibly by using verbose mode (which doesn't currently have this behavior, but could be changed to support it). Did you conclude that it would be preferable to write the log file as an artifact instead?

trynthink avatar Aug 29 '24 18:08 trynthink

@aspeake We also discussed exposing the log file contents in the STDOUT so that it would be immediately visible in the runner logs, possibly by using verbose mode (which doesn't currently have this behavior, but could be changed to support it). Did you conclude that it would be preferable to write the log file as an artifact instead?

Actually we did decide to output to the console with --verbose, I updated the issue to address both

aspeake avatar Aug 29 '24 18:08 aspeake