[GitHub] codecov-io edited a comment on issue #3814: wskadmin scala db command to export and import

2018-07-30 Thread GitBox
codecov-io edited a comment on issue #3814: wskadmin scala db command to export and import URL: https://github.com/apache/incubator-openwhisk/pull/3814#issuecomment-400666862 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3814?src=pr=h1) Report > Merging

[GitHub] chetanmeh commented on issue #3450: Introduce a AttachmentStore SPI

2018-07-30 Thread GitBox
chetanmeh commented on issue #3450: Introduce a AttachmentStore SPI URL: https://github.com/apache/incubator-openwhisk/issues/3450#issuecomment-409095830 All planned work for this issue done. Closing the issue This is an

[GitHub] chetanmeh closed issue #3450: Introduce a AttachmentStore SPI

2018-07-30 Thread GitBox
chetanmeh closed issue #3450: Introduce a AttachmentStore SPI URL: https://github.com/apache/incubator-openwhisk/issues/3450 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] chetanmeh closed issue #3477: Support extending the environment variables for docker in ansible

2018-07-30 Thread GitBox
chetanmeh closed issue #3477: Support extending the environment variables for docker in ansible URL: https://github.com/apache/incubator-openwhisk/issues/3477 This is an automated message from the Apache Git Service. To

[GitHub] KeonHee commented on issue #155: Encode database authentication information for issue with special characters

2018-07-30 Thread GitBox
KeonHee commented on issue #155: Encode database authentication information for issue with special characters URL: https://github.com/apache/incubator-openwhisk-package-alarms/pull/155#issuecomment-409089108 @jasonpet I modified the existing API to use it as it is :)

[GitHub] jasonpet closed issue #152: Why is the body of web action response base64 string, not object?

2018-07-30 Thread GitBox
jasonpet closed issue #152: Why is the body of web action response base64 string, not object? URL: https://github.com/apache/incubator-openwhisk-package-alarms/issues/152 This is an automated message from the Apache Git

[GitHub] jasonpet closed pull request #156: remove the use of base64 for webaction response

2018-07-30 Thread GitBox
jasonpet closed pull request #156: remove the use of base64 for webaction response URL: https://github.com/apache/incubator-openwhisk-package-alarms/pull/156 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 #2262: documentation for how to debug

2018-07-30 Thread GitBox
rabbah commented on issue #2262: documentation for how to debug URL: https://github.com/apache/incubator-openwhisk/issues/2262#issuecomment-409063818 Thank you @danidiez. Do you need pointers? This is an automated message

[GitHub] rabbah commented on issue #9: reduce system log output inside function

2018-07-30 Thread GitBox
rabbah commented on issue #9: reduce system log output inside function URL: https://github.com/apache/incubator-openwhisk-runtime-ruby/issues/9#issuecomment-409057965 It bleeds through the webserver elements underlying the function. It would be nice to eschew as such. What about using

[GitHub] remore edited a comment on issue #9: reduce system log output inside function

2018-07-30 Thread GitBox
remore edited a comment on issue #9: reduce system log output inside function URL: https://github.com/apache/incubator-openwhisk-runtime-ruby/issues/9#issuecomment-409057212 @rabbah it looks like there is no good way to prevent it.

[GitHub] remore commented on issue #9: reduce system log output inside function

2018-07-30 Thread GitBox
remore commented on issue #9: reduce system log output inside function URL: https://github.com/apache/incubator-openwhisk-runtime-ruby/issues/9#issuecomment-409057212 @rabbah it looks like there is no way to prevent it.

[GitHub] rabbah commented on a change in pull request #3919: Allow raw JSON string as response for webaction without base64 encoding.

2018-07-30 Thread GitBox
rabbah commented on a change in pull request #3919: Allow raw JSON string as response for webaction without base64 encoding. URL: https://github.com/apache/incubator-openwhisk/pull/3919#discussion_r206359836 ## File path:

[GitHub] rabbah commented on a change in pull request #3919: Allow raw JSON string as response for webaction without base64 encoding.

2018-07-30 Thread GitBox
rabbah commented on a change in pull request #3919: Allow raw JSON string as response for webaction without base64 encoding. URL: https://github.com/apache/incubator-openwhisk/pull/3919#discussion_r206359467 ## File path:

[GitHub] dgrove-oss commented on issue #263: virtual box must be installed for minikube

2018-07-30 Thread GitBox
dgrove-oss commented on issue #263: virtual box must be installed for minikube URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/263#issuecomment-409049457 I guess its a question of how much of the minikube documentation we want to replicate vs. incorporate by

[GitHub] dubee commented on a change in pull request #3919: Allow raw JSON string as response for webaction without base64 encoding.

2018-07-30 Thread GitBox
dubee commented on a change in pull request #3919: Allow raw JSON string as response for webaction without base64 encoding. URL: https://github.com/apache/incubator-openwhisk/pull/3919#discussion_r206355118 ## File path:

[GitHub] dubee commented on a change in pull request #3919: Allow raw JSON string as response for webaction without base64 encoding.

2018-07-30 Thread GitBox
dubee commented on a change in pull request #3919: Allow raw JSON string as response for webaction without base64 encoding. URL: https://github.com/apache/incubator-openwhisk/pull/3919#discussion_r206355118 ## File path:

[GitHub] dubee commented on a change in pull request #3919: Allow raw JSON string as response for webaction without base64 encoding.

2018-07-30 Thread GitBox
dubee commented on a change in pull request #3919: Allow raw JSON string as response for webaction without base64 encoding. URL: https://github.com/apache/incubator-openwhisk/pull/3919#discussion_r206353473 ## File path:

[GitHub] rabbah commented on a change in pull request #41: remove use of state in nodejs app

2018-07-30 Thread GitBox
rabbah commented on a change in pull request #41: remove use of state in nodejs app URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/41#discussion_r206352786 ## File path: core/nodejsActionBase/src/service.js ## @@ -112,10 +113,14 @@ function

[GitHub] rabbah commented on a change in pull request #41: remove use of state in nodejs app

2018-07-30 Thread GitBox
rabbah commented on a change in pull request #41: remove use of state in nodejs app URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/41#discussion_r206352786 ## File path: core/nodejsActionBase/src/service.js ## @@ -112,10 +113,14 @@ function

[GitHub] rabbah closed pull request #3799: Display proper error when sequence invocation fails due to missing component

2018-07-30 Thread GitBox
rabbah closed pull request #3799: Display proper error when sequence invocation fails due to missing component URL: https://github.com/apache/incubator-openwhisk/pull/3799 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for

[GitHub] rabbah commented on issue #3799: Display proper error when sequence invocation fails due to missing component

2018-07-30 Thread GitBox
rabbah commented on issue #3799: Display proper error when sequence invocation fails due to missing component URL: https://github.com/apache/incubator-openwhisk/pull/3799#issuecomment-409042516 Thanks for looking into that.

[GitHub] rabbah commented on issue #3919: Allow raw JSON string as response for webaction without base64 encoding.

2018-07-30 Thread GitBox
rabbah commented on issue #3919: Allow raw JSON string as response for webaction without base64 encoding. URL: https://github.com/apache/incubator-openwhisk/pull/3919#issuecomment-409042289 API gateway actions need to be updated @csantanapr.

[GitHub] dubee commented on issue #3799: Display proper error when sequence invocation fails due to missing component

2018-07-30 Thread GitBox
dubee commented on issue #3799: Display proper error when sequence invocation fails due to missing component URL: https://github.com/apache/incubator-openwhisk/pull/3799#issuecomment-409039708 @rabbah, how about I add the component names in an separate PR? Looks like there needs to be

[GitHub] tysonnorris commented on a change in pull request #41: remove use of state in nodejs app

2018-07-30 Thread GitBox
tysonnorris commented on a change in pull request #41: remove use of state in nodejs app URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/41#discussion_r206345903 ## File path: core/nodejsActionBase/app.js ## @@ -17,7 +17,8 @@ var config = {

[GitHub] codecov-io edited a comment on issue #3913: Bump kafka-clients version to 2.0.0.

2018-07-30 Thread GitBox
codecov-io edited a comment on issue #3913: Bump kafka-clients version to 2.0.0. URL: https://github.com/apache/incubator-openwhisk/pull/3913#issuecomment-408813148 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3913?src=pr=h1) Report > Merging

[GitHub] tysonnorris commented on a change in pull request #41: remove use of state in nodejs app

2018-07-30 Thread GitBox
tysonnorris commented on a change in pull request #41: remove use of state in nodejs app URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/41#discussion_r206345221 ## File path: core/nodejsActionBase/src/service.js ## @@ -112,10 +113,14 @@ function

[GitHub] csantanapr edited a comment on issue #3919: Allow raw JSON string as response for webaction without base64 encoding.

2018-07-30 Thread GitBox
csantanapr edited a comment on issue #3919: Allow raw JSON string as response for webaction without base64 encoding. URL: https://github.com/apache/incubator-openwhisk/pull/3919#issuecomment-409033615 This is a feature request from an user that is calling an external API, this API

[GitHub] csantanapr edited a comment on issue #3919: Allow raw JSON string as response for webaction without base64 encoding.

2018-07-30 Thread GitBox
csantanapr edited a comment on issue #3919: Allow raw JSON string as response for webaction without base64 encoding. URL: https://github.com/apache/incubator-openwhisk/pull/3919#issuecomment-409033615 This is a feature request from an end user that is calling an external API, this API

[GitHub] csantanapr commented on issue #3919: Allow raw JSON string as response for webaction without base64 encoding.

2018-07-30 Thread GitBox
csantanapr commented on issue #3919: Allow raw JSON string as response for webaction without base64 encoding. URL: https://github.com/apache/incubator-openwhisk/pull/3919#issuecomment-409033615 This change will allow the following to work, this is a feature request from an user that is

[GitHub] csantanapr edited a comment on issue #3919: Allow raw JSON string as response for webaction without base64 encoding.

2018-07-30 Thread GitBox
csantanapr edited a comment on issue #3919: Allow raw JSON string as response for webaction without base64 encoding. URL: https://github.com/apache/incubator-openwhisk/pull/3919#issuecomment-409033615 This is a feature request from an end user that is calling an external API, this API

[GitHub] csantanapr opened a new pull request #176: remove the use of base64 for webaction response

2018-07-30 Thread GitBox
csantanapr opened a new pull request #176: remove the use of base64 for webaction response URL: https://github.com/apache/incubator-openwhisk-package-cloudant/pull/176 This is an automated message from the Apache Git

[GitHub] csantanapr opened a new pull request #156: remove the use of base64 for webaction response

2018-07-30 Thread GitBox
csantanapr opened a new pull request #156: remove the use of base64 for webaction response URL: https://github.com/apache/incubator-openwhisk-package-alarms/pull/156 Fixes #152 This is an automated message from the Apache

[GitHub] tysonnorris commented on issue #3918: on "application error", return success=false, but should not destroy the container

2018-07-30 Thread GitBox
tysonnorris commented on issue #3918: on "application error", return success=false, but should not destroy the container URL: https://github.com/apache/incubator-openwhisk/issues/3918#issuecomment-409030389 #645 is different - I'm not suggesting changing the behavior of "what status is

[GitHub] rabbah opened a new issue #3920: do not base64 encode __ow_body for raw web actions

2018-07-30 Thread GitBox
rabbah opened a new issue #3920: do not base64 encode __ow_body for raw web actions URL: https://github.com/apache/incubator-openwhisk/issues/3920 #3919 removes the base64 encoding requirement when returning JSON as string (and setting the content-type). However we continue to encode JSON

[GitHub] rabbah commented on issue #3919: Allow raw JSON string as response for webaction without base64 encoding.

2018-07-30 Thread GitBox
rabbah commented on issue #3919: Allow raw JSON string as response for webaction without base64 encoding. URL: https://github.com/apache/incubator-openwhisk/pull/3919#issuecomment-409027066 Note that we continue to encode JSON for raw http actions receiving `__ow_body`. For symmetry, we

[GitHub] rabbah opened a new pull request #3919: Allow raw JSON string as response for webaction without base64 encoding.

2018-07-30 Thread GitBox
rabbah opened a new pull request #3919: Allow raw JSON string as response for webaction without base64 encoding. URL: https://github.com/apache/incubator-openwhisk/pull/3919 For legacy reasons, we required web actions to return a base 64 encoded string when returning JSON. This PR removes

[GitHub] dubee commented on issue #3799: Display proper error when sequence invocation fails due to missing component

2018-07-30 Thread GitBox
dubee commented on issue #3799: Display proper error when sequence invocation fails due to missing component URL: https://github.com/apache/incubator-openwhisk/pull/3799#issuecomment-409023711 @rabbah, makes sense to add the component name. I'll do some refactoring.

[GitHub] markusthoemmes commented on issue #3918: on "application error", return success=false, but should not destroy the container

2018-07-30 Thread GitBox
markusthoemmes commented on issue #3918: on "application error", return success=false, but should not destroy the container URL: https://github.com/apache/incubator-openwhisk/issues/3918#issuecomment-409021676 See #645 for the "old" discussion on this.

[GitHub] markusthoemmes commented on issue #3918: on "application error", return success=false, but should not destroy the container

2018-07-30 Thread GitBox
markusthoemmes commented on issue #3918: on "application error", return success=false, but should not destroy the container URL: https://github.com/apache/incubator-openwhisk/issues/3918#issuecomment-409021397 I think, on a broader perspective, this might also be related to our general

[GitHub] tysonnorris edited a comment on issue #3918: on "application error", return success=false, but should not destroy the container

2018-07-30 Thread GitBox
tysonnorris edited a comment on issue #3918: on "application error", return success=false, but should not destroy the container URL: https://github.com/apache/incubator-openwhisk/issues/3918#issuecomment-409014049 Any objection to changing this behavior? I'm not sure what the

[GitHub] tysonnorris commented on issue #3918: on "application error", return success=false, but should not destroy the container

2018-07-30 Thread GitBox
tysonnorris commented on issue #3918: on "application error", return success=false, but should not destroy the container URL: https://github.com/apache/incubator-openwhisk/issues/3918#issuecomment-409014049 Any objection to changing this behavior? I'm not sure what the considerations

[GitHub] codecov-io edited a comment on issue #3661: Restrict allowed namespaces when creating action of certain kinds

2018-07-30 Thread GitBox
codecov-io edited a comment on issue #3661: Restrict allowed namespaces when creating action of certain kinds URL: https://github.com/apache/incubator-openwhisk/pull/3661#issuecomment-393731610 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3661?src=pr=h1) Report >

[GitHub] codecov-io edited a comment on issue #3661: Restrict allowed namespaces when creating action of certain kinds

2018-07-30 Thread GitBox
codecov-io edited a comment on issue #3661: Restrict allowed namespaces when creating action of certain kinds URL: https://github.com/apache/incubator-openwhisk/pull/3661#issuecomment-393731610 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3661?src=pr=h1) Report >

[GitHub] codecov-io edited a comment on issue #3910: Allow non-json skip_pull_runtimes to be passed as True.

2018-07-30 Thread GitBox
codecov-io edited a comment on issue #3910: Allow non-json skip_pull_runtimes to be passed as True. URL: https://github.com/apache/incubator-openwhisk/pull/3910#issuecomment-408682227 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3910?src=pr=h1) Report > Merging

[GitHub] codecov-io edited a comment on issue #3760: Add API gateway to redo.

2018-07-30 Thread GitBox
codecov-io edited a comment on issue #3760: Add API gateway to redo. URL: https://github.com/apache/incubator-openwhisk/pull/3760#issuecomment-397050317 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3760?src=pr=h1) Report > Merging

[GitHub] codecov-io edited a comment on issue #3903: Manually create user log files

2018-07-30 Thread GitBox
codecov-io edited a comment on issue #3903: Manually create user log files URL: https://github.com/apache/incubator-openwhisk/pull/3903#issuecomment-407863054 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3903?src=pr=h1) Report > Merging

[GitHub] codecov-io edited a comment on issue #3903: Manually create user log files

2018-07-30 Thread GitBox
codecov-io edited a comment on issue #3903: Manually create user log files URL: https://github.com/apache/incubator-openwhisk/pull/3903#issuecomment-407863054 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3903?src=pr=h1) Report > Merging

[GitHub] rabbah closed issue #2572: Deploying OpenWhisk with Docker for Mac fails at apigateway.yml ansible step

2018-07-30 Thread GitBox
rabbah closed issue #2572: Deploying OpenWhisk with Docker for Mac fails at apigateway.yml ansible step URL: https://github.com/apache/incubator-openwhisk/issues/2572 This is an automated message from the Apache Git

[GitHub] codecov-io edited a comment on issue #3909: Refactor sentinel message into shared place.

2018-07-30 Thread GitBox
codecov-io edited a comment on issue #3909: Refactor sentinel message into shared place. URL: https://github.com/apache/incubator-openwhisk/pull/3909#issuecomment-408649977 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3909?src=pr=h1) Report > Merging

[GitHub] rabbah commented on issue #3799: Display proper error when sequence invocation fails due to missing component

2018-07-30 Thread GitBox
rabbah commented on issue #3799: Display proper error when sequence invocation fails due to missing component URL: https://github.com/apache/incubator-openwhisk/pull/3799#issuecomment-408986090 @dubee thanks for the improve test. one last thought: perhaps the error message should

[GitHub] rabbah closed pull request #3725: Add Ruby2.5 runtime support

2018-07-30 Thread GitBox
rabbah closed pull request #3725: Add Ruby2.5 runtime support URL: https://github.com/apache/incubator-openwhisk/pull/3725 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 pull

[GitHub] codecov-io edited a comment on issue #3725: Add Ruby2.5 runtime support

2018-07-30 Thread GitBox
codecov-io edited a comment on issue #3725: Add Ruby2.5 runtime support URL: https://github.com/apache/incubator-openwhisk/pull/3725#issuecomment-397487604 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3725?src=pr=h1) Report > Merging

[GitHub] codecov-io edited a comment on issue #3725: Add Ruby2.5 runtime support

2018-07-30 Thread GitBox
codecov-io edited a comment on issue #3725: Add Ruby2.5 runtime support URL: https://github.com/apache/incubator-openwhisk/pull/3725#issuecomment-397487604 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3725?src=pr=h1) Report > Merging

[GitHub] rabbah closed pull request #3914: Delete unused files in whisk.common.

2018-07-30 Thread GitBox
rabbah closed pull request #3914: Delete unused files in whisk.common. URL: https://github.com/apache/incubator-openwhisk/pull/3914 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 commented on a change in pull request #3909: Refactor sentinel message into shared place.

2018-07-30 Thread GitBox
rabbah commented on a change in pull request #3909: Refactor sentinel message into shared place. URL: https://github.com/apache/incubator-openwhisk/pull/3909#discussion_r206280563 ## File path: common/scala/src/main/scala/whisk/core/containerpool/Container.scala ## @@

[GitHub] rabbah opened a new issue #9: reduce system log output inside function

2018-07-30 Thread GitBox
rabbah opened a new issue #9: reduce system log output inside function URL: https://github.com/apache/incubator-openwhisk-runtime-ruby/issues/9 @remore is it possible to suppress this output? ``` "2018-07-30T18:35:08.842818218Z stdout: Puma starting in single mode...",

[GitHub] dubee commented on a change in pull request #3903: Manually create user log files

2018-07-30 Thread GitBox
dubee commented on a change in pull request #3903: Manually create user log files URL: https://github.com/apache/incubator-openwhisk/pull/3903#discussion_r206275476 ## File path: common/scala/src/main/scala/whisk/core/containerpool/logging/DockerToActivationFileLogStore.scala

[GitHub] dubee commented on a change in pull request #3903: Manually create user log files

2018-07-30 Thread GitBox
dubee commented on a change in pull request #3903: Manually create user log files URL: https://github.com/apache/incubator-openwhisk/pull/3903#discussion_r206275476 ## File path: common/scala/src/main/scala/whisk/core/containerpool/logging/DockerToActivationFileLogStore.scala

[GitHub] dubee commented on a change in pull request #3903: Manually create user log files

2018-07-30 Thread GitBox
dubee commented on a change in pull request #3903: Manually create user log files URL: https://github.com/apache/incubator-openwhisk/pull/3903#discussion_r206275476 ## File path: common/scala/src/main/scala/whisk/core/containerpool/logging/DockerToActivationFileLogStore.scala

[GitHub] rabbah closed pull request #4: encrypt slackbot URL using this repo's key

2018-07-30 Thread GitBox
rabbah closed pull request #4: encrypt slackbot URL using this repo's key URL: https://github.com/apache/incubator-openwhisk-runtime-ballerina/pull/4 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] rabbah commented on issue #4: encrypt slackbot URL using this repo's key

2018-07-30 Thread GitBox
rabbah commented on issue #4: encrypt slackbot URL using this repo's key URL: https://github.com/apache/incubator-openwhisk-runtime-ballerina/pull/4#issuecomment-408961371 Thanks @dgrove-oss. This is an automated message

[GitHub] dgrove-oss commented on issue #4: encrypt slackbot URL using this repo's key

2018-07-30 Thread GitBox
dgrove-oss commented on issue #4: encrypt slackbot URL using this repo's key URL: https://github.com/apache/incubator-openwhisk-runtime-ballerina/pull/4#issuecomment-408959721 ![screen shot 2018-07-30 at 2 15 07

[GitHub] markusthoemmes commented on issue #3894: Refactor invoker command line parsing.

2018-07-30 Thread GitBox
markusthoemmes commented on issue #3894: Refactor invoker command line parsing. URL: https://github.com/apache/incubator-openwhisk/pull/3894#issuecomment-408959334 PG3 2586  This is an automated message from the Apache Git

[GitHub] rabbah closed pull request #8: encrypt slackbot URL for proper repo

2018-07-30 Thread GitBox
rabbah closed pull request #8: encrypt slackbot URL for proper repo URL: https://github.com/apache/incubator-openwhisk-runtime-ruby/pull/8 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] markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk

2018-07-30 Thread GitBox
markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3886#discussion_r206266167 ## File path: core/lean/src/main/scala/whisk/core/loadBalancer/LeanBalancer.scala ## @@ -0,0 +1,256 @@

[GitHub] markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk

2018-07-30 Thread GitBox
markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3886#discussion_r206262909 ## File path: common/scala/src/main/scala/whisk/connector/lean/LeanProducer.scala ## @@ -0,0 +1,60 @@

[GitHub] markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk

2018-07-30 Thread GitBox
markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3886#discussion_r206266120 ## File path: core/lean/src/main/scala/whisk/core/loadBalancer/LeanBalancer.scala ## @@ -0,0 +1,256 @@

[GitHub] markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk

2018-07-30 Thread GitBox
markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3886#discussion_r206266249 ## File path: core/lean/src/main/scala/whisk/core/loadBalancer/LeanBalancer.scala ## @@ -0,0 +1,256 @@

[GitHub] markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk

2018-07-30 Thread GitBox
markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3886#discussion_r206266039 ## File path: core/lean/src/main/scala/whisk/core/loadBalancer/LeanBalancer.scala ## @@ -0,0 +1,256 @@

[GitHub] markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk

2018-07-30 Thread GitBox
markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3886#discussion_r206259882 ## File path: common/scala/src/main/scala/whisk/connector/lean/LeanProducer.scala ## @@ -0,0 +1,60 @@

[GitHub] markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk

2018-07-30 Thread GitBox
markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3886#discussion_r20626 ## File path: ansible/roles/lean/tasks/deploy.yml ## @@ -0,0 +1,265 @@ +# Licensed to the Apache

[GitHub] markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk

2018-07-30 Thread GitBox
markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3886#discussion_r206266383 ## File path: core/lean/src/main/scala/whisk/core/loadBalancer/LeanBalancer.scala ## @@ -0,0 +1,256 @@

[GitHub] markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk

2018-07-30 Thread GitBox
markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3886#discussion_r206263733 ## File path: core/lean/src/main/resources/reference.conf ## @@ -0,0 +1,15 @@ +# Licensed to the Apache

[GitHub] markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk

2018-07-30 Thread GitBox
markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3886#discussion_r206266556 ## File path: tests/build.gradle ## @@ -64,21 +77,14 @@ ext.testSets = [ ] ],

[GitHub] markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk

2018-07-30 Thread GitBox
markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3886#discussion_r206260789 ## File path: core/lean/src/main/scala/whisk/core/loadBalancer/LeanBalancer.scala ## @@ -0,0 +1,256 @@

[GitHub] markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk

2018-07-30 Thread GitBox
markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3886#discussion_r206263484 ## File path: core/lean/build.gradle ## @@ -0,0 +1,31 @@ +apply plugin: 'scala' +apply plugin:

[GitHub] markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk

2018-07-30 Thread GitBox
markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3886#discussion_r206262629 ## File path: common/scala/src/main/scala/whisk/connector/lean/LeanConsumer.scala ## @@ -0,0 +1,49 @@

[GitHub] markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk

2018-07-30 Thread GitBox
markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3886#discussion_r206266760 ## File path: core/lean/src/main/scala/whisk/core/loadBalancer/LeanBalancer.scala ## @@ -0,0 +1,256 @@

[GitHub] markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk

2018-07-30 Thread GitBox
markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3886#discussion_r206259234 ## File path: common/scala/src/main/scala/whisk/connector/lean/LeanMessagingProvider.scala ## @@ -0,0

[GitHub] markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk

2018-07-30 Thread GitBox
markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3886#discussion_r206260032 ## File path: common/scala/src/main/scala/whisk/connector/lean/LeanProducer.scala ## @@ -0,0 +1,60 @@

[GitHub] markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk

2018-07-30 Thread GitBox
markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3886#discussion_r206267591 ## File path: ansible/tasks/invoker_lean.yml ## @@ -0,0 +1,141 @@ +# Licensed to the Apache Software

[GitHub] markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk

2018-07-30 Thread GitBox
markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3886#discussion_r206263781 ## File path: core/lean/init.sh ## @@ -0,0 +1,10 @@ +#!/bin/bash + +# Licensed to the Apache Software

[GitHub] markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk

2018-07-30 Thread GitBox
markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3886#discussion_r206266961 ## File path: core/lean/src/main/scala/whisk/core/loadBalancer/LeanBalancer.scala ## @@ -0,0 +1,256 @@

[GitHub] markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk

2018-07-30 Thread GitBox
markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3886#discussion_r206259429 ## File path: common/scala/src/main/scala/whisk/connector/lean/LeanMessagingProvider.scala ## @@ -0,0

[GitHub] markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk

2018-07-30 Thread GitBox
markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3886#discussion_r206263282 ## File path: core/lean/build.gradle ## @@ -0,0 +1,31 @@ +apply plugin: 'scala' +apply plugin:

[GitHub] markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk

2018-07-30 Thread GitBox
markusthoemmes commented on a change in pull request #3886: Proposing Lean OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3886#discussion_r206259823 ## File path: common/scala/src/main/scala/whisk/connector/lean/LeanProducer.scala ## @@ -0,0 +1,60 @@

[GitHub] dgrove-oss opened a new pull request #4: encrypt slackbot URL using this repo's key

2018-07-30 Thread GitBox
dgrove-oss opened a new pull request #4: encrypt slackbot URL using this repo's key URL: https://github.com/apache/incubator-openwhisk-runtime-ballerina/pull/4 This is an automated message from the Apache Git Service. To

[GitHub] dgrove-oss commented on issue #8: encrypt slackbot URL for proper repo

2018-07-30 Thread GitBox
dgrove-oss commented on issue #8: encrypt slackbot URL for proper repo URL: https://github.com/apache/incubator-openwhisk-runtime-ruby/pull/8#issuecomment-408954150 ![screen shot 2018-07-30 at 1 56 32

[GitHub] dgrove-oss removed a comment on issue #8: encrypt slackbot URL for proper repo

2018-07-30 Thread GitBox
dgrove-oss removed a comment on issue #8: encrypt slackbot URL for proper repo URL: https://github.com/apache/incubator-openwhisk-runtime-ruby/pull/8#issuecomment-408953867 https://user-images.githubusercontent.com/5099100/43414343-7adf8538-9400-11e8-82ff-2300451b503f.png;>

[GitHub] dgrove-oss edited a comment on issue #8: encrypt slackbot URL for proper repo

2018-07-30 Thread GitBox
dgrove-oss edited a comment on issue #8: encrypt slackbot URL for proper repo URL: https://github.com/apache/incubator-openwhisk-runtime-ruby/pull/8#issuecomment-408953867 https://user-images.githubusercontent.com/5099100/43414343-7adf8538-9400-11e8-82ff-2300451b503f.png;>

[GitHub] dgrove-oss commented on issue #8: encrypt slackbot URL for proper repo

2018-07-30 Thread GitBox
dgrove-oss commented on issue #8: encrypt slackbot URL for proper repo URL: https://github.com/apache/incubator-openwhisk-runtime-ruby/pull/8#issuecomment-408953867 ![screen shot 2018-07-30 at 1 56 32

[GitHub] rabbah commented on issue #3918: on "application error", return success=false, but should not destroy the container

2018-07-30 Thread GitBox
rabbah commented on issue #3918: on "application error", return success=false, but should not destroy the container URL: https://github.com/apache/incubator-openwhisk/issues/3918#issuecomment-408951402 This is by design - all errors are treated equally and destroy the container. You make

[GitHub] rabbah closed pull request #3915: Fix misspelled or wrongly named types.

2018-07-30 Thread GitBox
rabbah closed pull request #3915: Fix misspelled or wrongly named types. URL: https://github.com/apache/incubator-openwhisk/pull/3915 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] tysonnorris opened a new issue #3918: on "application error", return success=false, but should not destroy the container

2018-07-30 Thread GitBox
tysonnorris opened a new issue #3918: on "application error", return success=false, but should not destroy the container URL: https://github.com/apache/incubator-openwhisk/issues/3918 ## Environment details: * local deployment ## Steps to reproduce the issue:

[GitHub] dgrove-oss opened a new pull request #8: encrypt slackbot URL for proper repo

2018-07-30 Thread GitBox
dgrove-oss opened a new pull request #8: encrypt slackbot URL for proper repo URL: https://github.com/apache/incubator-openwhisk-runtime-ruby/pull/8 This is an automated message from the Apache Git Service. To respond to the

[GitHub] codecov-io edited a comment on issue #3799: Display proper error when sequence invocation fails due to missing component

2018-07-30 Thread GitBox
codecov-io edited a comment on issue #3799: Display proper error when sequence invocation fails due to missing component URL: https://github.com/apache/incubator-openwhisk/pull/3799#issuecomment-399295422 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3799?src=pr=h1)

[GitHub] dubee commented on issue #3799: Display proper error when sequence invocation fails due to missing component

2018-07-30 Thread GitBox
dubee commented on issue #3799: Display proper error when sequence invocation fails due to missing component URL: https://github.com/apache/incubator-openwhisk/pull/3799#issuecomment-408940212 @rabbah, @markusthoemmes, thanks for all the feedback!!!

[GitHub] mdeuser commented on issue #3907: Fix for exception being thrown during api creation

2018-07-30 Thread GitBox
mdeuser commented on issue #3907: Fix for exception being thrown during api creation URL: https://github.com/apache/incubator-openwhisk/pull/3907#issuecomment-408940157 pg2/3426 :hourglass_flowing_sand: This is an

[GitHub] rabbah commented on issue #3: repo fix up

2018-07-30 Thread GitBox
rabbah commented on issue #3: repo fix up URL: https://github.com/apache/incubator-openwhisk-runtime-ballerina/issues/3#issuecomment-408938655 Need to encrypt travis still and noticed the travis badge is incorrect: https://travis-ci.org/apache/incubator-openwhisk-runtime-ballerina is the

[GitHub] rabbah opened a new issue #3: repo fix up

2018-07-30 Thread GitBox
rabbah opened a new issue #3: repo fix up URL: https://github.com/apache/incubator-openwhisk-runtime-ballerina/issues/3 in .travis.yml - [x] wrong repo name - [ ] re-encrypt docker hub credentials - [x] fix publishing/image name fix image name in - [x] readme.md ` wsk

  1   2   >