You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@cordova.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2016/02/11 23:01:18 UTC
[jira] [Commented] (CB-10510) medic-run doesn't check to make sure
the emulator started
[ https://issues.apache.org/jira/browse/CB-10510?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15143584#comment-15143584 ]
ASF GitHub Bot commented on CB-10510:
-------------------------------------
GitHub user riknoll opened a pull request:
https://github.com/apache/cordova-android/pull/258
CB-10510: Add an optional timeout to emu start script
The `emulator.js` script waits forever for the emulator to boot. If the emulator gets stuck, it will never terminate. The reason for adding this timeout is that the Android emulator on Windows occasionally hangs while booting up. This causes problems in our CI, where the run step would go on forever until buildbot eventually cut it off (see [here](https://ci.apache.org/builders/cordova-android-win/builds/1588/steps/running-tests/logs/stdio) for an example run). I have an upcoming PR for cordova-medic that switches from using `cordova run` to start the emulator to requiring this script directly so that we can add retry logic for the emulator start. I don't think it really makes sense to expose the timeout option in the CLI itself, so this PR does not alter the CLI's behavior at all (it will still hang). I'm not sure if we should add in a default timeout or not.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/MSOpenTech/cordova-android CB-10510
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/cordova-android/pull/258.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #258
----
commit 723393168127524f5f23d931bbcf171edbd08d39
Author: riknoll <ri...@gmail.com>
Date: 2016-02-11T21:37:00Z
CB-10510: Add an optional timeout to emu start script
The script used to wait forever for the emulator to boot.
If the emulator got stuck, it would never terminate.
This timeout is being added to support cordova-medic and
the CI.
----
> medic-run doesn't check to make sure the emulator started
> ---------------------------------------------------------
>
> Key: CB-10510
> URL: https://issues.apache.org/jira/browse/CB-10510
> Project: Apache Cordova
> Issue Type: Bug
> Components: Medic
> Environment: Windows, Android emulator
> Reporter: Richard B Knoll
> Assignee: Richard B Knoll
> Labels: android, found-by-ci
>
> Medic doesn't check to see if the emulator actually starts when invoking {{cordova run android}}. Instead, it starts polling cordova-vm immediately. This might also be considered a bug in cordova-android's run script, which just seems to hang if the emulator doesn't start. Example build is here:
> https://ci.apache.org/builders/cordova-android-win/builds/1488/steps/running-tests/logs/stdio
> And adb logs are here:
> https://ci.apache.org/builders/cordova-android-win/builds/1488/steps/gathering-logs/logs/stdio
> ====
> Log:
> C:\Users\buildbot\slaves\apache\cordova-android-win\mobilespec>node C:\Users\buildbot\slaves\apache\cordova-android-win\cordova-cli\bin\cordova run --emulator android -- --gradle
> ANDROID_HOME=C:\Android\android-sdk
> JAVA_HOME=C:\Program Files (x86)\Java\jdk1.7.0_55
> No emulator specified, defaulting to Galaxy_Nexus
> Waiting for emulator...
> WARNING: init: Missing WGL extension WGL_ARB_make_current_read
> emulator: warning: opening audio output failed
> emulator: emulator window was out of view and was recentered
> emulator: UpdateChecker: skipped version check
> Booting up emulator (this may take a while)........................................query failed for mobilespec_results { [Error: connect ETIMEDOUT] code: 'ETIMEDOUT', errno: 'ETIMEDOUT', syscall: 'connect' }
> .....................................query failed for mobilespec_results { [Error: connect ETIMEDOUT] code: 'ETIMEDOUT', errno: 'ETIMEDOUT', syscall: 'connect' }
> ..................query failed for mobilespec_results { [Error: connect ETIMEDOUT] code: 'ETIMEDOUT', errno: 'ETIMEDOUT', syscall: 'connect' }
> ....................................................................................................query failed for mobilespec_results { [Error: connect ETIMEDOUT] code: 'ETIMEDOUT', errno: 'ETIMEDOUT', syscall: 'connect' }
> ...................................................................................query failed for mobilespec_results { [Error: connect ETIMEDOUT] code: 'ETIMEDOUT', errno: 'ETIMEDOUT', syscall: 'connect' }
> .......................................................didn't get test results: timed out
> program finished with exit code 1
> elapsedTime=1150.659000
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org
For additional commands, e-mail: issues-help@cordova.apache.org