[GitHub] chetanmeh opened a new pull request #3884: Document metrics generated within OpenWhisk

chetanmeh opened a new pull request #3884: Document metrics generated within OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3884 OpenWhisk generates quite a few metrics via [Kamon](http://kamon.io). This PR aims to document all currently generated metrics such that

[GitHub] rabbah commented on issue #3884: Document metrics generated within OpenWhisk

rabbah commented on issue #3884: Document metrics generated within OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3884#issuecomment-405566993 Fantastic! This is an automated message from the Apache Git

[GitHub] codecov-io edited a comment on issue #3779: S3AttachmentStore

codecov-io edited a comment on issue #3779: S3AttachmentStore URL: https://github.com/apache/incubator-openwhisk/pull/3779#issuecomment-398322147 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3779?src=pr=h1) Report > Merging

[GitHub] remore commented on issue #3725: Add Ruby2.5 runtime support

remore commented on issue #3725: Add Ruby2.5 runtime support URL: https://github.com/apache/incubator-openwhisk/pull/3725#issuecomment-405635657 The first PR has been sent: https://github.com/apache/incubator-openwhisk-runtime-ruby/pull/1

[GitHub] drcariel commented on issue #97: remove references to bluemix

drcariel commented on issue #97: remove references to bluemix URL: https://github.com/apache/incubator-openwhisk-client-go/issues/97#issuecomment-405616640 @rabbah I see these bluemix references in cli and openwhisk repos too - should those be removed as well?

[GitHub] nerdguru commented on issue #258: Where should the come from for GKE nginx ingress?

nerdguru commented on issue #258: Where should the come from for GKE nginx ingress? URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/258#issuecomment-405605697 Hi Carlos and thanks for the quick reply. So does that imply that a domain name for my GKE cluster is

[GitHub] cduque89 commented on issue #258: Where should the come from for GKE nginx ingress?

cduque89 commented on issue #258: Where should the come from for GKE nginx ingress? URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/258#issuecomment-405621290 Hi @nerdguru. If that's the case you can still have your DNS domain as described in the

[GitHub] mdeuser commented on issue #3760: Add API gateway to redo.

mdeuser commented on issue #3760: Add API gateway to redo. URL: https://github.com/apache/incubator-openwhisk/pull/3760#issuecomment-405562162 pg4/1965 This is an automated message from the Apache Git Service. To respond to

[GitHub] cduque89 commented on issue #258: Where should the come from for GKE nginx ingress?

cduque89 commented on issue #258: Where should the come from for GKE nginx ingress? URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/258#issuecomment-405582272 Hi @nerdguru. You are the one defining the domain in your google cloud. Please give a look

[GitHub] dgrove-oss commented on issue #257: External zookeeper kafka

dgrove-oss commented on issue #257: External zookeeper kafka URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/257#issuecomment-405587098 Hi @StrayCat1, thanks for the contribution! If you haven't done so already, could you please sign an Apache CLA (see

[GitHub] dubee commented on a change in pull request #3799: Display proper error when sequence invocation fails due to missing component

dubee commented on a change in pull request #3799: Display proper error when sequence invocation fails due to missing component URL: https://github.com/apache/incubator-openwhisk/pull/3799#discussion_r203062593 ## File path:

[GitHub] remore opened a new pull request #1: Add the base source code for Ruby 2.5 Runtime

remore opened a new pull request #1: Add the base source code for Ruby 2.5 Runtime URL: https://github.com/apache/incubator-openwhisk-runtime-ruby/pull/1 Related Issue(s): https://github.com/apache/incubator-openwhisk/pull/3725

[GitHub] rabbah commented on issue #3884: Document metrics generated within OpenWhisk

rabbah commented on issue #3884: Document metrics generated within OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3884#issuecomment-405570857 @vvraskin @mhenke1 would you review this? @chetanmeh this is very useful thanks for doing this.

[GitHub] macdonst commented on issue #3883: Make it easier to use webpack with JS actions

macdonst commented on issue #3883: Make it easier to use webpack with JS actions URL: https://github.com/apache/incubator-openwhisk/issues/3883#issuecomment-405633048 @rabbah If I've open the issue in the wrong place than yeah, I'm more than happy to move this issue to another repo.

[GitHub] rabbah commented on issue #3883: Make it easier to use webpack with JS actions

rabbah commented on issue #3883: Make it easier to use webpack with JS actions URL: https://github.com/apache/incubator-openwhisk/issues/3883#issuecomment-405580411 @eweiter @csantanapr I think it would make sense to support this in the node runtime - should we move the issue there?

[GitHub] rabbah removed a comment on issue #3884: Document metrics generated within OpenWhisk

rabbah removed a comment on issue #3884: Document metrics generated within OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3884#issuecomment-405566993 Fantastic! This is an automated message from the

[GitHub] cduque89 opened a new pull request #259: ingress.md information about google cloud DNS

cduque89 opened a new pull request #259: ingress.md information about google cloud DNS URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/259 As stated in #258 it was not evident how to get a DNS name for the google cloud. I've updated the documentation with a link to the

[GitHub] eweiter opened a new pull request #73: Don't override runtime npm packages when user provides their own

eweiter opened a new pull request #73: Don't override runtime npm packages when user provides their own URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/73 See #58 When a user inherits from our runtimes and they run an npm install with their own package.json

[GitHub] StrayCat1 commented on issue #257: External zookeeper kafka

StrayCat1 commented on issue #257: External zookeeper kafka URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/257#issuecomment-405661256 Thanks @dgrove-oss , I just submitted the CLA and Ill let you know when I hear back. Also, I tested those changes to

[GitHub] rabbah commented on a change in pull request #3799: Display proper error when sequence invocation fails due to missing component

rabbah commented on a change in pull request #3799: Display proper error when sequence invocation fails due to missing component URL: https://github.com/apache/incubator-openwhisk/pull/3799#discussion_r203134728 ## File path:

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

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-405693390 Thanks @markusthoemmes. That's much simpler and preserves the pipelining between fetches and

[GitHub] codecov-io commented on issue #3884: Document metrics generated within OpenWhisk

codecov-io commented on issue #3884: Document metrics generated within OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3884#issuecomment-405653420 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3884?src=pr=h1) Report > Merging

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

markusthoemmes 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-405692374 I think this fix has moved a bit in the wrong direction. The mode @rabbah wants to

[GitHub] rabbah edited a comment on issue #3883: Make it easier to use webpack with JS actions

rabbah edited a comment on issue #3883: Make it easier to use webpack with JS actions URL: https://github.com/apache/incubator-openwhisk/issues/3883#issuecomment-405638973 no worries. We can move the issue if necessary. I think it’s worth having the discussion here first where there are

[GitHub] rabbah commented on issue #3883: Make it easier to use webpack with JS actions

rabbah commented on issue #3883: Make it easier to use webpack with JS actions URL: https://github.com/apache/incubator-openwhisk/issues/3883#issuecomment-405638973 no worries. We can move the issue of necessary. I think it’s worth having the discussion here first where there are more 

[GitHub] rabbah commented on a change in pull request #3799: Display proper error when sequence invocation fails due to missing component

rabbah commented on a change in pull request #3799: Display proper error when sequence invocation fails due to missing component URL: https://github.com/apache/incubator-openwhisk/pull/3799#discussion_r203134728 ## File path:

[GitHub] rabbah commented on a change in pull request #3799: Display proper error when sequence invocation fails due to missing component

rabbah commented on a change in pull request #3799: Display proper error when sequence invocation fails due to missing component URL: https://github.com/apache/incubator-openwhisk/pull/3799#discussion_r203134728 ## File path:

[GitHub] pritidesai closed pull request #223: Update the release folder on a repository basis

pritidesai closed pull request #223: Update the release folder on a repository basis URL: https://github.com/apache/incubator-openwhisk-release/pull/223 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] tysonnorris commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils

tysonnorris commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils URL: https://github.com/apache/incubator-openwhisk/pull/3812#discussion_r203219371 ## File path: common/scala/src/main/scala/whisk/http/PoolingRestClient.scala ##

[GitHub] tysonnorris commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils

tysonnorris commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils URL: https://github.com/apache/incubator-openwhisk/pull/3812#discussion_r203218116 ## File path:

[GitHub] tysonnorris commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils

tysonnorris commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils URL: https://github.com/apache/incubator-openwhisk/pull/3812#discussion_r203219670 ## File path:

[GitHub] jiangpengcheng commented on issue #3853: Set erlang magic cookie for couchdb

jiangpengcheng commented on issue #3853: Set erlang magic cookie for couchdb URL: https://github.com/apache/incubator-openwhisk/pull/3853#issuecomment-405773126 @vvraskin I use the `local_action` and `run_once` options, I think it does the same thing

[GitHub] tysonnorris commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils

tysonnorris commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils URL: https://github.com/apache/incubator-openwhisk/pull/3812#discussion_r203217369 ## File path:

[GitHub] tysonnorris commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils

tysonnorris commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils URL: https://github.com/apache/incubator-openwhisk/pull/3812#discussion_r203218558 ## File path: common/scala/src/main/scala/whisk/core/database/CouchDbRestStore.scala

[GitHub] tysonnorris commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils

tysonnorris commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils URL: https://github.com/apache/incubator-openwhisk/pull/3812#discussion_r203217847 ## File path:

[GitHub] codecov-io edited a comment on issue #3812: ContainerClient + akka http alternative to HttpUtils

codecov-io edited a comment on issue #3812: ContainerClient + akka http alternative to HttpUtils URL: https://github.com/apache/incubator-openwhisk/pull/3812#issuecomment-401212372 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3812?src=pr=h1) Report > Merging

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

upgle commented on issue #3880: Modify that web action in the bound package can be accessed. URL: https://github.com/apache/incubator-openwhisk/pull/3880#issuecomment-405807606 @rabbah I modified it to check the entitlement and added tests. Now, It returns the authentication

[GitHub] codecov-io edited a comment on issue #3880: Modify that web action in the bound package can be accessed.

codecov-io edited a comment on issue #3880: Modify that web action in the bound package can be accessed. URL: https://github.com/apache/incubator-openwhisk/pull/3880#issuecomment-405214016 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3880?src=pr=h1) Report >

[GitHub] codecov-io edited a comment on issue #3880: Modify that web action in the bound package can be accessed.

codecov-io edited a comment on issue #3880: Modify that web action in the bound package can be accessed. URL: https://github.com/apache/incubator-openwhisk/pull/3880#issuecomment-405214016 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3880?src=pr=h1) Report >

[GitHub] chetanmeh commented on a change in pull request #3882: Use invoker id for metrics instead of the hostname.

chetanmeh commented on a change in pull request #3882: Use invoker id for metrics instead of the hostname. URL: https://github.com/apache/incubator-openwhisk/pull/3882#discussion_r202908636 ## File path: common/scala/src/main/scala/whisk/common/Logging.scala ## @@ -212,6

[GitHub] codecov-io edited a comment on issue #3882: Use invoker id for metrics instead of the hostname.

codecov-io edited a comment on issue #3882: Use invoker id for metrics instead of the hostname. URL: https://github.com/apache/incubator-openwhisk/pull/3882#issuecomment-405291454 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3882?src=pr=h1) Report > Merging

[GitHub] chetanmeh commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils

chetanmeh commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils URL: https://github.com/apache/incubator-openwhisk/pull/3812#discussion_r202903907 ## File path: common/scala/src/main/scala/whisk/core/containerpool/ContainerClient.scala

[GitHub] chetanmeh commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils

chetanmeh commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils URL: https://github.com/apache/incubator-openwhisk/pull/3812#discussion_r202898726 ## File path: common/scala/src/main/scala/whisk/core/containerpool/ContainerClient.scala

[GitHub] chetanmeh commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils

chetanmeh commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils URL: https://github.com/apache/incubator-openwhisk/pull/3812#discussion_r202899807 ## File path: common/scala/src/main/scala/whisk/core/database/CouchDbRestStore.scala

[GitHub] chetanmeh commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils

chetanmeh commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils URL: https://github.com/apache/incubator-openwhisk/pull/3812#discussion_r202901242 ## File path: common/scala/src/main/scala/whisk/http/PoolingRestClient.scala ##

[GitHub] chetanmeh commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils

chetanmeh commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils URL: https://github.com/apache/incubator-openwhisk/pull/3812#discussion_r202898249 ## File path: common/scala/src/main/scala/whisk/core/containerpool/ContainerClient.scala

[GitHub] chetanmeh commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils

chetanmeh commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils URL: https://github.com/apache/incubator-openwhisk/pull/3812#discussion_r202896946 ## File path: common/scala/src/main/scala/whisk/core/containerpool/ContainerClient.scala

[GitHub] chetanmeh commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils

chetanmeh commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils URL: https://github.com/apache/incubator-openwhisk/pull/3812#discussion_r202897095 ## File path: common/scala/src/main/scala/whisk/core/containerpool/ContainerClient.scala

[GitHub] chetanmeh commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils

chetanmeh commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils URL: https://github.com/apache/incubator-openwhisk/pull/3812#discussion_r202898803 ## File path: common/scala/src/main/scala/whisk/core/containerpool/ContainerClient.scala

[GitHub] chetanmeh commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils

chetanmeh commented on a change in pull request #3812: ContainerClient + akka http alternative to HttpUtils URL: https://github.com/apache/incubator-openwhisk/pull/3812#discussion_r202901499 ## File path: common/scala/src/main/scala/whisk/http/PoolingRestClient.scala ##

[GitHub] codecov-io edited a comment on issue #3882: Use invoker id for metrics instead of the hostname.

codecov-io edited a comment on issue #3882: Use invoker id for metrics instead of the hostname. URL: https://github.com/apache/incubator-openwhisk/pull/3882#issuecomment-405291454 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3882?src=pr=h1) Report > Merging

[GitHub] codecov-io edited a comment on issue #3812: ContainerClient + akka http alternative to HttpUtils

codecov-io edited a comment on issue #3812: ContainerClient + akka http alternative to HttpUtils URL: https://github.com/apache/incubator-openwhisk/pull/3812#issuecomment-401212372 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3812?src=pr=h1) Report > Merging

[GitHub] jthomas commented on issue #3725: Add Ruby2.5 runtime support

jthomas commented on issue #3725: Add Ruby2.5 runtime support URL: https://github.com/apache/incubator-openwhisk/pull/3725#issuecomment-405538084 https://github.com/apache/incubator-openwhisk-runtime-ruby repo is now available.

[GitHub] rabbah commented on issue #97: remove references to bluemix

rabbah commented on issue #97: remove references to bluemix URL: https://github.com/apache/incubator-openwhisk-client-go/issues/97#issuecomment-405620225 yes i think we should remove them. This is an automated message from