Stop adding a "header" property to the text_log_summary

RESOLVED FIXED

Status

Tree Management
Treeherder: Data Ingestion
P3
normal
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: emorley, Assigned: emorley)

Tracking

Details

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
Bug 1057341 made the log viewer display job details from the 
use the job data /jobs/ endpoint, rather than from the "header" property of the text_log_summary.

As such, we can stop storing "header" when we parse the buildbot logs \o/

eg:
https://treeherder.mozilla.org/logviewer.html#?job_id=10925086&repo=mozilla-inbound
-> https://treeherder.mozilla.org/api/project/mozilla-inbound/artifact/?job_id=10925086&name=text_log_summary

    "header": {
        "slave": "tst-linux64-spot-1108",
        "buildid": "20150618021242",
        "builder": "mozilla-inbound_ubuntu64_vm-debug_test-mochitest-devtools-chrome-3",
        "results": "warnings (1)",
        "starttime": "1434620359.27",
        "builduid": "fe1094561aef45b6bdafdbdc8225822d",
        "revision": "fd2961b995ab2155e39ee16e623ec95d324fd11b"
    },
(Assignee)

Updated

3 years ago
Depends on: 1175882
(Assignee)

Comment 1

3 years ago
Created attachment 8624198 [details] [review]
Stop adding a "header" property to the text_log_summary
Attachment #8624198 - Flags: review?(cdawson)
Comment on attachment 8624198 [details] [review]
Stop adding a "header" property to the text_log_summary

This is awesome.  Thanks for removing it.  :)
Attachment #8624198 - Flags: review?(cdawson) → review+

Comment 3

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

https://github.com/mozilla/treeherder/commit/8637384d8cda467c742c7dceef18555d8f828e08
Bug 1175870 - Stop adding a "header" property to the text_log_summary

The header property is no longer used, since bug 1057341 made the log
viewer display job details from /jobs/ endpoint instead.
(Assignee)

Comment 4

3 years ago
:-)
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.