Update script which gathers test run times for typ switchover.

Update script which gathers test run times for typ switchover.

The script was correctly merging the tries, but needed to specifically
extract the per-test run times.

BUG=636153
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2708803002
Cr-Commit-Position: refs/heads/master@{#452362}
Committed: https://chromium.googlesource.com/chromium/src/+/85a8ce8dd67e4ed84fa7846b3f47f2b8b45b906e

[Ken Russell (switch to Gerrit), 2017-02-21 02:53:38]

Description was changed from

==========
Update script which gathers test run times for typ switchover.

The script was correctly merging the tries, but needed to specifically
extract the per-test run times.

BUG=636153
==========

to

==========
Update script which gathers test run times for typ switchover.

The script was correctly merging the tries, but needed to specifically
extract the per-test run times.

BUG=636153
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[Ken Russell (switch to Gerrit), 2017-02-21 02:55:26]

kbr@chromium.org changed reviewers:
+ nednguyen@google.com, zmo@chromium.org

[Ken Russell (switch to Gerrit), 2017-02-21 02:55:27]

Ned or Mo: please review at your leisure.

This should be landed after the typ cutover in
https://codereview.chromium.org/2663813003/ . It was tested against one of the
successful tryjobs from that CL:

./content/test/gpu/gather_swarming_json_results.py --waterfall
tryserver.chromium.mac --bot mac_optional_gpu_tests_rel --build 7198 --output
webgl2-output.json

[nednguyen, 2017-02-21 13:39:39]

lgtm with nit

https://codereview.chromium.org/2708803002/diff/1/content/test/gpu/gather_swa...
File content/test/gpu/gather_swarming_json_results.py (right):

https://codereview.chromium.org/2708803002/diff/1/content/test/gpu/gather_swa...
content/test/gpu/gather_swarming_json_results.py:150: dest[node_name] =
node['times'][0]
hmhh, should we use dest[node_name] = sum(node['times'])/len(node['times'])
instead?

[Ken Russell (switch to Gerrit), 2017-02-22 01:19:13]

https://codereview.chromium.org/2708803002/diff/1/content/test/gpu/gather_swa...
File content/test/gpu/gather_swarming_json_results.py (right):

https://codereview.chromium.org/2708803002/diff/1/content/test/gpu/gather_swa...
content/test/gpu/gather_swarming_json_results.py:150: dest[node_name] =
node['times'][0]
On 2017/02/21 13:39:39, nednguyen wrote:
> hmhh, should we use dest[node_name] = sum(node['times'])/len(node['times'])
> instead?

Thanks, good idea. Done.

[Zhenyao Mo, 2017-02-23 01:23:55]

lgtm

[Ken Russell (switch to Gerrit), 2017-02-23 01:29:15]

The CQ bit was checked by kbr@chromium.org

[Ken Russell (switch to Gerrit), 2017-02-23 01:29:15]

The patchset sent to the CQ was uploaded after l-g-t-m from nednguyen@google.com
Link to the patchset: https://codereview.chromium.org/2708803002/#ps20001
(title: "Addressed review feedback from nednguyen.")

[commit-bot: I haz the power, 2017-02-23 01:30:02]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-23 02:59:27]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1487813355317570,
"parent_rev": "1b901526d84b86839e58b340a117eef6b40a7706", "commit_rev":
"85a8ce8dd67e4ed84fa7846b3f47f2b8b45b906e"}

[commit-bot: I haz the power, 2017-02-23 03:00:01]

Description was changed from

==========
Update script which gathers test run times for typ switchover.

The script was correctly merging the tries, but needed to specifically
extract the per-test run times.

BUG=636153
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Update script which gathers test run times for typ switchover.

The script was correctly merging the tries, but needed to specifically
extract the per-test run times.

BUG=636153
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2708803002
Cr-Commit-Position: refs/heads/master@{#452362}
Committed:
https://chromium.googlesource.com/chromium/src/+/85a8ce8dd67e4ed84fa7846b3f47...
==========

[commit-bot: I haz the power, 2017-02-23 03:00:02]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/85a8ce8dd67e4ed84fa7846b3f47...