1175870 - Stop adding a "header" property to the text_log_summary

Status: RESOLVED FIXED

(Tree Management :: Treeherder: Data Ingestion, defect, P3)

Description

[Ed Morley [:emorley]]

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"
    },

Comment 1

[Ed Morley [:emorley]]

Attachment: Stop adding a "header" property to the text_log_summary

Comment 2

[Cameron Dawson [:camd]]

Comment on attachment 8624198 [details] [review]
Stop adding a "header" property to the text_log_summary

This is awesome.  Thanks for removing it.  :)

Comment 3

[Treeherder GitHub Bugbot]

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.

Comment 4

[Ed Morley [:emorley]]

:-)