[GitHub] mrutkows commented on a change in pull request #275: Fixing zip action creation

2017-05-25 Thread git
mrutkows commented on a change in pull request #275: Fixing zip action creation URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/275#discussion_r118634969 ## File path: parsers/manifest_parser.go ## @@ -262,6 +257,15 @@ func (dm *YAMLParser)

[GitHub] ddragosd commented on a change in pull request #2282: Distributed tracing support #2192

2017-05-25 Thread git
ddragosd commented on a change in pull request #2282: Distributed tracing support #2192 URL: https://github.com/apache/incubator-openwhisk/pull/2282#discussion_r118615003 ## File path: common/scala/src/main/scala/whisk/core/connector/Message.scala ## @@ -53,7 +54,8 @@

[GitHub] rabbah commented on issue #2298: temporary pinned apigateway to 0.7.0

2017-05-25 Thread git
rabbah commented on issue #2298: temporary pinned apigateway to 0.7.0 URL: https://github.com/apache/incubator-openwhisk/pull/2298#issuecomment-304158415 Sigh. Good point. For the open source communit at least that not a concern and we should not break master and prs. For the deployed

[GitHub] rabbah commented on issue #2298: temporary pinned apigateway to 0.7.0

2017-05-25 Thread git
rabbah commented on issue #2298: temporary pinned apigateway to 0.7.0 URL: https://github.com/apache/incubator-openwhisk/pull/2298#issuecomment-304158415 Sigh. Good point. For the open source communit at least that not a concern and we should not break master and prs. For the deployed

[GitHub] rabbah commented on a change in pull request #2282: Distributed tracing support #2192

2017-05-25 Thread git
rabbah commented on a change in pull request #2282: Distributed tracing support #2192 URL: https://github.com/apache/incubator-openwhisk/pull/2282#discussion_r118610118 ## File path: common/scala/src/main/scala/whisk/core/connector/Message.scala ## @@ -53,7 +54,8 @@ case

[GitHub] rabbah commented on a change in pull request #2282: Distributed tracing support #2192

2017-05-25 Thread git
rabbah commented on a change in pull request #2282: Distributed tracing support #2192 URL: https://github.com/apache/incubator-openwhisk/pull/2282#discussion_r118610118 ## File path: common/scala/src/main/scala/whisk/core/connector/Message.scala ## @@ -53,7 +54,8 @@ case

[GitHub] ddragosd commented on a change in pull request #2282: Distributed tracing support #2192

2017-05-25 Thread git
ddragosd commented on a change in pull request #2282: Distributed tracing support #2192 URL: https://github.com/apache/incubator-openwhisk/pull/2282#discussion_r118609460 ## File path: common/scala/src/main/scala/whisk/core/connector/Message.scala ## @@ -53,7 +54,8 @@

[GitHub] rabbah commented on issue #2298: temporary pinned apigateway to 0.7.0

2017-05-25 Thread git
rabbah commented on issue #2298: temporary pinned apigateway to 0.7.0 URL: https://github.com/apache/incubator-openwhisk/pull/2298#issuecomment-304152333 Perhaps this should stay pinned. We can't keep getting bit by this.

[incubator-openwhisk] branch master updated: temporary pinned apigateway to 0.7.0 (#2298)

2017-05-25 Thread rabbah
This is an automated email from the ASF dual-hosted git repository. rabbah pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-openwhisk.git The following commit(s) were added to refs/heads/master by this push: new 6509ed5 temporary pinned

[GitHub] taylorking commented on issue #212: fix v2 DELETE /apis

2017-05-25 Thread git
taylorking commented on issue #212: fix v2 DELETE /apis URL: https://github.com/apache/incubator-openwhisk-apigateway/pull/212#issuecomment-304151964 We don't have tests for this because we don't use this we manipulate the underlying data in redis, I ran our full regression suite against

[GitHub] rabbah commented on a change in pull request #2282: Distributed tracing support #2192

2017-05-25 Thread git
rabbah commented on a change in pull request #2282: Distributed tracing support #2192 URL: https://github.com/apache/incubator-openwhisk/pull/2282#discussion_r118606926 ## File path: core/controller/src/main/scala/whisk/core/controller/Controller.scala ## @@ -171,8

[GitHub] rabbah commented on a change in pull request #2282: Distributed tracing support #2192

2017-05-25 Thread git
rabbah commented on a change in pull request #2282: Distributed tracing support #2192 URL: https://github.com/apache/incubator-openwhisk/pull/2282#discussion_r118606926 ## File path: core/controller/src/main/scala/whisk/core/controller/Controller.scala ## @@ -171,8

[GitHub] ddragosd commented on a change in pull request #2282: Distributed tracing support #2192

2017-05-25 Thread git
ddragosd commented on a change in pull request #2282: Distributed tracing support #2192 URL: https://github.com/apache/incubator-openwhisk/pull/2282#discussion_r118605449 ## File path: core/controller/src/main/scala/whisk/core/controller/Controller.scala ## @@ -171,8

[GitHub] rabbah commented on a change in pull request #2282: Distributed tracing support #2192

2017-05-25 Thread git
rabbah commented on a change in pull request #2282: Distributed tracing support #2192 URL: https://github.com/apache/incubator-openwhisk/pull/2282#discussion_r118604808 ## File path: common/scala/src/main/scala/whisk/core/connector/Message.scala ## @@ -53,7 +54,8 @@ case

[GitHub] csantanapr opened a new pull request #2298: temporary pinned apigateway to 0.7.0

2017-05-25 Thread git
csantanapr opened a new pull request #2298: temporary pinned apigateway to 0.7.0 URL: https://github.com/apache/incubator-openwhisk/pull/2298 This is an automated message from the Apache Git Service. To respond to the

[GitHub] ddragosd commented on a change in pull request #2282: Distributed tracing support #2192

2017-05-25 Thread git
ddragosd commented on a change in pull request #2282: Distributed tracing support #2192 URL: https://github.com/apache/incubator-openwhisk/pull/2282#discussion_r118597128 ## File path: core/controller/src/main/scala/whisk/core/controller/Controller.scala ## @@ -171,8

[incubator-openwhisk-apigateway] tag 0.7.0 updated (1ba0f85 -> 1ba0f85)

2017-05-25 Thread csantanapr
This is an automated email from the ASF dual-hosted git repository. csantanapr pushed a change to tag 0.7.0 in repository https://gitbox.apache.org/repos/asf/incubator-openwhisk-apigateway.git. *** WARNING: tag 0.7.0 was modified! *** from 1ba0f85 (commit) to 1ba0f85

[GitHub] mdeuser commented on issue #2297: Are these known heisenbugs in the API gateway tests?

2017-05-25 Thread git
mdeuser commented on issue #2297: Are these known heisenbugs in the API gateway tests? URL: https://github.com/apache/incubator-openwhisk/issues/2297#issuecomment-304136223 Looking into the failure. There was a recent API GW change that may be part of the issue.

[GitHub] rabbah commented on issue #2128: change file name

2017-05-25 Thread git
rabbah commented on issue #2128: change file name URL: https://github.com/apache/incubator-openwhisk/pull/2128#issuecomment-304134304 @csantanapr @mrutkows is there a standard practice for how to name the license file? i see other apache repos use `LICENSE`.

[GitHub] rabbah opened a new issue #2297: Are these known heisenbugs in the API gateway tests?

2017-05-25 Thread git
rabbah opened a new issue #2297: Are these known heisenbugs in the API gateway tests? URL: https://github.com/apache/incubator-openwhisk/issues/2297 @mdeuser? ``` whisk.core.apigw.actions.test.ApiGwRoutemgmtActionTests > API Gateway routemgmt action parameter validation should

[incubator-openwhisk] branch master updated: update travis badge to point to apache (#2276)

2017-05-25 Thread rabbah
This is an automated email from the ASF dual-hosted git repository. rabbah pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-openwhisk.git The following commit(s) were added to refs/heads/master by this push: new 4888f99 update travis badge to

[GitHub] rabbah closed issue #2267: update travis badge in readme to point to apache travis location

2017-05-25 Thread git
rabbah closed issue #2267: update travis badge in readme to point to apache travis location URL: https://github.com/apache/incubator-openwhisk/issues/2267 This is an automated message from the Apache Git Service. To

[GitHub] rabbah closed pull request #2276: update travis badge to point to apache

2017-05-25 Thread git
rabbah closed pull request #2276: update travis badge to point to apache URL: https://github.com/apache/incubator-openwhisk/pull/2276 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] rabbah commented on a change in pull request #2282: Distributed tracing support #2192

2017-05-25 Thread git
rabbah commented on a change in pull request #2282: Distributed tracing support #2192 URL: https://github.com/apache/incubator-openwhisk/pull/2282#discussion_r118590922 ## File path: common/scala/src/main/scala/whisk/core/connector/Message.scala ## @@ -53,7 +54,8 @@ case

[GitHub] rabbah commented on a change in pull request #2282: Distributed tracing support #2192

2017-05-25 Thread git
rabbah commented on a change in pull request #2282: Distributed tracing support #2192 URL: https://github.com/apache/incubator-openwhisk/pull/2282#discussion_r118590441 ## File path: core/controller/src/main/scala/whisk/core/controller/Controller.scala ## @@ -171,8

[GitHub] bjustin-ibm closed pull request #100: Implement provider redundancy

2017-05-25 Thread git
bjustin-ibm closed pull request #100: Implement provider redundancy URL: https://github.com/apache/incubator-openwhisk-package-cloudant/pull/100 This is an automated message from the Apache Git Service. To respond to the

[incubator-openwhisk-package-cloudant] branch master updated: Implement provider redundancy (#100)

2017-05-25 Thread berstler
This is an automated email from the ASF dual-hosted git repository. berstler pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-openwhisk-package-cloudant.git The following commit(s) were added to refs/heads/master by this push: new 58617c5

[incubator-openwhisk-package-alarms] branch master updated: Implement provider redundancy (#62)

2017-05-25 Thread berstler
This is an automated email from the ASF dual-hosted git repository. berstler pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-openwhisk-package-alarms.git The following commit(s) were added to refs/heads/master by this push: new fcd3aea

[GitHub] bjustin-ibm commented on issue #160: HA - redundancy

2017-05-25 Thread git
bjustin-ibm commented on issue #160: HA - redundancy URL: https://github.com/apache/incubator-openwhisk-package-kafka/pull/160#issuecomment-304111257 @csantanapr ready to merge! This is an automated message from the Apache

[GitHub] rabbah closed issue #2295: excessive memory consumption in controller for zip actions

2017-05-25 Thread git
rabbah closed issue #2295: excessive memory consumption in controller for zip actions URL: https://github.com/apache/incubator-openwhisk/issues/2295 This is an automated message from the Apache Git Service. To respond to

[incubator-openwhisk] branch master updated: Controller memory tuning for zip actions: use Regex rather than Base64.decode (#2296)

2017-05-25 Thread rabbah
This is an automated email from the ASF dual-hosted git repository. rabbah pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-openwhisk.git The following commit(s) were added to refs/heads/master by this push: new 097b9e3 Controller memory

[GitHub] rabbah closed pull request #2296: Controller memory tuning for zip actions: use Regex rather than Base64.decode

2017-05-25 Thread git
rabbah closed pull request #2296: Controller memory tuning for zip actions: use Regex rather than Base64.decode URL: https://github.com/apache/incubator-openwhisk/pull/2296 This is an automated message from the Apache Git

[GitHub] csantanapr closed pull request #198: Dao

2017-05-25 Thread git
csantanapr closed pull request #198: Dao URL: https://github.com/apache/incubator-openwhisk-apigateway/pull/198 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and

[incubator-openwhisk-apigateway] branch master updated: data layer (#198)

2017-05-25 Thread csantanapr
This is an automated email from the ASF dual-hosted git repository. csantanapr pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-openwhisk-apigateway.git The following commit(s) were added to refs/heads/master by this push: new 47c3bbe data

[GitHub] taylorking commented on issue #198: Dao

2017-05-25 Thread git
taylorking commented on issue #198: Dao URL: https://github.com/apache/incubator-openwhisk-apigateway/pull/198#issuecomment-304079513 Yes This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] ddragosd commented on issue #198: Dao

2017-05-25 Thread git
ddragosd commented on issue #198: Dao URL: https://github.com/apache/incubator-openwhisk-apigateway/pull/198#issuecomment-304077098 so 4k/sec is the current baseline ? This is an automated message from the Apache Git

[GitHub] bjustin-ibm commented on a change in pull request #152: Verify migration

2017-05-25 Thread git
bjustin-ibm commented on a change in pull request #152: Verify migration URL: https://github.com/apache/incubator-openwhisk-package-kafka/pull/152#discussion_r118541250 ## File path: provider/app.py ## @@ -231,6 +231,8 @@ def main(): enable_generic_kafka =

[GitHub] starpit opened a new pull request #2296: Zip action memory tuning

2017-05-25 Thread git
starpit opened a new pull request #2296: Zip action memory tuning URL: https://github.com/apache/incubator-openwhisk/pull/2296 use a Regex to determine isBinaryCode, rather than going through with a full base64 decoding in the controller. this reduces peak memory consumption, and also

[incubator-openwhisk-cli] annotated tag latest updated (6f959ae -> ac03405)

2017-05-25 Thread csantanapr
This is an automated email from the ASF dual-hosted git repository. csantanapr pushed a change to annotated tag latest in repository https://gitbox.apache.org/repos/asf/incubator-openwhisk-cli.git. *** WARNING: tag latest was modified! *** from 6f959ae (tag) to ac03405 (tag)

[GitHub] houshengbo closed pull request #48: Fix display issue for <>

2017-05-25 Thread git
houshengbo closed pull request #48: Fix display issue for <> URL: https://github.com/apache/incubator-openwhisk-cli/pull/48 This is an automated message from the Apache Git Service. To respond to the message, please log on

[incubator-openwhisk-cli] branch master updated: Fix display issue for <>

2017-05-25 Thread houshengbo
This is an automated email from the ASF dual-hosted git repository. houshengbo pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-openwhisk-cli.git The following commit(s) were added to refs/heads/master by this push: new 94e500a Fix display

[GitHub] houshengbo opened a new pull request #48: Fix display issue for <>

2017-05-25 Thread git
houshengbo opened a new pull request #48: Fix display issue for <> URL: https://github.com/apache/incubator-openwhisk-cli/pull/48 This is an automated message from the Apache Git Service. To respond to the message, please

[incubator-openwhisk-cli] branch master updated: Update the README and CONTRIBUTING

2017-05-25 Thread houshengbo
This is an automated email from the ASF dual-hosted git repository. houshengbo pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-openwhisk-cli.git The following commit(s) were added to refs/heads/master by this push: new bd466a9 Update the

[GitHub] houshengbo opened a new pull request #47: Update the README and CONTRIBUTING

2017-05-25 Thread git
houshengbo opened a new pull request #47: Update the README and CONTRIBUTING URL: https://github.com/apache/incubator-openwhisk-cli/pull/47 This is an automated message from the Apache Git Service. To respond to the

[GitHub] csantanapr commented on a change in pull request #152: Verify migration

2017-05-25 Thread git
csantanapr commented on a change in pull request #152: Verify migration URL: https://github.com/apache/incubator-openwhisk-package-kafka/pull/152#discussion_r118517234 ## File path: provider/app.py ## @@ -231,6 +231,8 @@ def main(): enable_generic_kafka =

[GitHub] csantanapr commented on issue #152: Verify migration

2017-05-25 Thread git
csantanapr commented on issue #152: Verify migration URL: https://github.com/apache/incubator-openwhisk-package-kafka/pull/152#issuecomment-304046105 @bjustin-ibm do you want to merge this PR or close it? This is an

[GitHub] csantanapr commented on issue #88: Add the CouchDB support

2017-05-25 Thread git
csantanapr commented on issue #88: Add the CouchDB support URL: https://github.com/apache/incubator-openwhisk-package-kafka/pull/88#issuecomment-304045947 @bjustin-ibm do we want to merge this? If so @houshengbo would need to resolve the conflict

[GitHub] csantanapr closed pull request #173: test pr write access

2017-05-25 Thread git
csantanapr closed pull request #173: test pr write access URL: https://github.com/apache/incubator-openwhisk-package-kafka/pull/173 This is an automated message from the Apache Git Service. To respond to the message, please

[incubator-openwhisk-apigateway] branch master updated: Fix example swagger for v2 interface (#203)

2017-05-25 Thread csantanapr
This is an automated email from the ASF dual-hosted git repository. csantanapr pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-openwhisk-apigateway.git The following commit(s) were added to refs/heads/master by this push: new ca7dc6e Fix

[GitHub] csantanapr closed pull request #174: Ensure trigger payload is within specified size limit

2017-05-25 Thread git
csantanapr closed pull request #174: Ensure trigger payload is within specified size limit URL: https://github.com/apache/incubator-openwhisk-package-kafka/pull/174 This is an automated message from the Apache Git Service.

[GitHub] bjustin-ibm commented on issue #174: Ensure trigger payload is within specified size limit

2017-05-25 Thread git
bjustin-ibm commented on issue #174: Ensure trigger payload is within specified size limit URL: https://github.com/apache/incubator-openwhisk-package-kafka/pull/174#issuecomment-304033801 @csantanapr Ok done. Please merge when Travis finishes.

[GitHub] bjustin-ibm commented on issue #174: Ensure trigger payload is within specified size limit

2017-05-25 Thread git
bjustin-ibm commented on issue #174: Ensure trigger payload is within specified size limit URL: https://github.com/apache/incubator-openwhisk-package-kafka/pull/174#issuecomment-304033279 @csantanapr Actually... give me a minute to manually squash these commits.

[GitHub] bjustin-ibm commented on issue #174: Ensure trigger payload is within specified size limit

2017-05-25 Thread git
bjustin-ibm commented on issue #174: Ensure trigger payload is within specified size limit URL: https://github.com/apache/incubator-openwhisk-package-kafka/pull/174#issuecomment-304032801 @csantanapr please merge. This is

[incubator-openwhisk-deploy-kube] branch master updated (21dcca4 -> 42397f9)

2017-05-25 Thread mrutkowski
This is an automated email from the ASF dual-hosted git repository. mrutkowski pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/incubator-openwhisk-deploy-kube.git. from 21dcca4 Update README.md adds ad76b53 build kube 1.5.6 and 1.6.2 images

[incubator-openwhisk-deploy-kube] 01/01: Merge pull request #2 from DanLavine/kube-1.6

2017-05-25 Thread mrutkowski
This is an automated email from the ASF dual-hosted git repository. mrutkowski pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-openwhisk-deploy-kube.git commit 42397f9c9158c51c63c1967e3f5b749361d22961 Merge: 21dcca4 ad76b53 Author: Matt Rutkowski

[GitHub] Katiyman commented on issue #99: Document how to Build and Deploy

2017-05-25 Thread git
Katiyman commented on issue #99: Document how to Build and Deploy URL: https://github.com/apache/incubator-openwhisk-package-kafka/issues/99#issuecomment-303967556 In the above intructions. I request one clarification from you @bjustin-ibm that if my local openwhisk setup is using a

[GitHub] Katiyman commented on issue #99: Document how to Build and Deploy

2017-05-25 Thread git
Katiyman commented on issue #99: Document how to Build and Deploy URL: https://github.com/apache/incubator-openwhisk-package-kafka/issues/99#issuecomment-303967556 In the above intructions. I request one calrification from you @bjustin-ibm that if my local openwhisk setup is using a

[GitHub] Katiyman commented on issue #99: Document how to Build and Deploy

2017-05-25 Thread git
Katiyman commented on issue #99: Document how to Build and Deploy URL: https://github.com/apache/incubator-openwhisk-package-kafka/issues/99#issuecomment-303967556 In the above intructions. I request one calrification from you @bjustin-ibm that if my local openwhisk setup is using a

[GitHub] pritidesai opened a new pull request #275: Fixing zip action creation

2017-05-25 Thread git
pritidesai opened a new pull request #275: Fixing zip action creation URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/275 Base64 encoding zipfile content so that action payload is sent in expected format via HTTP request. Also, added integration test for a simple