1189498 - Log viewer does not display value of a 'text' content_type item from the job_details artifact

Status: VERIFIED FIXED

(Tree Management :: Treeherder, defect, P2)

Description

[Maja Frydrychowicz :impossibus (was :maja_zf) (needinfo me)]

All the job_details items are displayed correctly in the log viewer except for the second "artifact uploaded" item, which is:

{
   "content_type": "text",
   "value": "jenkins-mn-mse-youtube-medium1-60-nightly-win_32_64-62",
   "title": "artifact uploaded"
}

That is, the title is displayed in log viewer, but not the value.

The corresponding job_details artifact is here: https://treeherder.mozilla.org/api/project/mozilla-central/artifact/11231182/


(All the job details display correctly in the Job Details pane in the main Treeherder view.)

Comment 1

[Ed Morley [:emorley]]

The logviewer URL in question:
https://treeherder.mozilla.org/logviewer.html#?job_id=1916933&repo=mozilla-central

Of all the artifacts listed for that job, it's the only one with a content_type of "text", the others are "link" or "raw_html".

The logviewer doesn't have a fallback for !link/raw_html:
https://github.com/mozilla/treeherder/blob/66ee095257552ba2c7b830eba408cb474094cd7b/ui/logviewer.html#L96-L100

Unlike the job details panel in the main UI:
https://github.com/mozilla/treeherder/blob/29a704cc907ec8a64be02dc38d6bff5f555b5771/ui/plugins/job_details/main.html#L5-L12

Ideally we'd also consolidate those two partials (we'd need to add support for toggling the talos specific behaviour beforehand).

:jfrench, up for tweaking that? :-)

Comment 2

[Jonathan French (:jfrench)]

Sure thing :)

Comment 3

[Jonathan French (:jfrench)]

Attachment: PR 887

Tweak PR for review :) It now appears correctly with Maja's particular job.

I spoke with Ed on IRC and we agreed to consolidate the partials if possible, in a follow up.

Comment 4

[Ed Morley [:emorley]]

Comment on attachment 8649965 [details] [review]
PR 887

Thanks :-)

Comment 5

[Jonathan French (:jfrench)]

Follow up for a consolidated partial is bug 1196360.

Comment 6

[Treeherder GitHub Bugbot]

Commit pushed to master at https://github.com/mozilla/treeherder

https://github.com/mozilla/treeherder/commit/a4b0ae927ffaebb05679ddd5608660f2aa2c4b91
Bug 1189498 - Provide a text value fallback for logviewer job details

Comment 7

[Jonathan French (:jfrench)]

Marking fixed per above merge. Will verify on the next push to stage/prod.

Comment 8

[Jonathan French (:jfrench)]

Verified fixed on production. eg.
https://treeherder.mozilla.org/logviewer.html#?job_id=2283038&repo=mozilla-central