Buildernames currently used include: * mochitest-N * mochitests-N/T * mochitest-other * mochitest-browser-chrome and now on Cedar (post bug 804026): * mochitests-N We should: (a) Stick to always using the singular form (like we do for reftest/crashtest etc) (b) Get rid of the forward slash onwards This bug will make the buildbot related changes & I'll file dependants for TBPL/OrangeFactor etc.

Comment on attachment 674857 [details] [diff] [review] singular mochitest lgtm

Comment on attachment 674857 [details] [diff] [review] singular mochitest Thank you for the patch - unfortunately I didn't realise you were also working on this (I'd set myself as the assignee when filing, though this is probably not very clear in the bugmail), so have been creating patches myself too. The patches I have change the naming everywhere, rather than just in mozharness so are a superset of this one. Hope this didn't waste too much of your time, sorry!

* Gets rid of the forward slash suffix * Updates the trychooser tests to match the new output from this patch and the others in the series * Also removes the forward slashes from reftests in the trychooser tests, given that they've been gone for a while now

Comment on attachment 675101 [details] [diff] [review] buildbotcustom v1 We may also need to fix the try chooser logic here: http://hg.mozilla.org/build/buildbotcustom/file/a2afae086fe0/try_parser.py#l27

Ah yes, try_parser.py did need changing too (only difference from the v1 patch). I've taken out the mochitest-N special-casing, since the suite names will now be covered by the else clause :-) Requesting re-review for sanity check.

Comment on attachment 675209 [details] [diff] [review] buildbotcustom v2 I *think* that's good.

Thank you :-) https://hg.mozilla.org/build/buildbotcustom/rev/6d3d085eb0dd https://hg.mozilla.org/build/buildbot-configs/rev/0d2accf9795e https://hg.mozilla.org/build/buildapi/rev/33a9bfaf6264

In production.

Thank you :-)