[GitHub] kpavel closed pull request #4108: Making Redis password protected

2018-11-12 Thread GitBox
kpavel closed pull request #4108: Making Redis password protected URL: https://github.com/apache/incubator-openwhisk/pull/4108 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] kpavel opened a new pull request #4108: Making Redis password protected

2018-11-12 Thread GitBox
kpavel opened a new pull request #4108: Making Redis password protected URL: https://github.com/apache/incubator-openwhisk/pull/4108 ## Description Currently default openwhisk (apigateway) setup doesn't secure Redis with password. This PR adds password protection to Redis ansible

[GitHub] dgrove-oss closed pull request #350: Helm/Tiller 2.9 is acceptable

2018-11-12 Thread GitBox
dgrove-oss closed pull request #350: Helm/Tiller 2.9 is acceptable URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/350 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] dgrove-oss commented on issue #350: Helm/Tiller 2.9 is acceptable

2018-11-12 Thread GitBox
dgrove-oss commented on issue #350: Helm/Tiller 2.9 is acceptable URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/350#issuecomment-438088637 conflicts with #352 -- will change tiller version as part of that PR to simplify merging.,

[GitHub] chetanmeh commented on issue #3246: ActionLimitsTests memory test failing

2018-11-12 Thread GitBox
chetanmeh commented on issue #3246: ActionLimitsTests memory test failing URL: https://github.com/apache/incubator-openwhisk/issues/3246#issuecomment-437800584 Error logs from activation record ```json "entityType": "activation", "logs": [

[GitHub] chetanmeh commented on issue #4106: Update to Spray-json 1.3.5 security fix release

2018-11-12 Thread GitBox
chetanmeh commented on issue #4106: Update to Spray-json 1.3.5 security fix release URL: https://github.com/apache/incubator-openwhisk/pull/4106#issuecomment-437813710 PR is consistently ailing in system test (see #3246) which seems unrelated to this change (builds on master also failed

[GitHub] rabbah commented on a change in pull request #4106: Update to Spray-json 1.3.5 security fix release

2018-11-12 Thread GitBox
rabbah commented on a change in pull request #4106: Update to Spray-json 1.3.5 security fix release URL: https://github.com/apache/incubator-openwhisk/pull/4106#discussion_r232610524 ## File path: tests/src/test/scala/org/apache/openwhisk/core/entity/test/SchemaTests.scala

[GitHub] rabbah commented on a change in pull request #4106: Update to Spray-json 1.3.5 security fix release

2018-11-12 Thread GitBox
rabbah commented on a change in pull request #4106: Update to Spray-json 1.3.5 security fix release URL: https://github.com/apache/incubator-openwhisk/pull/4106#discussion_r232610524 ## File path: tests/src/test/scala/org/apache/openwhisk/core/entity/test/SchemaTests.scala

[GitHub] rabbah closed pull request #4106: Update to Spray-json 1.3.5 security fix release

2018-11-12 Thread GitBox
rabbah closed pull request #4106: Update to Spray-json 1.3.5 security fix release URL: https://github.com/apache/incubator-openwhisk/pull/4106 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] chetanmeh commented on issue #4106: Update to Spray-json 1.3.5 security fix release

2018-11-12 Thread GitBox
chetanmeh commented on issue #4106: Update to Spray-json 1.3.5 security fix release URL: https://github.com/apache/incubator-openwhisk/pull/4106#issuecomment-437825842 Finally its green after retrying build 3 times This is

[GitHub] codecov-io commented on issue #4106: Update to Spray-json 1.3.5 security fix release

2018-11-12 Thread GitBox
codecov-io commented on issue #4106: Update to Spray-json 1.3.5 security fix release URL: https://github.com/apache/incubator-openwhisk/pull/4106#issuecomment-437825639 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/4106?src=pr=h1) Report > Merging

[GitHub] chetanmeh commented on issue #4110: Exclude unused transitive dependencies

2018-11-12 Thread GitBox
chetanmeh commented on issue #4110: Exclude unused transitive dependencies URL: https://github.com/apache/incubator-openwhisk/pull/4110#issuecomment-438168027 Below are current dependencies from master branch ```

[GitHub] chetanmeh opened a new pull request #4110: Exclude unused transitive dependencies

2018-11-12 Thread GitBox
chetanmeh opened a new pull request #4110: Exclude unused transitive dependencies URL: https://github.com/apache/incubator-openwhisk/pull/4110 Excludes some of the transitive dependencies which are not used at runtime saving 14MB in disk space. ## Description Some of the

[GitHub] codecov-io edited a comment on issue #4104: Cache empty auth results to reduce db load

2018-11-12 Thread GitBox
codecov-io edited a comment on issue #4104: Cache empty auth results to reduce db load URL: https://github.com/apache/incubator-openwhisk/pull/4104#issuecomment-43700 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/4104?src=pr=h1) Report > Merging

[GitHub] chen116 opened a new pull request #4107: Enable Deploying Distributed OpenWhisk on Ubuntu 16.04 Cluster

2018-11-12 Thread GitBox
chen116 opened a new pull request #4107: Enable Deploying Distributed OpenWhisk on Ubuntu 16.04 Cluster URL: https://github.com/apache/incubator-openwhisk/pull/4107 enable native ubuntu 16.04 cluster deployment ## Description ubuntu 14.04 cluster is used in

[GitHub] dgrove-oss opened a new pull request #350: Helm/Tiller 2.9 is acceptable

2018-11-12 Thread GitBox
dgrove-oss opened a new pull request #350: Helm/Tiller 2.9 is acceptable URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/350 This is an automated message from the Apache Git Service. To respond to the

[GitHub] csantanapr commented on issue #351: Move smoketest into chart so it can be used for `helm test`

2018-11-12 Thread GitBox
csantanapr commented on issue #351: Move smoketest into chart so it can be used for `helm test` URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/351#issuecomment-437995905 I was expecting a Job not a Pod for the tests. I'm guessing that is any pod defined in any yaml

[GitHub] csantanapr closed pull request #351: Move smoketest into chart so it can be used for `helm test`

2018-11-12 Thread GitBox
csantanapr closed pull request #351: Move smoketest into chart so it can be used for `helm test` URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/351 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] dubee commented on a change in pull request #306: Update Tests to Wait for Producer to Finish

2018-11-12 Thread GitBox
dubee commented on a change in pull request #306: Update Tests to Wait for Producer to Finish URL: https://github.com/apache/incubator-openwhisk-package-kafka/pull/306#discussion_r232785519 ## File path: tests/src/test/scala/system/utils/KafkaUtils.scala ## @@ -176,7

[GitHub] codecov-io edited a comment on issue #4104: Cache empty auth results to reduce db load

2018-11-12 Thread GitBox
codecov-io edited a comment on issue #4104: Cache empty auth results to reduce db load URL: https://github.com/apache/incubator-openwhisk/pull/4104#issuecomment-43700 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/4104?src=pr=h1) Report > Merging

[GitHub] codecov-io edited a comment on issue #4104: Cache empty auth results to reduce db load

2018-11-12 Thread GitBox
codecov-io edited a comment on issue #4104: Cache empty auth results to reduce db load URL: https://github.com/apache/incubator-openwhisk/pull/4104#issuecomment-43700 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/4104?src=pr=h1) Report > Merging

[GitHub] csantanapr commented on issue #351: Move smoketest into chart so it can be used for `helm test`

2018-11-12 Thread GitBox
csantanapr commented on issue #351: Move smoketest into chart so it can be used for `helm test` URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/351#issuecomment-437996555 nvm I found that is based on ``` annotations: "helm.sh/hook": test-success

[GitHub] dgrove-oss opened a new pull request #351: Move smoketest into chart so it can be used for `helm test`

2018-11-12 Thread GitBox
dgrove-oss opened a new pull request #351: Move smoketest into chart so it can be used for `helm test` URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/351 Pull the logic for the simple hello action smoketest into the Helm chart for OpenWhisk so it can be used to

[GitHub] csantanapr opened a new pull request #45: publish docker image latest with git hash

2018-11-12 Thread GitBox
csantanapr opened a new pull request #45: publish docker image latest with git hash URL: https://github.com/apache/incubator-openwhisk-runtime-php/pull/45 This is an automated message from the Apache Git Service. To respond

[GitHub] tysonnorris opened a new pull request #4109: update mesos-actor; cleanup orphaned failed task launches

2018-11-12 Thread GitBox
tysonnorris opened a new pull request #4109: update mesos-actor; cleanup orphaned failed task launches URL: https://github.com/apache/incubator-openwhisk/pull/4109 ## Description When task launch times out, cleanup tasks that were sent to mesos (or not yet sent).

[GitHub] dgrove-oss opened a new pull request #352: Consolidate core and providers into a single chart

2018-11-12 Thread GitBox
dgrove-oss opened a new pull request #352: Consolidate core and providers into a single chart URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/352 Structurally, it makes more sense to deploy the core system and the providers as part of a single consolidated chart.

[GitHub] csantanapr commented on issue #352: Consolidate core and providers into a single chart

2018-11-12 Thread GitBox
csantanapr commented on issue #352: Consolidate core and providers into a single chart URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/352#issuecomment-438044844 hmmm @dgrove-oss How easy is to use a boolean in to enable or disable a specific provider now? Can

[GitHub] csantanapr edited a comment on issue #352: Consolidate core and providers into a single chart

2018-11-12 Thread GitBox
csantanapr edited a comment on issue #352: Consolidate core and providers into a single chart URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/352#issuecomment-438044844 hmmm @dgrove-oss How easy is to use a boolean in values include or exclude a specific provider

[GitHub] csantanapr edited a comment on issue #352: Consolidate core and providers into a single chart

2018-11-12 Thread GitBox
csantanapr edited a comment on issue #352: Consolidate core and providers into a single chart URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/352#issuecomment-438044844 hmmm @dgrove-oss How easy is to use a boolean in values enable or disable a specific provider

[GitHub] dgrove-oss commented on issue #352: Consolidate core and providers into a single chart

2018-11-12 Thread GitBox
dgrove-oss commented on issue #352: Consolidate core and providers into a single chart URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/352#issuecomment-438045857 @csantanapr -- yes, we can enable/disable providers by flipping a single boolean per provider. By

[GitHub] csantanapr commented on issue #352: Consolidate core and providers into a single chart

2018-11-12 Thread GitBox
csantanapr commented on issue #352: Consolidate core and providers into a single chart URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/352#issuecomment-438046225 Thank you @dgrove-oss This is an

[GitHub] csantanapr closed pull request #45: Update patch level for php & publish docker image latest with git hash

2018-11-12 Thread GitBox
csantanapr closed pull request #45: Update patch level for php & publish docker image latest with git hash URL: https://github.com/apache/incubator-openwhisk-runtime-php/pull/45 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed

[GitHub] csantanapr opened a new pull request #44: print sentinels in new line for php 7.2

2018-11-12 Thread GitBox
csantanapr opened a new pull request #44: print sentinels in new line for php 7.2 URL: https://github.com/apache/incubator-openwhisk-runtime-php/pull/44 Fixes #43 This is an automated message from the Apache Git Service. To

[GitHub] kpavel opened a new pull request #4108: Making Redis password protected

2018-11-12 Thread GitBox
kpavel opened a new pull request #4108: Making Redis password protected URL: https://github.com/apache/incubator-openwhisk/pull/4108 ## Description Currently default openwhisk (apigateway) setup doesn't secure Redis with password. This PR adds password protection to Redis ansible

[GitHub] mhamann commented on issue #4108: Making Redis password protected

2018-11-12 Thread GitBox
mhamann commented on issue #4108: Making Redis password protected URL: https://github.com/apache/incubator-openwhisk/pull/4108#issuecomment-438020008 This would affect other form factors as well: kubernetes, docker compose, etc. I think those should be updated too as part of this PR.

[GitHub] dubee commented on a change in pull request #306: Update Tests to Wait for Producer to Finish

2018-11-12 Thread GitBox
dubee commented on a change in pull request #306: Update Tests to Wait for Producer to Finish URL: https://github.com/apache/incubator-openwhisk-package-kafka/pull/306#discussion_r232803010 ## File path: tests/src/test/scala/system/packages/MessageHubFeedTests.scala ##

[GitHub] jasonpet closed pull request #306: Update Tests to Wait for Producer to Finish

2018-11-12 Thread GitBox
jasonpet closed pull request #306: Update Tests to Wait for Producer to Finish URL: https://github.com/apache/incubator-openwhisk-package-kafka/pull/306 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] jasonpet closed pull request #44: print sentinels in new line for php 7.2

2018-11-12 Thread GitBox
jasonpet closed pull request #44: print sentinels in new line for php 7.2 URL: https://github.com/apache/incubator-openwhisk-runtime-php/pull/44 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] jasonpet closed issue #43: PHP 7.2 stdout and stderr sentinels don't print in a new line

2018-11-12 Thread GitBox
jasonpet closed issue #43: PHP 7.2 stdout and stderr sentinels don't print in a new line URL: https://github.com/apache/incubator-openwhisk-runtime-php/issues/43 This is an automated message from the Apache Git Service. To