1092194 - add graph server targets for new talos names for e10s

Status: RESOLVED FIXED

(Webtools Graveyard :: Graph Server, defect)

Description

[Joel Maher ( :jmaher ) (UTC -8)]

the graph server requires us to have platforms and tests predefined for all data we upload.  Running e10s will need to have a unique way to identify the data so we don't mix it up with non-e10s.

So we can either change the platform names or the test names.  These changes will take place in the graph server and be hacked into talos when we upload the results to match the new values in graph server.

The platform name would be best to change as it wouldn't be too confusing.  In this  scenario we would post to linux e10s, instead of linux; likewise "windows 7 (ix) e10s" instead of "windows 7 (ix)".  Going this route would mean adding a new platform and duplicating the defined machine names as well.  

The other route is hacking the test name to be 'e10s'.  Why I am not a fan of this is the names are already so long in many cases that we could easily overlook the e10s names while viewing results, etc.  The advantage of going this route would be that we could have a clean set of names going forward (the current long names could be better worded).

Comment 1

[Joel Maher ( :jmaher ) (UTC -8)]

Attachment: linux 32 + 64 machine names + platforms for e10s (1.0)

this is a start, we would have to replicate this for osx 10.6, osx 10.8, winXP, win7, and win8.

I want to get the system working for these first and then tackle that problem next.

Comment 2

[Robert Helmer [:rhelmer]]

Comment on attachment 8515186 [details] [diff] [review]
linux 32 + 64 machine names + platforms for e10s (1.0)

Review of attachment 8515186 [details] [diff] [review]:
-----------------------------------------------------------------

::: sql/data.sql
@@ +30,5 @@
>  insert into os_list values (NULL, "Ubuntu HW 12.04");
>  insert into os_list values (NULL, "Ubuntu HW 12.04 x64");
>  insert into os_list values (39, "MacOSX 10.9");
> +insert into os_list values (NULL, "Ubuntu HW 12.04 (e10s)");
> +insert into os_list values (NULL, "Ubuntu HW 12.04 x64 (e10s");

Just cosmetic, but missing end paren before end quote (e10s)");

Comment 3

[Joel Maher ( :jmaher ) (UTC -8)]

Attachment: linux 32 + 64 machine names + platforms for e10s (1.1)

fix the cosmetic issue.

Comment 4

[Joel Maher ( :jmaher ) (UTC -8)]

https://hg.mozilla.org/graphs/rev/08857a00e52c

Comment 5

[Joel Maher ( :jmaher ) (UTC -8)]

Attachment: fix existing linux platform id's, add new platforms (1.0)

Comment 6

[Robert Helmer [:rhelmer]]

Comment on attachment 8517572 [details] [diff] [review]
fix existing linux platform id's, add new platforms (1.0)

Review of attachment 8517572 [details] [diff] [review]:
-----------------------------------------------------------------

lgtm, since the previous patch was run do we care about undoing those changes, to the machines table for example?

Comment 7

[Joel Maher ( :jmaher ) (UTC -8)]

http://hg.mozilla.org/graphs/rev/2008beeb3f24

Comment 8

[Joel Maher ( :jmaher ) (UTC -8)]

Attachment: support osx 10.6, 10.8, win xp, win 7, win 8 (1.0)

ok, lots of machines here- maybe this is the last one!

Comment 9

[Joel Maher ( :jmaher ) (UTC -8)]

http://hg.mozilla.org/graphs/rev/c9d016a168aa