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

Issue 332093003: ChromeDriver has pre-installed APK's, thus should not wipe during provisioning. (Closed)

Created:
6 years, 6 months ago by navabi
Modified:
6 years, 5 months ago
CC:
chromium-reviews, klundberg+watch_chromium.org, bulach+watch_chromium.org, yfriedman+watch_chromium.org, ilevy-cc_chromium.org
Project:
chromium
Visibility:
Public.

Description

ChromeDriver has pre-installed APK's, thus should not wipe during provisioning. BUG=383106 TBR=craigdh@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=277628

Patch Set 1 #

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

Messages

Total messages: 19 (0 generated)
navabi
6 years, 6 months ago (2014-06-16 08:14:32 UTC) #1
klundberg
lgtm Thanks, Armand!
6 years, 6 months ago (2014-06-16 15:43:09 UTC) #2
dnj
lgtm
6 years, 6 months ago (2014-06-16 15:45:01 UTC) #3
jbudorick
lgtm
6 years, 6 months ago (2014-06-16 18:08:04 UTC) #4
navabi
The CQ bit was checked by navabi@google.com
6 years, 6 months ago (2014-06-16 18:44:12 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/navabi@google.com/332093003/1
6 years, 6 months ago (2014-06-16 18:46:09 UTC) #6
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 6 months ago (2014-06-16 18:46:10 UTC) #7
commit-bot: I haz the power
No LGTM from a valid reviewer yet. Only full committers are accepted. Even if an ...
6 years, 6 months ago (2014-06-16 18:46:11 UTC) #8
navabi
need commiter LGTM
6 years, 6 months ago (2014-06-16 19:12:20 UTC) #9
zty
lgtm
6 years, 6 months ago (2014-06-16 19:18:00 UTC) #10
navabi
The CQ bit was checked by navabi@google.com
6 years, 6 months ago (2014-06-16 19:20:44 UTC) #11
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/navabi@google.com/332093003/1
6 years, 6 months ago (2014-06-16 19:21:33 UTC) #12
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 6 months ago (2014-06-16 19:21:35 UTC) #13
commit-bot: I haz the power
No LGTM from a valid reviewer yet. Only full committers are accepted. Even if an ...
6 years, 6 months ago (2014-06-16 19:21:36 UTC) #14
navabi
The CQ bit was checked by navabi@google.com
6 years, 6 months ago (2014-06-16 19:22:25 UTC) #15
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/navabi@google.com/332093003/1
6 years, 6 months ago (2014-06-16 19:24:48 UTC) #16
commit-bot: I haz the power
Change committed as 277628
6 years, 6 months ago (2014-06-17 02:40:16 UTC) #17
M-A Ruel
On 2014/06/17 02:40:16, I haz the power (commit-bot) wrote: > Change committed as 277628 I ...
6 years, 5 months ago (2014-07-09 15:35:16 UTC) #18
M-A Ruel
6 years, 5 months ago (2014-07-09 15:35:56 UTC) #19
Message was sent while issue was closed.
On 2014/07/09 15:35:16, M-A Ruel wrote:
> On 2014/06/17 02:40:16, I haz the power (commit-bot) wrote:
> > Change committed as 277628
> 
> I now get this:
> $ git cl upl
> Using 50% similarity for rename/copy detection. Override with --similarity.
> Loaded authentication cookies from ~/.codereview_upload_cookies
> Running presubmit upload checks ...
> 
> ** Presubmit Warnings **
> src/build/android/buildbot/tests/bb_run_bot_test.py (0.45s) failed
> Error running the bot script with id="try-tests-rel" Traceback (most recent
call
> last):
>   File "build/android/buildbot/bb_device_steps.py", line 698, in <module>
>     sys.exit(main(sys.argv))
>   File "build/android/buildbot/bb_device_steps.py", line 694, in main
>     MainTestWrapper(options)
>   File "build/android/buildbot/bb_device_steps.py", line 603, in
MainTestWrapper
>     bb_utils.RunSteps(options.test_filter, GetTestStepCmds(), options)
>   File "src/build/android/buildbot/bb_utils.py", line 100, in RunSteps
>     cmd(options)
>   File "build/android/buildbot/bb_device_steps.py", line 152, in
> RunTelemetryPerfUnitTests
>     devices = android_commands.GetAttachedDevices()
>   File "src/build/android/buildbot/../pylib/android_commands.py", line 158, in
> GetAttachedDevices
>     'devices'])
>   File "src/build/android/buildbot/../pylib/cmd_helper.py", line 67, in
> GetCmdOutput
>     (_, output) = GetCmdStatusAndOutput(args, cwd, shell)
>   File "src/build/android/buildbot/../pylib/cmd_helper.py", line 99, in
> GetCmdStatusAndOutput
>     shell=shell, cwd=cwd)
>   File "src/build/android/buildbot/../pylib/cmd_helper.py", line 27, in Popen
>     preexec_fn=lambda: signal.signal(signal.SIGPIPE, signal.SIG_DFL))
>   File "/usr/lib/python2.7/subprocess.py", line 679, in __init__
>     errread, errwrite)
>   File "/usr/lib/python2.7/subprocess.py", line 1249, in _execute_child
>     raise child_exception
> OSError: [Errno 2] No such file or directory
> 
> Error running the bot script with id="main-tests-rel" Traceback (most recent
> call last):
>   File "build/android/buildbot/bb_device_steps.py", line 698, in <module>
>     sys.exit(main(sys.argv))
>   File "build/android/buildbot/bb_device_steps.py", line 694, in main
>     MainTestWrapper(options)
>   File "build/android/buildbot/bb_device_steps.py", line 603, in
MainTestWrapper
>     bb_utils.RunSteps(options.test_filter, GetTestStepCmds(), options)
>   File "src/build/android/buildbot/bb_utils.py", line 100, in RunSteps
>     cmd(options)
>   File "build/android/buildbot/bb_device_steps.py", line 152, in
> RunTelemetryPerfUnitTests
>     devices = android_commands.GetAttachedDevices()
>   File "src/build/android/buildbot/../pylib/android_commands.py", line 158, in
> GetAttachedDevices
>     'devices'])
>   File "src/build/android/buildbot/../pylib/cmd_helper.py", line 67, in
> GetCmdOutput
>     (_, output) = GetCmdStatusAndOutput(args, cwd, shell)
>   File "src/build/android/buildbot/../pylib/cmd_helper.py", line 99, in
> GetCmdStatusAndOutput
>     shell=shell, cwd=cwd)
>   File "src/build/android/buildbot/../pylib/cmd_helper.py", line 27, in Popen
>     preexec_fn=lambda: signal.signal(signal.SIGPIPE, signal.SIG_DFL))
>   File "/usr/lib/python2.7/subprocess.py", line 679, in __init__
>     errread, errwrite)
>   File "/usr/lib/python2.7/subprocess.py", line 1249, in _execute_child
>     raise child_exception
> OSError: [Errno 2] No such file or directory
> 
> It's because I don't have adb in the path. Could either an error message be
> created or the test fixed?

To clarify, I don't think it's this CL particularly but it's relatively recent
and in this area.

Powered by Google App Engine
This is Rietveld 408576698