Open Bug 1471717 Opened 4 years ago Updated 2 years ago

Too many timestamps in linux and android unit test logs


(Testing :: General, enhancement, P3)

Version 3


(firefox63 fixed)

Tracking Status
firefox63 --- fixed


(Reporter: gbrown, Unassigned)



(Whiteboard: dev-prod-2020)


(1 file)

Tasks that run in linux docker containers -- notably linux and android unit tests -- have a timestamp added to every line in the log by run-task:

such as:

setup 2018-06-27T12:18:35.132Z] run-task started in /builds/worker

mozharness scripts also add a timestamp to everything they log, so we see:

[task 2018-06-27T12:22:31.480Z] 12:22:31     INFO - Using env: (same as previous command)

Timestamps are useful, but I don't think there is much value in 2 timestamps which are almost always identical (modulo formatting). On the downside, dual timestamps complicate reading and add load to log processing.
A simple config override of the mozharness log format eliminates the extra timestamps from most tasks.

[task 2018-06-27T20:42:12.318Z]     INFO -  TEST-PASS | ShowSSEConfig | took 11ms
[task 2018-06-27T20:42:12.319Z]     INFO -  TEST-START | TestPrintf
Attachment #8988313 - Flags: review?(jmaher)
Priority: -- → P1
Attachment #8988313 - Flags: review?(jmaher) → review+
Pushed by
Remove extra timestamp from mozharness logs for linux and android unit tests; r=jmaher
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
Depends on: 1473565
Re-opening for bug 1473565. I'll effectively back out changes until those issues can be addressed.
Resolution: FIXED → ---
Pushed by
Disable mozharness log format changes for android and linux; r=me,a=test-only
Keywords: leave-open
(In reply to Eliza Balazs [:ebalazs_] from comment #6)

:gbrown since this got to central, it should have been backed out of central and i cannot see that in treeherder. Can you please point out where the backout push? 

Thank you.
Flags: needinfo?(gbrown) was the original push, which caused trouble in bug 1473565. was not a "real" backout, but effectively backed out e82cf1aca043 by commenting out the change. Linux and Android test logs are back to normal now.
Flags: needinfo?(gbrown)
Assignee: gbrown → nobody
Priority: P1 → P3

The leave-open keyword is there and there is no activity for 6 months.
:gbrown, maybe it's time to close this bug?

Flags: needinfo?(gbrown)

I still hope to get back to this.

Flags: needinfo?(gbrown)
Keywords: leave-open
Whiteboard: dev-prod-2020
You need to log in before you can comment on or make changes to this bug.