[GitHub] tysonnorris commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils

2018-07-19 Thread GitBox
tysonnorris commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils URL: https://github.com/apache/incubator-openwhisk/pull/3812#discussion_r203930332 ## File path: common/scala/src/main/scala/whisk/http/PoolingRestClient.scala ##

[GitHub] codecov-io edited a comment on issue #3812: ContainerClient + akka http alternative to HttpUtils

2018-07-19 Thread GitBox
codecov-io edited a comment on issue #3812: ContainerClient + akka http alternative to HttpUtils URL: https://github.com/apache/incubator-openwhisk/pull/3812#issuecomment-401212372 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3812?src=pr=h1) Report > Merging

[GitHub] tysonnorris commented on issue #3812: ContainerClient + akka http alternative to HttpUtils

2018-07-19 Thread GitBox
tysonnorris commented on issue #3812: ContainerClient + akka http alternative to HttpUtils URL: https://github.com/apache/incubator-openwhisk/pull/3812#issuecomment-406480813 @chetanmeh this is close - let me know if you think I missed any of your comments? I think I got them all.

[GitHub] cbickel commented on issue #3882: Use invoker id for metrics instead of the hostname.

2018-07-19 Thread GitBox
cbickel commented on issue #3882: Use invoker id for metrics instead of the hostname. URL: https://github.com/apache/incubator-openwhisk/pull/3882#issuecomment-406272652 PG5#532 is  This is an automated message from the

[GitHub] gazben edited a comment on issue #139: Meet minio file access denied when waiting for the controller

2018-07-19 Thread GitBox
gazben edited a comment on issue #139: Meet minio file access denied when waiting for the controller URL: https://github.com/apache/incubator-openwhisk-devtools/issues/139#issuecomment-406263490 @edwardstudy Is this resolved for you? I have the same error? @ddragosd My invoker logs

[GitHub] gazben edited a comment on issue #139: Meet minio file access denied when waiting for the controller

2018-07-19 Thread GitBox
gazben edited a comment on issue #139: Meet minio file access denied when waiting for the controller URL: https://github.com/apache/incubator-openwhisk-devtools/issues/139#issuecomment-406263490 @edwardstudy Is this resolved for you? I have the same error? @ddragosd My invoker logs

[GitHub] rabbah commented on issue #3886: Proposing Lean OpenWhisk

2018-07-19 Thread GitBox
rabbah commented on issue #3886: Proposing Lean OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3886#issuecomment-406246696 It’ll save you some time to use a git commit hook https://github.com/apache/incubator-openwhisk/wiki/Contributing:-Git-guidelines#scala

[GitHub] codecov-io commented on issue #3886: Proposing Lean OpenWhisk

2018-07-19 Thread GitBox
codecov-io commented on issue #3886: Proposing Lean OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3886#issuecomment-406254669 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3886?src=pr=h1) Report > Merging

[GitHub] codecov-io edited a comment on issue #3886: Proposing Lean OpenWhisk

2018-07-19 Thread GitBox
codecov-io edited a comment on issue #3886: Proposing Lean OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3886#issuecomment-406254669 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3886?src=pr=h1) Report > Merging

[GitHub] dgrove-oss commented on issue #249: refactor of kube-deploy docker images

2018-07-19 Thread GitBox
dgrove-oss commented on issue #249: refactor of kube-deploy docker images URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/249#issuecomment-406259417 @rabbah -- would you be able to review and merge this PR?

[GitHub] gazben commented on issue #139: Meet minio file access denied when waiting for the controller

2018-07-19 Thread GitBox
gazben commented on issue #139: Meet minio file access denied when waiting for the controller URL: https://github.com/apache/incubator-openwhisk-devtools/issues/139#issuecomment-406263490 @edwardstudy Is this resolved for you? I have the same error? @ddragosd My invoker logs

[GitHub] jiangpengcheng opened a new pull request #3885: Fix typo dp_port in ansible/group_vars/all

2018-07-19 Thread GitBox
jiangpengcheng opened a new pull request #3885: Fix typo dp_port in ansible/group_vars/all URL: https://github.com/apache/incubator-openwhisk/pull/3885 ## Description ## Related issue and scope - [ ] I opened an issue to propose and discuss this change (#) ## My

[GitHub] vvraskin commented on a change in pull request #3876: Use separate db users for deployed components.

2018-07-19 Thread GitBox
vvraskin commented on a change in pull request #3876: Use separate db users for deployed components. URL: https://github.com/apache/incubator-openwhisk/pull/3876#discussion_r203640564 ## File path: ansible/tasks/db/grantPermissions.yml ## @@ -0,0 +1,55 @@ +# Licensed to

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

2018-07-19 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] codecov-io commented on issue #3885: Fix typo dp_port in ansible/group_vars/all

2018-07-19 Thread GitBox
codecov-io commented on issue #3885: Fix typo dp_port in ansible/group_vars/all URL: https://github.com/apache/incubator-openwhisk/pull/3885#issuecomment-406203611 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3885?src=pr=h1) Report > Merging

[GitHub] vvraskin commented on a change in pull request #3876: Use separate db users for deployed components.

2018-07-19 Thread GitBox
vvraskin commented on a change in pull request #3876: Use separate db users for deployed components. URL: https://github.com/apache/incubator-openwhisk/pull/3876#discussion_r203640925 ## File path: ansible/roles/controller/tasks/deploy.yml ## @@ -162,8 +164,8 @@

[GitHub] vvraskin commented on a change in pull request #3876: Use separate db users for deployed components.

2018-07-19 Thread GitBox
vvraskin commented on a change in pull request #3876: Use separate db users for deployed components. URL: https://github.com/apache/incubator-openwhisk/pull/3876#discussion_r203659834 ## File path: ansible/roles/controller/tasks/deploy.yml ## @@ -162,8 +164,8 @@

[GitHub] vvraskin closed pull request #3882: Use invoker id for metrics instead of the hostname.

2018-07-19 Thread GitBox
vvraskin closed pull request #3882: Use invoker id for metrics instead of the hostname. URL: https://github.com/apache/incubator-openwhisk/pull/3882 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] gazben commented on issue #139: Meet minio file access denied when waiting for the controller

2018-07-19 Thread GitBox
gazben commented on issue #139: Meet minio file access denied when waiting for the controller URL: https://github.com/apache/incubator-openwhisk-devtools/issues/139#issuecomment-406292895 FYI: the error does not occur on Ubuntu 14.04

[GitHub] codecov-io edited a comment on issue #3812: ContainerClient + akka http alternative to HttpUtils

2018-07-19 Thread GitBox
codecov-io edited a comment on issue #3812: ContainerClient + akka http alternative to HttpUtils URL: https://github.com/apache/incubator-openwhisk/pull/3812#issuecomment-401212372 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3812?src=pr=h1) Report > Merging

[GitHub] tysonnorris commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils

2018-07-19 Thread GitBox
tysonnorris commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils URL: https://github.com/apache/incubator-openwhisk/pull/3812#discussion_r203773479 ## File path:

[GitHub] gazben commented on issue #139: Meet minio file access denied when waiting for the controller

2018-07-19 Thread GitBox
gazben commented on issue #139: Meet minio file access denied when waiting for the controller URL: https://github.com/apache/incubator-openwhisk-devtools/issues/139#issuecomment-406284288 Another note: During the `make quick-start` command there is this error message: ``` checking

[GitHub] tysonnorris commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils

2018-07-19 Thread GitBox
tysonnorris commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils URL: https://github.com/apache/incubator-openwhisk/pull/3812#discussion_r203783691 ## File path:

[GitHub] rabbah closed pull request #249: refactor of kube-deploy docker images

2018-07-19 Thread GitBox
rabbah closed pull request #249: refactor of kube-deploy docker images URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/249 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] vvraskin commented on a change in pull request #3884: Document metrics generated within OpenWhisk

2018-07-19 Thread GitBox
vvraskin commented on a change in pull request #3884: Document metrics generated within OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3884#discussion_r203759126 ## File path: docs/metrics.md ## @@ -75,6 +75,183 @@ The docker image exposes StatsD via

[GitHub] vvraskin commented on a change in pull request #3884: Document metrics generated within OpenWhisk

2018-07-19 Thread GitBox
vvraskin commented on a change in pull request #3884: Document metrics generated within OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3884#discussion_r203763836 ## File path: docs/metrics.md ## @@ -75,6 +75,183 @@ The docker image exposes StatsD via

[GitHub] vvraskin commented on a change in pull request #3884: Document metrics generated within OpenWhisk

2018-07-19 Thread GitBox
vvraskin commented on a change in pull request #3884: Document metrics generated within OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3884#discussion_r203758403 ## File path: docs/metrics.md ## @@ -75,6 +75,183 @@ The docker image exposes StatsD via

[GitHub] vvraskin commented on a change in pull request #3884: Document metrics generated within OpenWhisk

2018-07-19 Thread GitBox
vvraskin commented on a change in pull request #3884: Document metrics generated within OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3884#discussion_r203758742 ## File path: docs/metrics.md ## @@ -75,6 +75,183 @@ The docker image exposes StatsD via

[GitHub] tysonnorris commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils

2018-07-19 Thread GitBox
tysonnorris commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils URL: https://github.com/apache/incubator-openwhisk/pull/3812#discussion_r203843977 ## File path:

[GitHub] dgrove-oss opened a new pull request #261: Init couchdb job

2018-07-19 Thread GitBox
dgrove-oss opened a new pull request #261: Init couchdb job URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/261 Restructure the db initialization into a separate Job using the whisk-ansible-runner docker image that can be optionally run on both "in cluster" and

[GitHub] codecov-io edited a comment on issue #3812: ContainerClient + akka http alternative to HttpUtils

2018-07-19 Thread GitBox
codecov-io edited a comment on issue #3812: ContainerClient + akka http alternative to HttpUtils URL: https://github.com/apache/incubator-openwhisk/pull/3812#issuecomment-401212372 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3812?src=pr=h1) Report > Merging

[GitHub] markusthoemmes commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils

2018-07-19 Thread GitBox
markusthoemmes commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils URL: https://github.com/apache/incubator-openwhisk/pull/3812#discussion_r203883243 ## File path:

[GitHub] markusthoemmes commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils

2018-07-19 Thread GitBox
markusthoemmes commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils URL: https://github.com/apache/incubator-openwhisk/pull/3812#discussion_r203882123 ## File path:

[GitHub] markusthoemmes commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils

2018-07-19 Thread GitBox
markusthoemmes commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils URL: https://github.com/apache/incubator-openwhisk/pull/3812#discussion_r203880928 ## File path:

[GitHub] markusthoemmes commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils

2018-07-19 Thread GitBox
markusthoemmes commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils URL: https://github.com/apache/incubator-openwhisk/pull/3812#discussion_r203882516 ## File path: common/scala/src/main/scala/whisk/core/containerpool/Container.scala

[GitHub] markusthoemmes commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils

2018-07-19 Thread GitBox
markusthoemmes commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils URL: https://github.com/apache/incubator-openwhisk/pull/3812#discussion_r203882259 ## File path:

[GitHub] markusthoemmes commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils

2018-07-19 Thread GitBox
markusthoemmes commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils URL: https://github.com/apache/incubator-openwhisk/pull/3812#discussion_r203881358 ## File path:

[GitHub] markusthoemmes commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils

2018-07-19 Thread GitBox
markusthoemmes commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils URL: https://github.com/apache/incubator-openwhisk/pull/3812#discussion_r203880878 ## File path:

[GitHub] markusthoemmes commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils

2018-07-19 Thread GitBox
markusthoemmes commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils URL: https://github.com/apache/incubator-openwhisk/pull/3812#discussion_r203883555 ## File path: core/invoker/src/main/resources/application.conf ## @@ -29,6

[GitHub] markusthoemmes commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils

2018-07-19 Thread GitBox
markusthoemmes commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils URL: https://github.com/apache/incubator-openwhisk/pull/3812#discussion_r203882811 ## File path:

[GitHub] markusthoemmes commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils

2018-07-19 Thread GitBox
markusthoemmes commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils URL: https://github.com/apache/incubator-openwhisk/pull/3812#discussion_r203881882 ## File path:

[GitHub] markusthoemmes commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils

2018-07-19 Thread GitBox
markusthoemmes commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils URL: https://github.com/apache/incubator-openwhisk/pull/3812#discussion_r203883483 ## File path: common/scala/src/main/scala/whisk/http/PoolingRestClient.scala

[GitHub] markusthoemmes commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils

2018-07-19 Thread GitBox
markusthoemmes commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils URL: https://github.com/apache/incubator-openwhisk/pull/3812#discussion_r203881076 ## File path:

[GitHub] codecov-io edited a comment on issue #3812: ContainerClient + akka http alternative to HttpUtils

2018-07-19 Thread GitBox
codecov-io edited a comment on issue #3812: ContainerClient + akka http alternative to HttpUtils URL: https://github.com/apache/incubator-openwhisk/pull/3812#issuecomment-401212372 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3812?src=pr=h1) Report > Merging

[GitHub] dgrove-oss commented on issue #252: Adding Ressource libcurl3-gnutls in couchDB Dockerfile uses deprecated link

2018-07-19 Thread GitBox
dgrove-oss commented on issue #252: Adding Ressource libcurl3-gnutls in couchDB Dockerfile uses deprecated link URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/252#issuecomment-406416567 #261 eliminates the problem by letting us use the upstream apache/couchdb

[GitHub] dgrove-oss opened a new pull request #262: add check for healthy invoker to build-helm.sh

2018-07-19 Thread GitBox
dgrove-oss opened a new pull request #262: add check for healthy invoker to build-helm.sh URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/262 Use the controller's /invokers/healthy/count route to verify that the controller has at least one healthy invoker before