[GitHub] mrutkows opened a new pull request #3598: Update all .md and .sh files to pass scancode for compliance.

2018-05-01 Thread GitBox
mrutkows opened a new pull request #3598: Update all .md and .sh files to pass scancode for compliance. URL: https://github.com/apache/incubator-openwhisk/pull/3598 ## Description Update all .md and .sh to comply with license and code scanning policy. ##

[GitHub] sciabarra commented on issue #14: Go actions not working using source and image

2018-05-01 Thread GitBox
sciabarra commented on issue #14: Go actions not working using source and image URL: https://github.com/apache/incubator-openwhisk-runtime-go/issues/14#issuecomment-385743411 I think you can close this This is an automated

[GitHub] sciabarra commented on issue #13: Add scala integration tests

2018-05-01 Thread GitBox
sciabarra commented on issue #13: Add scala integration tests URL: https://github.com/apache/incubator-openwhisk-runtime-go/issues/13#issuecomment-385743689 I provided the tests in a PR for apache/incubator-openwhisk so you can close this

[GitHub] eweiter opened a new pull request #50: update travis to push "master" tag to Docker on "master" branch merges

2018-05-01 Thread GitBox
eweiter opened a new pull request #50: update travis to push "master" tag to Docker on "master" branch merges URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/50 This PR adds the ability for Travis to push a "master" tag to Docker when any changes are merged on the

[GitHub] csantanapr commented on issue #15: replace github.com/sciabarracom

2018-05-01 Thread GitBox
csantanapr commented on issue #15: replace github.com/sciabarracom URL: https://github.com/apache/incubator-openwhisk-runtime-go/issues/15#issuecomment-385764838 Thanks @sciabarra for looking into this one. This is an

[GitHub] eweiter closed issue #44: Update Node runtime to push master tag to Docker from Travis build

2018-05-01 Thread GitBox
eweiter closed issue #44: Update Node runtime to push master tag to Docker from Travis build URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/issues/44 This is an automated message from the Apache Git

[GitHub] csantanapr commented on a change in pull request #50: update travis to push "master" tag to Docker on "master" branch merges

2018-05-01 Thread GitBox
csantanapr commented on a change in pull request #50: update travis to push "master" tag to Docker on "master" branch merges URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/50#discussion_r185317803 ## File path: .travis.yml ## @@ -30,16 +30,20 @@

[GitHub] codecov-io commented on issue #3598: Update all .md and .sh files to pass scancode for compliance.

2018-05-01 Thread GitBox
codecov-io commented on issue #3598: Update all .md and .sh files to pass scancode for compliance. URL: https://github.com/apache/incubator-openwhisk/pull/3598#issuecomment-385774845 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3598?src=pr=h1) Report > Merging

[GitHub] pritidesai closed pull request #292: Support ASF Header in .md and .sh files for scancode

2018-05-01 Thread GitBox
pritidesai closed pull request #292: Support ASF Header in .md and .sh files for scancode URL: https://github.com/apache/incubator-openwhisk-apigateway/pull/292 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] sciabarra commented on issue #17: Refactor and not use `core`

2018-05-01 Thread GitBox
sciabarra commented on issue #17: Refactor and not use `core` URL: https://github.com/apache/incubator-openwhisk-runtime-go/issues/17#issuecomment-385742870 I just followed the structure of other images This is an automated

[GitHub] sciabarra commented on issue #15: replace github.com/sciabarracom

2018-05-01 Thread GitBox
sciabarra commented on issue #15: replace github.com/sciabarracom URL: https://github.com/apache/incubator-openwhisk-runtime-go/issues/15#issuecomment-385743073 now that the code is available in github under apache you can do it

[GitHub] dubee commented on issue #3286: Treat action code as attachments for all runtimes and cache attachments

2018-05-01 Thread GitBox
dubee commented on issue #3286: Treat action code as attachments for all runtimes and cache attachments URL: https://github.com/apache/incubator-openwhisk/pull/3286#issuecomment-385749385 @rabbah, @markusthoemmes, a cache concurrency test fails on this one. I have figured out the failure

[GitHub] eweiter commented on issue #31: update travis to push "master" tag to Docker on "master" branch merges

2018-05-01 Thread GitBox
eweiter commented on issue #31: update travis to push "master" tag to Docker on "master" branch merges URL: https://github.com/apache/incubator-openwhisk-runtime-docker/pull/31#issuecomment-385758694 closes #30 This is an

[GitHub] csantanapr opened a new issue #20: split distDokcer, build and tests in separate tasks

2018-05-01 Thread GitBox
csantanapr opened a new issue #20: split distDokcer, build and tests in separate tasks URL: https://github.com/apache/incubator-openwhisk-runtime-go/issues/20 Today distDocker depends go build, and go build depends on go test We should split them in seperate tasks, then in Travis

[GitHub] eweiter commented on issue #3599: bump nodejs6 1.8.1 -> 1.9.0, bump nodejs8 1.5.1 -> 1.6.0

2018-05-01 Thread GitBox
eweiter commented on issue #3599: bump nodejs6 1.8.1 -> 1.9.0, bump nodejs8 1.5.1 -> 1.6.0 URL: https://github.com/apache/incubator-openwhisk/pull/3599#issuecomment-385770777 PG1 2380 is  This is an automated message from

[GitHub] csantanapr commented on a change in pull request #17: update Travis to push master tag on Docker when master branch has cha…

2018-05-01 Thread GitBox
csantanapr commented on a change in pull request #17: update Travis to push master tag on Docker when master branch has cha… URL: https://github.com/apache/incubator-openwhisk-runtime-php/pull/17#discussion_r185316151 ## File path: .travis.yml ## @@ -30,16 +30,20 @@

[GitHub] houshengbo opened a new pull request #3601: Change the LICENSE file into the correct format defined in openwhisk release

2018-05-01 Thread GitBox
houshengbo opened a new pull request #3601: Change the LICENSE file into the correct format defined in openwhisk release URL: https://github.com/apache/incubator-openwhisk/pull/3601 Partially-closes: apache/incubator-openwhisk-release#109

[GitHub] dubee commented on issue #3286: Treat action code as attachments for all runtimes and cache attachments

2018-05-01 Thread GitBox
dubee commented on issue #3286: Treat action code as attachments for all runtimes and cache attachments URL: https://github.com/apache/incubator-openwhisk/pull/3286#issuecomment-385749385 @rabbah, @markusthoemmes, a cache concurrency test fails on this one. I have figured out the failure

[GitHub] dubee commented on issue #3286: Treat action code as attachments for all runtimes and cache attachments

2018-05-01 Thread GitBox
dubee commented on issue #3286: Treat action code as attachments for all runtimes and cache attachments URL: https://github.com/apache/incubator-openwhisk/pull/3286#issuecomment-385749385 @rabbah, @markusthoemmes, a cache concurrency test fails on this one. I have figured out the failure

[GitHub] csantanapr commented on issue #10: Can't run tests in OSX

2018-05-01 Thread GitBox
csantanapr commented on issue #10: Can't run tests in OSX URL: https://github.com/apache/incubator-openwhisk-runtime-go/issues/10#issuecomment-385766817 What about running them using a docker container? I created a docker image using openjdk, then adding go 1.9.2 and try to run the

[GitHub] dubee opened a new pull request #287: Add retries to log integration test

2018-05-01 Thread GitBox
dubee opened a new pull request #287: Add retries to log integration test URL: https://github.com/apache/incubator-openwhisk-cli/pull/287 Prevents intermittent failures from eventual consistent databases. This is an automated

[GitHub] eweiter opened a new pull request #19: update travis to push "master" tag to Docker on "master" branch merges

2018-05-01 Thread GitBox
eweiter opened a new pull request #19: update travis to push "master" tag to Docker on "master" branch merges URL: https://github.com/apache/incubator-openwhisk-runtime-python/pull/19 This PR adds the ability for Travis to push a "master" tag to Docker when any changes are merged on the

[GitHub] eweiter commented on issue #19: update travis to push "master" tag to Docker on "master" branch merges

2018-05-01 Thread GitBox
eweiter commented on issue #19: update travis to push "master" tag to Docker on "master" branch merges URL: https://github.com/apache/incubator-openwhisk-runtime-python/pull/19#issuecomment-385754950 Closes #18 This is an

[GitHub] csantanapr closed pull request #46: update openwhisk npm package for nodejs6 and nodejs8

2018-05-01 Thread GitBox
csantanapr closed pull request #46: update openwhisk npm package for nodejs6 and nodejs8 URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/46 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake

[GitHub] csantanapr commented on issue #11: One test fails in Ubuntu Xenial

2018-05-01 Thread GitBox
csantanapr commented on issue #11: One test fails in Ubuntu Xenial URL: https://github.com/apache/incubator-openwhisk-runtime-go/issues/11#issuecomment-385768981 Also document as a prerequisites when running tests in the readme

[GitHub] csantanapr commented on issue #11: One test fails in Ubuntu Xenial

2018-05-01 Thread GitBox
csantanapr commented on issue #11: One test fails in Ubuntu Xenial URL: https://github.com/apache/incubator-openwhisk-runtime-go/issues/11#issuecomment-385768981 Also document as a prerequisites when running tests This is an

[GitHub] mrutkows opened a new pull request #29: Add .sh and .md scanning support in new .cfg file.

2018-05-01 Thread GitBox
mrutkows opened a new pull request #29: Add .sh and .md scanning support in new .cfg file. URL: https://github.com/apache/incubator-openwhisk-utilities/pull/29 This is an automated message from the Apache Git Service. To

[GitHub] sciabarra commented on issue #10: Can't run tests in OSX

2018-05-01 Thread GitBox
sciabarra commented on issue #10: Can't run tests in OSX URL: https://github.com/apache/incubator-openwhisk-runtime-go/issues/10#issuecomment-385745084 No, tests cannot run in OSX, there are many dependencies on an Linux environment. Since the target is actually Linux I did not try to

[GitHub] eweiter commented on issue #50: update travis to push "master" tag to Docker on "master" branch merges

2018-05-01 Thread GitBox
eweiter commented on issue #50: update travis to push "master" tag to Docker on "master" branch merges URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/50#issuecomment-385756399 closes #49 This is an

[GitHub] eweiter opened a new pull request #3599: bump nodejs6 1.8.1 -> 1.9.0, bump nodejs8 1.5.1 -> 1.6.0

2018-05-01 Thread GitBox
eweiter opened a new pull request #3599: bump nodejs6 1.8.1 -> 1.9.0, bump nodejs8 1.5.1 -> 1.6.0 URL: https://github.com/apache/incubator-openwhisk/pull/3599 This PR bumps the NodejS runtimes versions level to bring in the latest updates to that runtime. NodeJS6 is bumped from

[GitHub] houshengbo closed pull request #3598: Update all .md and .sh files to pass scancode for compliance.

2018-05-01 Thread GitBox
houshengbo closed pull request #3598: Update all .md and .sh files to pass scancode for compliance. URL: https://github.com/apache/incubator-openwhisk/pull/3598 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] jonpspri commented on issue #3534: Add abstraction to controller role in ansible

2018-05-01 Thread GitBox
jonpspri commented on issue #3534: Add abstraction to controller role in ansible URL: https://github.com/apache/incubator-openwhisk/pull/3534#issuecomment-385778654 Playground3/2192/  This is an automated message from the

[GitHub] sciabarra commented on issue #18: Use a more light weight base image

2018-05-01 Thread GitBox
sciabarra commented on issue #18: Use a more light weight base image URL: https://github.com/apache/incubator-openwhisk-runtime-go/issues/18#issuecomment-385742585 We can even use just a plain ubuntu for the base image. I could use even alpine but binaries need a libc generally so I would

[GitHub] eweiter commented on issue #17: update Travis to push master tag on Docker when master branch has cha…

2018-05-01 Thread GitBox
eweiter commented on issue #17: update Travis to push master tag on Docker when master branch has cha… URL: https://github.com/apache/incubator-openwhisk-runtime-php/pull/17#issuecomment-385752209 Related issue: https://github.com/apache/incubator-openwhisk-runtime-php/issues/16

[GitHub] csantanapr commented on issue #11: One test fails in Ubuntu Xenial

2018-05-01 Thread GitBox
csantanapr commented on issue #11: One test fails in Ubuntu Xenial URL: https://github.com/apache/incubator-openwhisk-runtime-go/issues/11#issuecomment-385768805 Ah easy to add `bc` I think I preferred to keep the test.

[GitHub] csantanapr commented on issue #11: One test fails in Ubuntu Xenial

2018-05-01 Thread GitBox
csantanapr commented on issue #11: One test fails in Ubuntu Xenial URL: https://github.com/apache/incubator-openwhisk-runtime-go/issues/11#issuecomment-385768875 Lets update vagrant file in core to install `bc` This is an

[GitHub] dubee opened a new pull request #3600: Remove unused variable from cache concurrency tests

2018-05-01 Thread GitBox
dubee opened a new pull request #3600: Remove unused variable from cache concurrency tests URL: https://github.com/apache/incubator-openwhisk/pull/3600 Variable `scriptPath` is not used and the fixture `CacheConcurrencyTests.sh` does not exist. ## Description ##

[GitHub] mrutkows opened a new pull request #81: Update .md files to pass scancode with latest .cfg

2018-05-01 Thread GitBox
mrutkows opened a new pull request #81: Update .md files to pass scancode with latest .cfg URL: https://github.com/apache/incubator-openwhisk-client-go/pull/81 This is an automated message from the Apache Git Service. To

[GitHub] mrutkows opened a new pull request #51: Update .md and .sh files to pass scancode with latest .cfg

2018-05-01 Thread GitBox
mrutkows opened a new pull request #51: Update .md and .sh files to pass scancode with latest .cfg URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/51 This is an automated message from the Apache Git

[GitHub] mrutkows opened a new pull request #47: Update .md and .sh files to pass scancode with latest .cfg

2018-05-01 Thread GitBox
mrutkows opened a new pull request #47: Update .md and .sh files to pass scancode with latest .cfg URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/47 This is an automated message from the Apache Git

[GitHub] mrutkows opened a new pull request #885: Update .md and .sh files to pass scancode with latest .cfg

2018-05-01 Thread GitBox
mrutkows opened a new pull request #885: Update .md and .sh files to pass scancode with latest .cfg URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/885 This is an automated message from the Apache Git

[GitHub] pritidesai closed pull request #81: Update .md files to pass scancode with latest .cfg

2018-05-01 Thread GitBox
pritidesai closed pull request #81: Update .md files to pass scancode with latest .cfg URL: https://github.com/apache/incubator-openwhisk-client-go/pull/81 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] rabbah closed pull request #3600: Remove unused variable from cache concurrency tests

2018-05-01 Thread GitBox
rabbah closed pull request #3600: Remove unused variable from cache concurrency tests URL: https://github.com/apache/incubator-openwhisk/pull/3600 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance:

[GitHub] eweiter commented on a change in pull request #17: update Travis to push master tag on Docker when master branch has cha…

2018-05-01 Thread GitBox
eweiter commented on a change in pull request #17: update Travis to push master tag on Docker when master branch has cha… URL: https://github.com/apache/incubator-openwhisk-runtime-php/pull/17#discussion_r185331360 ## File path: .travis.yml ## @@ -30,16 +30,20 @@

[GitHub] eweiter commented on a change in pull request #50: update travis to push "master" tag to Docker on "master" branch merges

2018-05-01 Thread GitBox
eweiter commented on a change in pull request #50: update travis to push "master" tag to Docker on "master" branch merges URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/50#discussion_r185331504 ## File path: .travis.yml ## @@ -30,16 +30,20 @@

[GitHub] eweiter commented on issue #3599: bump nodejs6 1.8.1 -> 1.9.0, bump nodejs8 1.5.1 -> 1.6.0

2018-05-01 Thread GitBox
eweiter commented on issue #3599: bump nodejs6 1.8.1 -> 1.9.0, bump nodejs8 1.5.1 -> 1.6.0 URL: https://github.com/apache/incubator-openwhisk/pull/3599#issuecomment-385770777 PG1 2831 is  This is an automated message from

[GitHub] pritidesai closed pull request #288: Update .md and .sh files to pass scancode.

2018-05-01 Thread GitBox
pritidesai closed pull request #288: Update .md and .sh files to pass scancode. URL: https://github.com/apache/incubator-openwhisk-cli/pull/288 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As

[GitHub] pritidesai closed pull request #885: Update .md and .sh files to pass scancode with latest .cfg

2018-05-01 Thread GitBox
pritidesai closed pull request #885: Update .md and .sh files to pass scancode with latest .cfg URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/885 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] codecov-io commented on issue #3600: Remove unused variable from cache concurrency tests

2018-05-01 Thread GitBox
codecov-io commented on issue #3600: Remove unused variable from cache concurrency tests URL: https://github.com/apache/incubator-openwhisk/pull/3600#issuecomment-385783686 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3600?src=pr=h1) Report > Merging

[GitHub] mrutkows opened a new pull request #271: Fix .md files to pass scanCode.py and comply with policy

2018-05-01 Thread GitBox
mrutkows opened a new pull request #271: Fix .md files to pass scanCode.py and comply with policy URL: https://github.com/apache/incubator-openwhisk-catalog/pull/271 This is an automated message from the Apache Git Service.

[GitHub] mrutkows opened a new pull request #32: Update .md and .sh files to pass scancode with latest .cfg

2018-05-01 Thread GitBox
mrutkows opened a new pull request #32: Update .md and .sh files to pass scancode with latest .cfg URL: https://github.com/apache/incubator-openwhisk-runtime-docker/pull/32 This is an automated message from the Apache Git

[GitHub] pritidesai closed pull request #47: Update .md and .sh files to pass scancode with latest .cfg

2018-05-01 Thread GitBox
pritidesai closed pull request #47: Update .md and .sh files to pass scancode with latest .cfg URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/47 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] ddragosd opened a new pull request #21: fixes #15

2018-05-01 Thread GitBox
ddragosd opened a new pull request #21: fixes #15 URL: https://github.com/apache/incubator-openwhisk-runtime-go/pull/21 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] codecov-io commented on issue #3602: Update build to use latest scancode .cfg for .md and .sh

2018-05-01 Thread GitBox
codecov-io commented on issue #3602: Update build to use latest scancode .cfg for .md and .sh URL: https://github.com/apache/incubator-openwhisk/pull/3602#issuecomment-385820676 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3602?src=pr=h1) Report > Merging

[GitHub] eweiter commented on issue #3599: bump nodejs6 1.8.1 -> 1.9.0, bump nodejs8 1.5.1 -> 1.6.0

2018-05-01 Thread GitBox
eweiter commented on issue #3599: bump nodejs6 1.8.1 -> 1.9.0, bump nodejs8 1.5.1 -> 1.6.0 URL: https://github.com/apache/incubator-openwhisk/pull/3599#issuecomment-385770777 PG1 2380 is  This is an automated message from

[GitHub] mrutkows opened a new pull request #288: Update .md and .sh files to pass scancode.

2018-05-01 Thread GitBox
mrutkows opened a new pull request #288: Update .md and .sh files to pass scancode. URL: https://github.com/apache/incubator-openwhisk-cli/pull/288 This is an automated message from the Apache Git Service. To respond to the

[GitHub] houshengbo closed pull request #29: Add .sh and .md scanning support in new .cfg file.

2018-05-01 Thread GitBox
houshengbo closed pull request #29: Add .sh and .md scanning support in new .cfg file. URL: https://github.com/apache/incubator-openwhisk-utilities/pull/29 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] ddragosd commented on issue #288: Apache release does not allow to have private license as the header

2018-05-01 Thread GitBox
ddragosd commented on issue #288: Apache release does not allow to have private license as the header URL: https://github.com/apache/incubator-openwhisk-apigateway/issues/288#issuecomment-385807163 I've assigned this issue to me as I'm planning on doing some work on config files towards

[GitHub] pritidesai closed pull request #271: Fix .md files to pass scanCode.py and comply with policy

2018-05-01 Thread GitBox
pritidesai closed pull request #271: Fix .md files to pass scanCode.py and comply with policy URL: https://github.com/apache/incubator-openwhisk-catalog/pull/271 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake

[GitHub] mrutkows opened a new pull request #3602: Update build to use latest scancode .cfg for .md and .sh

2018-05-01 Thread GitBox
mrutkows opened a new pull request #3602: Update build to use latest scancode .cfg for .md and .sh URL: https://github.com/apache/incubator-openwhisk/pull/3602 All .md and .sh files are now updated to conform to ASF license/policies as of merge of PR

[GitHub] mrutkows opened a new pull request #47: Update .md and .sh files to pass scancode with latest .cfg

2018-05-01 Thread GitBox
mrutkows opened a new pull request #47: Update .md and .sh files to pass scancode with latest .cfg URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/47 This is an automated message from the Apache Git

[GitHub] pritidesai closed pull request #32: Update .md and .sh files to pass scancode with latest .cfg

2018-05-01 Thread GitBox
pritidesai closed pull request #32: Update .md and .sh files to pass scancode with latest .cfg URL: https://github.com/apache/incubator-openwhisk-runtime-docker/pull/32 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] mrutkows opened a new pull request #20: Update .md and .sh files to pass scancode with latest .cfg

2018-05-01 Thread GitBox
mrutkows opened a new pull request #20: Update .md and .sh files to pass scancode with latest .cfg URL: https://github.com/apache/incubator-openwhisk-runtime-python/pull/20 This is an automated message from the Apache Git

[GitHub] sciabarra commented on issue #19: Mark readme as WIP

2018-05-01 Thread GitBox
sciabarra commented on issue #19: Mark readme as WIP URL: https://github.com/apache/incubator-openwhisk-runtime-go/issues/19#issuecomment-385742123 Now the README include documentation and the proxy is feature complete. Can close.

[GitHub] sciabarra commented on issue #11: One test fails in Ubuntu Xenial

2018-05-01 Thread GitBox
sciabarra commented on issue #11: One test fails in Ubuntu Xenial URL: https://github.com/apache/incubator-openwhisk-runtime-go/issues/11#issuecomment-385744681 That test needs the command "bc", and in Xenial is not installed by default (at lest in my own). This comment is also an

[GitHub] dubee commented on issue #3286: Treat action code as attachments for all runtimes and cache attachments

2018-05-01 Thread GitBox
dubee commented on issue #3286: Treat action code as attachments for all runtimes and cache attachments URL: https://github.com/apache/incubator-openwhisk/pull/3286#issuecomment-385749385 @rabbah, @markusthoemmes, a cache concurrency test fails on this one. I have figured out the failure

[GitHub] pritidesai commented on issue #884: passing parameters between manifest and deploy.yml

2018-05-01 Thread GitBox
pritidesai commented on issue #884: passing parameters between manifest and deploy.yml URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/884#issuecomment-385750203 Once the trigger is defined as alarm type, the trigger inputs are not propagated to actions: I

[GitHub] dubee commented on issue #3286: Treat action code as attachments for all runtimes and cache attachments

2018-05-01 Thread GitBox
dubee commented on issue #3286: Treat action code as attachments for all runtimes and cache attachments URL: https://github.com/apache/incubator-openwhisk/pull/3286#issuecomment-385751074 I suppose another option would be to change the test to accept a 200 or 404 from the `get` requests.

[GitHub] houshengbo opened a new pull request #270: Add the licenses of dependencies into LICENSE file

2018-05-01 Thread GitBox
houshengbo opened a new pull request #270: Add the licenses of dependencies into LICENSE file URL: https://github.com/apache/incubator-openwhisk-catalog/pull/270 This is an automated message from the Apache Git Service. To

[GitHub] csantanapr closed pull request #31: update travis to push "master" tag to Docker on "master" branch merges

2018-05-01 Thread GitBox
csantanapr closed pull request #31: update travis to push "master" tag to Docker on "master" branch merges URL: https://github.com/apache/incubator-openwhisk-runtime-docker/pull/31 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed

[GitHub] codecov-io commented on issue #3286: Treat action code as attachments for all runtimes and cache attachments

2018-05-01 Thread GitBox
codecov-io commented on issue #3286: Treat action code as attachments for all runtimes and cache attachments URL: https://github.com/apache/incubator-openwhisk/pull/3286#issuecomment-385776180 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3286?src=pr=h1) Report >

[GitHub] codecov-io commented on issue #2847: Treat action code as attachments for created/updated actions

2018-05-01 Thread GitBox
codecov-io commented on issue #2847: Treat action code as attachments for created/updated actions URL: https://github.com/apache/incubator-openwhisk/pull/2847#issuecomment-385775980 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/2847?src=pr=h1) Report > Merging

[GitHub] sciabarra commented on issue #23: FAIL: Example_badinit_nocompiler

2018-05-01 Thread GitBox
sciabarra commented on issue #23: FAIL: Example_badinit_nocompiler URL: https://github.com/apache/incubator-openwhisk-runtime-go/issues/23#issuecomment-385867780 I will This is an automated message from the Apache Git

[GitHub] pritidesai opened a new pull request #886: adding license files

2018-05-01 Thread GitBox
pritidesai opened a new pull request #886: adding license files URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/886 @mrutkows I reviewed the entire list of dependencies from Godeps.json and there is no reference to the following go libraries in whisk deploy source:

[GitHub] sciabarra commented on issue #10: Can't run tests in OSX

2018-05-01 Thread GitBox
sciabarra commented on issue #10: Can't run tests in OSX URL: https://github.com/apache/incubator-openwhisk-runtime-go/issues/10#issuecomment-385868558 Will try to make them work on OSX. I actually use OSX but I developed everything inside a vm with ubuntu 14.04.

[GitHub] pritidesai closed pull request #20: Update .md and .sh files to pass scancode with latest .cfg

2018-05-01 Thread GitBox
pritidesai closed pull request #20: Update .md and .sh files to pass scancode with latest .cfg URL: https://github.com/apache/incubator-openwhisk-runtime-python/pull/20 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] pritidesai closed pull request #51: Update .md and .sh files to pass scancode with latest .cfg

2018-05-01 Thread GitBox
pritidesai closed pull request #51: Update .md and .sh files to pass scancode with latest .cfg URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/51 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] daisy-ycguo commented on issue #96: Document LICENSE file Exclusions for incubator-openwhisk-catalog

2018-05-01 Thread GitBox
daisy-ycguo commented on issue #96: Document LICENSE file Exclusions for incubator-openwhisk-catalog URL: https://github.com/apache/incubator-openwhisk-release/issues/96#issuecomment-385846787 A PR has been merged https://github.com/apache/incubator-openwhisk-catalog/pull/262 to fix this

[GitHub] daisy-ycguo commented on issue #222: A list of source files missing Apache license headers

2018-05-01 Thread GitBox
daisy-ycguo commented on issue #222: A list of source files missing Apache license headers URL: https://github.com/apache/incubator-openwhisk-cli/issues/222#issuecomment-385847984 @mrutkows These dat files under tests are removed. They are not there. So we can close this issue.

[GitHub] csantanapr commented on issue #14: Go actions not working using source and image

2018-05-01 Thread GitBox
csantanapr commented on issue #14: Go actions not working using source and image URL: https://github.com/apache/incubator-openwhisk-runtime-go/issues/14#issuecomment-385851051 I was using the wrong file, now using `tests/src/hello/hello.go` works

[GitHub] csantanapr closed issue #14: Go actions not working using source and image

2018-05-01 Thread GitBox
csantanapr closed issue #14: Go actions not working using source and image URL: https://github.com/apache/incubator-openwhisk-runtime-go/issues/14 This is an automated message from the Apache Git Service. To respond to the

[GitHub] csantanapr closed issue #11: One test fails in Ubuntu Xenial

2018-05-01 Thread GitBox
csantanapr closed issue #11: One test fails in Ubuntu Xenial URL: https://github.com/apache/incubator-openwhisk-runtime-go/issues/11 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] csantanapr closed issue #17: Refactor and not use `core`

2018-05-01 Thread GitBox
csantanapr closed issue #17: Refactor and not use `core` URL: https://github.com/apache/incubator-openwhisk-runtime-go/issues/17 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] csantanapr closed pull request #22: switch to go 1.10

2018-05-01 Thread GitBox
csantanapr closed pull request #22: switch to go 1.10 URL: https://github.com/apache/incubator-openwhisk-runtime-go/pull/22 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign

[GitHub] csantanapr closed issue #9: Add ability to push master tag to Docker after Travis build

2018-05-01 Thread GitBox
csantanapr closed issue #9: Add ability to push master tag to Docker after Travis build URL: https://github.com/apache/incubator-openwhisk-runtime-go/issues/9 This is an automated message from the Apache Git Service. To

[GitHub] csantanapr closed issue #18: Use a more light weight base image

2018-05-01 Thread GitBox
csantanapr closed issue #18: Use a more light weight base image URL: https://github.com/apache/incubator-openwhisk-runtime-go/issues/18 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] csantanapr closed issue #20: split distDokcer, build and tests in separate tasks

2018-05-01 Thread GitBox
csantanapr closed issue #20: split distDokcer, build and tests in separate tasks URL: https://github.com/apache/incubator-openwhisk-runtime-go/issues/20 This is an automated message from the Apache Git Service. To respond to

[GitHub] csantanapr closed issue #16: Start support with go 1.10 instead of 1.9

2018-05-01 Thread GitBox
csantanapr closed issue #16: Start support with go 1.10 instead of 1.9 URL: https://github.com/apache/incubator-openwhisk-runtime-go/issues/16 This is an automated message from the Apache Git Service. To respond to the

[GitHub] csantanapr commented on issue #23: FAIL: Example_badinit_nocompiler

2018-05-01 Thread GitBox
csantanapr commented on issue #23: FAIL: Example_badinit_nocompiler URL: https://github.com/apache/incubator-openwhisk-runtime-go/issues/23#issuecomment-385864455 @sciabarracom could you take a look why this test is failing.

[GitHub] mrutkows opened a new pull request #18: Update .md and .sh files to pass scancode with latest .cfg

2018-05-01 Thread GitBox
mrutkows opened a new pull request #18: Update .md and .sh files to pass scancode with latest .cfg URL: https://github.com/apache/incubator-openwhisk-runtime-php/pull/18 This is an automated message from the Apache Git

[GitHub] csantanapr opened a new pull request #22: switch to go 1.10

2018-05-01 Thread GitBox
csantanapr opened a new pull request #22: switch to go 1.10 URL: https://github.com/apache/incubator-openwhisk-runtime-go/pull/22 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] csantanapr opened a new issue #23: FAIL: Example_badinit_nocompiler

2018-05-01 Thread GitBox
csantanapr opened a new issue #23: FAIL: Example_badinit_nocompiler URL: https://github.com/apache/incubator-openwhisk-runtime-go/issues/23 Build in travis is failing https://travis-ci.org/apache/incubator-openwhisk-runtime-go/builds/373762183#L1197 ``` --- FAIL:

[GitHub] pritidesai closed pull request #47: Update .md and .sh files to pass scancode with latest .cfg

2018-05-01 Thread GitBox
pritidesai closed pull request #47: Update .md and .sh files to pass scancode with latest .cfg URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/47 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] pritidesai closed pull request #3602: Update build to use latest scancode .cfg for .md and .sh

2018-05-01 Thread GitBox
pritidesai closed pull request #3602: Update build to use latest scancode .cfg for .md and .sh URL: https://github.com/apache/incubator-openwhisk/pull/3602 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] mrutkows commented on issue #93: Document LICENSE file Exclusions for incubator-openwhisk

2018-05-01 Thread GitBox
mrutkows commented on issue #93: Document LICENSE file Exclusions for incubator-openwhisk URL: https://github.com/apache/incubator-openwhisk-release/issues/93#issuecomment-385844995 @houshengbo is making updates on this PR https://github.com/apache/incubator-openwhisk/pull/3601

[GitHub] pritidesai closed pull request #18: Update .md and .sh files to pass scancode with latest .cfg

2018-05-01 Thread GitBox
pritidesai closed pull request #18: Update .md and .sh files to pass scancode with latest .cfg URL: https://github.com/apache/incubator-openwhisk-runtime-php/pull/18 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] ddragosd commented on issue #10: Can't run tests in OSX

2018-05-01 Thread GitBox
ddragosd commented on issue #10: Can't run tests in OSX URL: https://github.com/apache/incubator-openwhisk-runtime-go/issues/10#issuecomment-385832221  for running the tests in a docker container, ideally using the same base docker image used when running the action.

[GitHub] daisy-ycguo closed issue #125: Update RAT config to exclude some files in OpenWhisk repo

2018-05-01 Thread GitBox
daisy-ycguo closed issue #125: Update RAT config to exclude some files in OpenWhisk repo URL: https://github.com/apache/incubator-openwhisk-release/issues/125 This is an automated message from the Apache Git Service. To

[GitHub] mrutkows commented on issue #288: Apache release does not allow to have private license as the header

2018-05-01 Thread GitBox
mrutkows commented on issue #288: Apache release does not allow to have private license as the header URL: https://github.com/apache/incubator-openwhisk-apigateway/issues/288#issuecomment-385847511 @ddragosd IMO, if we keep these Adobe/MIT derivative licenses (and they cannot be changed

[GitHub] ddragosd commented on a change in pull request #22: switch to go 1.10

2018-05-01 Thread GitBox
ddragosd commented on a change in pull request #22: switch to go 1.10 URL: https://github.com/apache/incubator-openwhisk-runtime-go/pull/22#discussion_r185387159 ## File path: actionProxyLoop/Dockerfile ## @@ -0,0 +1,9 @@ +FROM golang:1.10.1 Review comment: :+1:

  1   2   >