388831 - Need to add new tinderboxes to the new-graphs pull script

Status: RESOLVED FIXED

(Webtools Graveyard :: Graph Server, defect)

Description

[Adam Guthrie]

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

Comment 1

[Adam Guthrie]

Attachment: patch

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?

Comment 2

[alice nodelman [:alice] [:anode]]

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.

Comment 3

[Adam Guthrie]

I filed bug 391450 for getting the content pushed once this lands.

Comment 4

[Reed Loden [:reed]]

ispiked, please submit an updated patch for the current tinderboxen. :)

Comment 5

[Robert Helmer [:rhelmer]]

Attachment: add bl-bldlnx03 and bm-xserve11

Comment 6

[Robert Helmer [:rhelmer]]

Note that "fxnewref" has been renamed so that's not necessary, but perf results moved from lnx01 to lnx03 (had to upgrade the OS).

Comment 7

[alice nodelman [:alice] [:anode]]

Attachment: combinations of first two patches

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.

Comment 8

[Robert Helmer [:rhelmer]]

Comment on attachment 279999 [details] [diff] [review]
combinations of first two patches

fxnewref-linux-tbox is gone now, r=rhelmer with that removed.

Comment 9

[alice nodelman [:alice] [:anode]]

Attachment: above patch minus fxnewref-linux-tbox

Removed fxnewref-linux-tbox from patch.

Comment 10

[alice nodelman [:alice] [:anode]]

Checking in pull.sh;
/cvsroot/mozilla/webtools/new-graph/utils/pull.sh,v  <--  pull.sh
new revision: 1.2; previous revision: 1.1
done