Make test_results_presentation.py merge swarming shards |
||
Issue descriptionRight now it fails when collecting the results of a swarming task split into multiple shards: https://build.chromium.org/p/tryserver.chromium.android/builders/linux_android_rel_ng/builds/318556
,
Jun 22 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/e82ecb088efcd8a4f8d28e6ff8b9185c2ba90f16 commit e82ecb088efcd8a4f8d28e6ff8b9185c2ba90f16 Author: hzl <hzl@google.com> Date: Thu Jun 22 08:51:58 2017 Merge test results presentation of multiple shards. BUG= 733824 Review-Url: https://codereview.chromium.org/2947603002 Cr-Commit-Position: refs/heads/master@{#481482} [add] https://crrev.com/e82ecb088efcd8a4f8d28e6ff8b9185c2ba90f16/build/android/pylib/results/presentation/__init__.py [add] https://crrev.com/e82ecb088efcd8a4f8d28e6ff8b9185c2ba90f16/build/android/pylib/results/presentation/standard_gtest_merge.py [modify] https://crrev.com/e82ecb088efcd8a4f8d28e6ff8b9185c2ba90f16/build/android/pylib/results/presentation/test_results_presentation.py
,
Jun 26 2017
Did #2 fix this? Should results_presentation now work with multiple shards?
,
Jul 24 2017
Is merging the json files of multiple shards the only thing we need to do to make results_presentation work? If so, then it should be working. Is it easy to test and see?
,
Jul 24 2017
I think this is working now, e.g. chrome_public_test_apk here: https://build.chromium.org/p/chromium.linux/builders/Android%20Tests/builds/44045
,
Jul 24 2017
|
||
►
Sign in to add a comment |
||
Comment 1 by bpastene@chromium.org
, Jun 15 2017