661495 - Remove superfluous chart data from endurance reports

Status: RESOLVED FIXED

(Mozilla QA Graveyard :: Mozmill Result Dashboard, defect)

Description

[Dave Hunt [:davehunt] [he/him] ⌚BST]

With the new metrics introduced for Firefox 6 in bug 657508 we now have additional series on the charts for all reports. We need to limit these to only those with relevant values.

Comment 1

[u415141]

A Pivotal Tracker story has been created for this Bug: https://www.pivotaltracker.com/story/show/16726099

Comment 2

[Dave Hunt [:davehunt] [he/him] ⌚BST]

Attachment: Remove mapped/allocated metrics from endurance test reports. v1.0

As discussed with Henrik, I've removed mapped/allocated from the dashboard reports as these only relate to unsupported versions of Firefox (4 & 5). The data will still be gathered and exist, but will not be displayed.

Comment 3

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Comment on attachment 579390 [details] [diff] [review]
Remove mapped/allocated metrics from endurance test reports. v1.0

Looks ok to me and I assume it has been tested. Feel free to land it.

Comment 4

[Dave Hunt [:davehunt] [he/him] ⌚BST]

Landed as:
https://github.com/whimboo/mozmill-dashboard/commit/67828b112f0b7ec64784356220af39ddf7799d0d

Comment 5

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Once I have cleaned-up the dashboards we will have to push to production. I target Monday next week.

Comment 6

[Dave Hunt [:davehunt] [he/him] ⌚BST]

Attachment: Follw-up patch to remove unused global constant. v1.0

I missed removing this global constant from the original patch.

Comment 7

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Former code has already been pushed. Will review in a bit.

Comment 8

[Dave Hunt [:davehunt] [he/him] ⌚BST]

Landed as:
https://github.com/whimboo/mozmill-dashboard/commit/5774a0db8f827fece94750e74c5ca054d1dc83be

Comment 9

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Landed on all dashboards.