[GitHub] markusthoemmes commented on a change in pull request #3941: to address #3918, reuse a container on applicationError

2018-08-06 Thread GitBox
markusthoemmes commented on a change in pull request #3941: to address #3918, reuse a container on applicationError URL: https://github.com/apache/incubator-openwhisk/pull/3941#discussion_r207806144 ## File path:

[GitHub] sven-lange-last commented on a change in pull request #3921: Synchronize access to the KafkaConsumer.

2018-08-06 Thread GitBox
sven-lange-last commented on a change in pull request #3921: Synchronize access to the KafkaConsumer. URL: https://github.com/apache/incubator-openwhisk/pull/3921#discussion_r207787705 ## File path: common/scala/src/main/scala/whisk/connector/kafka/KafkaConsumerConnector.scala

[GitHub] sven-lange-last commented on a change in pull request #3921: Synchronize access to the KafkaConsumer.

2018-08-06 Thread GitBox
sven-lange-last commented on a change in pull request #3921: Synchronize access to the KafkaConsumer. URL: https://github.com/apache/incubator-openwhisk/pull/3921#discussion_r207787705 ## File path: common/scala/src/main/scala/whisk/connector/kafka/KafkaConsumerConnector.scala

[GitHub] chetanmeh closed pull request #3921: Synchronize access to the KafkaConsumer.

2018-08-06 Thread GitBox
chetanmeh closed pull request #3921: Synchronize access to the KafkaConsumer. URL: https://github.com/apache/incubator-openwhisk/pull/3921 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] pritidesai opened a new issue #276: Add a reference to OpenWhisk Shell

2018-08-06 Thread GitBox
pritidesai opened a new issue #276: Add a reference to OpenWhisk Shell URL: https://github.com/apache/incubator-openwhisk-website/issues/276 Add a reference to OpenWhisk shell on doc page: https://openwhisk.ng.bluemix.net/api/v1/web/nickm_wskng_demos/public/index.html

[GitHub] codecov-io edited a comment on issue #3924: Fix CacheConcurrencyTest cleanup.

2018-08-06 Thread GitBox
codecov-io edited a comment on issue #3924: Fix CacheConcurrencyTest cleanup. URL: https://github.com/apache/incubator-openwhisk/pull/3924#issuecomment-409215228 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3924?src=pr=h1) Report > Merging

[GitHub] pritidesai commented on issue #262: Re-designing OpenWhisk website

2018-08-06 Thread GitBox
pritidesai commented on issue #262: Re-designing OpenWhisk website URL: https://github.com/apache/incubator-openwhisk-website/pull/262#issuecomment-410894740 thanks @csantanapr and @mrutkows. I am now merging this PR so that we can add many more improvements to this new design and

[GitHub] codecov-io edited a comment on issue #3924: Fix CacheConcurrencyTest cleanup.

2018-08-06 Thread GitBox
codecov-io edited a comment on issue #3924: Fix CacheConcurrencyTest cleanup. URL: https://github.com/apache/incubator-openwhisk/pull/3924#issuecomment-409215228 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3924?src=pr=h1) Report > Merging

[GitHub] rabbah commented on issue #3737: tests package needs some cleanup

2018-08-06 Thread GitBox
rabbah commented on issue #3737: tests package needs some cleanup URL: https://github.com/apache/incubator-openwhisk/issues/3737#issuecomment-410833464 This is great! Thanks for tackling it. The abstract classes exist to allow the CLI to reuse these test suites. Arguably the cli

[GitHub] rabbah commented on issue #3737: tests package needs some cleanup

2018-08-06 Thread GitBox
rabbah commented on issue #3737: tests package needs some cleanup URL: https://github.com/apache/incubator-openwhisk/issues/3737#issuecomment-410834411 For the sake of a more efficient review, id suggest either several prs or one pr with each commit refactoring one test suite.

[GitHub] houshengbo opened a new pull request #962: Update the version of Client Go library for 0.9.0

2018-08-06 Thread GitBox
houshengbo opened a new pull request #962: Update the version of Client Go library for 0.9.0 URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/962 This is an automated message from the Apache Git Service. To

[GitHub] rabbah commented on issue #275: Validate Roles

2018-08-06 Thread GitBox
rabbah commented on issue #275: Validate Roles URL: https://github.com/apache/incubator-openwhisk-website/issues/275#issuecomment-410834640 We should further disambiguate users and developers. This is an automated message

[GitHub] pritidesai closed pull request #280: Revert "Re-designing OpenWhisk website"

2018-08-06 Thread GitBox
pritidesai closed pull request #280: Revert "Re-designing OpenWhisk website" URL: https://github.com/apache/incubator-openwhisk-website/pull/280 This is an automated message from the Apache Git Service. To respond to the

[GitHub] mrutkows opened a new pull request #280: Revert "Re-designing OpenWhisk website"

2018-08-06 Thread GitBox
mrutkows opened a new pull request #280: Revert "Re-designing OpenWhisk website" URL: https://github.com/apache/incubator-openwhisk-website/pull/280 Reverts apache/incubator-openwhisk-website#262 This is an automated message

[GitHub] JiniousChoi opened a new issue #3947: Building Invoker fails on getting kubectl

2018-08-06 Thread GitBox
JiniousChoi opened a new issue #3947: Building Invoker fails on getting kubectl URL: https://github.com/apache/incubator-openwhisk/issues/3947 ## Environment details: * local deployment * I am currently on comit 8a3b4472d5ef420f0216cc8e42932d87f428ed30 ## Steps to

[GitHub] upgle commented on a change in pull request #3880: Modify that web action in the bound package can be accessed.

2018-08-06 Thread GitBox
upgle commented on a change in pull request #3880: Modify that web action in the bound package can be accessed. URL: https://github.com/apache/incubator-openwhisk/pull/3880#discussion_r208104271 ## File path: core/controller/src/main/scala/whisk/core/controller/WebActions.scala

[GitHub] mrutkows opened a new pull request #279: Revert "updating ruby version"

2018-08-06 Thread GitBox
mrutkows opened a new pull request #279: Revert "updating ruby version" URL: https://github.com/apache/incubator-openwhisk-website/pull/279 Reverts apache/incubator-openwhisk-website#278 This is an automated message from the

[GitHub] pritidesai closed pull request #279: Revert "updating ruby version"

2018-08-06 Thread GitBox
pritidesai closed pull request #279: Revert "updating ruby version" URL: https://github.com/apache/incubator-openwhisk-website/pull/279 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] tysonnorris opened a new issue #3948: ActorSystem prematurely terminates; cluster nodes cannot leave gracefully

2018-08-06 Thread GitBox
tysonnorris opened a new issue #3948: ActorSystem prematurely terminates; cluster nodes cannot leave gracefully URL: https://github.com/apache/incubator-openwhisk/issues/3948 ## Environment details: * local deployment, Mac OS ## Steps to reproduce the issue: 1.

[GitHub] tysonnorris opened a new pull request #3949: don't explicitly terminate actorSystem (let CoordinatedShutdown do it…

2018-08-06 Thread GitBox
tysonnorris opened a new pull request #3949: don't explicitly terminate actorSystem (let CoordinatedShutdown do it… URL: https://github.com/apache/incubator-openwhisk/pull/3949 Let CoordinatedShutdown run `ActorSystem.termination`, so cluster exit can be graceful fixes #3948

[GitHub] rabbah commented on issue #33: Error invoking wsk actions. "The supplied authentication is not authorized to access this resource"

2018-08-06 Thread GitBox
rabbah commented on issue #33: Error invoking wsk actions. "The supplied authentication is not authorized to access this resource" URL: https://github.com/apache/incubator-openwhisk/issues/33#issuecomment-410719010 can you share more details to help you diagnose this problem? in

[GitHub] fmaschler commented on issue #144: make quick-start hangs with message 'waiting for the Whisk invoker to come up..' on the screen

2018-08-06 Thread GitBox
fmaschler commented on issue #144: make quick-start hangs with message 'waiting for the Whisk invoker to come up..' on the screen URL: https://github.com/apache/incubator-openwhisk-devtools/issues/144#issuecomment-410666946 Maybe this is related to #139? I had the same issue and

[GitHub] emanuelr93 opened a new issue #145: Docker Compose Error with

2018-08-06 Thread GitBox
emanuelr93 opened a new issue #145: Docker Compose Error with URL: https://github.com/apache/incubator-openwhisk-devtools/issues/145 When try to run Docker Compose, i have this error on Invoker Container: ``` Picked up JAVA_TOOL_OPTIONS: -XX:+UnlockExperimentalVMOptions

[GitHub] emanuelr93 opened a new pull request #146: Modify Invoker to fix startup error

2018-08-06 Thread GitBox
emanuelr93 opened a new pull request #146: Modify Invoker to fix startup error URL: https://github.com/apache/incubator-openwhisk-devtools/pull/146 Solve this issue https://github.com/apache/incubator-openwhisk-devtools/issues/145

[GitHub] rabbah commented on issue #3943: Inconsistency between action and package list actions with invalid namespace

2018-08-06 Thread GitBox
rabbah commented on issue #3943: Inconsistency between action and package list actions with invalid namespace URL: https://github.com/apache/incubator-openwhisk/issues/3943#issuecomment-410717499 the reason - for the sake of discussion - is that you can query any namespace for its list

[GitHub] rabbah commented on issue #33: Lots and lots and lots of updates...

2018-08-06 Thread GitBox
rabbah commented on issue #33: Lots and lots and lots of updates... URL: https://github.com/apache/incubator-openwhisk-external-resources/pull/33#issuecomment-410719855 @jthomas can you resolve the conflict? This is an

[GitHub] rabbah commented on issue #3943: Inconsistency between action and package list actions with invalid namespace

2018-08-06 Thread GitBox
rabbah commented on issue #3943: Inconsistency between action and package list actions with invalid namespace URL: https://github.com/apache/incubator-openwhisk/issues/3943#issuecomment-410718494 furthermore, i'd say `wsk package get` rather than `wsk package list` should be scrutinized

[GitHub] mrutkows opened a new pull request #277: updates for index and layout on large sizes

2018-08-06 Thread GitBox
mrutkows opened a new pull request #277: updates for index and layout on large sizes URL: https://github.com/apache/incubator-openwhisk-website/pull/277 This is an automated message from the Apache Git Service. To respond

[GitHub] pritidesai opened a new pull request #278: updating ruby version

2018-08-06 Thread GitBox
pritidesai opened a new pull request #278: updating ruby version URL: https://github.com/apache/incubator-openwhisk-website/pull/278 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] upgle commented on a change in pull request #3880: Modify that web action in the bound package can be accessed.

2018-08-06 Thread GitBox
upgle commented on a change in pull request #3880: Modify that web action in the bound package can be accessed. URL: https://github.com/apache/incubator-openwhisk/pull/3880#discussion_r208104271 ## File path: core/controller/src/main/scala/whisk/core/controller/WebActions.scala

[GitHub] upgle commented on a change in pull request #3880: Modify that web action in the bound package can be accessed.

2018-08-06 Thread GitBox
upgle commented on a change in pull request #3880: Modify that web action in the bound package can be accessed. URL: https://github.com/apache/incubator-openwhisk/pull/3880#discussion_r208104271 ## File path: core/controller/src/main/scala/whisk/core/controller/WebActions.scala

[GitHub] markusthoemmes closed pull request #270: set invoker's --uniqueName to $INVOKER_NAME; eliminate setting of COM…

2018-08-06 Thread GitBox
markusthoemmes closed pull request #270: set invoker's --uniqueName to $INVOKER_NAME; eliminate setting of COM… URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/270 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is

[GitHub] chetanmeh commented on issue #143: Cannot install on windows

2018-08-06 Thread GitBox
chetanmeh commented on issue #143: Cannot install on windows URL: https://github.com/apache/incubator-openwhisk-devtools/issues/143#issuecomment-410750120 This is tracked in #119. So resolving this as closed This is an

[GitHub] chetanmeh closed issue #143: Cannot install on windows

2018-08-06 Thread GitBox
chetanmeh closed issue #143: Cannot install on windows URL: https://github.com/apache/incubator-openwhisk-devtools/issues/143 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] chetanmeh commented on issue #127: there are two 2888. it is duplicated.

2018-08-06 Thread GitBox
chetanmeh commented on issue #127: there are two 2888. it is duplicated. URL: https://github.com/apache/incubator-openwhisk-devtools/issues/127#issuecomment-410751482 Fixed via #128 This is an automated message from the

[GitHub] rabbah commented on a change in pull request #3880: Modify that web action in the bound package can be accessed.

2018-08-06 Thread GitBox
rabbah commented on a change in pull request #3880: Modify that web action in the bound package can be accessed. URL: https://github.com/apache/incubator-openwhisk/pull/3880#discussion_r207931005 ## File path: core/controller/src/main/scala/whisk/core/controller/WebActions.scala

[GitHub] chetanmeh commented on issue #28: [docker-compose] Create jenkins job that runs when a new a new commits lands in core openwhisk

2018-08-06 Thread GitBox
chetanmeh commented on issue #28: [docker-compose] Create jenkins job that runs when a new a new commits lands in core openwhisk URL: https://github.com/apache/incubator-openwhisk-devtools/issues/28#issuecomment-410750520 @csantanapr Given current travis cron to build every 24 hrs I think

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

2018-08-06 Thread GitBox
chetanmeh commented on issue #3286: Treat action code as attachments for created/updated actions URL: https://github.com/apache/incubator-openwhisk/pull/3286#issuecomment-410754258 PR #3945 supersedes this one This is an

[GitHub] dgrove-oss commented on issue #270: set invoker's --uniqueName to $INVOKER_NAME; eliminate setting of COM…

2018-08-06 Thread GitBox
dgrove-oss commented on issue #270: set invoker's --uniqueName to $INVOKER_NAME; eliminate setting of COM… URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/270#issuecomment-410743358 @markusthoemmes -- fixing the mistake you noted in #268 that I had missed when I

[GitHub] chetanmeh commented on issue #144: make quick-start hangs with message 'waiting for the Whisk invoker to come up..' on the screen

2018-08-06 Thread GitBox
chetanmeh commented on issue #144: make quick-start hangs with message 'waiting for the Whisk invoker to come up..' on the screen URL: https://github.com/apache/incubator-openwhisk-devtools/issues/144#issuecomment-410745786 This should be now fixed with #146

[GitHub] chetanmeh closed issue #127: there are two 2888. it is duplicated.

2018-08-06 Thread GitBox
chetanmeh closed issue #127: there are two 2888. it is duplicated. URL: https://github.com/apache/incubator-openwhisk-devtools/issues/127 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] chetanmeh closed issue #145: Docker Compose Error in Invoker Container

2018-08-06 Thread GitBox
chetanmeh closed issue #145: Docker Compose Error in Invoker Container URL: https://github.com/apache/incubator-openwhisk-devtools/issues/145 This is an automated message from the Apache Git Service. To respond to the

[GitHub] chetanmeh closed pull request #146: Modify Invoker to fix startup error

2018-08-06 Thread GitBox
chetanmeh closed pull request #146: Modify Invoker to fix startup error URL: https://github.com/apache/incubator-openwhisk-devtools/pull/146 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 commented on a change in pull request #3880: Modify that web action in the bound package can be accessed.

2018-08-06 Thread GitBox
rabbah commented on a change in pull request #3880: Modify that web action in the bound package can be accessed. URL: https://github.com/apache/incubator-openwhisk/pull/3880#discussion_r207928809 ## File path: core/controller/src/main/scala/whisk/core/controller/WebActions.scala

[GitHub] chetanmeh closed issue #144: make quick-start hangs with message 'waiting for the Whisk invoker to come up..' on the screen

2018-08-06 Thread GitBox
chetanmeh closed issue #144: make quick-start hangs with message 'waiting for the Whisk invoker to come up..' on the screen URL: https://github.com/apache/incubator-openwhisk-devtools/issues/144 This is an automated

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

2018-08-06 Thread GitBox
chetanmeh commented on issue #139: Meet minio file access denied when waiting for the controller URL: https://github.com/apache/incubator-openwhisk-devtools/issues/139#issuecomment-410748844 ```[2018-08-01T10:47:37.376Z] [ERROR] [#tid_sid_invoker] [Invoker] Error processing command line

[GitHub] chetanmeh commented on issue #2: integrate the openwhisk apigateway docker container

2018-08-06 Thread GitBox
chetanmeh commented on issue #2: integrate the openwhisk apigateway docker container URL: https://github.com/apache/incubator-openwhisk-devtools/issues/2#issuecomment-410750905 @ddragosd I see gateway is now in docker-compose #123. Should we close this issue?

[GitHub] chetanmeh opened a new issue #3944: Store action code as attachments

2018-08-06 Thread GitBox
chetanmeh opened a new issue #3944: Store action code as attachments URL: https://github.com/apache/incubator-openwhisk/issues/3944 See [dev thread][1] for background. All action code should be saved as attachments. This work would be done in following steps - [ ] Support storing

[GitHub] chetanmeh opened a new pull request #3945: Treat action code as attachments

2018-08-06 Thread GitBox
chetanmeh opened a new pull request #3945: Treat action code as attachments URL: https://github.com/apache/incubator-openwhisk/pull/3945 Treats action code as attachment for all kinds in default runtime. Supersedes #3268. ## Description This PR builds on #3268 and

[GitHub] chetanmeh closed pull request #3286: Treat action code as attachments for created/updated actions

2018-08-06 Thread GitBox
chetanmeh closed pull request #3286: Treat action code as attachments for created/updated actions URL: https://github.com/apache/incubator-openwhisk/pull/3286 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] jthomas commented on issue #3725: Add Ruby2.5 runtime support

2018-08-06 Thread GitBox
jthomas commented on issue #3725: Add Ruby2.5 runtime support URL: https://github.com/apache/incubator-openwhisk/pull/3725#issuecomment-410662436 Great to see this merged.  @remore Would you like to do an announcement blog post for the OpenWhisk Medium channel?

[GitHub] rabbah opened a new pull request #39: Remove unnecessary ansible files.

2018-08-06 Thread GitBox
rabbah opened a new pull request #39: Remove unnecessary ansible files. URL: https://github.com/apache/incubator-openwhisk-runtime-php/pull/39 This is an automated message from the Apache Git Service. To respond to the

[GitHub] houshengbo opened a new pull request #228: Update the documentation of 0.9.0 for Client Go and CLI

2018-08-06 Thread GitBox
houshengbo opened a new pull request #228: Update the documentation of 0.9.0 for Client Go and CLI URL: https://github.com/apache/incubator-openwhisk-release/pull/228 This is an automated message from the Apache Git

[GitHub] houshengbo closed pull request #228: Update the documentation of 0.9.0 for Client Go and CLI

2018-08-06 Thread GitBox
houshengbo closed pull request #228: Update the documentation of 0.9.0 for Client Go and CLI URL: https://github.com/apache/incubator-openwhisk-release/pull/228 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] violagao opened a new pull request #229: Update the documentation of 0.9.0 for Client Go and CLI

2018-08-06 Thread GitBox
violagao opened a new pull request #229: Update the documentation of 0.9.0 for Client Go and CLI URL: https://github.com/apache/incubator-openwhisk-release/pull/229 This is an automated message from the Apache Git Service.

[GitHub] pritidesai opened a new issue #275: Validate Roles

2018-08-06 Thread GitBox
pritidesai opened a new issue #275: Validate Roles URL: https://github.com/apache/incubator-openwhisk-website/issues/275 With this PR #262 we have defined roles as Developers, Contributors, and Operators. Operators: users who deploy, configure, and host the Apache OpenWhisk

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

2018-08-06 Thread GitBox
mdeuser 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_r207980938 ## File path: core/routemgmt/common/apigw-utils.js ## @@

[GitHub] mgencur commented on issue #3737: tests package needs some cleanup

2018-08-06 Thread GitBox
mgencur commented on issue #3737: tests package needs some cleanup URL: https://github.com/apache/incubator-openwhisk/issues/3737#issuecomment-410798821 At this point I'd suggest the following changes to the test suite so as to achieve a better logical grouping of the tests: 1) move

[GitHub] eweiter commented on issue #3946: update documentation for nodejs runtime

2018-08-06 Thread GitBox
eweiter commented on issue #3946: update documentation for nodejs runtime URL: https://github.com/apache/incubator-openwhisk/pull/3946#issuecomment-410757753 @csantanapr Please merge when you can. This is an automated

[GitHub] eweiter opened a new pull request #3946: update documentation for nodejs runtime

2018-08-06 Thread GitBox
eweiter opened a new pull request #3946: update documentation for nodejs runtime URL: https://github.com/apache/incubator-openwhisk/pull/3946 The NodeJS Runtime for Openwhisk was recently updated to include the latest version of the "openwhisk" NPM package. The documentation listing the

[GitHub] codecov-io commented on issue #3946: update documentation for nodejs runtime

2018-08-06 Thread GitBox
codecov-io commented on issue #3946: update documentation for nodejs runtime URL: https://github.com/apache/incubator-openwhisk/pull/3946#issuecomment-410774415 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3946?src=pr=h1) Report > Merging

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

2018-08-06 Thread GitBox
mdeuser 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_r207973962 ## File path: core/routemgmt/common/apigw-utils.js ## @@

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

2018-08-06 Thread GitBox
mdeuser 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_r207974264 ## File path: core/routemgmt/common/apigw-utils.js ## @@

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

2018-08-06 Thread GitBox
mdeuser 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_r207974264 ## File path: core/routemgmt/common/apigw-utils.js ## @@

[GitHub] csantanapr closed pull request #39: Remove unnecessary ansible files.

2018-08-06 Thread GitBox
csantanapr closed pull request #39: Remove unnecessary ansible files. URL: https://github.com/apache/incubator-openwhisk-runtime-php/pull/39 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] houshengbo closed pull request #229: Update the documentation of 0.9.0 for Client Go and CLI

2018-08-06 Thread GitBox
houshengbo closed pull request #229: Update the documentation of 0.9.0 for Client Go and CLI URL: https://github.com/apache/incubator-openwhisk-release/pull/229 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