Re: An idea for manually testing plugins

2016-12-06 Thread Filip Maj
Full disclosure: I contribute(d) to appium and worked for Sauce Labs,
so I am pretty biased ;)

Calabash, unfortunately, is iOS and Android only and does not support
Windows app, whereas Appium does via WinAppDriver [1].

I agree that appium, at least earlier on, was difficult to set up.
These days it's a simple `npm install`. I think appium has a much more
prolific committership to boot (MSFT contributes to it, for example).
The github network stats for each project back that up as well.

[1] https://github.com/microsoft/winappdriver

On Tue, Dec 6, 2016 at 4:02 PM, Trevor Brindle  wrote:
> In my experience with appium, it is troublesome to set up and maintain. We
> may investigate other frameworks that have similar functionality. I have
> had great luck with calabash.
>
>
> On Tue, Dec 6, 2016 at 5:30 PM julio cesar sanchez 
> wrote:
>
>> +1 to appium
>>
>>
>>
>> 2016-12-05 23:33 GMT+01:00 Filip Maj :
>>
>>
>>
>> > Hi, it's me again!
>>
>> >
>>
>> > How I'd like to contribute to Cordova is to help automate the stuff
>>
>> > that saves committers having to take the manual time to do themselves.
>>
>> > I think a good first goal would be to help automate as much of
>>
>> > platform release testing as possible [1]. The autotests seem to be
>>
>> > handled relatively well by the CI system so far (if anyone feels this
>>
>> > is not true, please speak up! I want to hear about what works and what
>>
>> > does not). The other part of mobile-spec-based platform release
>>
>> > testing is to "run the manual tests". After reading the
>>
>> > cordova-plugin-test-framework README [2], I thought this would be a
>>
>> > good place to give plugin developers some better tools to deal with
>>
>> > testing complex UI interactions that the autotests can't handle on
>>
>> > their own. I was thinking appium [3] would be a good tool to
>>
>> > complement that in this case. It gives us UI hooks into both web and
>>
>> > natives contexts within hybrid applications, plus it also allows us to
>>
>> > inject JavaScript into the web context. Wondering what others think?
>>
>> >
>>
>> > I could then foresee, with a little bit of scaffolding, a way to
>>
>> > string plugins' appium tests together to fully automate the 'manual'
>>
>> > testing of plugin tests during platform release testing.
>>
>> >
>>
>> > Let me know what y'all think!
>>
>> >
>>
>> > [1] https://github.com/apache/cordova-coho/blob/master/docs/
>>
>> > platforms-release-process.md#what-to-test
>>
>> > [2] https://github.com/apache/cordova-plugin-test-framework#
>>
>> > defining-manual-tests
>>
>> > [3] appium.io
>>
>> >
>>
>> > -
>>
>> > To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
>>
>> > For additional commands, e-mail: dev-h...@cordova.apache.org
>>
>> >
>>
>> >
>>
>>

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



Nightly build #249 for cordova has succeeded!

2016-12-06 Thread Apache Jenkins Server
Nightly build #249 for cordova has succeeded!
The latest nightly has been published and you can try it out with 'npm i -g 
cordova@nightly'

For details check build console at 
https://builds.apache.org/job/cordova-nightly/249/consoleFull

-
Jenkins for Apache Cordova

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org

Re: An idea for manually testing plugins

2016-12-06 Thread Trevor Brindle
In my experience with appium, it is troublesome to set up and maintain. We
may investigate other frameworks that have similar functionality. I have
had great luck with calabash.


On Tue, Dec 6, 2016 at 5:30 PM julio cesar sanchez 
wrote:

> +1 to appium
>
>
>
> 2016-12-05 23:33 GMT+01:00 Filip Maj :
>
>
>
> > Hi, it's me again!
>
> >
>
> > How I'd like to contribute to Cordova is to help automate the stuff
>
> > that saves committers having to take the manual time to do themselves.
>
> > I think a good first goal would be to help automate as much of
>
> > platform release testing as possible [1]. The autotests seem to be
>
> > handled relatively well by the CI system so far (if anyone feels this
>
> > is not true, please speak up! I want to hear about what works and what
>
> > does not). The other part of mobile-spec-based platform release
>
> > testing is to "run the manual tests". After reading the
>
> > cordova-plugin-test-framework README [2], I thought this would be a
>
> > good place to give plugin developers some better tools to deal with
>
> > testing complex UI interactions that the autotests can't handle on
>
> > their own. I was thinking appium [3] would be a good tool to
>
> > complement that in this case. It gives us UI hooks into both web and
>
> > natives contexts within hybrid applications, plus it also allows us to
>
> > inject JavaScript into the web context. Wondering what others think?
>
> >
>
> > I could then foresee, with a little bit of scaffolding, a way to
>
> > string plugins' appium tests together to fully automate the 'manual'
>
> > testing of plugin tests during platform release testing.
>
> >
>
> > Let me know what y'all think!
>
> >
>
> > [1] https://github.com/apache/cordova-coho/blob/master/docs/
>
> > platforms-release-process.md#what-to-test
>
> > [2] https://github.com/apache/cordova-plugin-test-framework#
>
> > defining-manual-tests
>
> > [3] appium.io
>
> >
>
> > -
>
> > To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
>
> > For additional commands, e-mail: dev-h...@cordova.apache.org
>
> >
>
> >
>
>


Re: An idea for manually testing plugins

2016-12-06 Thread julio cesar sanchez
+1 to appium

2016-12-05 23:33 GMT+01:00 Filip Maj :

> Hi, it's me again!
>
> How I'd like to contribute to Cordova is to help automate the stuff
> that saves committers having to take the manual time to do themselves.
> I think a good first goal would be to help automate as much of
> platform release testing as possible [1]. The autotests seem to be
> handled relatively well by the CI system so far (if anyone feels this
> is not true, please speak up! I want to hear about what works and what
> does not). The other part of mobile-spec-based platform release
> testing is to "run the manual tests". After reading the
> cordova-plugin-test-framework README [2], I thought this would be a
> good place to give plugin developers some better tools to deal with
> testing complex UI interactions that the autotests can't handle on
> their own. I was thinking appium [3] would be a good tool to
> complement that in this case. It gives us UI hooks into both web and
> natives contexts within hybrid applications, plus it also allows us to
> inject JavaScript into the web context. Wondering what others think?
>
> I could then foresee, with a little bit of scaffolding, a way to
> string plugins' appium tests together to fully automate the 'manual'
> testing of plugin tests during platform release testing.
>
> Let me know what y'all think!
>
> [1] https://github.com/apache/cordova-coho/blob/master/docs/
> platforms-release-process.md#what-to-test
> [2] https://github.com/apache/cordova-plugin-test-framework#
> defining-manual-tests
> [3] appium.io
>
> -
> To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
> For additional commands, e-mail: dev-h...@cordova.apache.org
>
>


[GitHub] cordova-plugin-statusbar issue #72: Update plugin.xml

2016-12-06 Thread jcesarmobile
Github user jcesarmobile commented on the issue:

https://github.com/apache/cordova-plugin-statusbar/pull/72
  
Those are the default values, if you provide your own values in the 
config.xml the app should use those.

If that isn't working then create an issue about it in 
http://issues.cordova.io/ so we can fix it, but removing them from the 
plugin.xml is not the solution.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



[GitHub] cordova-windows pull request #213: CB-12163 Make resource-file copy files ag...

2016-12-06 Thread ktop
GitHub user ktop opened a pull request:

https://github.com/apache/cordova-windows/pull/213

CB-12163 Make resource-file copy files again



### Platforms affected
- Windows

### What does this PR do?
- Revert the functionality of resource-file in plugin.xml. It will now copy 
files again instead of using the references. 

### What testing has been done on this change?
- Updated the spec tests and ran them
- Created a new app with the updated resource-file and check to see if it 
copies files. 

### Checklist
- [X] [Reported an issue](http://cordova.apache.org/contribute/issues.html) 
in the JIRA database
- [X] Commit message follows the format: "CB-3232: (android) Fix bug with 
resolving file paths", where CB- is the JIRA ID & "android" is the platform 
affected.
- [X] Added automated test coverage as appropriate for this change.



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/ktop/cordova-windows cb12163

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cordova-windows/pull/213.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 #213


commit d00e0c33d388f410d6444920d72b7ac1e86df3ef
Author: ktop 
Date:   2016-12-06T20:58:42Z

CB-12163 Make resource-file copy files again




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



[GitHub] cordova-plugin-statusbar issue #72: Update plugin.xml

2016-12-06 Thread cordova-qa
Github user cordova-qa commented on the issue:

https://github.com/apache/cordova-plugin-statusbar/pull/72
  
Cordova CI Build has completed successfully.

**Commit** - 
[Link](https://github.com/apache/cordova-plugin-statusbar/pull/72/commits/0d56d19d76c7a8f7cd5694fb69ea3258a1c2408f)
**Dashboard** - 
[Link](http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-statusbar-pr/38/)

| Builder Name  | Console Output | Test Report | Device Logs  |
| :---: | :---:  |   :---: | :---:|
| [Windows 8.1 Store]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-statusbar-pr/38//PLATFORM=windows-8.1-store/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-statusbar-pr/38//PLATFORM=windows-8.1-store/console)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-statusbar-pr/38//PLATFORM=windows-8.1-store/testReport/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-statusbar-pr/38//PLATFORM=windows-8.1-store/artifact/)
 |
| [Windows 10  Store]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-statusbar-pr/38//PLATFORM=windows-10-store/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-statusbar-pr/38//PLATFORM=windows-10-store/console)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-statusbar-pr/38//PLATFORM=windows-10-store/testReport/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-statusbar-pr/38//PLATFORM=windows-10-store/artifact/)
 |
| [Windows 8.1 Phone]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-statusbar-pr/38//PLATFORM=windows-8.1-phone/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-statusbar-pr/38//PLATFORM=windows-8.1-phone/console)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-statusbar-pr/38//PLATFORM=windows-8.1-phone/testReport/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-statusbar-pr/38//PLATFORM=windows-8.1-phone/artifact/)
 |
| [iOS]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-statusbar-pr/38//PLATFORM=ios/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-statusbar-pr/38//PLATFORM=ios/console)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-statusbar-pr/38//PLATFORM=ios/testReport/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-statusbar-pr/38//PLATFORM=ios/artifact/)
 |
| [Android]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-statusbar-pr/38//PLATFORM=android/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-statusbar-pr/38//PLATFORM=android/console)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-statusbar-pr/38//PLATFORM=android/testReport/)
 | [Link]( 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-statusbar-pr/38//PLATFORM=android/artifact/)
 |
 



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



[GitHub] cordova-plugin-statusbar pull request #72: Update plugin.xml

2016-12-06 Thread scavezze
GitHub user scavezze opened a pull request:

https://github.com/apache/cordova-plugin-statusbar/pull/72

Update plugin.xml



### IOS


### Removes prefrences that shoudl be controlled by the developer from the 
control of the plugin


### My app needs to not overlay the status bar and has a different style 
status bar .  I was unable to change them without creating a hook to remove 
these values.  once removed my status bar color changed and my app did overlay 
the status bar

These Values should be set in the config.xml by the app developer as needed 
for the app not set by the plugin.xml.  what happens is that if i want a 
different color or i want it to not overlay the web view I can't do it as the 
parser uses the last entry and overrides what needs to be there

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/scavezze/cordova-plugin-statusbar master

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cordova-plugin-statusbar/pull/72.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 #72


commit 0d56d19d76c7a8f7cd5694fb69ea3258a1c2408f
Author: scavezze 
Date:   2016-12-06T19:58:55Z

Update plugin.xml

These Values should be set in the config.xml by the app developer as needed 
for the app not set by the plugin.xml.  what happens is that if i want a 
different color or i want it to not overlay the web view I can't do it as the 
parser uses the last entry and overrides what needs to be there




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



Re: Access to Jenkins instance, questions about CI

2016-12-06 Thread Alexander Sorokin (Akvelon)
Yeah, my bad.


Fixed it now.


--Alex


From: Darryl Pogue 
Sent: Tuesday, December 6, 2016 9:26:11 PM
To: dev@cordova.apache.org
Subject: Re: Access to Jenkins instance, questions about CI

Hi Alexander,

It seems that public read permissions have broken for the Jenkins instance.
I think previously it was possible to see the list of jobs and job output
without logging in, but now I'm prompted immediately to auth with GitHub.
After doing that, I get an error message "dpogue is missing the
Overall/Read permission".

This means the failing builds that are linked in GitHub PR comments aren't
able to be viewed by the authors of those PRs.

~Darryl

On 6 December 2016 at 02:09, Alexander Sorokin (Akvelon) <
v-als...@microsoft.com> wrote:

> Hey Filip.
>
> I've granted you the permissions you need for the cloudapp Jenkins. You'll
> just need to log in with your "filmaj" Github account.
> Use your new powers well, because while trying to give them to you I've
> almost messed up all the permissions settings on the CI :)
>
> Regarding your findings:
> Your [1] link points to *-test job, these jobs are only meant to test
> things like new paramedic code before it gets into master and generally
> live for a couple of days, then get deleted (if I don't forget to do so).
> This particular one is meant to test my upcoming fix for paramedic to
> properly handle the app uninstall timeout and the phantom failures we now
> get for windows phone 8.1 builds.
>
> Your [2] link seems like a "cordova-coho npm-link" issue which needs to be
> investigated. It doesn't look like it's breaking things so it's not a high
> priority (for me) right now.
>
> [3]: When a node goes offline, "CI Sentinels" ™ get a notification via
> email, right now that would be me, Vladimir Kotikov and Sergey
> Shakhnazarov. If you'd like to be included to the caste, just let me know -
> but since the slave machines are inside Microsoft infrastructure it is only
> us who have the access to both of them and it is only us who can bring the
> slave back online if something bad happens.
>
> Thanks,
> Alexander Sorokin
>
> -Original Message-
> From: Filip Maj [mailto:maj@gmail.com]
> Sent: Tuesday, December 6, 2016 1:32 AM
> To: dev@cordova.apache.org
> Subject: Access to Jenkins instance, questions about CI
>
> Dearest Cordova devs,
>
> I would like to get admin-level access to the cloudapp jenkins
> instance: https://na01.safelinks.protection.outlook.com/?url=
> http%3A%2F%2Fcordova-ci.cloudapp.net%3A8080%2F=02%7C01%7Cv-alsoro%
> 40microsoft.com%7C9ab9ec2d342a40ee3aee08d41d5e8c4d%
> 7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636165739303098482=
> pdUokGPtsLwPvq6SynMtUlRt5iTOn2yY8S0SlA%2FCKio%3D=0
>
> Who can grant me this karma?
>
> I wanted to check out how the various jobs are configured, what triggers
> them and what the triggered code looks like.
>
> I also wanted to fix some things I've noticed:
>  - File Transfer tests relying on Alex Sorokin's fork of paramedic, which
> now seems to be gone, and thus fails the job [1]
>  - I noticed some popd/pushd errors in the periodic builds for specific
> plugins, as well as npm error output [2]. Seems to be related to a missing
> cordova-create directory or something.
>
> Had some followup questions too!
>  - When a worker node goes down or becomes inaccessible, that may fail a
> build. (see e.g. the Mac node graphs [3]). When this happens, do we get
> alerted? And how would do we fix it?
>  - In the Periodic Builds section [4], what is the difference between the
> "build" [5] and the "test" [6] jobs? The test jobs seem to only care about
> network information and file transfer plugins, but other than that, I'm
> wondering what the vision here is?
>
> [1] https://na01.safelinks.protection.outlook.com/?url=
> http%3A%2F%2Fcordova-ci.cloudapp.net%3A8080%2Fview%
> 2FPeriodic%2520builds%2Fjob%2Fcordova-periodic-test%
> 2FPLATFORM%3Dandroid-4.4%2CPLUGIN%3Dcordova-plugin-file-transfer%2F2%
> 2FconsoleText=02%7C01%7Cv-alsoro%40microsoft.com%
> 7C9ab9ec2d342a40ee3aee08d41d5e8c4d%7C72f988bf86f141af91ab2d7cd011
> db47%7C1%7C0%7C636165739303098482=XQq%2F4FDHoCdK51iqBNoQPlgbB%
> 2BFtLHA67yiKOtDS9ZU%3D=0
> [2] https://na01.safelinks.protection.outlook.com/?url=
> http%3A%2F%2Fcordova-ci.cloudapp.net%3A8080%2Fview%
> 2FPeriodic%2520builds%2Fjob%2Fcordova-periodic-build%
> 2FPLATFORM%3Dandroid-4.4%2CPLUGIN%3Dcordova-plugin-
> device-motion%2F247%2Fconsole=02%7C01%7Cv-alsoro%40microsoft.com%
> 7C9ab9ec2d342a40ee3aee08d41d5e8c4d%7C72f988bf86f141af91ab2d7cd011
> db47%7C1%7C0%7C636165739303098482=lXshEVOrBmnM4TXvlZIuOke4OcgjKO
> jx8NAiw3ASZy4%3D=0
> [3] https://na01.safelinks.protection.outlook.com/?url=
> http%3A%2F%2Fcordova-ci.cloudapp.net%3A8080%2Fcomputer%2Fmac-slave%2Fload-
> statistics%3Ftype%3Dhour=02%7C01%7Cv-alsoro%40microsoft.com%
> 7C9ab9ec2d342a40ee3aee08d41d5e8c4d%7C72f988bf86f141af91ab2d7cd011
> 

Re: Access to Jenkins instance, questions about CI

2016-12-06 Thread Darryl Pogue
Hi Alexander,

It seems that public read permissions have broken for the Jenkins instance.
I think previously it was possible to see the list of jobs and job output
without logging in, but now I'm prompted immediately to auth with GitHub.
After doing that, I get an error message "dpogue is missing the
Overall/Read permission".

This means the failing builds that are linked in GitHub PR comments aren't
able to be viewed by the authors of those PRs.

~Darryl

On 6 December 2016 at 02:09, Alexander Sorokin (Akvelon) <
v-als...@microsoft.com> wrote:

> Hey Filip.
>
> I've granted you the permissions you need for the cloudapp Jenkins. You'll
> just need to log in with your "filmaj" Github account.
> Use your new powers well, because while trying to give them to you I've
> almost messed up all the permissions settings on the CI :)
>
> Regarding your findings:
> Your [1] link points to *-test job, these jobs are only meant to test
> things like new paramedic code before it gets into master and generally
> live for a couple of days, then get deleted (if I don't forget to do so).
> This particular one is meant to test my upcoming fix for paramedic to
> properly handle the app uninstall timeout and the phantom failures we now
> get for windows phone 8.1 builds.
>
> Your [2] link seems like a "cordova-coho npm-link" issue which needs to be
> investigated. It doesn't look like it's breaking things so it's not a high
> priority (for me) right now.
>
> [3]: When a node goes offline, "CI Sentinels" ™ get a notification via
> email, right now that would be me, Vladimir Kotikov and Sergey
> Shakhnazarov. If you'd like to be included to the caste, just let me know -
> but since the slave machines are inside Microsoft infrastructure it is only
> us who have the access to both of them and it is only us who can bring the
> slave back online if something bad happens.
>
> Thanks,
> Alexander Sorokin
>
> -Original Message-
> From: Filip Maj [mailto:maj@gmail.com]
> Sent: Tuesday, December 6, 2016 1:32 AM
> To: dev@cordova.apache.org
> Subject: Access to Jenkins instance, questions about CI
>
> Dearest Cordova devs,
>
> I would like to get admin-level access to the cloudapp jenkins
> instance: https://na01.safelinks.protection.outlook.com/?url=
> http%3A%2F%2Fcordova-ci.cloudapp.net%3A8080%2F=02%7C01%7Cv-alsoro%
> 40microsoft.com%7C9ab9ec2d342a40ee3aee08d41d5e8c4d%
> 7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636165739303098482=
> pdUokGPtsLwPvq6SynMtUlRt5iTOn2yY8S0SlA%2FCKio%3D=0
>
> Who can grant me this karma?
>
> I wanted to check out how the various jobs are configured, what triggers
> them and what the triggered code looks like.
>
> I also wanted to fix some things I've noticed:
>  - File Transfer tests relying on Alex Sorokin's fork of paramedic, which
> now seems to be gone, and thus fails the job [1]
>  - I noticed some popd/pushd errors in the periodic builds for specific
> plugins, as well as npm error output [2]. Seems to be related to a missing
> cordova-create directory or something.
>
> Had some followup questions too!
>  - When a worker node goes down or becomes inaccessible, that may fail a
> build. (see e.g. the Mac node graphs [3]). When this happens, do we get
> alerted? And how would do we fix it?
>  - In the Periodic Builds section [4], what is the difference between the
> "build" [5] and the "test" [6] jobs? The test jobs seem to only care about
> network information and file transfer plugins, but other than that, I'm
> wondering what the vision here is?
>
> [1] https://na01.safelinks.protection.outlook.com/?url=
> http%3A%2F%2Fcordova-ci.cloudapp.net%3A8080%2Fview%
> 2FPeriodic%2520builds%2Fjob%2Fcordova-periodic-test%
> 2FPLATFORM%3Dandroid-4.4%2CPLUGIN%3Dcordova-plugin-file-transfer%2F2%
> 2FconsoleText=02%7C01%7Cv-alsoro%40microsoft.com%
> 7C9ab9ec2d342a40ee3aee08d41d5e8c4d%7C72f988bf86f141af91ab2d7cd011
> db47%7C1%7C0%7C636165739303098482=XQq%2F4FDHoCdK51iqBNoQPlgbB%
> 2BFtLHA67yiKOtDS9ZU%3D=0
> [2] https://na01.safelinks.protection.outlook.com/?url=
> http%3A%2F%2Fcordova-ci.cloudapp.net%3A8080%2Fview%
> 2FPeriodic%2520builds%2Fjob%2Fcordova-periodic-build%
> 2FPLATFORM%3Dandroid-4.4%2CPLUGIN%3Dcordova-plugin-
> device-motion%2F247%2Fconsole=02%7C01%7Cv-alsoro%40microsoft.com%
> 7C9ab9ec2d342a40ee3aee08d41d5e8c4d%7C72f988bf86f141af91ab2d7cd011
> db47%7C1%7C0%7C636165739303098482=lXshEVOrBmnM4TXvlZIuOke4OcgjKO
> jx8NAiw3ASZy4%3D=0
> [3] https://na01.safelinks.protection.outlook.com/?url=
> http%3A%2F%2Fcordova-ci.cloudapp.net%3A8080%2Fcomputer%2Fmac-slave%2Fload-
> statistics%3Ftype%3Dhour=02%7C01%7Cv-alsoro%40microsoft.com%
> 7C9ab9ec2d342a40ee3aee08d41d5e8c4d%7C72f988bf86f141af91ab2d7cd011
> db47%7C1%7C0%7C636165739303098482=mHO4enOyIscFocBf9mlZLSwuGGmfsN
> aoKETtdbR%2F0og%3D=0
> [4] https://na01.safelinks.protection.outlook.com/?url=
> http%3A%2F%2Fcordova-ci.cloudapp.net%3A8080%2Fview%
> 2FPeriodic%2520builds%2F=02%7C01%7Cv-alsoro%40microsoft.com%
> 

Re: Access to Jenkins instance, questions about CI

2016-12-06 Thread Filip Maj
Thanks Alexander!

I seem to have config access to the jobs now, which is what I needed.
Thank you :) It looks like your email will be notified on
configuration changes, so if I mess something up, you will know pretty
quickly who did it :D

I will take a look at issue #2 to start :)

As for [3], yeah, if I don't have access to the infrastructure, tough
for me to try to fix anything :D. Some level of public monitoring
would be helpful, at least to alert the rest of the committers to an
infrastructure problem, but drilling into CI execution log output also
reveals the infrastructure problem quickly, so perhaps it is not
needed.

On Tue, Dec 6, 2016 at 2:09 AM, Alexander Sorokin (Akvelon)
 wrote:
> Hey Filip.
>
> I've granted you the permissions you need for the cloudapp Jenkins. You'll 
> just need to log in with your "filmaj" Github account.
> Use your new powers well, because while trying to give them to you I've 
> almost messed up all the permissions settings on the CI :)
>
> Regarding your findings:
> Your [1] link points to *-test job, these jobs are only meant to test things 
> like new paramedic code before it gets into master and generally live for a 
> couple of days, then get deleted (if I don't forget to do so).
> This particular one is meant to test my upcoming fix for paramedic to 
> properly handle the app uninstall timeout and the phantom failures we now get 
> for windows phone 8.1 builds.
>
> Your [2] link seems like a "cordova-coho npm-link" issue which needs to be 
> investigated. It doesn't look like it's breaking things so it's not a high 
> priority (for me) right now.
>
> [3]: When a node goes offline, "CI Sentinels" ™ get a notification via email, 
> right now that would be me, Vladimir Kotikov and Sergey Shakhnazarov. If 
> you'd like to be included to the caste, just let me know - but since the 
> slave machines are inside Microsoft infrastructure it is only us who have the 
> access to both of them and it is only us who can bring the slave back online 
> if something bad happens.
>
> Thanks,
> Alexander Sorokin
>
> -Original Message-
> From: Filip Maj [mailto:maj@gmail.com]
> Sent: Tuesday, December 6, 2016 1:32 AM
> To: dev@cordova.apache.org
> Subject: Access to Jenkins instance, questions about CI
>
> Dearest Cordova devs,
>
> I would like to get admin-level access to the cloudapp jenkins
> instance: 
> https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fcordova-ci.cloudapp.net%3A8080%2F=02%7C01%7Cv-alsoro%40microsoft.com%7C9ab9ec2d342a40ee3aee08d41d5e8c4d%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636165739303098482=pdUokGPtsLwPvq6SynMtUlRt5iTOn2yY8S0SlA%2FCKio%3D=0
>
> Who can grant me this karma?
>
> I wanted to check out how the various jobs are configured, what triggers them 
> and what the triggered code looks like.
>
> I also wanted to fix some things I've noticed:
>  - File Transfer tests relying on Alex Sorokin's fork of paramedic, which now 
> seems to be gone, and thus fails the job [1]
>  - I noticed some popd/pushd errors in the periodic builds for specific 
> plugins, as well as npm error output [2]. Seems to be related to a missing 
> cordova-create directory or something.
>
> Had some followup questions too!
>  - When a worker node goes down or becomes inaccessible, that may fail a 
> build. (see e.g. the Mac node graphs [3]). When this happens, do we get 
> alerted? And how would do we fix it?
>  - In the Periodic Builds section [4], what is the difference between the 
> "build" [5] and the "test" [6] jobs? The test jobs seem to only care about 
> network information and file transfer plugins, but other than that, I'm 
> wondering what the vision here is?
>
> [1] 
> https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fcordova-ci.cloudapp.net%3A8080%2Fview%2FPeriodic%2520builds%2Fjob%2Fcordova-periodic-test%2FPLATFORM%3Dandroid-4.4%2CPLUGIN%3Dcordova-plugin-file-transfer%2F2%2FconsoleText=02%7C01%7Cv-alsoro%40microsoft.com%7C9ab9ec2d342a40ee3aee08d41d5e8c4d%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636165739303098482=XQq%2F4FDHoCdK51iqBNoQPlgbB%2BFtLHA67yiKOtDS9ZU%3D=0
> [2] 
> https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fcordova-ci.cloudapp.net%3A8080%2Fview%2FPeriodic%2520builds%2Fjob%2Fcordova-periodic-build%2FPLATFORM%3Dandroid-4.4%2CPLUGIN%3Dcordova-plugin-device-motion%2F247%2Fconsole=02%7C01%7Cv-alsoro%40microsoft.com%7C9ab9ec2d342a40ee3aee08d41d5e8c4d%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636165739303098482=lXshEVOrBmnM4TXvlZIuOke4OcgjKOjx8NAiw3ASZy4%3D=0
> [3] 
> https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fcordova-ci.cloudapp.net%3A8080%2Fcomputer%2Fmac-slave%2Fload-statistics%3Ftype%3Dhour=02%7C01%7Cv-alsoro%40microsoft.com%7C9ab9ec2d342a40ee3aee08d41d5e8c4d%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636165739303098482=mHO4enOyIscFocBf9mlZLSwuGGmfsNaoKETtdbR%2F0og%3D=0
> [4] 
> 

[GitHub] cordova-plugin-file-transfer issue #170: CB-12154 file-transfer progressEven...

2016-12-06 Thread cordova-qa
Github user cordova-qa commented on the issue:

https://github.com/apache/cordova-plugin-file-transfer/pull/170
  
Cordova CI Build has one or more failures. Link: 
http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-file-transfer-pr/47/

 312 tests run, 26 skipped, 50 failed.



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



[GitHub] cordova-plugin-file-transfer pull request #170: CB-12154 file-transfer progr...

2016-12-06 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/cordova-plugin-file-transfer/pull/170


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



[GitHub] cordova-plugin-file-transfer pull request #169: CB-10974 Cordova file transf...

2016-12-06 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/cordova-plugin-file-transfer/pull/169


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



[GitHub] cordova-plugin-inappbrowser issue #187: CB-11136: InAppBrowser fails to clos...

2016-12-06 Thread Glenndejaeger
Github user Glenndejaeger commented on the issue:

https://github.com/apache/cordova-plugin-inappbrowser/pull/187
  
Any idea when this PR will be merged into the official plugin's code base? 
Don't want to use a forked version for a long time


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org



RE: Access to Jenkins instance, questions about CI

2016-12-06 Thread Alexander Sorokin (Akvelon)
Hey Filip.

I've granted you the permissions you need for the cloudapp Jenkins. You'll just 
need to log in with your "filmaj" Github account.
Use your new powers well, because while trying to give them to you I've almost 
messed up all the permissions settings on the CI :)

Regarding your findings:
Your [1] link points to *-test job, these jobs are only meant to test things 
like new paramedic code before it gets into master and generally live for a 
couple of days, then get deleted (if I don't forget to do so).
This particular one is meant to test my upcoming fix for paramedic to properly 
handle the app uninstall timeout and the phantom failures we now get for 
windows phone 8.1 builds.

Your [2] link seems like a "cordova-coho npm-link" issue which needs to be 
investigated. It doesn't look like it's breaking things so it's not a high 
priority (for me) right now.

[3]: When a node goes offline, "CI Sentinels" ™ get a notification via email, 
right now that would be me, Vladimir Kotikov and Sergey Shakhnazarov. If you'd 
like to be included to the caste, just let me know - but since the slave 
machines are inside Microsoft infrastructure it is only us who have the access 
to both of them and it is only us who can bring the slave back online if 
something bad happens.

Thanks,
Alexander Sorokin

-Original Message-
From: Filip Maj [mailto:maj@gmail.com] 
Sent: Tuesday, December 6, 2016 1:32 AM
To: dev@cordova.apache.org
Subject: Access to Jenkins instance, questions about CI

Dearest Cordova devs,

I would like to get admin-level access to the cloudapp jenkins
instance: 
https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fcordova-ci.cloudapp.net%3A8080%2F=02%7C01%7Cv-alsoro%40microsoft.com%7C9ab9ec2d342a40ee3aee08d41d5e8c4d%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636165739303098482=pdUokGPtsLwPvq6SynMtUlRt5iTOn2yY8S0SlA%2FCKio%3D=0

Who can grant me this karma?

I wanted to check out how the various jobs are configured, what triggers them 
and what the triggered code looks like.

I also wanted to fix some things I've noticed:
 - File Transfer tests relying on Alex Sorokin's fork of paramedic, which now 
seems to be gone, and thus fails the job [1]
 - I noticed some popd/pushd errors in the periodic builds for specific 
plugins, as well as npm error output [2]. Seems to be related to a missing 
cordova-create directory or something.

Had some followup questions too!
 - When a worker node goes down or becomes inaccessible, that may fail a build. 
(see e.g. the Mac node graphs [3]). When this happens, do we get alerted? And 
how would do we fix it?
 - In the Periodic Builds section [4], what is the difference between the 
"build" [5] and the "test" [6] jobs? The test jobs seem to only care about 
network information and file transfer plugins, but other than that, I'm 
wondering what the vision here is?

[1] 
https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fcordova-ci.cloudapp.net%3A8080%2Fview%2FPeriodic%2520builds%2Fjob%2Fcordova-periodic-test%2FPLATFORM%3Dandroid-4.4%2CPLUGIN%3Dcordova-plugin-file-transfer%2F2%2FconsoleText=02%7C01%7Cv-alsoro%40microsoft.com%7C9ab9ec2d342a40ee3aee08d41d5e8c4d%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636165739303098482=XQq%2F4FDHoCdK51iqBNoQPlgbB%2BFtLHA67yiKOtDS9ZU%3D=0
[2] 
https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fcordova-ci.cloudapp.net%3A8080%2Fview%2FPeriodic%2520builds%2Fjob%2Fcordova-periodic-build%2FPLATFORM%3Dandroid-4.4%2CPLUGIN%3Dcordova-plugin-device-motion%2F247%2Fconsole=02%7C01%7Cv-alsoro%40microsoft.com%7C9ab9ec2d342a40ee3aee08d41d5e8c4d%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636165739303098482=lXshEVOrBmnM4TXvlZIuOke4OcgjKOjx8NAiw3ASZy4%3D=0
[3] 
https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fcordova-ci.cloudapp.net%3A8080%2Fcomputer%2Fmac-slave%2Fload-statistics%3Ftype%3Dhour=02%7C01%7Cv-alsoro%40microsoft.com%7C9ab9ec2d342a40ee3aee08d41d5e8c4d%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636165739303098482=mHO4enOyIscFocBf9mlZLSwuGGmfsNaoKETtdbR%2F0og%3D=0
[4] 
https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fcordova-ci.cloudapp.net%3A8080%2Fview%2FPeriodic%2520builds%2F=02%7C01%7Cv-alsoro%40microsoft.com%7C9ab9ec2d342a40ee3aee08d41d5e8c4d%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636165739303098482=QuFwqRkFV2mhb1PS3q5iKL90%2F2WGnbRtA0POejl3P6I%3D=0
[5] 
https://na01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fcordova-ci.cloudapp.net%3A8080%2Fview%2FPeriodic%2520builds%2Fjob%2Fcordova-periodic-build%2F=02%7C01%7Cv-alsoro%40microsoft.com%7C9ab9ec2d342a40ee3aee08d41d5e8c4d%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636165739303098482=zD3VA0rRay0EYKLfOHDUAhc0zYo1P%2BaKoV9z6bvTazQ%3D=0
[6] 

[GitHub] cordova-docs pull request #664: Update images.md

2016-12-06 Thread ttimmy
GitHub user ttimmy opened a pull request:

https://github.com/apache/cordova-docs/pull/664

Update images.md



### Platforms affected
Docs

### What does this PR do?
Add unofficial note to android icon docs.

### What testing has been done on this change?
None.

### Checklist
- [x] [Reported an issue](http://cordova.apache.org/contribute/issues.html) 
in the JIRA database
- [x] Commit message follows the format: "CB-3232: (android) Fix bug with 
resolving file paths", where CB- is the JIRA ID & "android" is the platform 
affected.
- [ ] Added automated test coverage as appropriate for this change.

CB-12221: (android) Fix documentation issue.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/ttimmy/cordova-docs patch-1

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cordova-docs/pull/664.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 #664


commit bc14b3503e7c34dc1410fa55657534c12b858c50
Author: ttimmy 
Date:   2016-12-06T09:31:18Z

Update images.md

Adding an unofficial (and not thoroughly tested) note about a bug in 
android icons.




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org