1143094 - talosprocess (mozprocess) fails to output needed data locally

Status: RESOLVED FIXED

(Testing :: Talos, defect)

Description

[Joel Maher ( :jmaher ) (UTC -8)]

while running talos locally, I found that the local.out and local.json didn't have results.  Looking into this further, we had failures in the browser_output.txt format causing our results processing to be wonky.  Why this is different locally vs in tree, no idea.

I have a patch that doesn't log by default, instead we use a processOutputHandler.

try run is here:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=4cbddf8c46fb

Comment 1

[Joel Maher ( :jmaher ) (UTC -8)]

Attachment: use processOutputHandler instead of logFile (1.0)

Comment 2

[William Lachance (:wlach)]

Comment on attachment 8577362 [details] [diff] [review]
use processOutputHandler instead of logFile (1.0)

Review of attachment 8577362 [details] [diff] [review]:
-----------------------------------------------------------------

So as discussed on irc, the proper solution is to just remove this class and call mozprocess directly. However, this fixes the immediate problem of us writing to the file from two different places, so we might as well put it in.

Comment 3

[Joel Maher ( :jmaher ) (UTC -8)]

https://hg.mozilla.org/build/talos/rev/4d31e0f61658