1311886 - Make build timeout error messages Treeherder-parseable

Status: RESOLVED DUPLICATE of bug 1333957

(Taskcluster :: Workers, enhancement)

Description

[Phil Ringnalda (:philor)]

https://public-artifacts.taskcluster.net/AzaARZ4eTxKdpNrFeUFB-w/0/public/logs/live_backing.log is an "Aborting task - max run time exceeded!" failure, which I know only because the dozens of lines of output that that's in the middle of are all the same thing, and thus the same length, so that stood out. Otherwise, having it neither be the last line in the log nor something treeherder recognizes as a failure line, I'd just think it was suddenly and inexplicably cut off.

Comment 1

[Pete Moore [:pmoore][:pete]]

Philor, do you remember if this a duplicate of bug 1333957 of was for a different worker (taskcluster-worker or docker-worker)?

Thanks

Comment 2

[Phil Ringnalda (:philor)]

No idea, I've gotten the impression that the difference between those two phrases matters a great deal, but I don't know what that difference is. I don't see any sign of builds starred against bug 1420078 in the last couple of weeks, so I can't point you at any fresh examples.

Comment 3

[Dustin J. Mitchell [:dustin] (he/him)]

Looks like this is just adding "[taskcluster:error]" or whatever the prefix is to https://github.com/taskcluster/generic-worker/blob/fe7c8f8ec63bd65f8160180135bec66b0658eaf9/taskstatus.go#L205

Comment 4

[Pete Moore [:pmoore][:pete]]

I think this probably was a duplicate of bug 1333957, which was resolved 16 months ago.

(In reply to Dustin J. Mitchell [:dustin] (he/him) from comment #3)

Looks like this is just adding "[taskcluster:error]" or whatever the prefix
is to
https://github.com/taskcluster/generic-worker/blob/fe7c8f8ec63bd65f8160180135bec66b0658eaf9/taskstatus.go#L205

No, that is what the task.Errorf method does already. :-)