788498 - add tbpl code for chromez suite

Status: RESOLVED FIXED

(Tree Management Graveyard :: TBPL, defect)

Description

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

we had to switch chromer to be chromez when adding tresize into the suite.  This is showing up as a 'T' on tbpl and should be a 'c'

Comment 1

[Ed Morley [:emorley]]

I presume branches are still running chromer, so we need both, not just s/chromer/chromez/ right? :-)

Comment 2

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

yes, older branches have chromer, but both could have a 'c' letter.

Comment 3

[Ed Morley [:emorley]]

We currently have all of the variants:
    /talos.*chrome(_mac)?(\.2)?(r)?$/i.test(name) ? "Talos chrome" :

Mapping to:
    "Talos chrome" : "c",

So presuming we are happy lumping chromez in with the others, we should just need to:
    s/(r)?/(r|z)?/

Comment 4

[Ed Morley [:emorley]]

Attachment: Patch v1

Tested locally and looks good :-)

Comment 5

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

Comment on attachment 658857 [details] [diff] [review]
Patch v1

Review of attachment 658857 [details] [diff] [review]:
-----------------------------------------------------------------

looks good to me.

Comment 6

[Ed Morley [:emorley]]

Checked in:


(But as normal, needs a push to prod; which is currently blocked on making sure bug 718632 is working as expected, now that we fixed the server dependency problems).

Comment 7

[Ed Morley [:emorley]]

(In reply to Ed Morley [:edmorley] from comment #6)
> Checked in:

http://hg.mozilla.org/users/mstange_themasta.com/tinderboxpushlog/rev/8f3bcd650aeb