[GitHub] style95 opened a new issue #3015: [Feature request] Hide codes or parameters from shared package

2017-11-27 Thread GitBox
style95 opened a new issue #3015: [Feature request] Hide codes or parameters from shared package URL: https://github.com/apache/incubator-openwhisk/issues/3015 Since it is common that someone want to share his features but does not want to reveal its internal or any secrets, it would be

[GitHub] ningyougang opened a new issue #3014: When decreasing component nodes, there will be duplicate or stable components

2017-11-27 Thread GitBox
ningyougang opened a new issue #3014: When decreasing component nodes, there will be duplicate or stable components URL: https://github.com/apache/incubator-openwhisk/issues/3014 Question desc before deploy, there has three invoker nodes on curren env. ``` invoker0

[GitHub] houshengbo opened a new pull request #3013: Change the idle timeout to 5 mins for client connection settings

2017-11-27 Thread GitBox
houshengbo opened a new pull request #3013: Change the idle timeout to 5 mins for client connection settings URL: https://github.com/apache/incubator-openwhisk/pull/3013 This is an automated message from the Apache Git

[GitHub] rabbah commented on a change in pull request #3005: Update invoke.py to reflect changes in repository

2017-11-27 Thread GitBox
rabbah commented on a change in pull request #3005: Update invoke.py to reflect changes in repository URL: https://github.com/apache/incubator-openwhisk/pull/3005#discussion_r153367050 ## File path: actionRuntimes/actionProxy/invoke.py ## @@ -3,9 +3,9 @@ This script

[GitHub] csantanapr commented on a change in pull request #3005: Update invoke.py to reflect changes in repository

2017-11-27 Thread GitBox
csantanapr commented on a change in pull request #3005: Update invoke.py to reflect changes in repository URL: https://github.com/apache/incubator-openwhisk/pull/3005#discussion_r153359628 ## File path: actionRuntimes/actionProxy/invoke.py ## @@ -3,9 +3,9 @@ This

[GitHub] dgrove-oss commented on issue #91: update for pureconfig (OpenWhisk PR 2576)

2017-11-27 Thread GitBox
dgrove-oss commented on issue #91: update for pureconfig (OpenWhisk PR 2576) URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/91#issuecomment-347352308 The nginx pod failed to start. Not obvious why from travis logs. Don't see why changing invoker/controller yml files

[GitHub] dubeejw commented on a change in pull request #2847: Treat action code as attachments for created/updated actions

2017-11-27 Thread GitBox
dubeejw commented on a change in pull request #2847: Treat action code as attachments for created/updated actions URL: https://github.com/apache/incubator-openwhisk/pull/2847#discussion_r153325540 ## File path: common/scala/src/main/scala/whisk/core/entity/Exec.scala ##

[GitHub] dubeejw commented on issue #2847: Treat action code as attachments for created/updated actions

2017-11-27 Thread GitBox
dubeejw commented on issue #2847: Treat action code as attachments for created/updated actions URL: https://github.com/apache/incubator-openwhisk/pull/2847#issuecomment-347328042 @rabbah, I believe I had originally planned to have https://github.com/apache/incubator-openwhisk/pull/2855

[GitHub] mdeuser commented on a change in pull request #45: WIP: Adding support for path parameters for api gateway

2017-11-27 Thread GitBox
mdeuser commented on a change in pull request #45: WIP: Adding support for path parameters for api gateway URL: https://github.com/apache/incubator-openwhisk-client-go/pull/45#discussion_r153316928 ## File path: whisk/api.go ## @@ -71,6 +71,32 @@ type Api struct {

[GitHub] mdeuser commented on a change in pull request #45: WIP: Adding support for path parameters for api gateway

2017-11-27 Thread GitBox
mdeuser commented on a change in pull request #45: WIP: Adding support for path parameters for api gateway URL: https://github.com/apache/incubator-openwhisk-client-go/pull/45#discussion_r153317535 ## File path: whisk/api.go ## @@ -71,6 +71,32 @@ type Api struct {

[GitHub] rabbah commented on a change in pull request #2: Typo: PHP does not include the letters s, w, i, f, or t

2017-11-27 Thread GitBox
rabbah commented on a change in pull request #2: Typo: PHP does not include the letters s, w, i, f, or t URL: https://github.com/apache/incubator-openwhisk-runtime-php/pull/2#discussion_r153304687 ## File path: README.md ## @@ -1,4 +1,4 @@ -# Apache OpenWhisk runtimes

[GitHub] trieloff opened a new pull request #2: Typo: PHP does not include the letters s, w, i, f, or t

2017-11-27 Thread GitBox
trieloff opened a new pull request #2: Typo: PHP does not include the letters s, w, i, f, or t URL: https://github.com/apache/incubator-openwhisk-runtime-php/pull/2 This is an automated message from the Apache Git Service.

[GitHub] jessealva opened a new pull request #170: WIP: Initial code checkin for path parameter support

2017-11-27 Thread GitBox
jessealva opened a new pull request #170: WIP: Initial code checkin for path parameter support URL: https://github.com/apache/incubator-openwhisk-cli/pull/170 Needs an update to incubator-openwhisk-client-go and an update to the Godeps.json before this will be functional.

[GitHub] bwmcadams commented on issue #2937: abrupt termination at the end of the REST tests

2017-11-27 Thread GitBox
bwmcadams commented on issue #2937: abrupt termination at the end of the REST tests URL: https://github.com/apache/incubator-openwhisk/issues/2937#issuecomment-347292774 @rabbah I'll have to take a look this afternoon and see if I can reproduce.

[GitHub] dgrove-oss commented on issue #90: Unable to create action - connection timed out

2017-11-27 Thread GitBox
dgrove-oss commented on issue #90: Unable to create action - connection timed out URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/90#issuecomment-347285752 maybe `kubectl describe nodes` This is

[GitHub] ngibanel commented on issue #90: Unable to create action - connection timed out

2017-11-27 Thread GitBox
ngibanel commented on issue #90: Unable to create action - connection timed out URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/90#issuecomment-347282787 `kubectl get nodes` does not give me the public IP address ? (maybe because I have only one node, the master ??)

[GitHub] dgrove-oss commented on issue #90: Unable to create action - connection timed out

2017-11-27 Thread GitBox
dgrove-oss commented on issue #90: Unable to create action - connection timed out URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/90#issuecomment-347279294 It might be worth trying `kubectl get nodes` to make sure that 10.110.238.5 really is the public IP address

[GitHub] davidbreitgand commented on issue #1273: Long-running actions

2017-11-27 Thread GitBox
davidbreitgand commented on issue #1273: Long-running actions URL: https://github.com/apache/incubator-openwhisk/issues/1273#issuecomment-347278582 Was there any subsequent discussion on this issue? This is an automated

[GitHub] ngibanel opened a new issue #90: Unable to create action - connection timed out

2017-11-27 Thread GitBox
ngibanel opened a new issue #90: Unable to create action - connection timed out URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/90 Hi, I have deployed openwhisk on a local kubernetes cluster (single node) following step-by-step the docs but I cannot reach the

[GitHub] csantanapr closed pull request #3009: Add the idle timeout configuration to the Akka client

2017-11-27 Thread GitBox
csantanapr closed pull request #3009: Add the idle timeout configuration to the Akka client URL: https://github.com/apache/incubator-openwhisk/pull/3009 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] rabbah commented on issue #3008: DockerContainerTests heseinbug

2017-11-27 Thread GitBox
rabbah commented on issue #3008: DockerContainerTests heseinbug URL: https://github.com/apache/incubator-openwhisk/issues/3008#issuecomment-347256437 @cbickel @markusthoemmes do you have a suggestion other than doing this? ```bash diff --git

[GitHub] jessealva commented on issue #45: WIP: Adding support for path parameters for api gateway

2017-11-27 Thread GitBox
jessealva commented on issue #45: WIP: Adding support for path parameters for api gateway URL: https://github.com/apache/incubator-openwhisk-client-go/pull/45#issuecomment-347256300 @mdeuser @dubeejw Can one of you review this as part of the changes for path parameter support? Need these

[GitHub] rabbah closed pull request #2576: Configure components via pureconfig.

2017-11-27 Thread GitBox
rabbah closed pull request #2576: Configure components via pureconfig. URL: https://github.com/apache/incubator-openwhisk/pull/2576 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] rabbah opened a new pull request #3012: Enhance wskadmin to list all spaces for a subject and add README

2017-11-27 Thread GitBox
rabbah opened a new pull request #3012: Enhance wskadmin to list all spaces for a subject and add README URL: https://github.com/apache/incubator-openwhisk/pull/3012 Added README describing `wskadmin` and examples how to use it. Also added utility for listing all namespaces for a

[GitHub] jessealva opened a new pull request #45: WIP: Adding support for path parameters for api gateway

2017-11-27 Thread GitBox
jessealva opened a new pull request #45: WIP: Adding support for path parameters for api gateway URL: https://github.com/apache/incubator-openwhisk-client-go/pull/45 This is an automated message from the Apache Git Service.

[GitHub] rabbah closed pull request #89: Update READMEs and kafka image to exploit PR-2948

2017-11-27 Thread GitBox
rabbah closed pull request #89: Update READMEs and kafka image to exploit PR-2948 URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/89 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] houshengbo opened a new pull request #3011: Remove WskApiGwTests

2017-11-27 Thread GitBox
houshengbo opened a new pull request #3011: Remove WskApiGwTests URL: https://github.com/apache/incubator-openwhisk/pull/3011 The test case has been moved to CLI repository as CLI-specific tests. This is an automated message

[GitHub] dgrove-oss commented on issue #2576: Configure components via pureconfig.

2017-11-27 Thread GitBox
dgrove-oss commented on issue #2576: Configure components via pureconfig. URL: https://github.com/apache/incubator-openwhisk/pull/2576#issuecomment-347232455 We'll need some kube updates to match the new naming convention for envars, but should be straightforward. Once this gets merged,

[GitHub] houshengbo commented on a change in pull request #3010: Adapt timeouts of controller and nginx.

2017-11-27 Thread GitBox
houshengbo commented on a change in pull request #3010: Adapt timeouts of controller and nginx. URL: https://github.com/apache/incubator-openwhisk/pull/3010#discussion_r153241816 ## File path: core/controller/src/main/resources/application.conf ## @@ -15,7 +15,7 @@

[GitHub] houshengbo commented on issue #2996: Remove WskBasicTests and WskBasicUsageTests

2017-11-27 Thread GitBox
houshengbo commented on issue #2996: Remove WskBasicTests and WskBasicUsageTests URL: https://github.com/apache/incubator-openwhisk/pull/2996#issuecomment-347230550 It looks like there are plenty of CLI related test cases existing in WskBasicTests and WskBasicUsageTests. Both of them has

[GitHub] cbickel commented on a change in pull request #3010: Adapt timeouts of controller and nginx.

2017-11-27 Thread GitBox
cbickel commented on a change in pull request #3010: Adapt timeouts of controller and nginx. URL: https://github.com/apache/incubator-openwhisk/pull/3010#discussion_r153240512 ## File path: core/controller/src/main/resources/application.conf ## @@ -15,7 +15,7 @@

[GitHub] dgrove-oss commented on issue #2117: couchdb-snapshot docker image is missing `subjects` and `whisks` DBs

2017-11-27 Thread GitBox
dgrove-oss commented on issue #2117: couchdb-snapshot docker image is missing `subjects` and `whisks` DBs URL: https://github.com/apache/incubator-openwhisk/issues/2117#issuecomment-347226654 Travis (for the deploy-kube project) is building an openwhisk/kube-couchdb image and publishing

[GitHub] rabbah closed issue #77: Nginx can not forward the following requests to controller once the major controller is restarted.

2017-11-27 Thread GitBox
rabbah closed issue #77: Nginx can not forward the following requests to controller once the major controller is restarted. URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/77 This is an automated

[GitHub] dgrove-oss commented on issue #77: Nginx can not forward the following requests to controller once the major controller is restarted.

2017-11-27 Thread GitBox
dgrove-oss commented on issue #77: Nginx can not forward the following requests to controller once the major controller is restarted. URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/77#issuecomment-347223768 thanks. Please go ahead and close it then.

[GitHub] cbickel commented on a change in pull request #3010: Adapt timeouts of controller and nginx.

2017-11-27 Thread GitBox
cbickel commented on a change in pull request #3010: Adapt timeouts of controller and nginx. URL: https://github.com/apache/incubator-openwhisk/pull/3010#discussion_r153233926 ## File path: core/controller/src/main/resources/application.conf ## @@ -15,7 +15,7 @@

[GitHub] houshengbo commented on a change in pull request #3010: Adapt timeouts of controller and nginx.

2017-11-27 Thread GitBox
houshengbo commented on a change in pull request #3010: Adapt timeouts of controller and nginx. URL: https://github.com/apache/incubator-openwhisk/pull/3010#discussion_r153232430 ## File path: core/controller/src/main/resources/application.conf ## @@ -15,7 +15,7 @@

[GitHub] houshengbo commented on issue #3010: Adapt timeouts of controller and nginx.

2017-11-27 Thread GitBox
houshengbo commented on issue #3010: Adapt timeouts of controller and nginx. URL: https://github.com/apache/incubator-openwhisk/pull/3010#issuecomment-347218285 @cbickel Have you got a wrong statement: ``` If the nginx proxy_read_timeout is higher than the controller timeout,

[GitHub] cbickel opened a new pull request #3010: Adapt timeouts of controller and nginx.

2017-11-27 Thread GitBox
cbickel opened a new pull request #3010: Adapt timeouts of controller and nginx. URL: https://github.com/apache/incubator-openwhisk/pull/3010 Lower the timeouts of the controller to match the proxy_read_timeout in nginx. If the nginx proxy_read_timeout is higher than the controller

[GitHub] houshengbo opened a new pull request #3009: Add the idle timeout configuration to the Akka client

2017-11-27 Thread GitBox
houshengbo opened a new pull request #3009: Add the idle timeout configuration to the Akka client URL: https://github.com/apache/incubator-openwhisk/pull/3009 This PR sets the idle timeout to 5 minutes in order to give enough time for sequence tests to finish.

[GitHub] dgrove-oss commented on issue #2991: controller creates health and cacheInvalidation topics

2017-11-27 Thread GitBox
dgrove-oss commented on issue #2991: controller creates health and cacheInvalidation topics URL: https://github.com/apache/incubator-openwhisk/pull/2991#issuecomment-347208311 PG2 2362 ? This is an automated message from

[GitHub] rabbah opened a new issue #3008: DockerContainerTests heseinbug

2017-11-27 Thread GitBox
rabbah opened a new issue #3008: DockerContainerTests heseinbug URL: https://github.com/apache/incubator-openwhisk/issues/3008 Failed in this Travis CI build 307526229 ``` whisk.core.containerpool.docker.test.DockerContainerTests > DockerContainer should eventually terminate even

[GitHub] sven-lange-last commented on a change in pull request #3007: Clean orphaned ifstate.veth* files on Ubuntu 14.04

2017-11-27 Thread GitBox
sven-lange-last commented on a change in pull request #3007: Clean orphaned ifstate.veth* files on Ubuntu 14.04 URL: https://github.com/apache/incubator-openwhisk/pull/3007#discussion_r153212813 ## File path: ansible/roles/invoker/tasks/clean.yml ## @@ -38,3 +38,14 @@

[GitHub] csantanapr commented on a change in pull request #3007: Clean orphaned ifstate.veth* files on Ubuntu 14.04

2017-11-27 Thread GitBox
csantanapr commented on a change in pull request #3007: Clean orphaned ifstate.veth* files on Ubuntu 14.04 URL: https://github.com/apache/incubator-openwhisk/pull/3007#discussion_r153210835 ## File path: ansible/roles/invoker/tasks/clean.yml ## @@ -38,3 +38,14 @@

[GitHub] csantanapr commented on a change in pull request #3007: Clean orphaned ifstate.veth* files on Ubuntu 14.04

2017-11-27 Thread GitBox
csantanapr commented on a change in pull request #3007: Clean orphaned ifstate.veth* files on Ubuntu 14.04 URL: https://github.com/apache/incubator-openwhisk/pull/3007#discussion_r153211065 ## File path: ansible/roles/invoker/tasks/clean.yml ## @@ -38,3 +38,14 @@

[GitHub] sven-lange-last commented on issue #3007: Clean orphaned ifstate.veth* files on Ubuntu 14.04

2017-11-27 Thread GitBox
sven-lange-last commented on issue #3007: Clean orphaned ifstate.veth* files on Ubuntu 14.04 URL: https://github.com/apache/incubator-openwhisk/pull/3007#issuecomment-347180437 PG3 / 1400 ? This is an automated message

[GitHub] sven-lange-last commented on issue #3007: Clean orphaned ifstate.veth* files on Ubuntu 14.04

2017-11-27 Thread GitBox
sven-lange-last commented on issue #3007: Clean orphaned ifstate.veth* files on Ubuntu 14.04 URL: https://github.com/apache/incubator-openwhisk/pull/3007#issuecomment-347147460 @csantanapr - can you please review this PR?

[GitHub] sven-lange-last commented on issue #3007: Clean orphaned ifstate.veth* files on Ubuntu 14.04

2017-11-27 Thread GitBox
sven-lange-last commented on issue #3007: Clean orphaned ifstate.veth* files on Ubuntu 14.04 URL: https://github.com/apache/incubator-openwhisk/pull/3007#issuecomment-347146168 PG3 / 1400 running This is an automated

[GitHub] sven-lange-last commented on issue #3007: Clean orphaned ifstate.veth* files on Ubuntu 14.04

2017-11-27 Thread GitBox
sven-lange-last commented on issue #3007: Clean orphaned ifstate.veth* files on Ubuntu 14.04 URL: https://github.com/apache/incubator-openwhisk/pull/3007#issuecomment-347144704 When deploying to a `docker-machine` environment, the cleaning procedure is properly skipped because it`s not

[GitHub] ServoKvd commented on issue #2911: Proposal to to reorganize core/ and root(openwhisk/) folders to reflect existing architecture better

2017-11-27 Thread GitBox
ServoKvd commented on issue #2911: Proposal to to reorganize core/ and root(openwhisk/) folders to reflect existing architecture better URL: https://github.com/apache/incubator-openwhisk/issues/2911#issuecomment-347144244 Yes - I guess the Install can happen in setup.sh of the Travis

[GitHub] sven-lange-last commented on issue #3007: Clean orphaned ifstate.veth* files on Ubuntu 14.04

2017-11-27 Thread GitBox
sven-lange-last commented on issue #3007: Clean orphaned ifstate.veth* files on Ubuntu 14.04 URL: https://github.com/apache/incubator-openwhisk/pull/3007#issuecomment-347143078 The updated playbook for cleaning invokers ran successfully on a Ubuntu 14.04 VM - see below. * This VM

[GitHub] sven-lange-last commented on issue #3007: Clean orphaned ifstate.veth* files on Ubuntu 14.04

2017-11-27 Thread GitBox
sven-lange-last commented on issue #3007: Clean orphaned ifstate.veth* files on Ubuntu 14.04 URL: https://github.com/apache/incubator-openwhisk/pull/3007#issuecomment-347143078 The updated playbook for cleaning invokers ran successfully on a Ubuntu 14.04 VM: ``` #

[GitHub] sven-lange-last commented on issue #3007: Clean orphaned ifstate.veth* files on Ubuntu 14.04

2017-11-27 Thread GitBox
sven-lange-last commented on issue #3007: Clean orphaned ifstate.veth* files on Ubuntu 14.04 URL: https://github.com/apache/incubator-openwhisk/pull/3007#issuecomment-347142421 ### Cleaning script design Build regular expression that matches all active virtual ethernet

[GitHub] sven-lange-last opened a new pull request #3007: Clean orphaned ifstate.veth* files on Ubuntu 14.04

2017-11-27 Thread GitBox
sven-lange-last opened a new pull request #3007: Clean orphaned ifstate.veth* files on Ubuntu 14.04 URL: https://github.com/apache/incubator-openwhisk/pull/3007 Ubuntu 14.04 has a well-known bug that leaves orphaned `ifstate.veth*` files in `/run/network` when creating / removing a

[GitHub] duynguyen commented on issue #65: travis build (and docker-compose based builds in general) fail with broken controller.env

2017-11-27 Thread GitBox
duynguyen commented on issue #65: travis build (and docker-compose based builds in general) fail with broken controller.env URL: https://github.com/apache/incubator-openwhisk-devtools/issues/65#issuecomment-347126224 I opened #64 to fix it. Can not merge yet as the build is failing (need

[GitHub] ningyougang commented on issue #2998: Increase limits.invocationsPerMinute default value

2017-11-27 Thread GitBox
ningyougang commented on issue #2998: Increase limits.invocationsPerMinute default value URL: https://github.com/apache/incubator-openwhisk/pull/2998#issuecomment-347112650 Oh, sadly, We use the latest codes, the problem is still exist(not very stable) which means happen somtimes and

[GitHub] rabbah commented on issue #2911: Proposal to to reorganize core/ and root(openwhisk/) folders to reflect existing architecture better

2017-11-27 Thread GitBox
rabbah commented on issue #2911: Proposal to to reorganize core/ and root(openwhisk/) folders to reflect existing architecture better URL: https://github.com/apache/incubator-openwhisk/issues/2911#issuecomment-347113873 I?d think the cli can stay as is wrt wsk api. It should assume these

[GitHub] rabbah commented on issue #2998: Increase limits.invocationsPerMinute default value

2017-11-27 Thread GitBox
rabbah commented on issue #2998: Increase limits.invocationsPerMinute default value URL: https://github.com/apache/incubator-openwhisk/pull/2998#issuecomment-347113510 Thanks for the catch. The proper fix is to settle the throttle properly in the test suite based on the controller

[GitHub] ningyougang commented on issue #2998: Increase limits.invocationsPerMinute default value

2017-11-27 Thread GitBox
ningyougang commented on issue #2998: Increase limits.invocationsPerMinute default value URL: https://github.com/apache/incubator-openwhisk/pull/2998#issuecomment-347112650 Oh, sadly, We use the latest codes, the problem is still exist(not very stable) which means happen somtimes and

[GitHub] ningyougang commented on issue #2998: Increase limits.invocationsPerMinute default value

2017-11-27 Thread GitBox
ningyougang commented on issue #2998: Increase limits.invocationsPerMinute default value URL: https://github.com/apache/incubator-openwhisk/pull/2998#issuecomment-347112650 Oh, sadly, We use the latest codes, the problem is still exist(not very stable) which means happen somtimes and

[GitHub] ningyougang opened a new pull request #2998: Increase limits.invocationsPerMinute default value

2017-11-27 Thread GitBox
ningyougang opened a new pull request #2998: Increase limits.invocationsPerMinute default value URL: https://github.com/apache/incubator-openwhisk/pull/2998 Sometimes, when run the test cases, the apigw request may exceed the default value(60), so it is necessary to increase the

[GitHub] devbv commented on issue #2744: Deploy kafka & zookeeper cluster with ansible

2017-11-27 Thread GitBox
devbv commented on issue #2744: Deploy kafka & zookeeper cluster with ansible URL: https://github.com/apache/incubator-openwhisk/pull/2744#issuecomment-347107329 Kafka issues are resolved, but HA mode of controller lead to fail travis. Many tests were designed for sequential command.