test suites w/ patch not displayed in testType selector for android try bots |
|||
Issue description^ e.g., there are options for "android_webview_test_apk" and "android_webview_test_apk:", but not for "android_webview_test_apk (with patch)" despite JSONs being uploaded for that: http://test-results.appspot.com/testfile?builder=linux_android_rel_ng&testtype=android_webview_test_apk%20(with%20patch)
,
Nov 1 2016
re how that list is generated: 😱
,
Nov 10 2016
So what's the next step here? Who's expected to act on this...? After looking for other results this week, it seems that this makes the flakiness dashboard unusable for the android try bots (at least).
,
Nov 10 2016
Also see issue 663713. If we allow searching by test name alone, this issue will become less urgent, but the fix suggested by Erik is probably much faster than fixing the issue 663713.
,
Dec 2 2016
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/ba0044d3eac27f216508c461ca615436f3892b2f commit ba0044d3eac27f216508c461ca615436f3892b2f Author: Erik Staab <estaab@google.com> Date: Thu Dec 01 14:01:21 2016 test-results: Remove dependency on buildbot / milo. This changes the way we generate the list of known (master, builder, test) tuples to use what has recently been uploaded instead of scraping step names from recent passing buildbot builds. The benefits to doing this are: 1) No more outages from empty builder data caused by milo queries that timeout from being too large. 2) The javascript client will now only fetch results for combinations that exist instead of an approximate set. 3) New test combinations will be able to upload and serve without needing to be whitelisted. BUG= 666889 ,659952, 655967 , 666810 , 661047 , 643944 , 405799 Change-Id: Icd0a8a88eecbd920677f704cf816b143e7437e50 Reviewed-on: https://chromium-review.googlesource.com/413142 Commit-Queue: Erik Staab <estaab@chromium.org> Reviewed-by: Sergiy Byelozyorov <sergiyb@chromium.org> [modify] https://crrev.com/ba0044d3eac27f216508c461ca615436f3892b2f/appengine/test_results/index.yaml [delete] https://crrev.com/fb6cd3035cc307a93948a87be99320e24ece75c0/go/src/infra/appengine/test-results/buildextract/buildextract.go [delete] https://crrev.com/fb6cd3035cc307a93948a87be99320e24ece75c0/go/src/infra/appengine/test-results/buildextract/buildextract.infra_testing [delete] https://crrev.com/fb6cd3035cc307a93948a87be99320e24ece75c0/go/src/infra/appengine/test-results/buildextract/buildextract_test.go [delete] https://crrev.com/fb6cd3035cc307a93948a87be99320e24ece75c0/go/src/infra/appengine/test-results/buildextract/testing.go [delete] https://crrev.com/fb6cd3035cc307a93948a87be99320e24ece75c0/go/src/infra/appengine/test-results/buildextract/testing_test.go [modify] https://crrev.com/ba0044d3eac27f216508c461ca615436f3892b2f/go/src/infra/appengine/test-results/cron.yaml [modify] https://crrev.com/ba0044d3eac27f216508c461ca615436f3892b2f/go/src/infra/appengine/test-results/frontend/builders.go [modify] https://crrev.com/ba0044d3eac27f216508c461ca615436f3892b2f/go/src/infra/appengine/test-results/frontend/builders_test.go [modify] https://crrev.com/ba0044d3eac27f216508c461ca615436f3892b2f/go/src/infra/appengine/test-results/frontend/handlers.go
,
Dec 5 2016
Looks correct now: http://test-results.appspot.com/testfile?builder=linux_android_rel_ng&testtype=android_webview_test_apk%20(with%20patch)
,
Dec 5 2016
:D |
|||
►
Sign in to add a comment |
|||
Comment 1 by estaab@chromium.org
, Nov 1 2016