Closed Bug 803357 Opened 9 years ago Closed 9 years ago

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

Categories

(Tree Management Graveyard :: TBPL, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: emorley, Assigned: emorley)

References

Details

Attachments

(1 file, 1 obsolete file)

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
Blocks: 731256
Attached patch Patch v1 (obsolete) — Splinter Review
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.
Could you take a look at the strings in comment 1 and see if they are suitable? :-)
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".
Attached patch Patch v2Splinter Review
(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.)
Attachment #673217 - Attachment is obsolete: true
Depends on: 805201
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Product: Webtools → Tree Management
Product: Tree Management → Tree Management Graveyard
You need to log in before you can comment on or make changes to this bug.