803357 - Add support for dxr-mozilla-central so it does not appear as "?"

Status: RESOLVED FIXED

(Tree Management Graveyard :: TBPL, defect)

Description

[Ed Morley [:emorley]]

Bug 802202 added support for showing unknown platforms/suites (is not yet in production).

Whilst testing the patch there I noticed we don't support dxr-mozilla-central.

Whilst we may not be too fussed about it - and can just hide it using the hidden builders list - we should still add support for it, so we don't get the "?" when using &noignore=1.

It's built on Linux64, so I guess we could put it on that row - though "Other" (which it is currently defaulting to) may actually be more appropriate.

Build log:
https://tbpl.mozilla.org/php/getParsedLog.php?id=16229815&tree=Firefox

Comment 1

[Ed Morley [:emorley]]

Attachment: Patch v1

The dxr builds are opt and performed on Linux64, so this patch makes them display on the existing Linux64 opt row as "Dxr", with tooltip "DXR Indices Build".

Happy to adjust the tooltip/'Dxr' or put on an "Other" row instead of Linux64 opt, if deemed more suitable.

Comment 2

[Ed Morley [:emorley]]

Could you take a look at the strings in comment 1 and see if they are suitable? :-)

Comment 3

[Joshua Cranmer [:jcranmer]]

Eventually, I think we want DXR indexing to run on multiple platforms, so keeping it on Linux64 opt makes sense to me.

I think I'd prefer "DXR Index Build" instead of "DXR Indices Build".

Comment 4

[Ed Morley [:emorley]]

Attachment: Patch v2

(In reply to Joshua Cranmer [:jcranmer] from comment #3)
> Eventually, I think we want DXR indexing to run on multiple platforms, so
> keeping it on Linux64 opt makes sense to me.
> 
> I think I'd prefer "DXR Index Build" instead of "DXR Indices Build".

Have s/Indices/Index/

Regarding other platforms, when we come to that point we'll need to update the buildername to include platform (on the buildbot side) and update TBPL appropriately - since it currently only specifies 'dxr-' and tree, but we can cross that bridge later :-)

(Will land this after the patches in bug 786314 land, so I don't have to rebase them all; also the DXR builds will need to stay hidden until they green up, due to bug 803530.)

Comment 5

[Ed Morley [:emorley]]

https://hg.mozilla.org/users/mstange_themasta.com/tinderboxpushlog/rev/3fd141d72137