[GitHub] chetanmeh commented on a change in pull request #2795: enable concurrent activation processing

2018-10-16 Thread GitBox
chetanmeh commented on a change in pull request #2795: enable concurrent activation processing URL: https://github.com/apache/incubator-openwhisk/pull/2795#discussion_r225774615 ## File path: core/invoker/src/main/scala/whisk/core/invoker/InvokerReactive.scala ## @@

[GitHub] chetanmeh commented on a change in pull request #2795: enable concurrent activation processing

2018-10-16 Thread GitBox
chetanmeh commented on a change in pull request #2795: enable concurrent activation processing URL: https://github.com/apache/incubator-openwhisk/pull/2795#discussion_r225774615 ## File path: core/invoker/src/main/scala/whisk/core/invoker/InvokerReactive.scala ## @@

[GitHub] rabbah commented on issue #4064: Fixes jinja warning from controller playbook

2018-10-16 Thread GitBox
rabbah commented on issue #4064: Fixes jinja warning from controller playbook URL: https://github.com/apache/incubator-openwhisk/pull/4064#issuecomment-430468132 Thanks @mptap for the fixes! This is an automated message

[GitHub] rabbah closed issue #3641: deprecation warning in couch playbook

2018-10-16 Thread GitBox
rabbah closed issue #3641: deprecation warning in couch playbook URL: https://github.com/apache/incubator-openwhisk/issues/3641 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] rabbah closed pull request #4064: Fixes jinja warning from controller playbook

2018-10-16 Thread GitBox
rabbah closed pull request #4064: Fixes jinja warning from controller playbook URL: https://github.com/apache/incubator-openwhisk/pull/4064 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] rabbah closed issue #3639: jinja warning from controller playbook

2018-10-16 Thread GitBox
rabbah closed issue #3639: jinja warning from controller playbook URL: https://github.com/apache/incubator-openwhisk/issues/3639 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] rabbah closed pull request #4063: Fixes deprecation warning in couch playbook

2018-10-16 Thread GitBox
rabbah closed pull request #4063: Fixes deprecation warning in couch playbook URL: https://github.com/apache/incubator-openwhisk/pull/4063 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] chen116 edited a comment on issue #313: SSL certificate issue with python

2018-10-16 Thread GitBox
chen116 edited a comment on issue #313: SSL certificate issue with python URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/313#issuecomment-430406845 Hi all, Solved it for native deployment, just need to add `verify=False` in your requests call

[GitHub] mrutkows commented on a change in pull request #991: Introducing include and exclude in zip action

2018-10-16 Thread GitBox
mrutkows commented on a change in pull request #991: Introducing include and exclude in zip action URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/991#discussion_r225713954 ## File path: parsers/manifest_parser.go ## @@ -649,7 +649,7 @@ func (dm

[GitHub] mrutkows commented on a change in pull request #991: Introducing include and exclude in zip action

2018-10-16 Thread GitBox
mrutkows commented on a change in pull request #991: Introducing include and exclude in zip action URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/991#discussion_r225713712 ## File path: parsers/manifest_parser.go ## @@ -649,7 +649,7 @@ func (dm

[GitHub] mrutkows commented on a change in pull request #991: Introducing include and exclude in zip action

2018-10-16 Thread GitBox
mrutkows commented on a change in pull request #991: Introducing include and exclude in zip action URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/991#discussion_r225714305 ## File path: utils/file.go ## @@ -0,0 +1,152 @@ +/* Review comment:

[GitHub] dgrove-oss closed issue #313: SSL certificate issue with python

2018-10-16 Thread GitBox
dgrove-oss closed issue #313: SSL certificate issue with python URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/313 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] chen116 commented on issue #313: SSL certificate issue with python

2018-10-16 Thread GitBox
chen116 commented on issue #313: SSL certificate issue with python URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/313#issuecomment-430406845 Hi all, Solved it, just need to add `verify=False` in your requests call

[GitHub] dgrove-oss commented on issue #203: travis smoke test should include apigw

2018-10-16 Thread GitBox
dgrove-oss commented on issue #203: travis smoke test should include apigw URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/203#issuecomment-430401374 Fixed before #127 by changes in #314 This is an

[GitHub] tysonnorris commented on a change in pull request #2795: enable concurrent activation processing

2018-10-16 Thread GitBox
tysonnorris commented on a change in pull request #2795: enable concurrent activation processing URL: https://github.com/apache/incubator-openwhisk/pull/2795#discussion_r225663644 ## File path: core/invoker/src/main/scala/whisk/core/containerpool/ContainerProxy.scala ##

[GitHub] tysonnorris commented on a change in pull request #2795: enable concurrent activation processing

2018-10-16 Thread GitBox
tysonnorris commented on a change in pull request #2795: enable concurrent activation processing URL: https://github.com/apache/incubator-openwhisk/pull/2795#discussion_r225663295 ## File path: core/invoker/src/main/scala/whisk/core/containerpool/ContainerProxy.scala ##

[GitHub] houshengbo commented on a change in pull request #73: Rename the package from openwhisk to org.apache.openwhisk

2018-10-16 Thread GitBox
houshengbo commented on a change in pull request #73: Rename the package from openwhisk to org.apache.openwhisk URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/73#discussion_r225662713 ## File path:

[GitHub] houshengbo commented on a change in pull request #73: Rename the package from openwhisk to org.apache.openwhisk

2018-10-16 Thread GitBox
houshengbo commented on a change in pull request #73: Rename the package from openwhisk to org.apache.openwhisk URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/73#discussion_r225662713 ## File path:

[GitHub] houshengbo commented on a change in pull request #73: Rename the package from openwhisk to org.apache.openwhisk

2018-10-16 Thread GitBox
houshengbo commented on a change in pull request #73: Rename the package from openwhisk to org.apache.openwhisk URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/73#discussion_r225662713 ## File path:

[GitHub] dgrove-oss commented on a change in pull request #73: Rename the package from openwhisk to org.apache.openwhisk

2018-10-16 Thread GitBox
dgrove-oss commented on a change in pull request #73: Rename the package from openwhisk to org.apache.openwhisk URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/73#discussion_r225662540 ## File path:

[GitHub] tysonnorris commented on a change in pull request #2795: enable concurrent activation processing

2018-10-16 Thread GitBox
tysonnorris commented on a change in pull request #2795: enable concurrent activation processing URL: https://github.com/apache/incubator-openwhisk/pull/2795#discussion_r225661971 ## File path: core/invoker/src/main/scala/whisk/core/containerpool/ContainerProxy.scala ##

[GitHub] rabbah commented on a change in pull request #73: Rename the package from openwhisk to org.apache.openwhisk

2018-10-16 Thread GitBox
rabbah commented on a change in pull request #73: Rename the package from openwhisk to org.apache.openwhisk URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/73#discussion_r225661493 ## File path:

[GitHub] houshengbo opened a new pull request #73: Rename the package from openwhisk to org.apache.openwhisk

2018-10-16 Thread GitBox
houshengbo opened a new pull request #73: Rename the package from openwhisk to org.apache.openwhisk URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/73 This is an automated message from the Apache Git

[GitHub] tysonnorris commented on a change in pull request #2795: enable concurrent activation processing

2018-10-16 Thread GitBox
tysonnorris commented on a change in pull request #2795: enable concurrent activation processing URL: https://github.com/apache/incubator-openwhisk/pull/2795#discussion_r225645207 ## File path: core/invoker/src/main/scala/whisk/core/invoker/InvokerReactive.scala ## @@

[GitHub] rabbah closed pull request #86: Remove the build of Swift 3.1.1

2018-10-16 Thread GitBox
rabbah closed pull request #86: Remove the build of Swift 3.1.1 URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/86 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

[GitHub] pritidesai closed pull request #52: Add helper scripts used to test mutiple github repos.

2018-10-16 Thread GitBox
pritidesai closed pull request #52: Add helper scripts used to test mutiple github repos. URL: https://github.com/apache/incubator-openwhisk-utilities/pull/52 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] tysonnorris commented on a change in pull request #2795: enable concurrent activation processing

2018-10-16 Thread GitBox
tysonnorris commented on a change in pull request #2795: enable concurrent activation processing URL: https://github.com/apache/incubator-openwhisk/pull/2795#discussion_r225633631 ## File path: tools/travis/setupSystem.sh ## @@ -25,7 +25,7 @@ ROOTDIR="$SCRIPTDIR/../.."

[GitHub] houshengbo closed pull request #85: Remove the build of swift 3.1.1 in travis

2018-10-16 Thread GitBox
houshengbo closed pull request #85: Remove the build of swift 3.1.1 in travis URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/85 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] chetanmeh commented on a change in pull request #2795: enable concurrent activation processing

2018-10-16 Thread GitBox
chetanmeh commented on a change in pull request #2795: enable concurrent activation processing URL: https://github.com/apache/incubator-openwhisk/pull/2795#discussion_r225603927 ## File path: core/invoker/src/main/scala/whisk/core/containerpool/ContainerProxy.scala ##

[GitHub] houshengbo opened a new pull request #86: Remove the build of Swift 3.1.1

2018-10-16 Thread GitBox
houshengbo opened a new pull request #86: Remove the build of Swift 3.1.1 URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/86 This is an automated message from the Apache Git Service. To respond to the

[GitHub] patpatpat123 commented on issue #4067: [Issue] 100% reproducible ok: invoked /guest/example, but the request has not yet finished, with id xxxx

2018-10-16 Thread GitBox
patpatpat123 commented on issue #4067: [Issue] 100% reproducible ok: invoked /guest/example, but the request has not yet finished, with id URL: https://github.com/apache/incubator-openwhisk/issues/4067#issuecomment-430280444 After some further tests, your investigations are I believe

[GitHub] dgrove-oss opened a new pull request #314: Switch from minikube to kubeadm-dind-cluster for TravisCI testing

2018-10-16 Thread GitBox
dgrove-oss opened a new pull request #314: Switch from minikube to kubeadm-dind-cluster for TravisCI testing URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/314 This is an automated message from the

[GitHub] patpatpat123 commented on issue #4067: [Issue] 100% reproducible ok: invoked /guest/example, but the request has not yet finished, with id xxxx

2018-10-16 Thread GitBox
patpatpat123 commented on issue #4067: [Issue] 100% reproducible ok: invoked /guest/example, but the request has not yet finished, with id URL: https://github.com/apache/incubator-openwhisk/issues/4067#issuecomment-430263322 Hello Neeraj, Thank you for the quick response.

[GitHub] neerajmangal commented on issue #4067: [Issue] 100% reproducible ok: invoked /guest/example, but the request has not yet finished, with id xxxx

2018-10-16 Thread GitBox
neerajmangal commented on issue #4067: [Issue] 100% reproducible ok: invoked /guest/example, but the request has not yet finished, with id URL: https://github.com/apache/incubator-openwhisk/issues/4067#issuecomment-430258553 You need to use "-t" during action creation. ```

[GitHub] patpatpat123 commented on issue #4067: [Issue] 100% reproducible ok: invoked /guest/example, but the request has not yet finished, with id xxxx

2018-10-16 Thread GitBox
patpatpat123 commented on issue #4067: [Issue] 100% reproducible ok: invoked /guest/example, but the request has not yet finished, with id URL: https://github.com/apache/incubator-openwhisk/issues/4067#issuecomment-430255277 Hello, Can you help with the -t command? I did a lot

[GitHub] saurabh1284 commented on issue #311: controller and invoker pods are stuck in PodInitializing

2018-10-16 Thread GitBox
saurabh1284 commented on issue #311: controller and invoker pods are stuck in PodInitializing URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/311#issuecomment-430237627 Re-deployed to sync and now Nginx pod is going in CrashLoopBackOff. -> kubectl get pods -n

[GitHub] sven-lange-last edited a comment on issue #4070: Diagnostic info + metrics for Docker command failures and timeouts

2018-10-16 Thread GitBox
sven-lange-last edited a comment on issue #4070: Diagnostic info + metrics for Docker command failures and timeouts URL: https://github.com/apache/incubator-openwhisk/pull/4070#issuecomment-430161875 PG 5 / 10719 completed successfully.

[GitHub] sven-lange-last commented on issue #4070: Diagnostic info + metrics for Docker command failures and timeouts

2018-10-16 Thread GitBox
sven-lange-last commented on issue #4070: Diagnostic info + metrics for Docker command failures and timeouts URL: https://github.com/apache/incubator-openwhisk/pull/4070#issuecomment-430234496 @csantanapr it's certainly a good idea to update the Docker CLI to something newer. A newer

[GitHub] patpatpat123 commented on issue #297: [Questions] Java based Kafka Producer to trigger OpenWhisk action

2018-10-16 Thread GitBox
patpatpat123 commented on issue #297: [Questions] Java based Kafka Producer to trigger OpenWhisk action URL: https://github.com/apache/incubator-openwhisk-package-kafka/issues/297#issuecomment-430215889 Hello Carlos, Thank you for your advices. I started with docker run -e

[GitHub] markusthoemmes commented on a change in pull request #4071: Fix mock-verification in Container-tests.

2018-10-16 Thread GitBox
markusthoemmes commented on a change in pull request #4071: Fix mock-verification in Container-tests. URL: https://github.com/apache/incubator-openwhisk/pull/4071#discussion_r225513299 ## File path:

[GitHub] codecov-io commented on issue #4071: Fix mock-verification in Container-tests.

2018-10-16 Thread GitBox
codecov-io commented on issue #4071: Fix mock-verification in Container-tests. URL: https://github.com/apache/incubator-openwhisk/pull/4071#issuecomment-430212490 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/4071?src=pr=h1) Report > Merging

[GitHub] csantanapr commented on issue #297: [Questions] Java based Kafka Producer to trigger OpenWhisk action

2018-10-16 Thread GitBox
csantanapr commented on issue #297: [Questions] Java based Kafka Producer to trigger OpenWhisk action URL: https://github.com/apache/incubator-openwhisk-package-kafka/issues/297#issuecomment-430207577 How are you setting up couchdb container? The problem is that the kafkaprovider can’t

[GitHub] cbickel opened a new pull request #4071: Fix mock-verification in Container-tests.

2018-10-16 Thread GitBox
cbickel opened a new pull request #4071: Fix mock-verification in Container-tests. URL: https://github.com/apache/incubator-openwhisk/pull/4071 ## Description These two tests are failing intermittently in our pipeline and travis. But unfortunately I'm not able to reproduce the

[GitHub] chetanmeh commented on a change in pull request #2795: enable concurrent activation processing

2018-10-16 Thread GitBox
chetanmeh commented on a change in pull request #2795: enable concurrent activation processing URL: https://github.com/apache/incubator-openwhisk/pull/2795#discussion_r225497006 ## File path: core/invoker/src/main/scala/whisk/core/containerpool/ContainerProxy.scala ##

[GitHub] chetanmeh commented on a change in pull request #2795: enable concurrent activation processing

2018-10-16 Thread GitBox
chetanmeh commented on a change in pull request #2795: enable concurrent activation processing URL: https://github.com/apache/incubator-openwhisk/pull/2795#discussion_r225466727 ## File path: core/invoker/src/main/scala/whisk/core/invoker/InvokerReactive.scala ## @@

[GitHub] chetanmeh commented on a change in pull request #2795: enable concurrent activation processing

2018-10-16 Thread GitBox
chetanmeh commented on a change in pull request #2795: enable concurrent activation processing URL: https://github.com/apache/incubator-openwhisk/pull/2795#discussion_r225481350 ## File path: core/invoker/src/main/scala/whisk/core/containerpool/ContainerProxy.scala ##

[GitHub] chetanmeh commented on a change in pull request #2795: enable concurrent activation processing

2018-10-16 Thread GitBox
chetanmeh commented on a change in pull request #2795: enable concurrent activation processing URL: https://github.com/apache/incubator-openwhisk/pull/2795#discussion_r225491099 ## File path: core/invoker/src/main/scala/whisk/core/containerpool/ContainerProxy.scala ##

[GitHub] chetanmeh commented on a change in pull request #2795: enable concurrent activation processing

2018-10-16 Thread GitBox
chetanmeh commented on a change in pull request #2795: enable concurrent activation processing URL: https://github.com/apache/incubator-openwhisk/pull/2795#discussion_r225416283 ## File path: common/scala/src/main/scala/whisk/core/containerpool/ContainerFactory.scala ##

[GitHub] chetanmeh commented on a change in pull request #2795: enable concurrent activation processing

2018-10-16 Thread GitBox
chetanmeh commented on a change in pull request #2795: enable concurrent activation processing URL: https://github.com/apache/incubator-openwhisk/pull/2795#discussion_r225473479 ## File path: core/invoker/src/main/scala/whisk/core/containerpool/ContainerProxy.scala ##

[GitHub] chetanmeh commented on a change in pull request #2795: enable concurrent activation processing

2018-10-16 Thread GitBox
chetanmeh commented on a change in pull request #2795: enable concurrent activation processing URL: https://github.com/apache/incubator-openwhisk/pull/2795#discussion_r225479830 ## File path: core/invoker/src/main/scala/whisk/core/containerpool/ContainerProxy.scala ##

[GitHub] csantanapr commented on issue #4070: Diagnostic info + metrics for Docker command failures and timeouts

2018-10-16 Thread GitBox
csantanapr commented on issue #4070: Diagnostic info + metrics for Docker command failures and timeouts URL: https://github.com/apache/incubator-openwhisk/pull/4070#issuecomment-430191059 Should we also update the docker CLI from 1.12 to one more current? Maybe not on this PR but

[GitHub] sven-lange-last edited a comment on issue #4070: Diagnostic info + metrics for Docker command failures and timeouts

2018-10-16 Thread GitBox
sven-lange-last edited a comment on issue #4070: Diagnostic info + metrics for Docker command failures and timeouts URL: https://github.com/apache/incubator-openwhisk/pull/4070#issuecomment-430161875 PG 5 / 10719 running

[GitHub] codecov-io commented on issue #4070: Diagnostic info + metrics for Docker command failures and timeouts

2018-10-16 Thread GitBox
codecov-io commented on issue #4070: Diagnostic info + metrics for Docker command failures and timeouts URL: https://github.com/apache/incubator-openwhisk/pull/4070#issuecomment-430174909 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/4070?src=pr=h1) Report >

[GitHub] sven-lange-last commented on issue #4070: Diagnostic info + metrics for Docker command failures and timeouts

2018-10-16 Thread GitBox
sven-lange-last commented on issue #4070: Diagnostic info + metrics for Docker command failures and timeouts URL: https://github.com/apache/incubator-openwhisk/pull/4070#issuecomment-430161875 PG 5 / 10718 running This is

[GitHub] sven-lange-last opened a new pull request #4070: Diagnostic info + metrics for Docker command failures and timeouts

2018-10-16 Thread GitBox
sven-lange-last opened a new pull request #4070: Diagnostic info + metrics for Docker command failures and timeouts URL: https://github.com/apache/incubator-openwhisk/pull/4070 Provide more diagnostic information for Docker command failures as well as timeouts. Also emit new counter

[GitHub] patpatpat123 edited a comment on issue #297: [Questions] Java based Kafka Producer to trigger OpenWhisk action

2018-10-16 Thread GitBox
patpatpat123 edited a comment on issue #297: [Questions] Java based Kafka Producer to trigger OpenWhisk action URL: https://github.com/apache/incubator-openwhisk-package-kafka/issues/297#issuecomment-430028745 Hello, Thank you for this reference. May I ask a quick follow up

[GitHub] patpatpat123 edited a comment on issue #297: [Questions] Java based Kafka Producer to trigger OpenWhisk action

2018-10-16 Thread GitBox
patpatpat123 edited a comment on issue #297: [Questions] Java based Kafka Producer to trigger OpenWhisk action URL: https://github.com/apache/incubator-openwhisk-package-kafka/issues/297#issuecomment-430028745 Hello Carlos, Thank you for this reference. May I ask a quick follow

[GitHub] markusthoemmes commented on issue #4069: DockerContainerTests heisenbug

2018-10-16 Thread GitBox
markusthoemmes commented on issue #4069: DockerContainerTests heisenbug URL: https://github.com/apache/incubator-openwhisk/issues/4069#issuecomment-430114602 I'd propose to just kill the stubbed function and replace it with a `LoggingFunction` (a self-rolled stubbing mechanism). I had