add graph server targets for new talos names for e10s

RESOLVED FIXED

Status

Webtools Graveyard
Graph Server
RESOLVED FIXED
3 years ago
a year ago

People

(Reporter: jmaher, Assigned: jmaher)

Tracking

(Blocks: 1 bug)

Dependency tree / graph

Details

Attachments

(3 attachments, 1 obsolete attachment)

(Assignee)

Description

3 years ago
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).
(Assignee)

Comment 1

3 years ago
Created attachment 8515186 [details] [diff] [review]
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.
Assignee: nobody → jmaher
Status: NEW → ASSIGNED
Attachment #8515186 - Flags: review?(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)");
(Assignee)

Comment 3

3 years ago
Created attachment 8515190 [details] [diff] [review]
linux 32 + 64 machine names + platforms for e10s (1.1)

fix the cosmetic issue.
Attachment #8515186 - Attachment is obsolete: true
Attachment #8515186 - Flags: review?(rhelmer)
Attachment #8515190 - Flags: review?(rhelmer)
Attachment #8515190 - Flags: review?(rhelmer) → review+
(Assignee)

Comment 4

3 years ago
https://hg.mozilla.org/graphs/rev/08857a00e52c
(Assignee)

Updated

3 years ago
Blocks: 1050706
Component: Graph Server → DMD
Product: Webtools → Core
Blocks: 1058354
tracking-e10s: --- → +
Component: DMD → Graph Server
Product: Core → Webtools
(Assignee)

Comment 5

3 years ago
Created attachment 8517572 [details] [diff] [review]
fix existing linux platform id's, add new platforms (1.0)
Attachment #8517572 - Flags: review?(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?
Attachment #8517572 - Flags: review?(rhelmer) → review+
(Assignee)

Comment 7

3 years ago
http://hg.mozilla.org/graphs/rev/2008beeb3f24
(Assignee)

Comment 8

3 years ago
Created attachment 8517694 [details] [diff] [review]
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!
Attachment #8517694 - Flags: review?(rhelmer)
(Assignee)

Updated

3 years ago
Blocks: 1078391
Attachment #8517694 - Flags: review?(rhelmer) → review+
(Assignee)

Comment 9

3 years ago
http://hg.mozilla.org/graphs/rev/c9d016a168aa
(Assignee)

Updated

3 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Product: Webtools → Webtools Graveyard
You need to log in before you can comment on or make changes to this bug.