New issue
Advanced search Search tips

Issue 815062 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Apr 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

job reporter should not dump its log in the root results directory

Project Member Reported by akes...@chromium.org, Feb 23 2018

Issue description

Example of a results directory, randomly selected:

https://pantheon.corp.google.com/storage/browser/chromeos-autotest-results/178648390-chromeos-test/chromeos6-row22-jetstream-host8/

A new type of file is now being dumped here, job_reporter_output.log

This log is only siginicant to infra, has no meaning to other developers trying to understand test results. We should avoid adding yet more distracting logs to the result directory.

Options:
 - Rename to .job_reporter_output.log
 - Move to a separate and clearly named infra-only log directory.
 
I'd prefer creating a harness/ directory and dumping this and other such logs there. This would include tko parse logs, and future skylab-drone logs.
It's hard for me to appreciate how developers see the results, but it seems like a lot of stuff in the directly aren't immediately applicable to just the test results, e.g. results_summary, spong_invocation, control file, pid files, keyvals, host info.

Certainly job reporter's logs could be moved to an infra-logs dir, but the benefits won't be fully appreciated without fixing all of the junk.
Agreed but that should not prevent us from fixing this.
Labels: Hotlist-Lucifer
Status: Fixed (was: Assigned)

Sign in to add a comment