[GitHub] daisy-ycguo closed pull request #368: ingress cleanup: allow default value for httpsNodePort

2018-11-20 Thread GitBox
daisy-ycguo closed pull request #368: ingress cleanup: allow default value for httpsNodePort URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/368 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] jiangpengcheng commented on a change in pull request #4115: Don't react on Completion message of blocking requests.

2018-11-20 Thread GitBox
jiangpengcheng commented on a change in pull request #4115: Don't react on Completion message of blocking requests. URL: https://github.com/apache/incubator-openwhisk/pull/4115#discussion_r235243689 ## File path:

[GitHub] jiangpengcheng commented on a change in pull request #4115: Don't react on Completion message of blocking requests.

2018-11-20 Thread GitBox
jiangpengcheng commented on a change in pull request #4115: Don't react on Completion message of blocking requests. URL: https://github.com/apache/incubator-openwhisk/pull/4115#discussion_r235243689 ## File path:

[GitHub] SamHjelmfelt commented on issue #4129: Adding YARNContainerFactory. This allows OpenWhisk to run actions on Apache Hadoop clusters.

2018-11-20 Thread GitBox
SamHjelmfelt commented on issue #4129: Adding YARNContainerFactory. This allows OpenWhisk to run actions on Apache Hadoop clusters. URL: https://github.com/apache/incubator-openwhisk/pull/4129#issuecomment-440477208 For anyone wanting to test this out, here is a quick guide for enabling

[GitHub] rabbah commented on issue #4129: Adding YARNContainerFactory. This allows OpenWhisk to run actions on Apache Hadoop clusters.

2018-11-20 Thread GitBox
rabbah commented on issue #4129: Adding YARNContainerFactory. This allows OpenWhisk to run actions on Apache Hadoop clusters. URL: https://github.com/apache/incubator-openwhisk/pull/4129#issuecomment-440475063 勞 this is excellent!

[GitHub] SamHjelmfelt opened a new pull request #4129: Adding YARNContainerFactory. This allows OpenWhisk to run actions on Apache Hadoop clusters.

2018-11-20 Thread GitBox
SamHjelmfelt opened a new pull request #4129: Adding YARNContainerFactory. This allows OpenWhisk to run actions on Apache Hadoop clusters. URL: https://github.com/apache/incubator-openwhisk/pull/4129 Thousands of organizations have Apache Hadoop clusters today. By implementing a

[GitHub] dgrove-oss commented on a change in pull request #367: Helm chart changes

2018-11-20 Thread GitBox
dgrove-oss commented on a change in pull request #367: Helm chart changes URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/367#discussion_r235192131 ## File path: helm/openwhisk/values.yaml ## @@ -161,6 +163,7 @@ nginx: httpPort: 80 httpsPort: 443

[GitHub] dgrove-oss opened a new pull request #368: ingress cleanup: allow default value for httpsNodePort

2018-11-20 Thread GitBox
dgrove-oss opened a new pull request #368: ingress cleanup: allow default value for httpsNodePort URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/368 Make usage of nginx.httpsNodePort templates conditional on ingress.type being NodePort to enable a default value

[GitHub] neeraj-laad commented on a change in pull request #367: Helm chart changes

2018-11-20 Thread GitBox
neeraj-laad commented on a change in pull request #367: Helm chart changes URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/367#discussion_r235152398 ## File path: helm/openwhisk/values.yaml ## @@ -161,6 +163,7 @@ nginx: httpPort: 80 httpsPort:

[GitHub] neeraj-laad commented on a change in pull request #367: Helm chart changes

2018-11-20 Thread GitBox
neeraj-laad commented on a change in pull request #367: Helm chart changes URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/367#discussion_r235152134 ## File path: helm/openwhisk/README.md ## @@ -0,0 +1,178 @@ +# OpenWhisk + +Apache OpenWhisk is an open

[GitHub] neeraj-laad commented on a change in pull request #367: Helm chart changes

2018-11-20 Thread GitBox
neeraj-laad commented on a change in pull request #367: Helm chart changes URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/367#discussion_r235152043 ## File path: helm/openwhisk/README.md ## @@ -0,0 +1,178 @@ +# OpenWhisk + +Apache OpenWhisk is an open

[GitHub] dgrove-oss commented on a change in pull request #367: Helm chart changes

2018-11-20 Thread GitBox
dgrove-oss commented on a change in pull request #367: Helm chart changes URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/367#discussion_r235147321 ## File path: helm/openwhisk/README.md ## @@ -0,0 +1,178 @@ +# OpenWhisk + +Apache OpenWhisk is an open

[GitHub] dgrove-oss commented on a change in pull request #367: Helm chart changes

2018-11-20 Thread GitBox
dgrove-oss commented on a change in pull request #367: Helm chart changes URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/367#discussion_r235148952 ## File path: helm/openwhisk/values.yaml ## @@ -161,6 +163,7 @@ nginx: httpPort: 80 httpsPort: 443

[GitHub] dgrove-oss commented on a change in pull request #367: Helm chart changes

2018-11-20 Thread GitBox
dgrove-oss commented on a change in pull request #367: Helm chart changes URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/367#discussion_r235146838 ## File path: helm/openwhisk/README.md ## @@ -0,0 +1,178 @@ +# OpenWhisk + +Apache OpenWhisk is an open

[GitHub] kpavel opened a new pull request #3886: Proposing Lean OpenWhisk

2018-11-20 Thread GitBox
kpavel opened a new pull request #3886: Proposing Lean OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3886 The proposition is to get rid of Kafka, have controller and invoker compiled together into a "lean" controller-invoker (have a Gradle project for that) with

[GitHub] kpavel closed pull request #3886: Proposing Lean OpenWhisk

2018-11-20 Thread GitBox
kpavel closed pull request #3886: Proposing Lean OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3886 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 pull

[GitHub] mdeuser commented on a change in pull request #383: enable use of concurrency action limits via cli

2018-11-20 Thread GitBox
mdeuser commented on a change in pull request #383: enable use of concurrency action limits via cli URL: https://github.com/apache/incubator-openwhisk-cli/pull/383#discussion_r235136981 ## File path:

[GitHub] rabbah commented on a change in pull request #4115: Don't react on Completion message of blocking requests.

2018-11-20 Thread GitBox
rabbah commented on a change in pull request #4115: Don't react on Completion message of blocking requests. URL: https://github.com/apache/incubator-openwhisk/pull/4115#discussion_r235121779 ## File path:

[GitHub] dgrove-oss opened a new pull request #239: composer 0.9.0 release

2018-11-20 Thread GitBox
dgrove-oss opened a new pull request #239: composer 0.9.0 release URL: https://github.com/apache/incubator-openwhisk-release/pull/239 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] dgrove-oss closed pull request #9: Fix openwhisk-composer module resolution in compose command

2018-11-20 Thread GitBox
dgrove-oss closed pull request #9: Fix openwhisk-composer module resolution in compose command URL: https://github.com/apache/incubator-openwhisk-composer/pull/9 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] Juice10 commented on a change in pull request #60: Some basic changes to README to better conform to runtime template.

2018-11-20 Thread GitBox
Juice10 commented on a change in pull request #60: Some basic changes to README to better conform to runtime template. URL: https://github.com/apache/incubator-openwhisk-runtime-go/pull/60#discussion_r235115317 ## File path: README.md ## @@ -16,21 +16,32 @@ #

[GitHub] tardieu opened a new pull request #9: Fix openwhisk-composer module resolution in compose command

2018-11-20 Thread GitBox
tardieu opened a new pull request #9: Fix openwhisk-composer module resolution in compose command URL: https://github.com/apache/incubator-openwhisk-composer/pull/9 The compose command now resolves the `openwhisk-composer` module to the parent folder if not found in the search path.

[GitHub] neeraj-laad opened a new pull request #367: Helm chart changes

2018-11-20 Thread GitBox
neeraj-laad opened a new pull request #367: Helm chart changes URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/367 Added readme for helm catalog Updated chart decrtiption for helm catalog updated notes.txt to include instrcutsion on testing the release included

[GitHub] dgrove-oss commented on issue #104: "wsk api list" is not working

2018-11-20 Thread GitBox
dgrove-oss commented on issue #104: "wsk api list" is not working URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/104#issuecomment-440340820 It looks like the job (or jobs depending on how recently you cloned the repo) that install the openwhisk catalog and the

[GitHub] xgenvn opened a new issue #177: CouchDB shows error of database not exist after make restart

2018-11-20 Thread GitBox
xgenvn opened a new issue #177: CouchDB shows error of database not exist after make restart URL: https://github.com/apache/incubator-openwhisk-devtools/issues/177 Hi, Is it by design we have to re-init couchdb right after every restart? I'm using PhotonOS VM for testing.

[GitHub] dgrove-oss commented on issue #238: add incubator-openwhisk-composer to big table of repos

2018-11-20 Thread GitBox
dgrove-oss commented on issue #238: add incubator-openwhisk-composer to big table of repos URL: https://github.com/apache/incubator-openwhisk-release/pull/238#issuecomment-440328813 @mrutkows @houshengbo -- travis CI still appears to be broken for the openwhisk-release repository. Could

[GitHub] csantanapr closed pull request #184: Add support for IAM based Cloudant DB instances

2018-11-20 Thread GitBox
csantanapr closed pull request #184: Add support for IAM based Cloudant DB instances URL: https://github.com/apache/incubator-openwhisk-package-cloudant/pull/184 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] KeonHee commented on issue #4128: Delete unnecessary code comments

2018-11-20 Thread GitBox
KeonHee commented on issue #4128: Delete unnecessary code comments URL: https://github.com/apache/incubator-openwhisk/pull/4128#issuecomment-440316330 I don't understand why this error occurs. How do I run the build test again?

[GitHub] jasonpet opened a new pull request #184: Add support for IAM based Cloudant DB instances

2018-11-20 Thread GitBox
jasonpet opened a new pull request #184: Add support for IAM based Cloudant DB instances URL: https://github.com/apache/incubator-openwhisk-package-cloudant/pull/184 With this PR users can now listen to changes on Cloudant accounts that are IAM only enabled (the recommended approach).

[GitHub] mhenke1 commented on issue #4119: Fix test failures in case min and max concurrency settings are identical

2018-11-20 Thread GitBox
mhenke1 commented on issue #4119: Fix test failures in case min and max concurrency settings are identical URL: https://github.com/apache/incubator-openwhisk/pull/4119#issuecomment-440301761 superseded by https://github.com/apache/incubator-openwhisk/pull/4125

[GitHub] codecov-io edited a comment on issue #4126: Fix netty leak in CosmosDBArtifactStore`

2018-11-20 Thread GitBox
codecov-io edited a comment on issue #4126: Fix netty leak in CosmosDBArtifactStore` URL: https://github.com/apache/incubator-openwhisk/pull/4126#issuecomment-440267651 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/4126?src=pr=h1) Report > Merging

[GitHub] KeonHee opened a new pull request #4128: Delete unnecessary code comments

2018-11-20 Thread GitBox
KeonHee opened a new pull request #4128: Delete unnecessary code comments URL: https://github.com/apache/incubator-openwhisk/pull/4128 ## Description It looks like a code comment that accidentally entered. so I erased it and put up pr. ## My changes affect the

[GitHub] mhenke1 closed pull request #4127: WIP- Test will be deleted

2018-11-20 Thread GitBox
mhenke1 closed pull request #4127: WIP- Test will be deleted URL: https://github.com/apache/incubator-openwhisk/pull/4127 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 pull

[GitHub] mhenke1 commented on issue #4125: System test succeeds when concurrency is not enabled

2018-11-20 Thread GitBox
mhenke1 commented on issue #4125: System test succeeds when concurrency is not enabled URL: https://github.com/apache/incubator-openwhisk/pull/4125#issuecomment-440273733 The non concurrent case got tested with https://travis-ci.org/apache/incubator-openwhisk/builds/457432688

[GitHub] codecov-io commented on issue #4126: Fix netty leak in CosmosDBArtifactStore`

2018-11-20 Thread GitBox
codecov-io commented on issue #4126: Fix netty leak in CosmosDBArtifactStore` URL: https://github.com/apache/incubator-openwhisk/pull/4126#issuecomment-440267651 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/4126?src=pr=h1) Report > Merging

[GitHub] chetanmeh commented on issue #4111: Update to Gradle 4.10.2

2018-11-20 Thread GitBox
chetanmeh commented on issue #4111: Update to Gradle 4.10.2 URL: https://github.com/apache/incubator-openwhisk/pull/4111#issuecomment-440263221 No change required in other repos This is an automated message from the Apache

[GitHub] codecov-io edited a comment on issue #4125: System test succeeds when concurrency is not enabled

2018-11-20 Thread GitBox
codecov-io edited a comment on issue #4125: System test succeeds when concurrency is not enabled URL: https://github.com/apache/incubator-openwhisk/pull/4125#issuecomment-440220094 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/4125?src=pr=h1) Report > Merging

[GitHub] mhenke1 opened a new pull request #4127: WIP- Test will be deleted

2018-11-20 Thread GitBox
mhenke1 opened a new pull request #4127: WIP- Test will be deleted URL: https://github.com/apache/incubator-openwhisk/pull/4127 ## Description ## Related issue and scope - [ ] I opened an issue to propose and discuss this change (#) ## My changes

[GitHub] codecov-io edited a comment on issue #4125: System succeeds when concurrency is not enabled

2018-11-20 Thread GitBox
codecov-io edited a comment on issue #4125: System succeeds when concurrency is not enabled URL: https://github.com/apache/incubator-openwhisk/pull/4125#issuecomment-440220094 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/4125?src=pr=h1) Report > Merging

[GitHub] csantanapr commented on issue #4111: Update to Gradle 4.10.2

2018-11-20 Thread GitBox
csantanapr commented on issue #4111: Update to Gradle 4.10.2 URL: https://github.com/apache/incubator-openwhisk/pull/4111#issuecomment-440246713 @chetanmeh Are all repos should be updated? Or not necessary for now This is

[GitHub] saurabh1284 commented on issue #104: "wsk api list" is not working

2018-11-20 Thread GitBox
saurabh1284 commented on issue #104: "wsk api list" is not working URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/104#issuecomment-440246246 I had recently cloned and deployed it, but still hitting the same issue--> # wsk api list -i error: Unable to obtain

[GitHub] chetanmeh commented on issue #4111: Update to Gradle 4.10.2

2018-11-20 Thread GitBox
chetanmeh commented on issue #4111: Update to Gradle 4.10.2 URL: https://github.com/apache/incubator-openwhisk/pull/4111#issuecomment-440245442 @csantanapr Can you do a PG run for this? This is an automated message from the

[GitHub] chetanmeh commented on issue #4126: Fix netty leak in CosmosDBArtifactStore`

2018-11-20 Thread GitBox
chetanmeh commented on issue #4126: Fix netty leak in CosmosDBArtifactStore` URL: https://github.com/apache/incubator-openwhisk/pull/4126#issuecomment-440244878 Note that test case fails only intermittently as Netty leak detection logic relies on callbacks post GC. Reducing the max memory

[GitHub] chetanmeh opened a new pull request #4126: Fix netty leak in CosmosDBArtifactStore`

2018-11-20 Thread GitBox
chetanmeh opened a new pull request #4126: Fix netty leak in CosmosDBArtifactStore` URL: https://github.com/apache/incubator-openwhisk/pull/4126 Fixes a netty leak due to how query is done in `ComosDBArtifactStore` ## Description In some of the setups we are observing Netty

[GitHub] codecov-io commented on issue #4115: Don't react on Completion message of blocking requests.

2018-11-20 Thread GitBox
codecov-io commented on issue #4115: Don't react on Completion message of blocking requests. URL: https://github.com/apache/incubator-openwhisk/pull/4115#issuecomment-440231764 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/4115?src=pr=h1) Report > Merging

[GitHub] codecov-io commented on issue #4125: WIP - Test without concurrency

2018-11-20 Thread GitBox
codecov-io commented on issue #4125: WIP - Test without concurrency URL: https://github.com/apache/incubator-openwhisk/pull/4125#issuecomment-440220094 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/4125?src=pr=h1) Report > Merging

[GitHub] mhenke1 closed pull request #4121: WIP - Configure local to default concurrency

2018-11-20 Thread GitBox
mhenke1 closed pull request #4121: WIP - Configure local to default concurrency URL: https://github.com/apache/incubator-openwhisk/pull/4121 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] mhenke1 opened a new pull request #4125: WIP - Test without concurrency

2018-11-20 Thread GitBox
mhenke1 opened a new pull request #4125: WIP - Test without concurrency URL: https://github.com/apache/incubator-openwhisk/pull/4125 ## Description ## Related issue and scope - [ ] I opened an issue to propose and discuss this change (#) ## My

[GitHub] markusthoemmes commented on a change in pull request #4115: Don't react on Completion message of blocking requests.

2018-11-20 Thread GitBox
markusthoemmes commented on a change in pull request #4115: Don't react on Completion message of blocking requests. URL: https://github.com/apache/incubator-openwhisk/pull/4115#discussion_r234927460 ## File path:

[GitHub] cbickel commented on a change in pull request #4115: Don't react on Completion message of blocking requests.

2018-11-20 Thread GitBox
cbickel commented on a change in pull request #4115: Don't react on Completion message of blocking requests. URL: https://github.com/apache/incubator-openwhisk/pull/4115#discussion_r234925458 ## File path:

[GitHub] atsag closed issue #4113: Documentation corrections

2018-11-20 Thread GitBox
atsag closed issue #4113: Documentation corrections URL: https://github.com/apache/incubator-openwhisk/issues/4113 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and