The new platform mode test runner architecture will simplify the code base, so we want to move the android perf test runner to this as well.
Any progress here?
CL is in review. Its a big one so there has been quite a bit of back and forth. https://codereview.chromium.org/2012323002/
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/8db92cc2828e9757b91662a1ed7057f17f8024ed commit 8db92cc2828e9757b91662a1ed7057f17f8024ed Author: rnephew <rnephew@chromium.org> Date: Wed Jul 13 03:46:17 2016 [Android] Implement perf tests to platform mode. BUG= 615157 , 590229 Review-Url: https://codereview.chromium.org/2012323002 Cr-Commit-Position: refs/heads/master@{#404978} [modify] https://crrev.com/8db92cc2828e9757b91662a1ed7057f17f8024ed/build/android/pylib/base/test_instance_factory.py [modify] https://crrev.com/8db92cc2828e9757b91662a1ed7057f17f8024ed/build/android/pylib/base/test_run_factory.py [modify] https://crrev.com/8db92cc2828e9757b91662a1ed7057f17f8024ed/build/android/pylib/local/device/local_device_environment.py [add] https://crrev.com/8db92cc2828e9757b91662a1ed7057f17f8024ed/build/android/pylib/local/device/local_device_perf_test_run.py [modify] https://crrev.com/8db92cc2828e9757b91662a1ed7057f17f8024ed/build/android/pylib/local/device/local_device_test_run.py [add] https://crrev.com/8db92cc2828e9757b91662a1ed7057f17f8024ed/build/android/pylib/perf/perf_test_instance.py [modify] https://crrev.com/8db92cc2828e9757b91662a1ed7057f17f8024ed/build/android/test_runner.py [modify] https://crrev.com/8db92cc2828e9757b91662a1ed7057f17f8024ed/build/android/test_runner.pydeps
The following revision refers to this bug: https://chromium.googlesource.com/chromium/tools/build.git/+/dfdf1345605f32aad5c842247e94e130c274154b commit dfdf1345605f32aad5c842247e94e130c274154b Author: rnephew <rnephew@chromium.org> Date: Wed Jul 20 17:15:12 2016 [Android] Add option to run perf tests in platform mode. BUG= 615157 Review-Url: https://codereview.chromium.org/2146783003 [modify] https://crrev.com/dfdf1345605f32aad5c842247e94e130c274154b/scripts/slave/recipe_modules/chromium_android/api.py [add] https://crrev.com/dfdf1345605f32aad5c842247e94e130c274154b/scripts/slave/recipe_modules/chromium_android/example.expected/enable_platform_mode_basic.json [modify] https://crrev.com/dfdf1345605f32aad5c842247e94e130c274154b/scripts/slave/recipe_modules/chromium_android/example.py [modify] https://crrev.com/dfdf1345605f32aad5c842247e94e130c274154b/scripts/slave/recipe_modules/chromium_tests/steps.py [modify] https://crrev.com/dfdf1345605f32aad5c842247e94e130c274154b/scripts/slave/recipes/android/perf.py
The following revision refers to this bug: https://chrome-internal.googlesource.com/chrome/tools/build_limited/scripts/slave/+/3db7e649b3f1f8a4b820450177baa3b4e0a76be4 commit 3db7e649b3f1f8a4b820450177baa3b4e0a76be4 Author: recipe-roller <recipe-roller@chromium.org> Date: Wed Jul 20 17:18:02 2016
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/9e05300fcb133b7772ae7839ec25e0eafffbeb69 commit 9e05300fcb133b7772ae7839ec25e0eafffbeb69 Author: recipe-roller <recipe-roller@chromium.org> Date: Wed Jul 20 17:36:30 2016 Roll recipe dependencies (trivial). This is an automated CL created by the recipe roller. This CL rolls recipe changes from upstream projects (e.g. depot_tools) into downstream projects (e.g. tools/build). More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug (or complain) build: https://crrev.com/dfdf1345605f32aad5c842247e94e130c274154b [Android] Add option to run perf tests in platform mode. (rnephew@chromium.org) R=rnephew@chromium.org BUG= 615157 TBR=martiniss@chromium.org,phajdan.jr@chromium.org Review-Url: https://codereview.chromium.org/2166793003 [modify] https://crrev.com/9e05300fcb133b7772ae7839ec25e0eafffbeb69/infra/config/recipes.cfg
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/e7e78b17bec648e227b1fe37f45fa45f9e72548b commit e7e78b17bec648e227b1fe37f45fa45f9e72548b Author: recipe-roller <recipe-roller@chromium.org> Date: Wed Jul 20 17:56:30 2016 Roll recipe dependencies (trivial). This is an automated CL created by the recipe roller. This CL rolls recipe changes from upstream projects (e.g. depot_tools) into downstream projects (e.g. tools/build). More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug (or complain) build: https://crrev.com/dfdf1345605f32aad5c842247e94e130c274154b [Android] Add option to run perf tests in platform mode. (rnephew@chromium.org) https://crrev.com/cdb4007803a43e21800f0597372fa627466c3db0 Roll recipe dependencies (trivial). (recipe-roller@chromium.org) https://crrev.com/b3e62f30e334a0eb3c8dfb1a85744f7bdaedcc44 Pass buildbucket.build.id to cbuildbot as --buildbucket-id. (nxia@google.com) depot_tools: https://crrev.com/1f2c9b78aeda31d7abbda56e2c9cc3f08bdb772b Add config to gclient to pull in Chromedriver test deps. (mikecase@chromium.org) R=rnephew@chromium.org,recipe-roller@chromium.org,mikecase@chromium.org,martiniss@chromium.org,phajdan.jr@chromium.org,nxia@google.com BUG= chromium:629688 , 615157 TBR=martiniss@chromium.org,phajdan.jr@chromium.org Review-Url: https://codereview.chromium.org/2161403003 [modify] https://crrev.com/e7e78b17bec648e227b1fe37f45fa45f9e72548b/infra/config/recipes.cfg
The following revision refers to this bug: https://chromium.googlesource.com/chromium/tools/build.git/+/bf806e109ffa60364a3ec9983371555b69b662b4 commit bf806e109ffa60364a3ec9983371555b69b662b4 Author: rnephew <rnephew@chromium.org> Date: Wed Jul 20 21:42:12 2016 [Android] Enable platform mode for perf tests on Galaxy S5 BUG= 615157 Review-Url: https://codereview.chromium.org/2171463002 [modify] https://crrev.com/bf806e109ffa60364a3ec9983371555b69b662b4/scripts/slave/recipes/android/perf.expected/full_chromium_perf_Android_Galaxy_S5_Perf__1_.json [modify] https://crrev.com/bf806e109ffa60364a3ec9983371555b69b662b4/scripts/slave/recipes/android/perf.expected/full_chromium_perf_Android_Galaxy_S5_Perf__2_.json [modify] https://crrev.com/bf806e109ffa60364a3ec9983371555b69b662b4/scripts/slave/recipes/android/perf.expected/full_chromium_perf_Android_Galaxy_S5_Perf__3_.json [modify] https://crrev.com/bf806e109ffa60364a3ec9983371555b69b662b4/scripts/slave/recipes/android/perf.py
The following revision refers to this bug: https://chrome-internal.googlesource.com/chrome/tools/build_limited/scripts/slave/+/3954a514e0ce320aefc0ded8201a47a3af236904 commit 3954a514e0ce320aefc0ded8201a47a3af236904 Author: recipe-roller <recipe-roller@chromium.org> Date: Wed Jul 20 21:48:00 2016
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/bd4f4066e85f71c30f54ea117cf704c73407e7b3 commit bd4f4066e85f71c30f54ea117cf704c73407e7b3 Author: recipe-roller <recipe-roller@chromium.org> Date: Wed Jul 20 22:08:12 2016 Roll recipe dependencies (trivial). This is an automated CL created by the recipe roller. This CL rolls recipe changes from upstream projects (e.g. depot_tools) into downstream projects (e.g. tools/build). More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug (or complain) build: https://crrev.com/342c7abb4ec378b3af0c38316f6b03626c3a0461 Roll recipe dependencies (trivial). (recipe-roller@chromium.org) https://crrev.com/bf806e109ffa60364a3ec9983371555b69b662b4 [Android] Enable platform mode for perf tests on Galaxy S5 (rnephew@chromium.org) depot_tools: https://crrev.com/fac10e1c9a6e6cbf0031a9b4a04c3316803607c4 [Docs] Remove mention of the 'blink' fetch recipe, and re-run asciidoc. (jbroman@chromium.org) https://crrev.com/d224cb7e497a4bf950404320328b906d57ee2a71 Update README a bit and add Contributing info (techtonik@gmail.com) R=rnephew@chromium.org,techtonik@gmail.com,jbroman@chromium.org,recipe-roller@chromium.org,martiniss@chromium.org,phajdan.jr@chromium.org BUG= 615157 TBR=martiniss@chromium.org,phajdan.jr@chromium.org Review-Url: https://codereview.chromium.org/2167863002 [modify] https://crrev.com/bd4f4066e85f71c30f54ea117cf704c73407e7b3/infra/config/recipes.cfg
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/4f07c1664f22135b4dd2ba39847456396d6b11c3 commit 4f07c1664f22135b4dd2ba39847456396d6b11c3 Author: recipe-roller <recipe-roller@chromium.org> Date: Wed Jul 20 22:27:56 2016 Roll recipe dependencies (trivial). This is an automated CL created by the recipe roller. This CL rolls recipe changes from upstream projects (e.g. depot_tools) into downstream projects (e.g. tools/build). More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug (or complain) build: https://crrev.com/bf806e109ffa60364a3ec9983371555b69b662b4 [Android] Enable platform mode for perf tests on Galaxy S5 (rnephew@chromium.org) https://crrev.com/4d30cb1a645ad31468a2af390bbbc9b550b391d5 Add up and down arrows for ascend and descend sort. (hzl@google.com) R=rnephew@chromium.org,hzl@google.com BUG=605572, 615157 TBR=martiniss@chromium.org,phajdan.jr@chromium.org Review-Url: https://codereview.chromium.org/2171543003 [modify] https://crrev.com/4f07c1664f22135b4dd2ba39847456396d6b11c3/infra/config/recipes.cfg
The following revision refers to this bug: https://chrome-internal.googlesource.com/chrome/tools/build_limited/scripts/slave/+/e4f50172b04e07316610a9833604c5c40794766c commit e4f50172b04e07316610a9833604c5c40794766c Author: recipe-roller <recipe-roller@chromium.org> Date: Wed Jul 20 22:58:08 2016
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/6a79faf16ac5bf3055f79e160c9e664480f30e65 commit 6a79faf16ac5bf3055f79e160c9e664480f30e65 Author: recipe-roller <recipe-roller@chromium.org> Date: Wed Jul 20 23:27:24 2016 Roll recipe dependencies (trivial). This is an automated CL created by the recipe roller. This CL rolls recipe changes from upstream projects (e.g. depot_tools) into downstream projects (e.g. tools/build). More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug (or complain) build: https://crrev.com/c24914b0f7d3df5373190d0b5950da82f08673b7 Revert of [Android] Enable platform mode for perf tests on Galaxy S5 (patchset #1 id:1 of https://codereview.chromium.org/2171463002/ ) (jbudorick@chromium.org) R=dtu@chromium.org,jbudorick@chromium.org,rnephew@chromium.org,stip@chromium.org BUG= 615157 TBR=martiniss@chromium.org,phajdan.jr@chromium.org Review-Url: https://codereview.chromium.org/2164173002 [modify] https://crrev.com/6a79faf16ac5bf3055f79e160c9e664480f30e65/infra/config/recipes.cfg
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/a84f32ea4f16bb2c41ed45f933b2f11348f7bf91 commit a84f32ea4f16bb2c41ed45f933b2f11348f7bf91 Author: rnephew <rnephew@chromium.org> Date: Thu Jul 21 02:23:30 2016 [Android] Fix blacklist handling for perf tests in platform mode. BUG= 615157 Review-Url: https://codereview.chromium.org/2172463003 Cr-Commit-Position: refs/heads/master@{#406743} [modify] https://crrev.com/a84f32ea4f16bb2c41ed45f933b2f11348f7bf91/build/android/pylib/local/device/local_device_perf_test_run.py
The following revision refers to this bug: https://chrome-internal.googlesource.com/chrome/tools/build_limited/scripts/slave/+/0447f11433b81e3a15d6c416cf74b296785e92fd commit 0447f11433b81e3a15d6c416cf74b296785e92fd Author: recipe-roller <recipe-roller@chromium.org> Date: Thu Jul 21 15:21:24 2016
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/9dbb8b7b1271804a792e311ecba6344a6b957d34 commit 9dbb8b7b1271804a792e311ecba6344a6b957d34 Author: recipe-roller <recipe-roller@chromium.org> Date: Thu Jul 21 15:38:14 2016 Roll recipe dependencies (trivial). This is an automated CL created by the recipe roller. This CL rolls recipe changes from upstream projects (e.g. depot_tools) into downstream projects (e.g. tools/build). More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug (or complain) build: https://crrev.com/bbfbad4ff1c20c25e0b3df42fd30c5d0b69e154b chromium_tests: use correct cwd for deapplying patch (phajdan.jr@chromium.org) https://crrev.com/e222f56e1e92b532e577cdc36f9e60543aef6e98 chromium.fyi: use chromium recipe in chromium/src via gitiles on remote_run builders (phajdan.jr@chromium.org) https://crrev.com/f91d5337a0d67c79c22ce9ebb71d00e19161e724 Reland of [Android] Enable platform mode for perf tests on Galaxy S5 (patchset #1 id:1 of https://codereview.chromium.org/2172463002/ ) (rnephew@chromium.org) R=dtu@chromium.org,rnephew@chromium.org,jbudorick@chromium.org,stip@chromium.org,phajdan.jr@chromium.org BUG= 630065 ,584079, 615157 TBR=martiniss@chromium.org,phajdan.jr@chromium.org Review-Url: https://codereview.chromium.org/2163373007 [modify] https://crrev.com/9dbb8b7b1271804a792e311ecba6344a6b957d34/infra/config/recipes.cfg
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/430fa14b4178317246b45eac9a6930d5958f5fe5 commit 430fa14b4178317246b45eac9a6930d5958f5fe5 Author: recipe-roller <recipe-roller@chromium.org> Date: Thu Jul 21 15:58:40 2016 Roll recipe dependencies (trivial). This is an automated CL created by the recipe roller. This CL rolls recipe changes from upstream projects (e.g. depot_tools) into downstream projects (e.g. tools/build). More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug (or complain) build: https://crrev.com/bbfbad4ff1c20c25e0b3df42fd30c5d0b69e154b chromium_tests: use correct cwd for deapplying patch (phajdan.jr@chromium.org) https://crrev.com/e222f56e1e92b532e577cdc36f9e60543aef6e98 chromium.fyi: use chromium recipe in chromium/src via gitiles on remote_run builders (phajdan.jr@chromium.org) https://crrev.com/f91d5337a0d67c79c22ce9ebb71d00e19161e724 Reland of [Android] Enable platform mode for perf tests on Galaxy S5 (patchset #1 id:1 of https://codereview.chromium.org/2172463002/ ) (rnephew@chromium.org) https://crrev.com/1160d66500b544685e537800c77a4071e35912da Set nest step status when test failure for Chromedriver recipe. (mikecase@chromium.org) R=rnephew@chromium.org,jbudorick@chromium.org,stip@chromium.org,dtu@chromium.org,mikecase@chromium.org,phajdan.jr@chromium.org BUG= 627633 , 630065 ,584079, 615157 TBR=martiniss@chromium.org,phajdan.jr@chromium.org Review-Url: https://codereview.chromium.org/2166043006 [modify] https://crrev.com/430fa14b4178317246b45eac9a6930d5958f5fe5/infra/config/recipes.cfg
The following revision refers to this bug: https://chromium.googlesource.com/chromium/tools/build.git/+/f91d5337a0d67c79c22ce9ebb71d00e19161e724 commit f91d5337a0d67c79c22ce9ebb71d00e19161e724 Author: rnephew <rnephew@chromium.org> Date: Thu Jul 21 15:16:13 2016 Reland of [Android] Enable platform mode for perf tests on Galaxy S5 (patchset #1 id:1 of https://codereview.chromium.org/2172463002/ ) Reason for revert: Fix landed. Original issue's description: > Revert of [Android] Enable platform mode for perf tests on Galaxy S5 (patchset #1 id:1 of https://codereview.chromium.org/2171463002/ ) > > Reason for revert: > https://build.chromium.org/p/chromium.perf/builders/Android%20Galaxy%20S5%20Perf%20%283%29/builds/2703 > > Original issue's description: > > [Android] Enable platform mode for perf tests on Galaxy S5 > > > > BUG= 615157 > > > > Committed: https://chromium.googlesource.com/chromium/tools/build/+/bf806e109ffa60364a3ec9983371555b69b662b4 > > TBR=dtu@chromium.org,stip@chromium.org,rnephew@chromium.org > # Skipping CQ checks because original CL landed less than 1 days ago. > NOPRESUBMIT=true > NOTREECHECKS=true > NOTRY=true > BUG= 615157 > > Committed: https://chromium.googlesource.com/chromium/tools/build/+/c24914b0f7d3df5373190d0b5950da82f08673b7 TBR=dtu@chromium.org,stip@chromium.org,jbudorick@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= 615157 Review-Url: https://codereview.chromium.org/2167113003 [modify] https://crrev.com/f91d5337a0d67c79c22ce9ebb71d00e19161e724/scripts/slave/recipes/android/perf.expected/full_chromium_perf_Android_Galaxy_S5_Perf__1_.json [modify] https://crrev.com/f91d5337a0d67c79c22ce9ebb71d00e19161e724/scripts/slave/recipes/android/perf.expected/full_chromium_perf_Android_Galaxy_S5_Perf__2_.json [modify] https://crrev.com/f91d5337a0d67c79c22ce9ebb71d00e19161e724/scripts/slave/recipes/android/perf.expected/full_chromium_perf_Android_Galaxy_S5_Perf__3_.json [modify] https://crrev.com/f91d5337a0d67c79c22ce9ebb71d00e19161e724/scripts/slave/recipes/android/perf.py
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/bec4caf4bf87bdc03f93e5e27c8ab7c14370df8b commit bec4caf4bf87bdc03f93e5e27c8ab7c14370df8b Author: rnephew <rnephew@chromium.org> Date: Thu Jul 21 22:27:27 2016 [Android] Fix return code for Sharded Perf step. BUG= 615157 Review-Url: https://codereview.chromium.org/2169093003 Cr-Commit-Position: refs/heads/master@{#406970} [modify] https://crrev.com/bec4caf4bf87bdc03f93e5e27c8ab7c14370df8b/build/android/test_runner.py
quick and dirty DD here: https://docs.google.com/document/d/1E8a2x1CWcxDOid-9_OVodHRmr0II-4v5MxEPKDdlV34/edit
The following revision refers to this bug: https://chromium.googlesource.com/chromium/tools/build.git/+/109b217f3aaed1ee8a3deabcee603943aea21082 commit 109b217f3aaed1ee8a3deabcee603943aea21082 Author: rnephew <rnephew@chromium.org> Date: Fri Jul 22 22:26:50 2016 [Android] Revert Galaxy s5 perf bots to not using platform mode test runner. BUG= 630627 , 615157 Review-Url: https://codereview.chromium.org/2179513002 [modify] https://crrev.com/109b217f3aaed1ee8a3deabcee603943aea21082/scripts/slave/recipes/android/perf.expected/full_chromium_perf_Android_Galaxy_S5_Perf__1_.json [modify] https://crrev.com/109b217f3aaed1ee8a3deabcee603943aea21082/scripts/slave/recipes/android/perf.expected/full_chromium_perf_Android_Galaxy_S5_Perf__2_.json [modify] https://crrev.com/109b217f3aaed1ee8a3deabcee603943aea21082/scripts/slave/recipes/android/perf.expected/full_chromium_perf_Android_Galaxy_S5_Perf__3_.json [modify] https://crrev.com/109b217f3aaed1ee8a3deabcee603943aea21082/scripts/slave/recipes/android/perf.py
The following revision refers to this bug: https://chrome-internal.googlesource.com/chrome/tools/build_limited/scripts/slave/+/1889152f9d3042d1277cda64e284b3ab91d5a66c commit 1889152f9d3042d1277cda64e284b3ab91d5a66c Author: recipe-roller <recipe-roller@chromium.org> Date: Fri Jul 22 22:38:26 2016
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/20987985f90a1403ca57df2310853ef7836a994f commit 20987985f90a1403ca57df2310853ef7836a994f Author: recipe-roller <recipe-roller@chromium.org> Date: Fri Jul 22 22:46:23 2016 Roll recipe dependencies (trivial). This is an automated CL created by the recipe roller. This CL rolls recipe changes from upstream projects (e.g. depot_tools) into downstream projects (e.g. tools/build). More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug (or complain) build: https://crrev.com/109b217f3aaed1ee8a3deabcee603943aea21082 [Android] Revert Galaxy s5 perf bots to not using platform mode test runner. (rnephew@chromium.org) R=rnephew@chromium.org BUG= 630627 , 615157 TBR=martiniss@chromium.org,phajdan.jr@chromium.org Review-Url: https://codereview.chromium.org/2175863004 [modify] https://crrev.com/20987985f90a1403ca57df2310853ef7836a994f/infra/config/recipes.cfg
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/e8555d3785393f8bf04579226a9e39f4e574c599 commit e8555d3785393f8bf04579226a9e39f4e574c599 Author: rnephew <rnephew@chromium.org> Date: Mon Jul 25 21:17:46 2016 [Android] Add heartbeat of platform mode perf test runner. Extremely long running tests can cause timeouts due to lack of output. This adds a heartbeat message to stop that while tests are still running. BUG= 615157 , 630640 Review-Url: https://codereview.chromium.org/2182563002 Cr-Commit-Position: refs/heads/master@{#407576} [modify] https://crrev.com/e8555d3785393f8bf04579226a9e39f4e574c599/build/android/pylib/local/device/local_device_perf_test_run.py
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/6c4a773fb2b4dc7b71051460a534ae8c0c3ac006 commit 6c4a773fb2b4dc7b71051460a534ae8c0c3ac006 Author: rnephew <rnephew@chromium.org> Date: Mon Jul 25 21:48:47 2016 [Android] Make perf test runner in platform mode check that return value is not None. BUG= 630627 , 615157 Review-Url: https://codereview.chromium.org/2175313003 Cr-Commit-Position: refs/heads/master@{#407598} [modify] https://crrev.com/6c4a773fb2b4dc7b71051460a534ae8c0c3ac006/build/android/pylib/local/device/local_device_perf_test_run.py
The following revision refers to this bug: https://chromium.googlesource.com/chromium/tools/build.git/+/772e098a3f629dfeadab58212623a07e54ee902a commit 772e098a3f629dfeadab58212623a07e54ee902a Author: rnephew <rnephew@chromium.org> Date: Mon Jul 25 22:04:39 2016 [Android] Reland Galaxy s5 perf bots to run using platform mode. BUG= 615157 , 630627 , 630640 Review-Url: https://codereview.chromium.org/2177863004 [modify] https://crrev.com/772e098a3f629dfeadab58212623a07e54ee902a/scripts/slave/recipes/android/perf.expected/full_chromium_perf_Android_Galaxy_S5_Perf__1_.json [modify] https://crrev.com/772e098a3f629dfeadab58212623a07e54ee902a/scripts/slave/recipes/android/perf.expected/full_chromium_perf_Android_Galaxy_S5_Perf__2_.json [modify] https://crrev.com/772e098a3f629dfeadab58212623a07e54ee902a/scripts/slave/recipes/android/perf.expected/full_chromium_perf_Android_Galaxy_S5_Perf__3_.json [modify] https://crrev.com/772e098a3f629dfeadab58212623a07e54ee902a/scripts/slave/recipes/android/perf.py
The following revision refers to this bug: https://chrome-internal.googlesource.com/chrome/tools/build_limited/scripts/slave/+/ce6b0ce17d5c82a02b3972492150a886a5e19dcf commit ce6b0ce17d5c82a02b3972492150a886a5e19dcf Author: recipe-roller <recipe-roller@chromium.org> Date: Mon Jul 25 22:07:59 2016
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/81693cf3baf302312bc2e5b6185c7f2740b339b5 commit 81693cf3baf302312bc2e5b6185c7f2740b339b5 Author: recipe-roller <recipe-roller@chromium.org> Date: Mon Jul 25 22:27:51 2016 Roll recipe dependencies (trivial). This is an automated CL created by the recipe roller. This CL rolls recipe changes from upstream projects (e.g. depot_tools) into downstream projects (e.g. tools/build). More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug (or complain) build: https://crrev.com/772e098a3f629dfeadab58212623a07e54ee902a [Android] Reland Galaxy s5 perf bots to run using platform mode. (rnephew@chromium.org) R=rnephew@chromium.org BUG= 630627 , 630640 , 615157 TBR=martiniss@chromium.org,phajdan.jr@chromium.org Review-Url: https://codereview.chromium.org/2178073004 [modify] https://crrev.com/81693cf3baf302312bc2e5b6185c7f2740b339b5/infra/config/recipes.cfg
The following revision refers to this bug: https://chromium.googlesource.com/chromium/tools/build.git/+/93b972af676e8098ebc020db5600b1197a761982 commit 93b972af676e8098ebc020db5600b1197a761982 Author: jbudorick <jbudorick@chromium.org> Date: Tue Jul 26 13:44:09 2016 Revert of [Android] Reland Galaxy s5 perf bots to run using platform mode. (patchset #1 id:1 of https://codereview.chromium.org/2177863004/ ) Reason for revert: the sharded perf tests step fails when one or more devices are blacklisted. Original issue's description: > [Android] Reland Galaxy s5 perf bots to run using platform mode. > > BUG= 615157 , 630627 , 630640 > > Committed: https://chromium.googlesource.com/chromium/tools/build/+/772e098a3f629dfeadab58212623a07e54ee902a TBR=stip@chromium.org,rnephew@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= 615157 , 630627 , 630640 Review-Url: https://codereview.chromium.org/2187483002 [modify] https://crrev.com/93b972af676e8098ebc020db5600b1197a761982/scripts/slave/recipes/android/perf.expected/full_chromium_perf_Android_Galaxy_S5_Perf__1_.json [modify] https://crrev.com/93b972af676e8098ebc020db5600b1197a761982/scripts/slave/recipes/android/perf.expected/full_chromium_perf_Android_Galaxy_S5_Perf__2_.json [modify] https://crrev.com/93b972af676e8098ebc020db5600b1197a761982/scripts/slave/recipes/android/perf.expected/full_chromium_perf_Android_Galaxy_S5_Perf__3_.json [modify] https://crrev.com/93b972af676e8098ebc020db5600b1197a761982/scripts/slave/recipes/android/perf.py
The following revision refers to this bug: https://chrome-internal.googlesource.com/chrome/tools/build_limited/scripts/slave/+/1db1515ad68074bdb31c1aa1d4bf0784e8d9ce50 commit 1db1515ad68074bdb31c1aa1d4bf0784e8d9ce50 Author: recipe-roller <recipe-roller@chromium.org> Date: Tue Jul 26 13:48:27 2016
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/5b9186bb318fad23806c4115489984ef1a58e828 commit 5b9186bb318fad23806c4115489984ef1a58e828 Author: recipe-roller <recipe-roller@chromium.org> Date: Tue Jul 26 14:08:05 2016 Roll recipe dependencies (trivial). This is an automated CL created by the recipe roller. This CL rolls recipe changes from upstream projects (e.g. depot_tools) into downstream projects (e.g. tools/build). More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug (or complain) build: https://crrev.com/93b972af676e8098ebc020db5600b1197a761982 Revert of [Android] Reland Galaxy s5 perf bots to run using platform mode. (patchset #1 id:1 of https://codereview.chromium.org/2177863004/ ) (jbudorick@chromium.org) R=jbudorick@chromium.org,rnephew@chromium.org,stip@chromium.org BUG= 630627 , 630640 , 615157 TBR=martiniss@chromium.org,phajdan.jr@chromium.org Review-Url: https://codereview.chromium.org/2178213004 [modify] https://crrev.com/5b9186bb318fad23806c4115489984ef1a58e828/infra/config/recipes.cfg
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/b506bd9515fe22c04f5bdd77e1024efdb7f5274c commit b506bd9515fe22c04f5bdd77e1024efdb7f5274c Author: rnephew <rnephew@chromium.org> Date: Tue Jul 26 16:36:26 2016 [Android] Fix perf test runner in platform mode return value for when devices are blacklisted. BUG= 630627 , 615157 Review-Url: https://codereview.chromium.org/2187513002 Cr-Commit-Position: refs/heads/master@{#407829} [modify] https://crrev.com/b506bd9515fe22c04f5bdd77e1024efdb7f5274c/build/android/pylib/local/device/local_device_perf_test_run.py
The following revision refers to this bug: https://chromium.googlesource.com/chromium/tools/build.git/+/525217b0d1284f1b2a9ddd691b099f9a4f894349 commit 525217b0d1284f1b2a9ddd691b099f9a4f894349 Author: rnephew <rnephew@chromium.org> Date: Tue Jul 26 16:44:46 2016 Reland of [Android] Reland Galaxy s5 perf bots to run using platform mode. (patchset #1 id:1 of https://codereview.chromium.org/2187483002/ ) Reason for revert: Fixed with https://codereview.chromium.org/2187513002/ Original issue's description: > Revert of [Android] Reland Galaxy s5 perf bots to run using platform mode. (patchset #1 id:1 of https://codereview.chromium.org/2177863004/ ) > > Reason for revert: > the sharded perf tests step fails when one or more devices are blacklisted. > > Original issue's description: > > [Android] Reland Galaxy s5 perf bots to run using platform mode. > > > > BUG= 615157 , 630627 , 630640 > > > > Committed: https://chromium.googlesource.com/chromium/tools/build/+/772e098a3f629dfeadab58212623a07e54ee902a > > TBR=stip@chromium.org,rnephew@chromium.org > # Skipping CQ checks because original CL landed less than 1 days ago. > NOPRESUBMIT=true > NOTREECHECKS=true > NOTRY=true > BUG= 615157 , 630627 , 630640 > > Committed: https://chromium.googlesource.com/chromium/tools/build/+/93b972af676e8098ebc020db5600b1197a761982 TBR=stip@chromium.org,jbudorick@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= 615157 , 630627 , 630640 Review-Url: https://codereview.chromium.org/2184803002 [modify] https://crrev.com/525217b0d1284f1b2a9ddd691b099f9a4f894349/scripts/slave/recipes/android/perf.expected/full_chromium_perf_Android_Galaxy_S5_Perf__1_.json [modify] https://crrev.com/525217b0d1284f1b2a9ddd691b099f9a4f894349/scripts/slave/recipes/android/perf.expected/full_chromium_perf_Android_Galaxy_S5_Perf__2_.json [modify] https://crrev.com/525217b0d1284f1b2a9ddd691b099f9a4f894349/scripts/slave/recipes/android/perf.expected/full_chromium_perf_Android_Galaxy_S5_Perf__3_.json [modify] https://crrev.com/525217b0d1284f1b2a9ddd691b099f9a4f894349/scripts/slave/recipes/android/perf.py
The following revision refers to this bug: https://chrome-internal.googlesource.com/chrome/tools/build_limited/scripts/slave/+/8052d18580548dbcd820e90a638c873e9b283995 commit 8052d18580548dbcd820e90a638c873e9b283995 Author: recipe-roller <recipe-roller@chromium.org> Date: Tue Jul 26 16:48:26 2016
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/374c564fecdd069a6c1b0bf73b6541f21d6dee2f commit 374c564fecdd069a6c1b0bf73b6541f21d6dee2f Author: recipe-roller <recipe-roller@chromium.org> Date: Tue Jul 26 17:06:05 2016 Roll recipe dependencies (trivial). This is an automated CL created by the recipe roller. This CL rolls recipe changes from upstream projects (e.g. depot_tools) into downstream projects (e.g. tools/build). More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug (or complain) build: https://crrev.com/525217b0d1284f1b2a9ddd691b099f9a4f894349 Reland of [Android] Reland Galaxy s5 perf bots to run using platform mode. (patchset #1 id:1 of https://codereview.chromium.org/2187483002/ ) (rnephew@chromium.org) R=rnephew@chromium.org,jbudorick@chromium.org,stip@chromium.org BUG= 630627 , 630640 , 615157 TBR=martiniss@chromium.org,phajdan.jr@chromium.org Review-Url: https://codereview.chromium.org/2178353004 [modify] https://crrev.com/374c564fecdd069a6c1b0bf73b6541f21d6dee2f/infra/config/recipes.cfg
The following revision refers to this bug: https://chromium.googlesource.com/chromium/tools/build.git/+/b0a4417ce4066c82582d8d28daf4679edcc373e7 commit b0a4417ce4066c82582d8d28daf4679edcc373e7 Author: rnephew <rnephew@chromium.org> Date: Wed Jul 27 16:12:02 2016 [Android] Enable platform mode on more perf bots. Enable platorm mode on N5, N5X, and N7 bots. BUG= 615157 Review-Url: https://codereview.chromium.org/2169993002 [modify] https://crrev.com/b0a4417ce4066c82582d8d28daf4679edcc373e7/scripts/slave/recipes/android/perf.expected/full_chromium_perf_Android_Nexus5X_Perf__1_.json [modify] https://crrev.com/b0a4417ce4066c82582d8d28daf4679edcc373e7/scripts/slave/recipes/android/perf.expected/full_chromium_perf_Android_Nexus5X_Perf__2_.json [modify] https://crrev.com/b0a4417ce4066c82582d8d28daf4679edcc373e7/scripts/slave/recipes/android/perf.expected/full_chromium_perf_Android_Nexus5X_Perf__3_.json [modify] https://crrev.com/b0a4417ce4066c82582d8d28daf4679edcc373e7/scripts/slave/recipes/android/perf.expected/full_chromium_perf_Android_Nexus5_Perf__1_.json [modify] https://crrev.com/b0a4417ce4066c82582d8d28daf4679edcc373e7/scripts/slave/recipes/android/perf.expected/full_chromium_perf_Android_Nexus5_Perf__2_.json [modify] https://crrev.com/b0a4417ce4066c82582d8d28daf4679edcc373e7/scripts/slave/recipes/android/perf.expected/full_chromium_perf_Android_Nexus5_Perf__3_.json [modify] https://crrev.com/b0a4417ce4066c82582d8d28daf4679edcc373e7/scripts/slave/recipes/android/perf.expected/full_chromium_perf_Android_Nexus7v2_Perf__1_.json [modify] https://crrev.com/b0a4417ce4066c82582d8d28daf4679edcc373e7/scripts/slave/recipes/android/perf.expected/full_chromium_perf_Android_Nexus7v2_Perf__2_.json [modify] https://crrev.com/b0a4417ce4066c82582d8d28daf4679edcc373e7/scripts/slave/recipes/android/perf.expected/full_chromium_perf_Android_Nexus7v2_Perf__3_.json [modify] https://crrev.com/b0a4417ce4066c82582d8d28daf4679edcc373e7/scripts/slave/recipes/android/perf.expected/get_perf_test_list_old_data.json [modify] https://crrev.com/b0a4417ce4066c82582d8d28daf4679edcc373e7/scripts/slave/recipes/android/perf.expected/missing_device.json [modify] https://crrev.com/b0a4417ce4066c82582d8d28daf4679edcc373e7/scripts/slave/recipes/android/perf.expected/src_side_script_fails.json [modify] https://crrev.com/b0a4417ce4066c82582d8d28daf4679edcc373e7/scripts/slave/recipes/android/perf.expected/test_failure.json [modify] https://crrev.com/b0a4417ce4066c82582d8d28daf4679edcc373e7/scripts/slave/recipes/android/perf.py
The following revision refers to this bug: https://chrome-internal.googlesource.com/chrome/tools/build_limited/scripts/slave/+/410a879b5531e613776334d89cae61bc8d4c79da commit 410a879b5531e613776334d89cae61bc8d4c79da Author: recipe-roller <recipe-roller@chromium.org> Date: Wed Jul 27 16:18:03 2016
The following revision refers to this bug: https://chromium.googlesource.com/infra/infra.git/+/487ca452249d18e56e117f192c54e15b01b5a0bd commit 487ca452249d18e56e117f192c54e15b01b5a0bd Author: recipe-roller <recipe-roller@chromium.org> Date: Wed Jul 27 16:28:29 2016 Roll recipe dependencies (trivial). This is an automated CL created by the recipe roller. This CL rolls recipe changes from upstream projects (e.g. depot_tools) into downstream projects (e.g. tools/build). More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug (or complain) build: https://crrev.com/b0a4417ce4066c82582d8d28daf4679edcc373e7 [Android] Enable platform mode on more perf bots. (rnephew@chromium.org) R=rnephew@chromium.org BUG= 615157 TBR=martiniss@chromium.org,phajdan.jr@chromium.org Review-Url: https://codereview.chromium.org/2190663002 [modify] https://crrev.com/487ca452249d18e56e117f192c54e15b01b5a0bd/infra/config/recipes.cfg
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/9ea896bb6a840dcf84381fea4fece6069d497159 commit 9ea896bb6a840dcf84381fea4fece6069d497159 Author: rnephew <rnephew@chromium.org> Date: Fri Jul 29 17:06:07 2016 [Android] Set platform mode as default for perf tests. BUG= 615157 Review-Url: https://codereview.chromium.org/2189773002 Cr-Commit-Position: refs/heads/master@{#408665} [modify] https://crrev.com/9ea896bb6a840dcf84381fea4fece6069d497159/build/android/test_runner.py
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/088334431c3f59cbd6af3cb868036331f41a1997 commit 088334431c3f59cbd6af3cb868036331f41a1997 Author: rnephew <rnephew@chromium.org> Date: Sat Jul 30 00:49:03 2016 Revert of [Android] Set platform mode as default for perf tests. (patchset #1 id:1 of https://codereview.chromium.org/2189773002/ ) Reason for revert: https://bugs.chromium.org/p/chromium/issues/detail?id=632922 Original issue's description: > [Android] Set platform mode as default for perf tests. > > BUG= 615157 > > Committed: https://crrev.com/9ea896bb6a840dcf84381fea4fece6069d497159 > Cr-Commit-Position: refs/heads/master@{#408665} TBR=mikecase@chromium.org,jbudorick@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG= 615157 Review-Url: https://codereview.chromium.org/2201563002 Cr-Commit-Position: refs/heads/master@{#408823} [modify] https://crrev.com/088334431c3f59cbd6af3cb868036331f41a1997/build/android/test_runner.py
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/a00fe127bfd2be32a412e4c0c73b4ced7b8bf6ab commit a00fe127bfd2be32a412e4c0c73b4ced7b8bf6ab Author: rnephew <rnephew@chromium.org> Date: Mon Aug 01 19:16:23 2016 [Android] Fix backwards compatibility flags with perf tests running in platform mode. BUG= 615157 ,632922 Review-Url: https://codereview.chromium.org/2196343002 Cr-Commit-Position: refs/heads/master@{#409016} [modify] https://crrev.com/a00fe127bfd2be32a412e4c0c73b4ced7b8bf6ab/build/android/pylib/perf/perf_test_instance.py
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/17b73d696569a139b614b057bbddb13784c81ca9 commit 17b73d696569a139b614b057bbddb13784c81ca9 Author: rnephew <rnephew@chromium.org> Date: Mon Aug 01 20:16:15 2016 Reland [Android] Set platform mode as default for perf tests. BUG= 615157 , 632922 Review-Url: https://codereview.chromium.org/2201843002 Cr-Commit-Position: refs/heads/master@{#409040} [modify] https://crrev.com/17b73d696569a139b614b057bbddb13784c81ca9/build/android/test_runner.py
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/3a1dfbd039fcb3472bda2a565df92f4d35e79c55 commit 3a1dfbd039fcb3472bda2a565df92f4d35e79c55 Author: rnephew <rnephew@chromium.org> Date: Wed Aug 24 21:53:01 2016 [Android] Get rid of old perf test runner. BUG= 615157 Review-Url: https://codereview.chromium.org/2198613002 Cr-Commit-Position: refs/heads/master@{#414158} [delete] https://crrev.com/b8cd385fb7004724f431d99af4f410adcd51888e/build/android/pylib/perf/setup.py [delete] https://crrev.com/b8cd385fb7004724f431d99af4f410adcd51888e/build/android/pylib/perf/test_options.py [delete] https://crrev.com/b8cd385fb7004724f431d99af4f410adcd51888e/build/android/pylib/perf/test_runner.py [delete] https://crrev.com/b8cd385fb7004724f431d99af4f410adcd51888e/build/android/pylib/utils/test_environment.py [modify] https://crrev.com/3a1dfbd039fcb3472bda2a565df92f4d35e79c55/build/android/test_runner.py [modify] https://crrev.com/3a1dfbd039fcb3472bda2a565df92f4d35e79c55/build/android/test_runner.pydeps
Comment 1 by jbudorick@chromium.org
, May 26 2016Labels: -Restrict-View-Google