Closed Bug 1393655 Opened 4 years ago Closed 4 years ago

run-task: strip microsecond

Categories

(Taskcluster :: General, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jonasfj, Assigned: jonasfj)

Details

Attachments

(1 file)

microsecond timing probably doesn't have much use.
Let's strip it and reclaim some log margin...

Note: we could round it to two decimals, if people really want it.
Assignee: nobody → jopsen
Status: NEW → ASSIGNED
Fix here: https://hg.mozilla.org/try/rev/6ec968bbca0016c93be1dd524de78a9c5a9f19bc
And try push: https://treeherder.mozilla.org/#/jobs?repo=try&revision=5b164b63de4cc4dc76c0026e2e3d73f00ea2ff79

For some reason |hg push review| doesn't work... probably bad config on my end, I'll dig into that tomorrow...
Attachment #8901280 - Flags: review?(gps)
Comment on attachment 8901280 [details]
Bug 1393655 - Strip microsecond from run-task logs

https://reviewboard.mozilla.org/r/172732/#review178244

I agree that microsecond isn't that useful. But this change brings us down to second latency and that is too coarse. I'll accept a patch to reduce this to millisecond latency.
Attachment #8901280 - Flags: review?(gps) → review-
Comment on attachment 8901280 [details]
Bug 1393655 - Strip microsecond from run-task logs

https://reviewboard.mozilla.org/r/172732/#review184126

I would prefer full millisecond resolution. But centisecond is probably good enough.
Attachment #8901280 - Flags: review?(gps) → review+
Then full millisecond you shall have :)

 - I don't know what I was thinking... centiseconds... tsk tsk...
Keywords: checkin-needed
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/9c24bd90faa1
Strip microsecond from run-task logs r=gps
Keywords: checkin-needed
Thanks :)
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.