|
|
Chromium Code Reviews
Description[tools/perf] Enable smoke coverage for all telemetry benchmarks
BUG=548652
Patch Set 1 #Patch Set 2 : Add Webkit/PerformanceTests deps #Patch Set 3 : Add path to pywebsocket lib #
Total comments: 1
Messages
Total messages: 23 (13 generated)
Description was changed from ========== [tools/perf] Enable smoke coverage for all telemetry benchmarks BUG=548652 ========== to ========== [tools/perf] Enable smoke coverage for all telemetry benchmarks BUG=548652 CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:android_s5_perf_cq;tryserver.chromium.perf:winx64_10_perf_cq;tryserver.chromium.perf:mac_retina_perf_cq ==========
Description was changed from ========== [tools/perf] Enable smoke coverage for all telemetry benchmarks BUG=548652 CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:android_s5_perf_cq;tryserver.chromium.perf:winx64_10_perf_cq;tryserver.chromium.perf:mac_retina_perf_cq ========== to ========== [tools/perf] Enable smoke coverage for all telemetry benchmarks BUG=548652 ==========
The CQ bit was checked by nednguyen@google.com to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/2081403003/1
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: Try jobs failed on following builders: linux_android_rel_ng on tryserver.chromium.android (JOB_FAILED, https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)
Description was changed from ========== [tools/perf] Enable smoke coverage for all telemetry benchmarks BUG=548652 ========== to ========== [tools/perf] Enable smoke coverage for all telemetry benchmarks BUG=548652 CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:android_s5_perf_cq;tryserver.chromium.perf:winx64_10_perf_cq;tryserver.chromium.perf:mac_retina_perf_cq ==========
Description was changed from ========== [tools/perf] Enable smoke coverage for all telemetry benchmarks BUG=548652 CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:android_s5_perf_cq;tryserver.chromium.perf:winx64_10_perf_cq;tryserver.chromium.perf:mac_retina_perf_cq ========== to ========== [tools/perf] Enable smoke coverage for all telemetry benchmarks BUG=548652 ==========
Description was changed from ========== [tools/perf] Enable smoke coverage for all telemetry benchmarks BUG=548652 ========== to ========== [tools/perf] Enable smoke coverage for all telemetry benchmarks BUG=548652 CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:android_s5_perf_cq;tryserver.chromium.perf:winx64_10_perf_cq;tryserver.chromium.perf:mac_retina_perf_cq ==========
Description was changed from ========== [tools/perf] Enable smoke coverage for all telemetry benchmarks BUG=548652 CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:android_s5_perf_cq;tryserver.chromium.perf:winx64_10_perf_cq;tryserver.chromium.perf:mac_retina_perf_cq ========== to ========== [tools/perf] Enable smoke coverage for all telemetry benchmarks BUG=548652 ==========
nednguyen@google.com changed reviewers: + dtu@chromium.org, kbr@chromium.org, sullivan@chromium.org
Not ready to land yet, some issues remain: 1) Some benchmarks are failing due to browser crashes. 2) Android is not yet sharded, taking 32 minutes in total. For 1), not sure what is the best way to disable those benchmarks. It's weird that v8.infinite_scroll is only crashing on cq (https://chromium-swarm.appspot.com/user/task/2f92d7f81557dd10) and not on the perf waterfall. Is that because the chromium build on CQ is different from the one on waterfall? For 2), Ken: we haven't support sharding for Android yet?
nednguyen@google.com changed reviewers: + jbudorick@chromium.org
On 2016/06/22 22:17:49, nednguyen wrote: > Not ready to land yet, some issues remain: > 1) Some benchmarks are failing due to browser crashes. > 2) Android is not yet sharded, taking 32 minutes in total. > > For 1), not sure what is the best way to disable those benchmarks. It's weird > that v8.infinite_scroll is only crashing on cq > (https://chromium-swarm.appspot.com/user/task/2f92d7f81557dd10) and not on the > perf waterfall. Is that because the chromium build on CQ is different from the > one on waterfall? > > For 2), Ken: we haven't support sharding for Android yet? +John on the status of sharding Android
On 2016/06/22 22:17:49, nednguyen wrote: > Not ready to land yet, some issues remain: > 1) Some benchmarks are failing due to browser crashes. I am suspecting the crash happens because we skip wait action: https://codereview.chromium.org/2090893002/ may fix it.
On 2016/06/22 22:45:00, nednguyen wrote: > On 2016/06/22 22:17:49, nednguyen wrote: > > Not ready to land yet, some issues remain: > > 1) Some benchmarks are failing due to browser crashes. > > 2) Android is not yet sharded, taking 32 minutes in total. > > > > For 1), not sure what is the best way to disable those benchmarks. It's weird > > that v8.infinite_scroll is only crashing on cq > > (https://chromium-swarm.appspot.com/user/task/2f92d7f81557dd10) and not on the > > perf waterfall. Is that because the chromium build on CQ is different from the > > one on waterfall? > > > > For 2), Ken: we haven't support sharding for Android yet? > > +John on the status of sharding Android Ned, I'm not sure what you're asking. It looks like you enabled sharding yesterday in https://codereview.chromium.org/2089633003
On 2016/06/22 22:17:49, nednguyen wrote: > Not ready to land yet, some issues remain: > 1) Some benchmarks are failing due to browser crashes. > 2) Android is not yet sharded, taking 32 minutes in total. > > For 1), not sure what is the best way to disable those benchmarks. It's weird > that v8.infinite_scroll is only crashing on cq > (https://chromium-swarm.appspot.com/user/task/2f92d7f81557dd10) and not on the > perf waterfall. Is that because the chromium build on CQ is different from the > one on waterfall? The perf waterfall runs release official builds. So there are differences. Maybe try emailing chrome-windows@ for help? > For 2), Ken: we haven't support sharding for Android yet?
The CQ bit was checked by nednguyen@google.com to run a CQ dry run
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/2081403003/40001
The CQ bit was unchecked by commit-bot@chromium.org
Dry run: Try jobs failed on following builders: linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
nednguyen@google.com changed reviewers: + eyaich@chromium.org
https://codereview.chromium.org/2081403003/diff/40001/tools/perf/BUILD.gn File tools/perf/BUILD.gn (right): https://codereview.chromium.org/2081403003/diff/40001/tools/perf/BUILD.gn#new... tools/perf/BUILD.gn:17: +Emily: this CL uncovers some of the missing deps |
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
