cordova-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (CB-10510) medic-run doesn't check to make sure the emulator started
Date Thu, 11 Feb 2016 22:01:18 GMT

    [ 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 <richard.b.knoll@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


Mime
View raw message