Closed
Bug 766084
Opened 12 years ago
Closed 12 years ago
Add the mozilla-esr10 & fx-team trees to OrangeFactor
Categories
(Tree Management Graveyard :: OrangeFactor, defect)
Tree Management Graveyard
OrangeFactor
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: emorley, Assigned: emorley)
References
Details
(Whiteboard: [sheriff-want])
Attachments
(3 files)
(deleted),
patch
|
jgriffin
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
jgriffin
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
jgriffin
:
review+
|
Details | Diff | Splinter Review |
Ok, I know what you are thinking... but we don't care much about ESR right?
Well true, but we're stuck with the virtually unchanged repo for a year, so it would be nice to identify the top oranges (that we still end up having to star) and uplift any test-only fixes for them (/just disable the things).
At the moment I have no easy way of working out what are the top failures in order to do this.
Assignee | ||
Comment 1•12 years ago
|
||
mcote, I presume this just needs mozilla-esr10 adding to server/orangefactor.conf on prod+stage yeah?
Comment 2•12 years ago
|
||
We'll also need to add the mozilla-esr branch to the WOO logparser.
Assignee | ||
Updated•12 years ago
|
Whiteboard: [sheriff-want]
Assignee | ||
Comment 3•12 years ago
|
||
(In reply to Jonathan Griffin (:jgriffin) from comment #2)
> We'll also need to add the mozilla-esr branch to the WOO logparser.
Is that in another repo?
Comment 4•12 years ago
|
||
(In reply to Ed Morley [:edmorley] from comment #3)
> (In reply to Jonathan Griffin (:jgriffin) from comment #2)
> > We'll also need to add the mozilla-esr branch to the WOO logparser.
>
> Is that in another repo?
I meant mozilla-esr10, http://hg.mozilla.org/releases/mozilla-esr10/. We're currently not parsing logs from tests on this repo, so we'll have to start doing that.
Assignee | ||
Comment 5•12 years ago
|
||
Sorry, by 'it' I was referring to the text "WOO logparser", since my search of the orangefactor repo hadn't found a log parser.
Comment 6•12 years ago
|
||
(In reply to Ed Morley [:edmorley] from comment #5)
> Sorry, by 'it' I was referring to the text "WOO logparser", since my search
> of the orangefactor repo hadn't found a log parser.
Ah yes, that lives at http://hg.mozilla.org/automation/logparser/
Assignee | ||
Comment 7•12 years ago
|
||
Ah, thank you.
I've just had a look, and I spotted the 'pine' repo is listed there:
http://hg.mozilla.org/automation/logparser/file/25c47477904c/logparser/config.py#l25
Do we still need that?
Perhaps we could s/pine/fx-team/ and add mozilla-esr10 ?
Comment 8•12 years ago
|
||
(In reply to Ed Morley [:edmorley] from comment #7)
> Ah, thank you.
>
> I've just had a look, and I spotted the 'pine' repo is listed there:
> http://hg.mozilla.org/automation/logparser/file/25c47477904c/logparser/
> config.py#l25
>
> Do we still need that?
> Perhaps we could s/pine/fx-team/ and add mozilla-esr10 ?
We don't need pine. Those changes would be fine with me.
Assignee | ||
Updated•12 years ago
|
Summary: Add the mozilla-esr10 tree to OrangeFactor → Add the mozilla-esr10 & fx-team trees to OrangeFactor
Assignee | ||
Comment 9•12 years ago
|
||
Assignee: nobody → bmo
Status: NEW → ASSIGNED
Assignee | ||
Comment 10•12 years ago
|
||
+Removes 'places', since no longer used.
We'll need to update the live copy of orangefactor.conf on dev/staging/prod too :-)
Attachment #650104 -
Flags: review?(jgriffin)
Comment 11•12 years ago
|
||
Comment on attachment 650104 [details] [diff] [review]
orangefactor.conf.example changes
Review of attachment 650104 [details] [diff] [review]:
-----------------------------------------------------------------
Looks good, thanks!
Attachment #650104 -
Flags: review?(jgriffin) → review+
Comment 12•12 years ago
|
||
(In reply to Jonathan Griffin (:jgriffin) from comment #11)
> Comment on attachment 650104 [details] [diff] [review]
> orangefactor.conf.example changes
>
> Review of attachment 650104 [details] [diff] [review]:
> -----------------------------------------------------------------
>
> Looks good, thanks!
Landed as http://hg.mozilla.org/automation/orangefactor/rev/f7fbe255bfc9, and deployed on the server. Note that the new trees will have to be added to http://hg.mozilla.org/automation/orangefactor/file/f7fbe255bfc9/html/scripts/woo.utils.js#l27 in order for them to appear in the dropdown in the UI. But the data for the new trunk_trees is already present when viewing "Trunk" data in the UI.
Assignee | ||
Comment 13•12 years ago
|
||
Comment on attachment 650099 [details] [diff] [review]
Logparser changes
Oops, forgot to request review for this one.
Attachment #650099 -
Flags: review?(jgriffin)
Assignee | ||
Comment 14•12 years ago
|
||
Thank you, forgot about the client-side changes :-)
Assignee | ||
Updated•12 years ago
|
Attachment #650505 -
Flags: review?(jgriffin)
Comment 15•12 years ago
|
||
Comment on attachment 650099 [details] [diff] [review]
Logparser changes
Landed as http://hg.mozilla.org/automation/logparser/rev/025da2685aeb and deployed.
Attachment #650099 -
Flags: review?(jgriffin) → review+
Comment 16•12 years ago
|
||
Comment on attachment 650505 [details] [diff] [review]
Update branches in woo.utils.js
Landed as http://hg.mozilla.org/automation/orangefactor/rev/aeff407a34c5 and deployed. You may have to shift+reload to pick up the changes.
Attachment #650505 -
Flags: review?(jgriffin) → review+
Assignee | ||
Comment 17•12 years ago
|
||
Thank you :-)
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•10 years ago
|
Product: Testing → Tree Management
Updated•4 years ago
|
Product: Tree Management → Tree Management Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•