Closed
Bug 388831
Opened 17 years ago
Closed 17 years ago
Need to add new tinderboxes to the new-graphs pull script
Categories
(Webtools Graveyard :: Graph Server, defect)
Webtools Graveyard
Graph Server
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: ispiked, Assigned: anodelman)
Details
Attachments
(1 file, 3 obsolete files)
(deleted),
patch
|
Details | Diff | Splinter Review |
It would be nice if we could get the new Linux perf. box's numbers as well as the new Mac OS X leak box's numbers on graphs.mozilla.org.
Their names are as follows:
bl-bldlnx03_fxnewref-linux-tbox-head
bm-xserve11.build.mozilla.org
Reporter | ||
Comment 1•17 years ago
|
||
I added the two new Linux tinderboxen, the Mac OS X leak box and added the codesighs and codesighs embed tests to the data to pull. Adding these two tests should work automagically, right?
Attachment #275626 -
Flags: review?(anodelman)
Assignee | ||
Comment 2•17 years ago
|
||
Comment on attachment 275626 [details] [diff] [review]
patch
This looks good to me - both of the tinderboxes added are actively being reported from.
As to getting this actually working, after checking in the code we need to ensure that it gets pushed to graphs-stage.m.o and graphs.m.o. After that everything is automagic.
Attachment #275626 -
Flags: review?(anodelman) → review+
Updated•17 years ago
|
Assignee: nobody → ispiked
OS: Windows XP → All
Hardware: PC → All
Reporter | ||
Comment 3•17 years ago
|
||
I filed bug 391450 for getting the content pushed once this lands.
Keywords: checkin-needed
Comment 4•17 years ago
|
||
ispiked, please submit an updated patch for the current tinderboxen. :)
Updated•17 years ago
|
Keywords: checkin-needed
Comment 5•17 years ago
|
||
Attachment #279953 -
Flags: review?(anodelman)
Comment 6•17 years ago
|
||
Note that "fxnewref" has been renamed so that's not necessary, but perf results moved from lnx01 to lnx03 (had to upgrade the OS).
Assignee | ||
Comment 7•17 years ago
|
||
I took the given two patches and combined them into one. Should now handle the machine name changes that have occurred along with pulling in more tests.
Attachment #275626 -
Attachment is obsolete: true
Attachment #279953 -
Attachment is obsolete: true
Attachment #279999 -
Flags: review?(rhelmer)
Attachment #279953 -
Flags: review?(anodelman)
Comment 8•17 years ago
|
||
Comment on attachment 279999 [details] [diff] [review]
combinations of first two patches
fxnewref-linux-tbox is gone now, r=rhelmer with that removed.
Assignee | ||
Comment 9•17 years ago
|
||
Removed fxnewref-linux-tbox from patch.
Assignee: ispiked → anodelman
Attachment #279999 -
Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #279999 -
Flags: review?(rhelmer)
Assignee | ||
Comment 10•17 years ago
|
||
Checking in pull.sh;
/cvsroot/mozilla/webtools/new-graph/utils/pull.sh,v <-- pull.sh
new revision: 1.2; previous revision: 1.1
done
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Updated•8 years ago
|
Product: Webtools → Webtools Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•