1179958 - Make treeherder aware of a GTest job

Status: RESOLVED FIXED

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

Description

[Chris Manchester (limited bugmail, email directly)]

I'll make a PR once I get far enough with bug 1179955 to know the builder name.

Comment 1

[Chris Manchester (limited bugmail, email directly)]

Attachment: PR for GTest + Cpp symbol

I'm open to suggestions on a better symbol :)

Comment 2

[Cameron Dawson [:camd]]

Comment on attachment 8629138 [details] [review]
PR for GTest + Cpp symbol

Suggested a new symbol.  Nothing personal!!  :)

Comment 3

[Chris Manchester (limited bugmail, email directly)]

Comment on attachment 8629138 [details] [review]
PR for GTest + Cpp symbol

Thanks for the review, updated with your legit suggestion :)

Comment 4

[Cameron Dawson [:camd]]

Comment on attachment 8629138 [details] [review]
PR for GTest + Cpp symbol

Looks good to me.  I don't remember if you usually merge these yourself.  If not, please let me know and I'll do it for you.  :)

Comment 5

[Chris Manchester (limited bugmail, email directly)]

Great, please merge or give me permission to do so.

Comment 6

[Treeherder GitHub Bugbot]

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

https://github.com/mozilla/treeherder/commit/3431e61614810cf02047fe0c50b98eb5003ffe98
Bug 1179958 - Add Cpp + GTest symbol to treeherder for combined Cpp and Gtest job. r=camd

https://github.com/mozilla/treeherder/commit/f4acb8777b04d062378f5b811bffbf1f111d8a5b
Merge pull request #704 from chmanchester/cpp_gtest

Bug 1179958 - Add Cpp + GTest symbol to treeherder for combined Cpp a…

Comment 7

[Cameron Dawson [:camd]]

:chmanchester - Merged.  Yours to resolve.  :)