[GitHub] csantanapr closed pull request #47: Add ActionProxyContainer tests

2018-06-19 Thread GitBox
csantanapr closed pull request #47: Add ActionProxyContainer tests URL: https://github.com/apache/incubator-openwhisk-runtime-docker/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 sake of provenance: As this

[GitHub] mhamann commented on issue #312: awk script in init.sh matches wrong lines in /etc/resolv.conf

2018-06-19 Thread GitBox
mhamann commented on issue #312: awk script in init.sh matches wrong lines in /etc/resolv.conf URL: https://github.com/apache/incubator-openwhisk-apigateway/issues/312#issuecomment-398609617 Thanks for the issue and PR! Nice catch. Merged!

[GitHub] mhamann closed issue #312: awk script in init.sh matches wrong lines in /etc/resolv.conf

2018-06-19 Thread GitBox
mhamann closed issue #312: awk script in init.sh matches wrong lines in /etc/resolv.conf URL: https://github.com/apache/incubator-openwhisk-apigateway/issues/312 This is an automated message from the Apache Git Service. To

[GitHub] mhamann closed pull request #313: Fix awk command in init.sh generating resolvers.conf file

2018-06-19 Thread GitBox
mhamann closed pull request #313: Fix awk command in init.sh generating resolvers.conf file URL: https://github.com/apache/incubator-openwhisk-apigateway/pull/313 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] codecov-io edited a comment on issue #3656: Adjust the ForcibleSemaphore mechanism

2018-06-19 Thread GitBox
codecov-io edited a comment on issue #3656: Adjust the ForcibleSemaphore mechanism URL: https://github.com/apache/incubator-openwhisk/pull/3656#issuecomment-389115907 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3656?src=pr=h1) Report > Merging

[GitHub] csantanapr opened a new pull request #47: Add ActionProxyContainer tests

2018-06-19 Thread GitBox
csantanapr opened a new pull request #47: Add ActionProxyContainer tests URL: https://github.com/apache/incubator-openwhisk-runtime-docker/pull/47 This is an automated message from the Apache Git Service. To respond to the

[GitHub] codecov-io edited a comment on issue #3745: Remove runtime builds

2018-06-19 Thread GitBox
codecov-io edited a comment on issue #3745: Remove runtime builds URL: https://github.com/apache/incubator-openwhisk/pull/3745#issuecomment-396133122 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3745?src=pr=h1) Report > Merging

[GitHub] codecov-io edited a comment on issue #3745: Remove runtime builds

2018-06-19 Thread GitBox
codecov-io edited a comment on issue #3745: Remove runtime builds URL: https://github.com/apache/incubator-openwhisk/pull/3745#issuecomment-396133122 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3745?src=pr=h1) Report > Merging

[GitHub] jiangpengcheng commented on issue #3656: Adjust the ForcableSemaphore mechanism

2018-06-19 Thread GitBox
jiangpengcheng commented on issue #3656: Adjust the ForcableSemaphore mechanism URL: https://github.com/apache/incubator-openwhisk/pull/3656#issuecomment-398602587 @rabbah, yes, I set the `currentInvokerThreshold` as the hard max, so the total slots for an invoker won't exceeded that

[GitHub] rabbah commented on a change in pull request #3745: Remove runtime builds

2018-06-19 Thread GitBox
rabbah commented on a change in pull request #3745: Remove runtime builds URL: https://github.com/apache/incubator-openwhisk/pull/3745#discussion_r196629750 ## File path: actionRuntimes/actionProxy/build.gradle ## @@ -1,19 +0,0 @@ -/* - * Licensed to the Apache Software

[GitHub] rabbah commented on a change in pull request #3745: Remove runtime builds

2018-06-19 Thread GitBox
rabbah commented on a change in pull request #3745: Remove runtime builds URL: https://github.com/apache/incubator-openwhisk/pull/3745#discussion_r196629829 ## File path: tools/build/redo ## @@ -274,12 +274,6 @@ Components = [ yaml = 'postdeploy.yml'),

[GitHub] houshengbo closed pull request #191: Add the script to retrieve the latest commit hashes for all repos

2018-06-19 Thread GitBox
houshengbo closed pull request #191: Add the script to retrieve the latest commit hashes for all repos URL: https://github.com/apache/incubator-openwhisk-release/pull/191 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for

[GitHub] houshengbo opened a new pull request #191: Add script get latest commit

2018-06-19 Thread GitBox
houshengbo opened a new pull request #191: Add script get latest commit URL: https://github.com/apache/incubator-openwhisk-release/pull/191 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] markusthoemmes commented on issue #2932: refactoring loadbalacing service with an overflow queue

2018-06-19 Thread GitBox
markusthoemmes commented on issue #2932: refactoring loadbalacing service with an overflow queue URL: https://github.com/apache/incubator-openwhisk/pull/2932#issuecomment-398558406 @tysonnorris I'll close this then for now. Please reopen a PR once this is being worked on again.

[GitHub] markusthoemmes closed pull request #2932: refactoring loadbalacing service with an overflow queue

2018-06-19 Thread GitBox
markusthoemmes closed pull request #2932: refactoring loadbalacing service with an overflow queue URL: https://github.com/apache/incubator-openwhisk/pull/2932 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] codecov-io commented on issue #3785: Refactor invoker roles

2018-06-19 Thread GitBox
codecov-io commented on issue #3785: Refactor invoker roles URL: https://github.com/apache/incubator-openwhisk/pull/3785#issuecomment-398556807 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3785?src=pr=h1) Report > Merging

[GitHub] houshengbo opened a new pull request #190: Prepare the incubator-1.0.0 rc2 artifacts

2018-06-19 Thread GitBox
houshengbo opened a new pull request #190: Prepare the incubator-1.0.0 rc2 artifacts URL: https://github.com/apache/incubator-openwhisk-release/pull/190 This is an automated message from the Apache Git Service. To respond

[GitHub] csantanapr closed pull request #66: Add skip_pull_runtimes for Travis CI

2018-06-19 Thread GitBox
csantanapr closed pull request #66: Add skip_pull_runtimes for Travis CI URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/66 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] rabbah commented on issue #344: add OS and CPU architecture to user agent header

2018-06-19 Thread GitBox
rabbah commented on issue #344: add OS and CPU architecture to user agent header URL: https://github.com/apache/incubator-openwhisk-cli/pull/344#issuecomment-398547127 sounds good re the client. Thanks for doing this. This

[GitHub] dubee opened a new pull request #3785: Refactor invoker roles

2018-06-19 Thread GitBox
dubee opened a new pull request #3785: Refactor invoker roles URL: https://github.com/apache/incubator-openwhisk/pull/3785 Refactors the invoker Ansible roles to make the invoker have plugable tasks, and simplifies invoker name and an index retrieval within Ansible scripts. ##

[GitHub] drcariel opened a new pull request #344: add OS and CPU architecture to user agent header

2018-06-19 Thread GitBox
drcariel opened a new pull request #344: add OS and CPU architecture to user agent header URL: https://github.com/apache/incubator-openwhisk-cli/pull/344 Closes https://github.com/apache/incubator-openwhisk-cli/issues/165 Also, should I make a similar change to the client?

[GitHub] tysonnorris commented on issue #2932: refactoring loadbalacing service with an overflow queue

2018-06-19 Thread GitBox
tysonnorris commented on issue #2932: refactoring loadbalacing service with an overflow queue URL: https://github.com/apache/incubator-openwhisk/pull/2932#issuecomment-398520497 No news, too many other things in flight. Will get back to this at some point.

[GitHub] csantanapr closed pull request #3782: Make test-instances of Exec depend on the loaded manifest.

2018-06-19 Thread GitBox
csantanapr closed pull request #3782: Make test-instances of Exec depend on the loaded manifest. URL: https://github.com/apache/incubator-openwhisk/pull/3782 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] dgrove-oss opened a new pull request #228: only run deploy script for apache/incubator-openwhisk-deploy-kube:master

2018-06-19 Thread GitBox
dgrove-oss opened a new pull request #228: only run deploy script for apache/incubator-openwhisk-deploy-kube:master URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/228 This is an automated message from

[GitHub] codecov-io edited a comment on issue #3722: Scala based admin tooling

2018-06-19 Thread GitBox
codecov-io edited a comment on issue #3722: Scala based admin tooling URL: https://github.com/apache/incubator-openwhisk/pull/3722#issuecomment-394275882 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3722?src=pr=h1) Report > Merging

[GitHub] eweiter opened a new pull request #67: add to travis.yml the runtime repo in the deploy block

2018-06-19 Thread GitBox
eweiter opened a new pull request #67: add to travis.yml the runtime repo in the deploy block URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/67 This PR is to lock down the Deploy step of the Travis.yml so that it only executes when the repo matches the "upstream"

[GitHub] chetanmeh commented on issue #3722: Scala based admin tooling

2018-06-19 Thread GitBox
chetanmeh commented on issue #3722: Scala based admin tooling URL: https://github.com/apache/incubator-openwhisk/pull/3722#issuecomment-398493802 > Can the existing wskadmin tests pass against wskadmin-next May be not all as in some cases the message format may have changed. Would

[GitHub] chetanmeh commented on a change in pull request #3722: Scala based admin tooling

2018-06-19 Thread GitBox
chetanmeh commented on a change in pull request #3722: Scala based admin tooling URL: https://github.com/apache/incubator-openwhisk/pull/3722#discussion_r196526808 ## File path: tools/admin/src/main/scala/whisk/core/cli/Main.scala ## @@ -0,0 +1,163 @@ +/* + * Licensed to

[GitHub] houshengbo closed pull request #29: Add skip_pull_runtimes for Travis CI

2018-06-19 Thread GitBox
houshengbo closed pull request #29: Add skip_pull_runtimes for Travis CI URL: https://github.com/apache/incubator-openwhisk-runtime-python/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 provenance:

[GitHub] rabbah closed pull request #31: skip invoker pull runtimes

2018-06-19 Thread GitBox
rabbah closed pull request #31: skip invoker pull runtimes URL: https://github.com/apache/incubator-openwhisk-runtime-php/pull/31 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

[GitHub] houshengbo closed pull request #59: Add skip_pull_runtimes for Travis CI

2018-06-19 Thread GitBox
houshengbo closed pull request #59: Add skip_pull_runtimes for Travis CI URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/59 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] akrabat commented on issue #31: skip invoker pull runtimes

2018-06-19 Thread GitBox
akrabat commented on issue #31: skip invoker pull runtimes URL: https://github.com/apache/incubator-openwhisk-runtime-php/pull/31#issuecomment-398489150 I think that this a duplicate of #30 This is an automated message from

[GitHub] houshengbo closed pull request #30: Add skip_pull_runtimes for Travis CI

2018-06-19 Thread GitBox
houshengbo closed pull request #30: Add skip_pull_runtimes for Travis CI URL: https://github.com/apache/incubator-openwhisk-runtime-php/pull/30 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] houshengbo closed pull request #45: Add skip_pull_runtimes for Travis CI

2018-06-19 Thread GitBox
houshengbo closed pull request #45: Add skip_pull_runtimes for Travis CI URL: https://github.com/apache/incubator-openwhisk-runtime-docker/pull/45 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] rabbah commented on a change in pull request #48: Allow file-level exclusions.

2018-06-19 Thread GitBox
rabbah commented on a change in pull request #48: Allow file-level exclusions. URL: https://github.com/apache/incubator-openwhisk-utilities/pull/48#discussion_r196520349 ## File path: scancode/scanCode.py ## @@ -409,17 +409,22 @@ def all_paths(root_dir): # For every

[GitHub] rabbah opened a new pull request #48: Allow file-level exclusions.

2018-06-19 Thread GitBox
rabbah opened a new pull request #48: Allow file-level exclusions. URL: https://github.com/apache/incubator-openwhisk-utilities/pull/48 A small change to allow file level exclusions to complement the whole sale directory exclusion.

[GitHub] rabbah edited a comment on issue #3622: make containers' timezone sync with the docker host

2018-06-19 Thread GitBox
rabbah edited a comment on issue #3622: make containers' timezone sync with the docker host URL: https://github.com/apache/incubator-openwhisk/pull/3622#issuecomment-398481996 While this sets the localtime zone note that from the system log's pov, they are all relative to UTC.

[GitHub] rabbah commented on issue #3622: make containers' timezone sync with the docker host

2018-06-19 Thread GitBox
rabbah commented on issue #3622: make containers' timezone sync with the docker host URL: https://github.com/apache/incubator-openwhisk/pull/3622#issuecomment-398481996 While this sets the localtime zone note that from the system log's pov, they are all relative to UTC.

[GitHub] rabbah commented on issue #2941: Add api gateway test that uses custom package for action

2018-06-19 Thread GitBox
rabbah commented on issue #2941: Add api gateway test that uses custom package for action URL: https://github.com/apache/incubator-openwhisk/pull/2941#issuecomment-398481483 @drcariel will you take care of that after I merge this or do you want to add to this branch?

[GitHub] houshengbo opened a new pull request #59: Add skip_pull_runtimes for Travis CI

2018-06-19 Thread GitBox
houshengbo opened a new pull request #59: Add skip_pull_runtimes for Travis CI URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/59 This is an automated message from the Apache Git Service. To respond to

[GitHub] csantanapr opened a new pull request #31: skip invoker pull runtimes

2018-06-19 Thread GitBox
csantanapr opened a new pull request #31: skip invoker pull runtimes URL: https://github.com/apache/incubator-openwhisk-runtime-php/pull/31 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] houshengbo closed pull request #64: add skip_pull_runtimes for travis CI

2018-06-19 Thread GitBox
houshengbo closed pull request #64: add skip_pull_runtimes for travis CI URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/64 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] houshengbo opened a new pull request #30: Add skip_pull_runtimes for Travis CI

2018-06-19 Thread GitBox
houshengbo opened a new pull request #30: Add skip_pull_runtimes for Travis CI URL: https://github.com/apache/incubator-openwhisk-runtime-php/pull/30 This is an automated message from the Apache Git Service. To respond to

[GitHub] houshengbo opened a new pull request #29: Add skip_pull_runtimes for Travis CI

2018-06-19 Thread GitBox
houshengbo opened a new pull request #29: Add skip_pull_runtimes for Travis CI URL: https://github.com/apache/incubator-openwhisk-runtime-python/pull/29 This is an automated message from the Apache Git Service. To respond to

[GitHub] tysonnorris commented on issue #41: remove use of state in nodejs app

2018-06-19 Thread GitBox
tysonnorris commented on issue #41: remove use of state in nodejs app URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/41#issuecomment-398478702 @csantanapr let me know if these changes address your concerns.

[GitHub] rabbah closed pull request #3783: Fixes for vagrant installation versus oracle jdk

2018-06-19 Thread GitBox
rabbah closed pull request #3783: Fixes for vagrant installation versus oracle jdk URL: https://github.com/apache/incubator-openwhisk/pull/3783 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] rabbah closed issue #3765: Vagrant installation waits for user accepts java 8 license

2018-06-19 Thread GitBox
rabbah closed issue #3765: Vagrant installation waits for user accepts java 8 license URL: https://github.com/apache/incubator-openwhisk/issues/3765 This is an automated message from the Apache Git Service. To respond to

[GitHub] houshengbo closed pull request #3784: skip pull runtimes when skip_pull_runtimes is true

2018-06-19 Thread GitBox
houshengbo closed pull request #3784: skip pull runtimes when skip_pull_runtimes is true URL: https://github.com/apache/incubator-openwhisk/pull/3784 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] csantanapr opened a new pull request #64: add skip_pull_runtimes for travis CI

2018-06-19 Thread GitBox
csantanapr opened a new pull request #64: add skip_pull_runtimes for travis CI URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/64 This is an automated message from the Apache Git Service. To respond to

[GitHub] codecov-io commented on issue #3784: skip pull runtimes when skip_pull_runtimes is true

2018-06-19 Thread GitBox
codecov-io commented on issue #3784: skip pull runtimes when skip_pull_runtimes is true URL: https://github.com/apache/incubator-openwhisk/pull/3784#issuecomment-398470855 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3784?src=pr=h1) Report > Merging

[GitHub] jthomas opened a new issue #63: Thrown errors from async function missing details

2018-06-19 Thread GitBox
jthomas opened a new issue #63: Thrown errors from async function missing details URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/issues/63 ## Environment details: IBM Cloud Functions ## Steps to reproduce the issue: Create a function using the

[GitHub] houshengbo closed pull request #954: disabling export integration tests

2018-06-19 Thread GitBox
houshengbo closed pull request #954: disabling export integration tests URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/954 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] csantanapr opened a new pull request #3784: skip pull runtimes when skip_pull_runtimes is true

2018-06-19 Thread GitBox
csantanapr opened a new pull request #3784: skip pull runtimes when skip_pull_runtimes is true URL: https://github.com/apache/incubator-openwhisk/pull/3784 I think there is a very minor logic problem with ansible variable `skip_pull_runtimes` in invoker role This would allow to skip

[GitHub] chetanmeh commented on a change in pull request #3722: Scala based admin tooling

2018-06-19 Thread GitBox
chetanmeh commented on a change in pull request #3722: Scala based admin tooling URL: https://github.com/apache/incubator-openwhisk/pull/3722#discussion_r196483516 ## File path: tools/admin/README-NEXT.md ## @@ -0,0 +1,126 @@ + + +## Administrative Operations + +The

[GitHub] chetanmeh commented on a change in pull request #3722: Scala based admin tooling

2018-06-19 Thread GitBox
chetanmeh commented on a change in pull request #3722: Scala based admin tooling URL: https://github.com/apache/incubator-openwhisk/pull/3722#discussion_r196483743 ## File path: tools/admin/README-NEXT.md ## @@ -0,0 +1,126 @@ + + +## Administrative Operations + +The

[GitHub] codecov-io commented on issue #3783: Fixes for vagrant installation versus oracle jdk

2018-06-19 Thread GitBox
codecov-io commented on issue #3783: Fixes for vagrant installation versus oracle jdk URL: https://github.com/apache/incubator-openwhisk/pull/3783#issuecomment-398440556 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3783?src=pr=h1) Report > Merging

[GitHub] chetanmeh commented on a change in pull request #3722: Scala based admin tooling

2018-06-19 Thread GitBox
chetanmeh commented on a change in pull request #3722: Scala based admin tooling URL: https://github.com/apache/incubator-openwhisk/pull/3722#discussion_r196470422 ## File path: tools/admin/src/main/scala/whisk/core/database/UserCommand.scala ## @@ -0,0 +1,352 @@ +/* + *

[GitHub] chetanmeh commented on a change in pull request #3722: Scala based admin tooling

2018-06-19 Thread GitBox
chetanmeh commented on a change in pull request #3722: Scala based admin tooling URL: https://github.com/apache/incubator-openwhisk/pull/3722#discussion_r196466560 ## File path: tools/admin/src/main/scala/whisk/core/cli/Main.scala ## @@ -0,0 +1,163 @@ +/* + * Licensed to

[GitHub] eweiter commented on issue #58: add to travis.yml the runtime repo in the deploy block

2018-06-19 Thread GitBox
eweiter commented on issue #58: add to travis.yml the runtime repo in the deploy block URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/58#issuecomment-398435400 @csantanapr Please review This is an

[GitHub] eweiter opened a new pull request #58: add to travis.yml the runtime repo in the deploy block

2018-06-19 Thread GitBox
eweiter opened a new pull request #58: add to travis.yml the runtime repo in the deploy block URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/58 This PR is to lock down the Deploy step of the Travis.yml so that it only executes when the repo matches the "upstream"

[GitHub] eweiter commented on issue #46: add to travis.yml the runtime repo in the deploy block

2018-06-19 Thread GitBox
eweiter commented on issue #46: add to travis.yml the runtime repo in the deploy block URL: https://github.com/apache/incubator-openwhisk-runtime-docker/pull/46#issuecomment-398434507 @csantanapr Please review This is an

[GitHub] eweiter opened a new pull request #46: add to travis.yml the runtime repo in the deploy block

2018-06-19 Thread GitBox
eweiter opened a new pull request #46: add to travis.yml the runtime repo in the deploy block URL: https://github.com/apache/incubator-openwhisk-runtime-docker/pull/46 This PR is to lock down the Deploy step of the Travis.yml so that it only executes when the repo matches the "upstream"

[GitHub] eweiter commented on issue #62: add to travis.yml the runtime repo to the deploy block

2018-06-19 Thread GitBox
eweiter commented on issue #62: add to travis.yml the runtime repo to the deploy block URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/62#issuecomment-398433126 @csantanapr Please review This is an

[GitHub] eweiter opened a new pull request #62: add to travis.yml the runtime repo to the deploy block

2018-06-19 Thread GitBox
eweiter opened a new pull request #62: add to travis.yml the runtime repo to the deploy block URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/62 This PR is to lock down the Deploy step of the Travis.yml so that it only executes when the repo matches the "upstream"

[GitHub] codecov-io edited a comment on issue #2282: Distributed tracing support #2192

2018-06-19 Thread GitBox
codecov-io edited a comment on issue #2282: Distributed tracing support #2192 URL: https://github.com/apache/incubator-openwhisk/pull/2282#issuecomment-391308983 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/2282?src=pr=h1) Report > Merging

[GitHub] starpit commented on issue #3765: Vagrant installation waits for user accepts java 8 license

2018-06-19 Thread GitBox
starpit commented on issue #3765: Vagrant installation waits for user accepts java 8 license URL: https://github.com/apache/incubator-openwhisk/issues/3765#issuecomment-398423297 done, via https://github.com/apache/incubator-openwhisk/pull/3783 however, as i noted in the PR

[GitHub] starpit opened a new pull request #3783: Fixes for vagrant installation versus oracle jdk

2018-06-19 Thread GitBox
starpit opened a new pull request #3783: Fixes for vagrant installation versus oracle jdk URL: https://github.com/apache/incubator-openwhisk/pull/3783 Vagrant installation is broken, due to a hang waiting for acceptance of the Oracle JDK license. This PR adds an acceptance script, and

[GitHub] baxtersa opened a new pull request #313: Fix awk command in init.sh generating resolvers.conf file

2018-06-19 Thread GitBox
baxtersa opened a new pull request #313: Fix awk command in init.sh generating resolvers.conf file URL: https://github.com/apache/incubator-openwhisk-apigateway/pull/313 Fixes #312 This is an automated message from the

[GitHub] houshengbo opened a new pull request #45: WIP: Test travis

2018-06-19 Thread GitBox
houshengbo opened a new pull request #45: WIP: Test travis URL: https://github.com/apache/incubator-openwhisk-runtime-docker/pull/45 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] markusthoemmes commented on issue #3782: Make test-instances of Exec depend on the loaded manifest.

2018-06-19 Thread GitBox
markusthoemmes commented on issue #3782: Make test-instances of Exec depend on the loaded manifest. URL: https://github.com/apache/incubator-openwhisk/pull/3782#issuecomment-398404370 @rabbah if no manifest is defined, the whole tests will fail anyway. Take a look at `Exec.serdes`, it

[GitHub] houshengbo opened a new pull request #61: WIP: Test Travis

2018-06-19 Thread GitBox
houshengbo opened a new pull request #61: WIP: Test Travis URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/61 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] rabbah commented on issue #3781: vagrant plugin disk-size not installed automatically

2018-06-19 Thread GitBox
rabbah commented on issue #3781: vagrant plugin disk-size not installed automatically URL: https://github.com/apache/incubator-openwhisk/issues/3781#issuecomment-398394785 Wasn’t obvious/vagrant up was more instinctive. Anyway it still didn’t work

[GitHub] codecov-io commented on issue #3782: Make test-instances of Exec depend on the loaded manifest.

2018-06-19 Thread GitBox
codecov-io commented on issue #3782: Make test-instances of Exec depend on the loaded manifest. URL: https://github.com/apache/incubator-openwhisk/pull/3782#issuecomment-398394320 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3782?src=pr=h1) Report > Merging

[GitHub] eweiter opened a new pull request #60: update nodejs 6 & 8 to latest security patch

2018-06-19 Thread GitBox
eweiter opened a new pull request #60: update nodejs 6 & 8 to latest security patch URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/60 The NodeJS runtime had a security patch released. This bumped the versions for NodeJS 6 & 8 from NodeJS6 went from 6.14.2

[GitHub] eweiter closed pull request #60: update nodejs 6 & 8 to latest security patch

2018-06-19 Thread GitBox
eweiter closed pull request #60: update nodejs 6 & 8 to latest security patch URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/60 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] csantanapr commented on issue #3781: vagrant plugin disk-size not installed automatically

2018-06-19 Thread GitBox
csantanapr commented on issue #3781: vagrant plugin disk-size not installed automatically URL: https://github.com/apache/incubator-openwhisk/issues/3781#issuecomment-398389998 Use `./hello` don't use `vargrant up`, it's in the doc to use `./hello`

[GitHub] markusthoemmes opened a new pull request #3782: Make test-instances of Exec depend on the loaded manifest.

2018-06-19 Thread GitBox
markusthoemmes opened a new pull request #3782: Make test-instances of Exec depend on the loaded manifest. URL: https://github.com/apache/incubator-openwhisk/pull/3782 Exec.serdes very implicitly relies on a manifest to be loaded, which makes the test-instances of Exec take values from

[GitHub] dgrove-oss commented on issue #3778: Add documentation to the loadbalancer.

2018-06-19 Thread GitBox
dgrove-oss commented on issue #3778: Add documentation to the loadbalancer. URL: https://github.com/apache/incubator-openwhisk/pull/3778#issuecomment-398379236 Absolutely only want one copy of thisotherwise they will get out of synch. But maybe it is more discoverable as an

[GitHub] rabbah commented on a change in pull request #3722: Scala based admin tooling

2018-06-19 Thread GitBox
rabbah commented on a change in pull request #3722: Scala based admin tooling URL: https://github.com/apache/incubator-openwhisk/pull/3722#discussion_r196399585 ## File path: core/admin/src/main/scala/whisk/core/cli/Main.scala ## @@ -0,0 +1,161 @@ +/* + * Licensed to the

[GitHub] rabbah commented on a change in pull request #3722: Scala based admin tooling

2018-06-19 Thread GitBox
rabbah commented on a change in pull request #3722: Scala based admin tooling URL: https://github.com/apache/incubator-openwhisk/pull/3722#discussion_r196399585 ## File path: core/admin/src/main/scala/whisk/core/cli/Main.scala ## @@ -0,0 +1,161 @@ +/* + * Licensed to the

[GitHub] rabbah commented on issue #3781: vagrant plugin disk-size not installed automatically

2018-06-19 Thread GitBox
rabbah commented on issue #3781: vagrant plugin disk-size not installed automatically URL: https://github.com/apache/incubator-openwhisk/issues/3781#issuecomment-398378170 It looks like using `hello` does the install for you - suggest we document it explicitly (if not already); leaving

[GitHub] rabbah opened a new issue #3781: vagrant plugin disk-size not installed automatically

2018-06-19 Thread GitBox
rabbah opened a new issue #3781: vagrant plugin disk-size not installed automatically URL: https://github.com/apache/incubator-openwhisk/issues/3781 This is on a fresh install of VB and Vagrant with vagrant up running for the first time: ``` > vagrant up Bringing machine

[GitHub] rabbah commented on a change in pull request #3722: Scala based admin tooling

2018-06-19 Thread GitBox
rabbah commented on a change in pull request #3722: Scala based admin tooling URL: https://github.com/apache/incubator-openwhisk/pull/3722#discussion_r196397408 ## File path: tools/admin/src/main/scala/whisk/core/cli/Main.scala ## @@ -0,0 +1,163 @@ +/* + * Licensed to the

[GitHub] rabbah commented on a change in pull request #3722: Scala based admin tooling

2018-06-19 Thread GitBox
rabbah commented on a change in pull request #3722: Scala based admin tooling URL: https://github.com/apache/incubator-openwhisk/pull/3722#discussion_r196396579 ## File path: tools/admin/README-NEXT.md ## @@ -0,0 +1,126 @@ + + +## Administrative Operations + +The

[GitHub] rabbah commented on a change in pull request #3722: Scala based admin tooling

2018-06-19 Thread GitBox
rabbah commented on a change in pull request #3722: Scala based admin tooling URL: https://github.com/apache/incubator-openwhisk/pull/3722#discussion_r196396839 ## File path: tools/admin/README-NEXT.md ## @@ -0,0 +1,126 @@ + + +## Administrative Operations + +The

[GitHub] rabbah commented on a change in pull request #3722: Scala based admin tooling

2018-06-19 Thread GitBox
rabbah commented on a change in pull request #3722: Scala based admin tooling URL: https://github.com/apache/incubator-openwhisk/pull/3722#discussion_r196396301 ## File path: tools/admin/README-NEXT.md ## @@ -0,0 +1,126 @@ + + +## Administrative Operations + +The

[GitHub] rabbah commented on a change in pull request #3722: Scala based admin tooling

2018-06-19 Thread GitBox
rabbah commented on a change in pull request #3722: Scala based admin tooling URL: https://github.com/apache/incubator-openwhisk/pull/3722#discussion_r196396447 ## File path: tools/admin/README-NEXT.md ## @@ -0,0 +1,126 @@ + + +## Administrative Operations + +The

[GitHub] rabbah commented on a change in pull request #3722: Scala based admin tooling

2018-06-19 Thread GitBox
rabbah commented on a change in pull request #3722: Scala based admin tooling URL: https://github.com/apache/incubator-openwhisk/pull/3722#discussion_r196394859 ## File path: tools/admin/src/main/scala/whisk/core/database/UserCommand.scala ## @@ -0,0 +1,352 @@ +/* + *

[GitHub] rabbah commented on a change in pull request #3722: Scala based admin tooling

2018-06-19 Thread GitBox
rabbah commented on a change in pull request #3722: Scala based admin tooling URL: https://github.com/apache/incubator-openwhisk/pull/3722#discussion_r196394040 ## File path: tools/admin/src/main/scala/whisk/core/cli/Main.scala ## @@ -0,0 +1,163 @@ +/* + * Licensed to the

[GitHub] rabbah commented on a change in pull request #3722: Scala based admin tooling

2018-06-19 Thread GitBox
rabbah commented on a change in pull request #3722: Scala based admin tooling URL: https://github.com/apache/incubator-openwhisk/pull/3722#discussion_r196394744 ## File path: tools/admin/src/main/scala/whisk/core/database/UserCommand.scala ## @@ -0,0 +1,352 @@ +/* + *

[GitHub] markusthoemmes commented on issue #3778: Add documentation to the loadbalancer.

2018-06-19 Thread GitBox
markusthoemmes commented on issue #3778: Add documentation to the loadbalancer. URL: https://github.com/apache/incubator-openwhisk/pull/3778#issuecomment-398370719 Rethinking that, it's a pain to keep two documents in sync, and I'm in favor of keeping the docs as close to the code as

[GitHub] codecov-io edited a comment on issue #3767: introduce timeout factor to control invoker queue behavior

2018-06-19 Thread GitBox
codecov-io edited a comment on issue #3767: introduce timeout factor to control invoker queue behavior URL: https://github.com/apache/incubator-openwhisk/pull/3767#issuecomment-397564811 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3767?src=pr=h1) Report >

[GitHub] markusthoemmes commented on issue #3778: Add documentation to the loadbalancer.

2018-06-19 Thread GitBox
markusthoemmes commented on issue #3778: Add documentation to the loadbalancer. URL: https://github.com/apache/incubator-openwhisk/pull/3778#issuecomment-398369728 @mrutkows sure, it might make sense to add the docs in `.md` format to our dev docs.

[GitHub] markusthoemmes commented on a change in pull request #3747: Memory based loadbalancing

2018-06-19 Thread GitBox
markusthoemmes commented on a change in pull request #3747: Memory based loadbalancing URL: https://github.com/apache/incubator-openwhisk/pull/3747#discussion_r196387058 ## File path: tests/src/test/scala/whisk/core/containerpool/test/ContainerPoolTests.scala ## @@

[GitHub] markusthoemmes commented on a change in pull request #3747: Memory based loadbalancing

2018-06-19 Thread GitBox
markusthoemmes commented on a change in pull request #3747: Memory based loadbalancing URL: https://github.com/apache/incubator-openwhisk/pull/3747#discussion_r196388846 ## File path: tests/src/test/scala/whisk/core/containerpool/test/ContainerPoolTests.scala ## @@

[GitHub] markusthoemmes commented on a change in pull request #3747: Memory based loadbalancing

2018-06-19 Thread GitBox
markusthoemmes commented on a change in pull request #3747: Memory based loadbalancing URL: https://github.com/apache/incubator-openwhisk/pull/3747#discussion_r196387705 ## File path: tests/src/test/scala/whisk/core/containerpool/test/ContainerPoolTests.scala ## @@

[GitHub] markusthoemmes commented on a change in pull request #3747: Memory based loadbalancing

2018-06-19 Thread GitBox
markusthoemmes commented on a change in pull request #3747: Memory based loadbalancing URL: https://github.com/apache/incubator-openwhisk/pull/3747#discussion_r196386635 ## File path: tests/src/test/scala/whisk/core/containerpool/test/ContainerPoolTests.scala ## @@

[GitHub] codecov-io edited a comment on issue #3767: introduce timeout factor to control invoker queue behavior

2018-06-19 Thread GitBox
codecov-io edited a comment on issue #3767: introduce timeout factor to control invoker queue behavior URL: https://github.com/apache/incubator-openwhisk/pull/3767#issuecomment-397564811 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3767?src=pr=h1) Report >

[GitHub] rabbah commented on a change in pull request #3617: Move version information to pureconfig and remove `whisk_version_name`.

2018-06-19 Thread GitBox
rabbah commented on a change in pull request #3617: Move version information to pureconfig and remove `whisk_version_name`. URL: https://github.com/apache/incubator-openwhisk/pull/3617#discussion_r196382936 ## File path: ansible/environments/vagrant/group_vars/all ## @@

[GitHub] codecov-io edited a comment on issue #3777: Avoid converting ByteString to bytes in attachment inlining flow

2018-06-19 Thread GitBox
codecov-io edited a comment on issue #3777: Avoid converting ByteString to bytes in attachment inlining flow URL: https://github.com/apache/incubator-openwhisk/pull/3777#issuecomment-398009157 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3777?src=pr=h1) Report >

  1   2   >