1355810 - the log viewer should hide taskcluster timestamps by default

Status: RESOLVED INCOMPLETE

(Tree Management :: Treeherder, defect)

Description

[Nathan Froyd [:froydnj]]

It's super annoying to copy and paste sections of logfiles from the log viewer and wind up with:

[task 2017-04-12T11:57:04.335910Z] 11:57:04     INFO - Hit MOZ_CRASH(gfxUserFontSet not thread-safe) at /home/worker/workspace/build/src/gfx/thebes/gfxUserFontSet.h:178
[task 2017-04-12T11:57:04.336831Z] 11:57:04     INFO - #01: gfxFontGroup::gfxFontGroup [gfx/thebes/gfxTextRun.cpp:1713]
[task 2017-04-12T11:57:04.337553Z] 11:57:04     INFO - 
[task 2017-04-12T11:57:04.338807Z] 11:57:04     INFO - #02: gfxPlatformGtk::CreateFontGroup [gfx/thebes/gfxPlatformGtk.cpp:281]
[task 2017-04-12T11:57:04.340467Z] 11:57:04     INFO - 
[task 2017-04-12T11:57:04.342410Z] 11:57:04     INFO - #03: nsFontMetrics::nsFontMetrics [mfbt/RefPtr.h:53]
[task 2017-04-12T11:57:04.344022Z] 11:57:04     INFO - 
[task 2017-04-12T11:57:04.345656Z] 11:57:04     INFO - #04: nsFontCache::GetMetricsFor [mfbt/RefPtr.h:110]

...and so forth, where roughly half of the text is [task 2017-04-12T...] junk that's really not important.  I'm sure it's useful for some cases, but it would be nice if the log viewer automatically suppressed those bits, maybe with a toggle to turn them back on.

Comment 1

[Ed Morley [:emorley]]

I agree that would be useful :-)

The treeherder log viewer is now just an iframe pointing to the unified log viewer (apart from the metadata shown at the top of the page):
https://github.com/taskcluster/unified-logviewer

It doesn't have a Bugzilla component (bug 1315808 comment 20 onwards) otherwise I'd move this bug there.
I don't suppose you could refile over on GitHub?

Comment 2

[Nathan Froyd [:froydnj]]

(In reply to Ed Morley [:emorley] from comment #1)
> I don't suppose you could refile over on GitHub?

Done!  https://github.com/taskcluster/unified-logviewer/issues/37

So I should file bugs there, and not in bugzilla anymore, despite the "Treeherder: Log Viewer" component?

Comment 3

[Ed Morley [:emorley]]

Good point. There's still technically the parts around the iframe, but perhaps it would be less confusing to merge the component into the main Treeherder component. I'll have a think :-)

Comment 4

[Ed Morley [:emorley]]

In bug 1397698 I've had a bit of a spring clean of the Treeherder Bugzilla components. I decided to leave the "Treeherder: Log Viewer" component since there are parts of it that are still controlled by Treeherder, but have updated the description to say:

"""
Bugs regarding the page for viewing build & test job logs. The scrollable log content section in the centre is an iframe holding the Taskcluster Unified Log Viewer - bugs for that should be filed [here](https://github.com/taskcluster/unified-logviewer). 
"""

I'll mark this bug as incomplete for now, since any fix will need to come from the unified-logviewer side, for which there's the issue linked in comment 2.