1211905 - Treeherder changes for the splitting of mochitest-oth into a11y and chrome jobs

Status: RESOLVED FIXED

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

Description

[Ed Morley [:emorley]]

Once bug 1211889 has decided the new buildernames, Treeherder will need to be updated before the changes there are deployed.

We'll also need to know desired job symbols and descriptions.

Comment 1

[Joel Maher ( :jmaher ) (UTC -8)]

job symbols:
M(a)
M(c1 c2 c2)

buildernames- still have to figure that out

Comment 2

[Joel Maher ( :jmaher ) (UTC -8)]

buildernames in patch up for review:
+ Windows XP 32-bit mozilla-central opt test mochitest-chrome-1
+ Windows XP 32-bit mozilla-central opt test mochitest-chrome-2
+ Windows XP 32-bit mozilla-central opt test mochitest-chrome-3
+ Windows XP 32-bit mozilla-central pgo test mochitest-a11y

this would be for all platforms and many different branches, just note:
mochitest-chrome-[1-3] -> M(c1/c2/c3)
mochitest-a11y -> M(a)

Comment 3

[Joel Maher ( :jmaher ) (UTC -8)]

is this something you want me to do, if so can you point me to a similar patch which I can then fine the code to edit via the example?

If you want to handle it, please let me know what else you need and a general time window (i.e. this week, next year, etc.)

Comment 4

[Ed Morley [:emorley]]

I'm happy to do it, will fix this week.

Comment 5

[Joel Maher ( :jmaher ) (UTC -8)]

Thanks Ed!  When this is deployed, I will land the buildbot patch!

Comment 6

[Ed Morley [:emorley]]

Attachment: Add support for mochitest-ally

mochitest-chrome was already supported, so I just had to add a test for that part.

The review here is more about symbol/description/buildername; the treeherder implementation parts are pretty standard (and the tests pass) so don't worry about that.

Comment 7

[Joel Maher ( :jmaher ) (UTC -8)]

Comment on attachment 8672719 [details] [review]
Add support for mochitest-ally

I see the test for c2, but I don't see any code for M(c[x]).  I assume this already exists?

Comment 8

[Treeherder GitHub Bugbot]

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

https://github.com/mozilla/treeherder/commit/900ee1a1e42acbcea1aee21cb40323ed76258a32
Bug 1211905 - Add support for mochitest-ally

And add a test for chunked mochitest-chrome.

Comment 9

[Ed Morley [:emorley]]

(In reply to Joel Maher (:jmaher) from comment #7)
> I see the test for c2, but I don't see any code for M(c[x]).  I assume this
> already exists?

See comment 6 :-)

I'll try and do a prod deploy tomorrow.