[GitHub] dinogun commented on issue #18: Rebase OW JavaRuntime to use OpenJDK 8

2018-02-28 Thread GitBox
dinogun commented on issue #18: Rebase OW JavaRuntime to use OpenJDK 8 URL: https://github.com/apache/incubator-openwhisk-runtime-java/issues/18#issuecomment-369499254 @csantanapr One option to look into would be the [AdoptOpenJDK](https://adoptopenjdk.net/index.html) project. It is a

[GitHub] chetanmeh opened a new pull request #3374: Use Docker ADD command to untar

2018-02-28 Thread GitBox
chetanmeh opened a new pull request #3374: Use Docker ADD command to untar URL: https://github.com/apache/incubator-openwhisk/pull/3374 This PR has two changes ### Use ADD instead of COPY Currently * controller - The tar file is copied but not removed thus increasing the

[GitHub] upgle commented on issue #3343: Support time-series DB using Kamon Tag, and optional granular metric.

2018-02-28 Thread GitBox
upgle commented on issue #3343: Support time-series DB using Kamon Tag, and optional granular metric. URL: https://github.com/apache/incubator-openwhisk/pull/3343#issuecomment-369462987 The test is passed, and some test code has modified to use log parser which is changed the signature.

[GitHub] RetardedPigeon opened a new issue #3373: The security of running arbitrary code in docker containers

2018-02-28 Thread GitBox
RetardedPigeon opened a new issue #3373: The security of running arbitrary code in docker containers URL: https://github.com/apache/incubator-openwhisk/issues/3373 I couldn't really find any documentation on how the security regarding docker containers is addressed. As far as I was aware,

[GitHub] devbv opened a new issue #3372: Couchbase support

2018-02-28 Thread GitBox
devbv opened a new issue #3372: Couchbase support URL: https://github.com/apache/incubator-openwhisk/issues/3372 Hi all, I found that I wrote spi module of artifact store supporting Couchbase in last year but not did PR. It's based on pretty old revision so It may take a few time to

[GitHub] upgle commented on issue #3343: Support time-series DB using Kamon Tag, and optional granular metric.

2018-02-28 Thread GitBox
upgle commented on issue #3343: Support time-series DB using Kamon Tag, and optional granular metric. URL: https://github.com/apache/incubator-openwhisk/pull/3343#issuecomment-369462987 The test is passed, and some test code has modified to use log parser which changed the signature.

[GitHub] dgrove-oss commented on a change in pull request #3338: implement suspend/resume for KubernetesContainer

2018-02-28 Thread GitBox
dgrove-oss commented on a change in pull request #3338: implement suspend/resume for KubernetesContainer URL: https://github.com/apache/incubator-openwhisk/pull/3338#discussion_r171450061 ## File path:

[GitHub] csantanapr closed pull request #3: Switch to Maven based dependency

2018-02-28 Thread GitBox
csantanapr closed pull request #3: Switch to Maven based dependency URL: https://github.com/apache/incubator-openwhisk-runtime-php/pull/3 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

[GitHub] csantanapr closed pull request #7: Switch to Maven based dependency

2018-02-28 Thread GitBox
csantanapr closed pull request #7: Switch to Maven based dependency URL: https://github.com/apache/incubator-openwhisk-runtime-python/pull/7 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] csantanapr closed pull request #21: Switch to Maven based dependency

2018-02-28 Thread GitBox
csantanapr closed pull request #21: Switch to Maven based dependency URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/21 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

[GitHub] csantanapr commented on issue #2915: AWS deployment

2018-02-28 Thread GitBox
csantanapr commented on issue #2915: AWS deployment URL: https://github.com/apache/incubator-openwhisk/pull/2915#issuecomment-369423318 there is no current way of testing this. I direction is to move away of using ansible VMs to deploy and instead use more native container/docker way by

[GitHub] csantanapr closed pull request #2915: AWS deployment

2018-02-28 Thread GitBox
csantanapr closed pull request #2915: AWS deployment URL: https://github.com/apache/incubator-openwhisk/pull/2915 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 request

[GitHub] jonpspri closed pull request #3216: Download Nginx Tarball from OpenWhisk CLI release, rather than constructing file-by-file

2018-02-28 Thread GitBox
jonpspri closed pull request #3216: Download Nginx Tarball from OpenWhisk CLI release, rather than constructing file-by-file URL: https://github.com/apache/incubator-openwhisk/pull/3216 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is

[GitHub] jonpspri commented on issue #3216: Download Nginx Tarball from OpenWhisk CLI release, rather than constructing file-by-file

2018-02-28 Thread GitBox
jonpspri commented on issue #3216: Download Nginx Tarball from OpenWhisk CLI release, rather than constructing file-by-file URL: https://github.com/apache/incubator-openwhisk/pull/3216#issuecomment-369422993 Closing and reopening to re-execute Travis. I suspect the Kafka test case is a

[GitHub] jonpspri opened a new pull request #3216: Download Nginx Tarball from OpenWhisk CLI release, rather than constructing file-by-file

2018-02-28 Thread GitBox
jonpspri opened a new pull request #3216: Download Nginx Tarball from OpenWhisk CLI release, rather than constructing file-by-file URL: https://github.com/apache/incubator-openwhisk/pull/3216 Ansible build of Nginx edge now downloads the Openwhisk CLI tarball from the GitHub release

[GitHub] csantanapr commented on issue #2622: ansible playbooks for deploying each of the providers

2018-02-28 Thread GitBox
csantanapr commented on issue #2622: ansible playbooks for deploying each of the providers URL: https://github.com/apache/incubator-openwhisk/pull/2622#issuecomment-369422256 I think we are going to wait on doing this and do it the kube way, closing for now

[GitHub] csantanapr closed pull request #2622: ansible playbooks for deploying each of the providers

2018-02-28 Thread GitBox
csantanapr closed pull request #2622: ansible playbooks for deploying each of the providers URL: https://github.com/apache/incubator-openwhisk/pull/2622 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] csantanapr commented on a change in pull request #3133: WIP: Subtle changes to support S390X deployment of OpenWhisk

2018-02-28 Thread GitBox
csantanapr commented on a change in pull request #3133: WIP: Subtle changes to support S390X deployment of OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3133#discussion_r171422134 ## File path: ansible/extra/s390x.yml ## @@ -0,0 +1,17 @@ +### S390X

[GitHub] csantanapr commented on a change in pull request #3133: WIP: Subtle changes to support S390X deployment of OpenWhisk

2018-02-28 Thread GitBox
csantanapr commented on a change in pull request #3133: WIP: Subtle changes to support S390X deployment of OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3133#discussion_r171421639 ## File path: ansible/extra/s390x.yml ## @@ -0,0 +1,17 @@ +### S390X

[GitHub] csantanapr commented on a change in pull request #3133: WIP: Subtle changes to support S390X deployment of OpenWhisk

2018-02-28 Thread GitBox
csantanapr commented on a change in pull request #3133: WIP: Subtle changes to support S390X deployment of OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3133#discussion_r171421421 ## File path: core/invoker/Dockerfile.s390x ## @@ -0,0 +1,15 @@ +FROM

[GitHub] csantanapr commented on a change in pull request #3133: WIP: Subtle changes to support S390X deployment of OpenWhisk

2018-02-28 Thread GitBox
csantanapr commented on a change in pull request #3133: WIP: Subtle changes to support S390X deployment of OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3133#discussion_r171421293 ## File path: core/invoker/Dockerfile.s390x ## @@ -0,0 +1,15 @@ +FROM

[GitHub] csantanapr commented on issue #2729: add support for multiple versions of ubuntu

2018-02-28 Thread GitBox
csantanapr commented on issue #2729: add support for multiple versions of ubuntu URL: https://github.com/apache/incubator-openwhisk/pull/2729#issuecomment-369420070 This is now Fix by https://github.com/apache/incubator-openwhisk/pull/3358

[GitHub] csantanapr closed pull request #2729: add support for multiple versions of ubuntu

2018-02-28 Thread GitBox
csantanapr closed pull request #2729: add support for multiple versions of ubuntu URL: https://github.com/apache/incubator-openwhisk/pull/2729 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

[GitHub] csantanapr closed pull request #3358: Update to tools/ubuntu-setup to support xenial and later Ubuntu releases

2018-02-28 Thread GitBox
csantanapr closed pull request #3358: Update to tools/ubuntu-setup to support xenial and later Ubuntu releases URL: https://github.com/apache/incubator-openwhisk/pull/3358 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for

[GitHub] ddragosd opened a new issue #762: wskdeploy only supports /whisk.system bindings or github.com packages

2018-02-28 Thread GitBox
ddragosd opened a new issue #762: wskdeploy only supports /whisk.system bindings or github.com packages URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/762 In my case I'm dealing with 2 different namespaces, and I'd like to setup a package binding between these 2

[GitHub] markusthoemmes commented on issue #3370: WskBasicTests failed due to a caught exception during the log fetching process

2018-02-28 Thread GitBox
markusthoemmes commented on issue #3370: WskBasicTests failed due to a caught exception during the log fetching process URL: https://github.com/apache/incubator-openwhisk/issues/3370#issuecomment-369397447 Yep. Hint for 1: It will be a matter of inserting a logline at

[GitHub] csantanapr commented on issue #3358: Update to tools/ubuntu-setup to support xenial and later Ubuntu releases

2018-02-28 Thread GitBox
csantanapr commented on issue #3358: Update to tools/ubuntu-setup to support xenial and later Ubuntu releases URL: https://github.com/apache/incubator-openwhisk/pull/3358#issuecomment-369396350 It worked after updating @jonpspri ``` default: ok: whisk auth set. Run 'wsk

[GitHub] houshengbo commented on issue #3216: Download Nginx Tarball from OpenWhisk CLI release, rather than constructing file-by-file

2018-02-28 Thread GitBox
houshengbo commented on issue #3216: Download Nginx Tarball from OpenWhisk CLI release, rather than constructing file-by-file URL: https://github.com/apache/incubator-openwhisk/pull/3216#issuecomment-369394615 Still failed the test case: services.KafkaConnectorTests > Kafka connector

[GitHub] houshengbo opened a new pull request #277: Add license notice

2018-02-28 Thread GitBox
houshengbo opened a new pull request #277: Add license notice URL: https://github.com/apache/incubator-openwhisk-apigateway/pull/277 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] tysonnorris commented on issue #2833: MesosContainerFactory

2018-02-28 Thread GitBox
tysonnorris commented on issue #2833: MesosContainerFactory URL: https://github.com/apache/incubator-openwhisk/pull/2833#issuecomment-369376482 Thanks @glikson - 1. Marathon has a different notion of lifecycle for containers than openwhisk, so I don't think that using it is a good

[GitHub] mhamann commented on issue #276: API GW should adhere to "Accept" header dictates as much as possible

2018-02-28 Thread GitBox
mhamann commented on issue #276: API GW should adhere to "Accept" header dictates as much as possible URL: https://github.com/apache/incubator-openwhisk-apigateway/issues/276#issuecomment-369375125 I think 406 isn't terribly useful, since we don't know exactly what types are allowable

[GitHub] mdeuser commented on issue #3370: WskBasicTests failed due to a caught exception during the log fetching process

2018-02-28 Thread GitBox
mdeuser commented on issue #3370: WskBasicTests failed due to a caught exception during the log fetching process URL: https://github.com/apache/incubator-openwhisk/issues/3370#issuecomment-369374057 @markusthoemmes - do you agree that there are 2 issues at play - (1) more logging failure

[GitHub] mdeuser commented on issue #276: API GW should adhere to "Accept" header dictates as much as possible

2018-02-28 Thread GitBox
mdeuser commented on issue #276: API GW should adhere to "Accept" header dictates as much as possible URL: https://github.com/apache/incubator-openwhisk-apigateway/issues/276#issuecomment-369371794 I guess the server might be able to support a few "standard" media types and respond with

[GitHub] markusthoemmes commented on issue #3371: Make AuthKey parsing leaner.

2018-02-28 Thread GitBox
markusthoemmes commented on issue #3371: Make AuthKey parsing leaner. URL: https://github.com/apache/incubator-openwhisk/pull/3371#issuecomment-369365951 PG5 125 ? This is an automated message from the Apache Git Service.

[GitHub] csantanapr closed pull request #3357: Remove usages of replace and replaceAll on the hotpath.

2018-02-28 Thread GitBox
csantanapr closed pull request #3357: Remove usages of replace and replaceAll on the hotpath. URL: https://github.com/apache/incubator-openwhisk/pull/3357 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] dubeejw closed pull request #234: Switch to Maven based dependency

2018-02-28 Thread GitBox
dubeejw closed pull request #234: Switch to Maven based dependency URL: https://github.com/apache/incubator-openwhisk-cli/pull/234 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] dubeejw closed pull request #159: Switch to Maven based dependency

2018-02-28 Thread GitBox
dubeejw closed pull request #159: Switch to Maven based dependency URL: https://github.com/apache/incubator-openwhisk-package-cloudant/pull/159 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

[GitHub] mhamann commented on issue #276: API GW should adhere to "Accept" header dictates as much as possible

2018-02-28 Thread GitBox
mhamann commented on issue #276: API GW should adhere to "Accept" header dictates as much as possible URL: https://github.com/apache/incubator-openwhisk-apigateway/issues/276#issuecomment-369356342 So, in your scenario, let's say you pass an Accept header of `text/html`. If an API key is

[GitHub] mhamann commented on issue #276: API GW should adhere to "Accept" header dictates as much as possible

2018-02-28 Thread GitBox
mhamann commented on issue #276: API GW should adhere to "Accept" header dictates as much as possible URL: https://github.com/apache/incubator-openwhisk-apigateway/issues/276#issuecomment-369356342 So, in your scenario, let's say you pass an Accept header of `text/html`. If an API key is

[GitHub] markusthoemmes commented on issue #2855: Cache database attachments

2018-02-28 Thread GitBox
markusthoemmes commented on issue #2855: Cache database attachments URL: https://github.com/apache/incubator-openwhisk/pull/2855#issuecomment-369355147 Does this mean we're going to cache attachments in the controller? Is that intended?

[GitHub] glikson commented on issue #155: config files to use kubernetes container pool and invoker-agent

2018-02-28 Thread GitBox
glikson commented on issue #155: config files to use kubernetes container pool and invoker-agent URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/155#issuecomment-369353425 Would it make sense to have invoker-agent in a different repo? It doesn't seem to be too related

[GitHub] glikson commented on issue #155: config files to use kubernetes container pool and invoker-agent

2018-02-28 Thread GitBox
glikson commented on issue #155: config files to use kubernetes container pool and invoker-agent URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/155#issuecomment-369351785 Would it make sense to (eventually) use CRI instead of direct interaction with the docker

[GitHub] csantanapr commented on issue #3358: Update to tools/ubuntu-setup to support xenial and later Ubuntu releases

2018-02-28 Thread GitBox
csantanapr commented on issue #3358: Update to tools/ubuntu-setup to support xenial and later Ubuntu releases URL: https://github.com/apache/incubator-openwhisk/pull/3358#issuecomment-369351700 Upgrading to virtualbox 5.2.6 and vagrant 2.0.2 Will report results

[GitHub] markusthoemmes commented on issue #3357: Remove usages of replace and replaceAll on the hotpath.

2018-02-28 Thread GitBox
markusthoemmes commented on issue #3357: Remove usages of replace and replaceAll on the hotpath. URL: https://github.com/apache/incubator-openwhisk/pull/3357#issuecomment-369349122 PG4 1427 ? This is an automated message

[GitHub] csantanapr closed pull request #132: Switch to Maven based dependency

2018-02-28 Thread GitBox
csantanapr closed pull request #132: Switch to Maven based dependency URL: https://github.com/apache/incubator-openwhisk-package-alarms/pull/132 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

[GitHub] csantanapr commented on issue #132: Switch to Maven based dependency

2018-02-28 Thread GitBox
csantanapr commented on issue #132: Switch to Maven based dependency URL: https://github.com/apache/incubator-openwhisk-package-alarms/pull/132#issuecomment-369347674 FYI @jasonpet anything internal in CID running tests will need to run maven install first

[GitHub] csantanapr closed pull request #3360: Set mode of wsk to executable

2018-02-28 Thread GitBox
csantanapr closed pull request #3360: Set mode of wsk to executable URL: https://github.com/apache/incubator-openwhisk/pull/3360 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] csantanapr closed pull request #131: Add health monitoring support

2018-02-28 Thread GitBox
csantanapr closed pull request #131: Add health monitoring support URL: https://github.com/apache/incubator-openwhisk-package-alarms/pull/131 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

[GitHub] kameshsampath commented on issue #18: Rebase OW JavaRuntime to use OpenJDK 8

2018-02-28 Thread GitBox
kameshsampath commented on issue #18: Rebase OW JavaRuntime to use OpenJDK 8 URL: https://github.com/apache/incubator-openwhisk-runtime-java/issues/18#issuecomment-369343263 @csantanapr - the reason for using fabric8 images was that it right now works around with the JDK's inability to

[GitHub] mdeuser commented on issue #225: wsk trigger update on a feed trigger calls the feed action again

2018-02-28 Thread GitBox
mdeuser commented on issue #225: wsk trigger update on a feed trigger calls the feed action again URL: https://github.com/apache/incubator-openwhisk-cli/issues/225#issuecomment-369342948 see also https://github.com/apache/incubator-openwhisk/pull/3318 for a related CLI change to `wsk

[GitHub] glikson commented on issue #3219: Kubernetes ContainerFactoryProvider implementation

2018-02-28 Thread GitBox
glikson commented on issue #3219: Kubernetes ContainerFactoryProvider implementation URL: https://github.com/apache/incubator-openwhisk/pull/3219#issuecomment-369335995 should have "kubernetes" label? This is an automated

[GitHub] glikson commented on issue #2833: MesosContainerFactory

2018-02-28 Thread GitBox
glikson commented on issue #2833: MesosContainerFactory URL: https://github.com/apache/incubator-openwhisk/pull/2833#issuecomment-369329528 Great job! Couple of thoughts: 1) I wonder whether you have considered leveraging Marathon rather than Mesos directly? Obviously there might be

[GitHub] markusthoemmes commented on issue #3325: Refactor Kafka clients for more generalized configuration reading.

2018-02-28 Thread GitBox
markusthoemmes commented on issue #3325: Refactor Kafka clients for more generalized configuration reading. URL: https://github.com/apache/incubator-openwhisk/pull/3325#issuecomment-369317679 PG3 1934 ? This is an

[GitHub] mdeuser opened a new issue #276: API GW should adhere to "Accept" header dictates as much as possible

2018-02-28 Thread GitBox
mdeuser opened a new issue #276: API GW should adhere to "Accept" header dictates as much as possible URL: https://github.com/apache/incubator-openwhisk-apigateway/issues/276 Here is one specific example, although other scenarios likely have the same issue. 1. Create an API that is

[GitHub] markusthoemmes closed pull request #3085: Update akka-http to 10.0.11

2018-02-28 Thread GitBox
markusthoemmes closed pull request #3085: Update akka-http to 10.0.11 URL: https://github.com/apache/incubator-openwhisk/pull/3085 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] markusthoemmes closed pull request #2956: Introduce separate Akka dispatchers for CouchDB and Kafka Clients

2018-02-28 Thread GitBox
markusthoemmes closed pull request #2956: Introduce separate Akka dispatchers for CouchDB and Kafka Clients URL: https://github.com/apache/incubator-openwhisk/pull/2956 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] markusthoemmes commented on issue #3085: Update akka-http to 10.0.11

2018-02-28 Thread GitBox
markusthoemmes commented on issue #3085: Update akka-http to 10.0.11 URL: https://github.com/apache/incubator-openwhisk/pull/3085#issuecomment-369297313 Closing this for now, let's revisit once the other changes are made.

[GitHub] markusthoemmes commented on issue #2956: Introduce separate Akka dispatchers for CouchDB and Kafka Clients

2018-02-28 Thread GitBox
markusthoemmes commented on issue #2956: Introduce separate Akka dispatchers for CouchDB and Kafka Clients URL: https://github.com/apache/incubator-openwhisk/pull/2956#issuecomment-369297155 @bwmcadams I'll close this one for now, please reopen once you get to it.

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

2018-02-28 Thread GitBox
markusthoemmes commented on issue #2847: Treat action code as attachments for created/updated actions URL: https://github.com/apache/incubator-openwhisk/pull/2847#issuecomment-369296614 @dubeejw What's the status here? The latest Travis failed.

[GitHub] eweiter commented on issue #3370: WskBasicTests failed due to a caught exception during the log fetching process

2018-02-28 Thread GitBox
eweiter commented on issue #3370: WskBasicTests failed due to a caught exception during the log fetching process URL: https://github.com/apache/incubator-openwhisk/issues/3370#issuecomment-369291237 Updated title to no longer reflect an uncaught exception. Rereading the error message I

[GitHub] markusthoemmes opened a new pull request #3371: Make AuthKey parsing leaner.

2018-02-28 Thread GitBox
markusthoemmes opened a new pull request #3371: Make AuthKey parsing leaner. URL: https://github.com/apache/incubator-openwhisk/pull/3371 Like in the case of ActivationIds, for AuthKeys we do not care if the user really gives us a UUID. For the system internal it doesn't matter.

[GitHub] markusthoemmes commented on issue #3370: WskBasicTests failed due to an uncaught exception during the log fetching process

2018-02-28 Thread GitBox
markusthoemmes commented on issue #3370: WskBasicTests failed due to an uncaught exception during the log fetching process URL: https://github.com/apache/incubator-openwhisk/issues/3370#issuecomment-369278125 Thanks for investigating so deeply. Note though: This is not "an uncaught

[GitHub] csantanapr commented on issue #18: Rebase OW JavaRuntime to use OpenJDK 8

2018-02-28 Thread GitBox
csantanapr commented on issue #18: Rebase OW JavaRuntime to use OpenJDK 8 URL: https://github.com/apache/incubator-openwhisk-runtime-java/issues/18#issuecomment-369272873 I would preferred to base it on a community based image instead So I propose we use:

[GitHub] eweiter opened a new issue #3370: WskBasicTests failed due to an uncaught exception during the log fetching process

2018-02-28 Thread GitBox
eweiter opened a new issue #3370: WskBasicTests failed due to an uncaught exception during the log fetching process URL: https://github.com/apache/incubator-openwhisk/issues/3370 While running the test `Wsk Action REST should create, and invoke an action using a parameter file` from

[GitHub] csantanapr commented on issue #18: Rebase OW JavaRuntime to use OpenJDK 8

2018-02-28 Thread GitBox
csantanapr commented on issue #18: Rebase OW JavaRuntime to use OpenJDK 8 URL: https://github.com/apache/incubator-openwhisk-runtime-java/issues/18#issuecomment-369273149 Let's do the image rebase on one PR first. and then build on top of it any other improvements.

[GitHub] csantanapr commented on issue #18: Rebase OW JavaRuntime to use OpenJDK 8

2018-02-28 Thread GitBox
csantanapr commented on issue #18: Rebase OW JavaRuntime to use OpenJDK 8 URL: https://github.com/apache/incubator-openwhisk-runtime-java/issues/18#issuecomment-369272873 I would preferred to base it on a community based image instead So I propose we use:

[GitHub] jonpspri commented on a change in pull request #3358: Update to tools/ubuntu-setup to support xenial and later Ubuntu releases

2018-02-28 Thread GitBox
jonpspri commented on a change in pull request #3358: Update to tools/ubuntu-setup to support xenial and later Ubuntu releases URL: https://github.com/apache/incubator-openwhisk/pull/3358#discussion_r171272313 ## File path: tools/ubuntu-setup/java8.sh ## @@ -2,8 +2,10 @@

[GitHub] jonpspri commented on a change in pull request #3358: Update to tools/ubuntu-setup to support xenial and later Ubuntu releases

2018-02-28 Thread GitBox
jonpspri commented on a change in pull request #3358: Update to tools/ubuntu-setup to support xenial and later Ubuntu releases URL: https://github.com/apache/incubator-openwhisk/pull/3358#discussion_r171272124 ## File path: tools/ubuntu-setup/all.sh ## @@ -1,4 +1,12 @@

[GitHub] csantanapr commented on a change in pull request #3358: Update to tools/ubuntu-setup to support xenial and later Ubuntu releases

2018-02-28 Thread GitBox
csantanapr commented on a change in pull request #3358: Update to tools/ubuntu-setup to support xenial and later Ubuntu releases URL: https://github.com/apache/incubator-openwhisk/pull/3358#discussion_r171271053 ## File path: tools/ubuntu-setup/all.sh ## @@ -1,4 +1,12 @@

[GitHub] csantanapr commented on a change in pull request #3358: Update to tools/ubuntu-setup to support xenial and later Ubuntu releases

2018-02-28 Thread GitBox
csantanapr commented on a change in pull request #3358: Update to tools/ubuntu-setup to support xenial and later Ubuntu releases URL: https://github.com/apache/incubator-openwhisk/pull/3358#discussion_r171270770 ## File path: tools/ubuntu-setup/java8.sh ## @@ -2,8 +2,10

[GitHub] chetanmeh commented on issue #3277: Use Maven based dependency instead of gradle project based dependency

2018-02-28 Thread GitBox
chetanmeh commented on issue #3277: Use Maven based dependency instead of gradle project based dependency URL: https://github.com/apache/incubator-openwhisk/issues/3277#issuecomment-369260990 Opened PR on all 13 repo which need changes. Once those are merged we can make changes in core

[GitHub] Himavanth commented on issue #3232: Making prewarm kind (and count) configurable

2018-02-28 Thread GitBox
Himavanth commented on issue #3232: Making prewarm kind (and count) configurable URL: https://github.com/apache/incubator-openwhisk/pull/3232#issuecomment-369259455 Thanks @rabbah for letting me know about the PureConfig doc. I have made the changes required. I have also submitted the

[GitHub] jonpspri commented on issue #3363: Ansible image name overrides and bump NGinx to 1.13.9

2018-02-28 Thread GitBox
jonpspri commented on issue #3363: Ansible image name overrides and bump NGinx to 1.13.9 URL: https://github.com/apache/incubator-openwhisk/pull/3363#issuecomment-369259339 @markusthoemmes @cbickel Further edits made. I discussed this with @csantanapr on Slack yesterday.

[GitHub] chetanmeh opened a new pull request #234: Switch to Maven based dependency

2018-02-28 Thread GitBox
chetanmeh opened a new pull request #234: Switch to Maven based dependency URL: https://github.com/apache/incubator-openwhisk-cli/pull/234 This is required for apache/incubator-openwhisk#3277 This is an automated message from

[GitHub] rabbah commented on a change in pull request #20: [WIP] Java Improvements

2018-02-28 Thread GitBox
rabbah commented on a change in pull request #20: [WIP] Java Improvements URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/20#discussion_r171253472 ## File path: tests/src/test/scala/actionContainers/JavaActionContainerTests.scala ## @@ -87,7 +87,7 @@

[GitHub] upgle commented on issue #3343: Support time-series DB using Kamon Tag, and optional granular metric.

2018-02-28 Thread GitBox
upgle commented on issue #3343: Support time-series DB using Kamon Tag, and optional granular metric. URL: https://github.com/apache/incubator-openwhisk/pull/3343#issuecomment-369247867 @markusthoemmes has it been resolved?

[GitHub] kameshsampath commented on a change in pull request #20: [WIP] Java Improvements

2018-02-28 Thread GitBox
kameshsampath commented on a change in pull request #20: [WIP] Java Improvements URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/20#discussion_r171252055 ## File path: tests/src/test/scala/actionContainers/JavaActionContainerTests.scala ## @@ -87,7

[GitHub] kameshsampath commented on a change in pull request #20: [WIP] Java Improvements

2018-02-28 Thread GitBox
kameshsampath commented on a change in pull request #20: [WIP] Java Improvements URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/20#discussion_r171249786 ## File path: tests/src/test/scala/actionContainers/JavaActionContainerTests.scala ## @@ -87,7

[GitHub] rabbah commented on a change in pull request #20: [WIP] Java Improvements

2018-02-28 Thread GitBox
rabbah commented on a change in pull request #20: [WIP] Java Improvements URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/20#discussion_r171250363 ## File path: tests/src/test/scala/actionContainers/JavaActionContainerTests.scala ## @@ -87,7 +87,7 @@

[GitHub] kameshsampath commented on a change in pull request #20: [WIP] Java Improvements

2018-02-28 Thread GitBox
kameshsampath commented on a change in pull request #20: [WIP] Java Improvements URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/20#discussion_r171250335 ## File path: core/javaAction/proxy/src/test/java/openwhisk/java/action/JarLoaderTest.java ## @@

[GitHub] kameshsampath commented on a change in pull request #20: [WIP] Java Improvements

2018-02-28 Thread GitBox
kameshsampath commented on a change in pull request #20: [WIP] Java Improvements URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/20#discussion_r171249786 ## File path: tests/src/test/scala/actionContainers/JavaActionContainerTests.scala ## @@ -87,7

[GitHub] kameshsampath commented on a change in pull request #20: [WIP] Java Improvements

2018-02-28 Thread GitBox
kameshsampath commented on a change in pull request #20: [WIP] Java Improvements URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/20#discussion_r171249455 ## File path: core/javaAction/proxy/src/test/java/openwhisk/java/action/JarLoaderTest.java ## @@

[GitHub] rabbah commented on a change in pull request #20: [WIP] Java Improvements

2018-02-28 Thread GitBox
rabbah commented on a change in pull request #20: [WIP] Java Improvements URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/20#discussion_r171249003 ## File path: tests/src/test/scala/actionContainers/JavaActionContainerTests.scala ## @@ -87,7 +87,7 @@

[GitHub] rabbah commented on a change in pull request #20: [WIP] Java Improvements

2018-02-28 Thread GitBox
rabbah commented on a change in pull request #20: [WIP] Java Improvements URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/20#discussion_r171248815 ## File path: core/javaAction/proxy/src/test/java/openwhisk/java/action/JarLoaderTest.java ## @@ -0,0

[GitHub] rabbah commented on a change in pull request #20: [WIP] Java Improvements

2018-02-28 Thread GitBox
rabbah commented on a change in pull request #20: [WIP] Java Improvements URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/20#discussion_r171248460 ## File path: core/javaAction/proxy/src/test/java/openwhisk/java/action/JarLoaderTest.java ## @@ -0,0

[GitHub] kameshsampath commented on issue #20: [WIP] Java Improvements

2018-02-28 Thread GitBox
kameshsampath commented on issue #20: [WIP] Java Improvements URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/20#issuecomment-369243117 @rabbah > Looks OK. Why the change to the logging? Can that be avoided? Which one you refer to ? the logging from

[GitHub] chetanmeh closed issue #3312: Configure Gradle build scan plugin

2018-02-28 Thread GitBox
chetanmeh closed issue #3312: Configure Gradle build scan plugin URL: https://github.com/apache/incubator-openwhisk/issues/3312 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] chetanmeh commented on issue #3312: Configure Gradle build scan plugin

2018-02-28 Thread GitBox
chetanmeh commented on issue #3312: Configure Gradle build scan plugin URL: https://github.com/apache/incubator-openwhisk/issues/3312#issuecomment-369242695 Closed via #3314 This is an automated message from the Apache Git

[GitHub] chetanmeh opened a new pull request #261: Switch to Maven based dependency

2018-02-28 Thread GitBox
chetanmeh opened a new pull request #261: Switch to Maven based dependency URL: https://github.com/apache/incubator-openwhisk-catalog/pull/261 This is required for apache/incubator-openwhisk#3277 This is an automated message

[GitHub] kameshsampath commented on issue #20: [WIP] Java Improvements

2018-02-28 Thread GitBox
kameshsampath commented on issue #20: [WIP] Java Improvements URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/20#issuecomment-369238477 @csantanapr - can you please review and let me know what you think ?

[GitHub] chetanmeh opened a new pull request #87: Switch to Maven based dependency

2018-02-28 Thread GitBox
chetanmeh opened a new pull request #87: Switch to Maven based dependency URL: https://github.com/apache/incubator-openwhisk-package-pushnotifications/pull/87 This is required for apache/incubator-openwhisk#3277 This is an

[GitHub] chetanmeh opened a new pull request #252: Switch to Maven based dependency

2018-02-28 Thread GitBox
chetanmeh opened a new pull request #252: Switch to Maven based dependency URL: https://github.com/apache/incubator-openwhisk-package-kafka/pull/252 This is required for apache/incubator-openwhisk#3277 This is an automated

[GitHub] chetanmeh opened a new pull request #159: Switch to Maven based dependency

2018-02-28 Thread GitBox
chetanmeh opened a new pull request #159: Switch to Maven based dependency URL: https://github.com/apache/incubator-openwhisk-package-cloudant/pull/159 This is required for apache/incubator-openwhisk#3277 This is an automated

[GitHub] cbickel closed pull request #3368: Soften DockerExampleContainerTests.

2018-02-28 Thread GitBox
cbickel closed pull request #3368: Soften DockerExampleContainerTests. URL: https://github.com/apache/incubator-openwhisk/pull/3368 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] markusthoemmes commented on issue #3368: Soften DockerExampleContainerTests.

2018-02-28 Thread GitBox
markusthoemmes commented on issue #3368: Soften DockerExampleContainerTests. URL: https://github.com/apache/incubator-openwhisk/pull/3368#issuecomment-369221503 PG2 2869 ? This is an automated message from the Apache Git

[GitHub] chetanmeh commented on a change in pull request #3369: Use pureconfig for CouchDbRestStore

2018-02-28 Thread GitBox
chetanmeh commented on a change in pull request #3369: Use pureconfig for CouchDbRestStore URL: https://github.com/apache/incubator-openwhisk/pull/3369#discussion_r171225153 ## File path: common/scala/src/main/scala/whisk/core/database/CouchDbStoreProvider.scala ## @@

[GitHub] chetanmeh commented on issue #3369: Use pureconfig for CouchDbRestStore

2018-02-28 Thread GitBox
chetanmeh commented on issue #3369: Use pureconfig for CouchDbRestStore URL: https://github.com/apache/incubator-openwhisk/pull/3369#issuecomment-369220307 Travis failure in [Kafka test](https://scans.gradle.com/s/lxxoxz74thj74/tests/ubbzynwftyf6i-6k6x5jmp5qe2c). Does not look related to

[GitHub] chetanmeh commented on issue #3369: Use pureconfig for CouchDbRestStore

2018-02-28 Thread GitBox
chetanmeh commented on issue #3369: Use pureconfig for CouchDbRestStore URL: https://github.com/apache/incubator-openwhisk/pull/3369#issuecomment-369220307 Travis failure in Kafka test. Does not look related to this change and possibly timing related. Would trigger a rebuild ```

[GitHub] markusthoemmes closed pull request #3361: Monitor kafka queue for all consumers

2018-02-28 Thread GitBox
markusthoemmes closed pull request #3361: Monitor kafka queue for all consumers URL: https://github.com/apache/incubator-openwhisk/pull/3361 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] cbickel closed pull request #3367: Narrow exclude to run needed tests.

2018-02-28 Thread GitBox
cbickel closed pull request #3367: Narrow exclude to run needed tests. URL: https://github.com/apache/incubator-openwhisk/pull/3367 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

  1   2   >