651625 - Add ability to override Tinderbox tree name in TBPL

Status: RESOLVED FIXED

(Tree Management Graveyard :: TBPL, defect)

Description

[Mounir Lamouri (:mounir)]

Attachment: Patch v1

Comment 1

[Mounir Lamouri (:mounir)]

Pushed:
https://hg.mozilla.org/users/mstange_themasta.com/tinderboxpushlog/rev/30f230b0cdf9

Comment 2

[Phil Ringnalda (:philor)]

Backed out in http://hg.mozilla.org/users/mstange_themasta.com/tinderboxpushlog/rev/9f46fb26f51e - it breaks the links to other trees (treeName=Foo works just fine as long as you want to see the pushlog and tinderbox of Firefox, not of Foo), and it didn't fix up wrongtree.html, giving differently-broken links in the two different places.

Comment 3

[Lukas Blakk [:lsblakk] use ?needinfo]

Is there an ETA on this? I'm just going through the jetpack un-hiding bugs and noticed that it's still not in tbpl and that's because of this bug.

Comment 4

[Phil Ringnalda (:philor)]

Not really, bug 652491 is considerably more of a blocker than this is. If that made it possible for tbpl to display anything on the jetpack tree, a cunning person could just pick a less particular reviewer for bug 649375 and shove it through as "Jetpack" by explaining that it's the name of the tree and it's the name of the test suite, so it's actually a better name than "Addon SDK."

Comment 5

[Markus Stange [:mstange]]

Relanded with fix:
http://hg.mozilla.org/users/mstange_themasta.com/tinderboxpushlog/rev/7de6cd8f6c91

Comment 6

[Lukas Blakk [:lsblakk] use ?needinfo]

(In reply to comment #4)
> Not really, bug 652491 is considerably more of a blocker than this is

My bad, I read the dep bugs in the reverse order and caught that post-commenting.  So I've updated bug 649375 regarding printing revs as that would look like for this sort of script automation.

Comment 7

[Markus Stange [:mstange]]

Bustage fix for the interaction with bug 652386:
http://hg.mozilla.org/users/mstange_themasta.com/tinderboxpushlog/rev/136f7fe35da2