Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(349)

Issue 16391007: Add an Android perf bisect bot and its host steps. (Closed)

Created:
7 years, 6 months ago by Siva Chandra
Modified:
7 years, 6 months ago
Reviewers:
Isaac (away), shatch
CC:
chromium-reviews, craigdh+watch_chromium.org, bulach+watch_chromium.org, yfriedman+watch_chromium.org, ilevy-cc_chromium.org, klundberg+watch_chromium.org, frankf+watch_chromium.org
Visibility:
Public.

Description

Add an Android perf bisect bots and its host steps. BUG=245361 R=ilevy@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=205641

Patch Set 1 #

Patch Set 2 : Remove the runhooks step #

Total comments: 1

Patch Set 3 : Make bisection a single annotated step #

Patch Set 4 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+14 lines, -0 lines) Patch
M build/android/buildbot/bb_host_steps.py View 1 2 3 3 chunks +13 lines, -0 lines 0 comments Download
M build/android/buildbot/bb_run_bot.py View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 9 (0 generated)
Siva Chandra
The trybots are failing on 'apply issue' because they apply on LKGR which is way ...
7 years, 6 months ago (2013-06-10 21:30:04 UTC) #1
Siva Chandra
I should also follow this CL with a master/slaves.cfg change. I am waiting for the ...
7 years, 6 months ago (2013-06-10 21:31:21 UTC) #2
Siva Chandra
PTaL at patch set 2.
7 years, 6 months ago (2013-06-11 00:14:55 UTC) #3
Isaac (away)
lgtm w/ nit https://chromiumcodereview.appspot.com/16391007/diff/5001/build/android/buildbot/bb_host_steps.py File build/android/buildbot/bb_host_steps.py (right): https://chromiumcodereview.appspot.com/16391007/diff/5001/build/android/buildbot/bb_host_steps.py#newcode96 build/android/buildbot/bb_host_steps.py:96: buildbot_report.PrintNamedStep('Preparing for Bisection') Can you make ...
7 years, 6 months ago (2013-06-11 02:06:44 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/sivachandra@chromium.org/16391007/16001
7 years, 6 months ago (2013-06-11 19:11:05 UTC) #5
commit-bot: I haz the power
Retried try job too often on chromium_presubmit for step(s) presubmit http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_presubmit&number=8377
7 years, 6 months ago (2013-06-11 19:20:08 UTC) #6
Siva Chandra
On 2013/06/11 19:20:08, I haz the power (commit-bot) wrote: > Retried try job too often ...
7 years, 6 months ago (2013-06-11 19:37:14 UTC) #7
Isaac (away)
Please dcommit for now, which will run presubmit locally.
7 years, 6 months ago (2013-06-11 20:47:27 UTC) #8
Siva Chandra
7 years, 6 months ago (2013-06-11 21:47:40 UTC) #9
Message was sent while issue was closed.
Committed patchset #4 manually as r205641 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698