[GitHub] dgrove-oss commented on a change in pull request #179: put db_host and db_port in db.config configmap

2018-04-06 Thread GitBox
dgrove-oss commented on a change in pull request #179: put db_host and db_port in db.config configmap URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/179#discussion_r179834914 ## File path: kubernetes/couchdb/README.md ## @@ -9,7 +9,7 @@ The db.auth

[GitHub] drcariel opened a new pull request #270: use utils and apigw tests from open and rename duplicate tests to be …

2018-04-06 Thread GitBox
drcariel opened a new pull request #270: use utils and apigw tests from open and rename duplicate tests to be … URL: https://github.com/apache/incubator-openwhisk-cli/pull/270 …accurate Getting rid of TestCLIUtils entirely to rely on TestUtils in open repo. Moving timestamp

[GitHub] pritidesai commented on issue #267: Install catalog with wskdeploy

2018-04-06 Thread GitBox
pritidesai commented on issue #267: Install catalog with wskdeploy URL: https://github.com/apache/incubator-openwhisk-catalog/issues/267#issuecomment-379331885 I had thought about the structure few weeks ago to add manifest in each package, I will have PR for one next week, starting from

[GitHub] dgrove-oss closed pull request #177: Test on multiple Kubernetes versions

2018-04-06 Thread GitBox
dgrove-oss closed pull request #177: Test on multiple Kubernetes versions URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/177 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] mrutkows commented on issue #171: mirror upstream ansible version upgrade to 2.4.2.0

2018-04-06 Thread GitBox
mrutkows commented on issue #171: mirror upstream ansible version upgrade to 2.4.2.0 URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/171#issuecomment-379343592 @rabbah @dgrove-oss has travis been enabled for this repo. ? If not, in the past I have opened a JIRA

[GitHub] mrutkows commented on issue #171: mirror upstream ansible version upgrade to 2.4.2.0

2018-04-06 Thread GitBox
mrutkows commented on issue #171: mirror upstream ansible version upgrade to 2.4.2.0 URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/171#issuecomment-379343592 @rabbah @dgrove-oss has travis been enabled for this repo. ? If not, in the past I have opened a JIRA

[GitHub] houshengbo commented on issue #844: Add the LICENSE and NOTICE into the binary packages

2018-04-06 Thread GitBox
houshengbo commented on issue #844: Add the LICENSE and NOTICE into the binary packages URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/844#issuecomment-379370853 @pritidesai For i18n_resources.go, wskdeploy already generates it before we release anything, so users do

[GitHub] pritidesai opened a new pull request #858: adding support to create api gateway based on sequences

2018-04-06 Thread GitBox
pritidesai opened a new pull request #858: adding support to create api gateway based on sequences URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/858 Closes #854 This is an automated message from the

[GitHub] mrutkows opened a new issue #859: Change Tag release to use Apache Bot id/account

2018-04-06 Thread GitBox
mrutkows opened a new issue #859: Change Tag release to use Apache Bot id/account URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/859 It would be good if our tagged releases were not always indicating Carlos Santana produced them. Now that we have an approved bot

[GitHub] houshengbo opened a new pull request #860: Remove the variable clibuild

2018-04-06 Thread GitBox
houshengbo opened a new pull request #860: Remove the variable clibuild URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/860 This is an automated message from the Apache Git Service. To respond to the

[GitHub] brunogirin opened a new pull request #861: Partial fix for issue #847

2018-04-06 Thread GitBox
brunogirin opened a new pull request #861: Partial fix for issue #847 URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/861 This example shows a simple sequence exposed via the API Gateway. It doesn't handle the `web: raw` option as I haven't got experience with this option

[GitHub] mrutkows closed pull request #858: adding support to create api gateway based on sequences

2018-04-06 Thread GitBox
mrutkows closed pull request #858: adding support to create api gateway based on sequences URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/858 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] mrutkows closed issue #854: API Gateway doesn't take sequences into account

2018-04-06 Thread GitBox
mrutkows closed issue #854: API Gateway doesn't take sequences into account URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/854 This is an automated message from the Apache Git Service. To respond to the

[GitHub] drcariel opened a new pull request #3531: More test clean up

2018-04-06 Thread GitBox
drcariel opened a new pull request #3531: More test clean up URL: https://github.com/apache/incubator-openwhisk/pull/3531 My attempt to finsih up the final 3 files in https://github.com/apache/incubator-openwhisk-cli/issues/259 along with some other test clean up ##

[GitHub] drcariel commented on issue #270: use utils and apigw tests from open and rename duplicate tests to be …

2018-04-06 Thread GitBox
drcariel commented on issue #270: use utils and apigw tests from open and rename duplicate tests to be … URL: https://github.com/apache/incubator-openwhisk-cli/pull/270#issuecomment-379370116 tests will not compile until companion is merged

[GitHub] rabbah closed pull request #179: put db_host and db_port in db.config configmap

2018-04-06 Thread GitBox
rabbah closed pull request #179: put db_host and db_port in db.config configmap URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/179 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 #179: put db_host and db_port in db.config configmap

2018-04-06 Thread GitBox
rabbah commented on issue #179: put db_host and db_port in db.config configmap URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/179#issuecomment-379333286 @csantanapr this repo might not be configured correctly - I was able to bypass travis.  

[GitHub] beemarie opened a new pull request #23: WIP: updating wskdeploy version

2018-04-06 Thread GitBox
beemarie opened a new pull request #23: WIP: updating wskdeploy version URL: https://github.com/apache/incubator-openwhisk-package-deploy/pull/23 This is an automated message from the Apache Git Service. To respond to the

[GitHub] mrutkows commented on issue #844: Add the LICENSE and NOTICE into the binary packages

2018-04-06 Thread GitBox
mrutkows commented on issue #844: Add the LICENSE and NOTICE into the binary packages URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/844#issuecomment-379397843 @houshengbo surely, there is an Apache exception for generated files after all they are not "licensed" by

[GitHub] pritidesai closed pull request #860: Remove the variable clibuild

2018-04-06 Thread GitBox
pritidesai closed pull request #860: Remove the variable clibuild URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/860 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 #3527: Add build matrix to execute perormance tests.

2018-04-06 Thread GitBox
markusthoemmes closed pull request #3527: Add build matrix to execute perormance tests. URL: https://github.com/apache/incubator-openwhisk/pull/3527 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] dgrove-oss commented on issue #104: update travis.yml to send notifications to Slack on build failures

2018-04-06 Thread GitBox
dgrove-oss commented on issue #104: update travis.yml to send notifications to Slack on build failures URL: https://github.com/apache/incubator-openwhisk-devtools/issues/104#issuecomment-379238743 No, travis uses a repo-specific key to encrypt the URL. I can take care of doing it today.

[GitHub] sciabarracom commented on issue #6: Multiarch framework for Go runtime

2018-04-06 Thread GitBox
sciabarracom commented on issue #6: Multiarch framework for Go runtime URL: https://github.com/apache/incubator-openwhisk-runtime-go/pull/6#issuecomment-379237870 Ok add the read me norpblems This is an automated message

[GitHub] sciabarracom commented on issue #6: Multiarch framework for Go runtime

2018-04-06 Thread GitBox
sciabarracom commented on issue #6: Multiarch framework for Go runtime URL: https://github.com/apache/incubator-openwhisk-runtime-go/pull/6#issuecomment-379237870 Ok add the read me no problem This is an automated message

[GitHub] mdeuser commented on a change in pull request #267: --web-secure generated integer must be within api gw max int range

2018-04-06 Thread GitBox
mdeuser commented on a change in pull request #267: --web-secure generated integer must be within api gw max int range URL: https://github.com/apache/incubator-openwhisk-cli/pull/267#discussion_r179744124 ## File path: commands/action.go ## @@ -1194,10 +1195,12 @@ func

[GitHub] mdeuser commented on issue #3529: Authentication issues when installing openwhisk

2018-04-06 Thread GitBox
mdeuser commented on issue #3529: Authentication issues when installing openwhisk URL: https://github.com/apache/incubator-openwhisk/issues/3529#issuecomment-379245063 @elenagarrido - can you run the following in your vagrant vm and post the results here ``` export

[GitHub] chetanmeh commented on issue #3523: KafkaConnectorTests heisenbug

2018-04-06 Thread GitBox
chetanmeh commented on issue #3523: KafkaConnectorTests heisenbug URL: https://github.com/apache/incubator-openwhisk/issues/3523#issuecomment-379211351 https://scans.gradle.com/s/kjbvfxcbw3j7y/tests Here failure was due to timeout ``` java.util.concurrent.TimeoutException:

[GitHub] chetanmeh commented on issue #3523: KafkaConnectorTests heisenbug

2018-04-06 Thread GitBox
chetanmeh commented on issue #3523: KafkaConnectorTests heisenbug URL: https://github.com/apache/incubator-openwhisk/issues/3523#issuecomment-379211351 https://scans.gradle.com/s/kjbvfxcbw3j7y/tests https://travis-ci.org/chetanmeh/incubator-openwhisk/builds/363019106 Here failure

[GitHub] rabbah opened a new issue #3530: factor out max code blob size to a deployment configuration

2018-04-06 Thread GitBox
rabbah opened a new issue #3530: factor out max code blob size to a deployment configuration URL: https://github.com/apache/incubator-openwhisk/issues/3530 From a conversation on slack, it was pointed out by @chetanmeh that we are hardcoding the exec max size

[GitHub] rabbah commented on issue #3106: Couchdb persistency

2018-04-06 Thread GitBox
rabbah commented on issue #3106: Couchdb persistency URL: https://github.com/apache/incubator-openwhisk/pull/3106#issuecomment-379219123 For the custom image you’re thinking to add the dockerfile in this repo? If so I think that works then can publish the image to dockerhub. I’d check

[GitHub] rabbah closed pull request #5: The go proxy

2018-04-06 Thread GitBox
rabbah closed pull request #5: The go proxy URL: https://github.com/apache/incubator-openwhisk-runtime-go/pull/5 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] elenagarrido opened a new issue #3529: Authentication issues when installing openwhisk

2018-04-06 Thread GitBox
elenagarrido opened a new issue #3529: Authentication issues when installing openwhisk URL: https://github.com/apache/incubator-openwhisk/issues/3529 ## Environment details: * windows 7 * Vagrant 2.0.2 * Virtual Box Version 5.2.8 r121009 ## Steps to reproduce the

[GitHub] kameshsampath commented on issue #102: Maven Java example does not work

2018-04-06 Thread GitBox
kameshsampath commented on issue #102: Maven Java example does not work URL: https://github.com/apache/incubator-openwhisk-devtools/issues/102#issuecomment-379173972 We can close it I have fixed it for @galderz it was more of setup and config of OW

[GitHub] rabbah commented on a change in pull request #5: The go proxy

2018-04-06 Thread GitBox
rabbah commented on a change in pull request #5: The go proxy URL: https://github.com/apache/incubator-openwhisk-runtime-go/pull/5#discussion_r179725739 ## File path: openwhisk/executor.go ## @@ -0,0 +1,208 @@ +/* + * Licensed to the Apache Software Foundation (ASF)

[GitHub] rabbah commented on issue #5: The go proxy

2018-04-06 Thread GitBox
rabbah commented on issue #5: The go proxy URL: https://github.com/apache/incubator-openwhisk-runtime-go/pull/5#issuecomment-378333255 hehe - no worries you can leave it. I was able to follow the code nicely even though I’ve written a grand total of 100 lines of go code.

[GitHub] rabbah commented on issue #5: The go proxy

2018-04-06 Thread GitBox
rabbah commented on issue #5: The go proxy URL: https://github.com/apache/incubator-openwhisk-runtime-go/pull/5#issuecomment-378331138 I meant that you factor you the loop into a function and call it twice. First for stdout then for stderr. Anyway it’s a small nit.

[GitHub] rabbah commented on issue #5: The go proxy

2018-04-06 Thread GitBox
rabbah commented on issue #5: The go proxy URL: https://github.com/apache/incubator-openwhisk-runtime-go/pull/5#issuecomment-379221583 There is more to come on this runtime enablement and this is a good first step. I am going merge it so unblock and allow some more iterations to proceed

[GitHub] rabbah commented on issue #6: Multiarch framework for Go runtime

2018-04-06 Thread GitBox
rabbah commented on issue #6: Multiarch framework for Go runtime URL: https://github.com/apache/incubator-openwhisk-runtime-go/pull/6#issuecomment-379223210 @jonpspri @sciabarracom i resolved the conflicts in the PR and can merge it.

[GitHub] rabbah commented on issue #6: Multiarch framework for Go runtime

2018-04-06 Thread GitBox
rabbah commented on issue #6: Multiarch framework for Go runtime URL: https://github.com/apache/incubator-openwhisk-runtime-go/pull/6#issuecomment-379223473 i'd like to put a disclaimer on this repo/in the readme that this is a work in progress and not yet fully integrated with openwhisk

[GitHub] markusthoemmes closed pull request #5: Depreaction message.

2018-04-06 Thread GitBox
markusthoemmes closed pull request #5: Depreaction message. URL: https://github.com/apache/incubator-openwhisk-performance/pull/5 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] brunogirin commented on issue #31: Java action that depends on packages jar file fails

2018-04-06 Thread GitBox
brunogirin commented on issue #31: Java action that depends on packages jar file fails URL: https://github.com/apache/incubator-openwhisk-runtime-java/issues/31#issuecomment-379170866 When I check the jar file, here's what's inside, the additional jar are in the `lib` folder in the main

[GitHub] dgrove-oss commented on issue #111: add url for travis2slack integration

2018-04-06 Thread GitBox
dgrove-oss commented on issue #111: add url for travis2slack integration URL: https://github.com/apache/incubator-openwhisk-devtools/pull/111#issuecomment-379276331 I'll post a screenshot of the slack notification here to verify it is working. please wait for that before approving (in

[GitHub] mrutkows commented on issue #267: Install catalog with wskdeploy

2018-04-06 Thread GitBox
mrutkows commented on issue #267: Install catalog with wskdeploy URL: https://github.com/apache/incubator-openwhisk-catalog/issues/267#issuecomment-379295244 @ddragosd agreed... @pritidesai and some of us have talked about this for some time; perhaps when we finish preparing all the

[GitHub] mstoodle commented on issue #3504: use alpine-java base image (alpine 3.7 + oracle 8u162b12_server-jre)

2018-04-06 Thread GitBox
mstoodle commented on issue #3504: use alpine-java base image (alpine 3.7 + oracle 8u162b12_server-jre) URL: https://github.com/apache/incubator-openwhisk/pull/3504#issuecomment-379282892 @tysonnorris @markusthoemmes For information about Eclipse OpenJ9 and why it might be interesting

[GitHub] mstoodle commented on issue #3504: use alpine-java base image (alpine 3.7 + oracle 8u162b12_server-jre)

2018-04-06 Thread GitBox
mstoodle commented on issue #3504: use alpine-java base image (alpine 3.7 + oracle 8u162b12_server-jre) URL: https://github.com/apache/incubator-openwhisk/pull/3504#issuecomment-379282892 @tysonnorris @markusthoemmes For information about Eclipse OpenJ9 and why it might be interesting

[GitHub] mstoodle commented on issue #3504: use alpine-java base image (alpine 3.7 + oracle 8u162b12_server-jre)

2018-04-06 Thread GitBox
mstoodle commented on issue #3504: use alpine-java base image (alpine 3.7 + oracle 8u162b12_server-jre) URL: https://github.com/apache/incubator-openwhisk/pull/3504#issuecomment-379282892 @tysonnorris @markusthoemmes For information about Eclipse OpenJ9 and why it might be interesting

[GitHub] mstoodle commented on issue #3504: use alpine-java base image (alpine 3.7 + oracle 8u162b12_server-jre)

2018-04-06 Thread GitBox
mstoodle commented on issue #3504: use alpine-java base image (alpine 3.7 + oracle 8u162b12_server-jre) URL: https://github.com/apache/incubator-openwhisk/pull/3504#issuecomment-379282892 @tysonnorris For information about Eclipse OpenJ9 and why it might be interesting for a project like

[GitHub] mstoodle commented on issue #3504: use alpine-java base image (alpine 3.7 + oracle 8u162b12_server-jre)

2018-04-06 Thread GitBox
mstoodle commented on issue #3504: use alpine-java base image (alpine 3.7 + oracle 8u162b12_server-jre) URL: https://github.com/apache/incubator-openwhisk/pull/3504#issuecomment-379282892 @tysonnorris @markusthoemmes For information about OpenJDK with Eclipse OpenJ9 and why it might be

[GitHub] csantanapr closed issue #104: update travis.yml to send notifications to Slack on build failures

2018-04-06 Thread GitBox
csantanapr closed issue #104: update travis.yml to send notifications to Slack on build failures URL: https://github.com/apache/incubator-openwhisk-devtools/issues/104 This is an automated message from the Apache Git

[GitHub] csantanapr closed pull request #111: add url for travis2slack integration

2018-04-06 Thread GitBox
csantanapr closed pull request #111: add url for travis2slack integration URL: https://github.com/apache/incubator-openwhisk-devtools/pull/111 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] dgrove-oss commented on issue #3106: Couchdb persistency

2018-04-06 Thread GitBox
dgrove-oss commented on issue #3106: Couchdb persistency URL: https://github.com/apache/incubator-openwhisk/pull/3106#issuecomment-379272260 It would be nice if we could have a single openwhisk/couchdb image that would work for both the main project and the downstream projects like kube.

[GitHub] dgrove-oss opened a new pull request #111: add url for travis2slack integration

2018-04-06 Thread GitBox
dgrove-oss opened a new pull request #111: add url for travis2slack integration URL: https://github.com/apache/incubator-openwhisk-devtools/pull/111 Fixes #104 This is an automated message from the Apache Git Service. To

[GitHub] dgrove-oss commented on issue #111: add url for travis2slack integration

2018-04-06 Thread GitBox
dgrove-oss commented on issue #111: add url for travis2slack integration URL: https://github.com/apache/incubator-openwhisk-devtools/pull/111#issuecomment-379278180 working. ![screen shot 2018-04-06 at 10 52 17

[GitHub] mstoodle commented on issue #3504: use alpine-java base image (alpine 3.7 + oracle 8u162b12_server-jre)

2018-04-06 Thread GitBox
mstoodle commented on issue #3504: use alpine-java base image (alpine 3.7 + oracle 8u162b12_server-jre) URL: https://github.com/apache/incubator-openwhisk/pull/3504#issuecomment-379282892 @tysonnorris @markusthoemmes For information about OpenJDK with Eclipse OpenJ9 and why it might be

[GitHub] mrutkows commented on issue #3264: A list of source files miss Apache license headers

2018-04-06 Thread GitBox
mrutkows commented on issue #3264: A list of source files miss Apache license headers URL: https://github.com/apache/incubator-openwhisk/issues/3264#issuecomment-379315586 @daisy-ycguo @csantanapr I believe Daisy put together a slide/chart which she was going to publish to

[GitHub] chetanmeh commented on a change in pull request #3249: #3243 - Use gradle scoverage to collect test coverage stats

2018-04-06 Thread GitBox
chetanmeh commented on a change in pull request #3249: #3243 - Use gradle scoverage to collect test coverage stats URL: https://github.com/apache/incubator-openwhisk/pull/3249#discussion_r179813125 ## File path: tools/travis/build.sh ## @@ -66,10 +66,12 @@ $ANSIBLE_CMD

[GitHub] chetanmeh commented on a change in pull request #3249: #3243 - Use gradle scoverage to collect test coverage stats

2018-04-06 Thread GitBox
chetanmeh commented on a change in pull request #3249: #3243 - Use gradle scoverage to collect test coverage stats URL: https://github.com/apache/incubator-openwhisk/pull/3249#discussion_r179813148 ## File path: settings.gradle ## @@ -29,3 +29,8 @@ gradle.ext.scalafmt =

[GitHub] chetanmeh commented on a change in pull request #3249: #3243 - Use gradle scoverage to collect test coverage stats

2018-04-06 Thread GitBox
chetanmeh commented on a change in pull request #3249: #3243 - Use gradle scoverage to collect test coverage stats URL: https://github.com/apache/incubator-openwhisk/pull/3249#discussion_r179813302 ## File path: tests/build.gradle ## @@ -93,7 +102,63 @@ task

[GitHub] rabbah commented on a change in pull request #179: put db_host and db_port in db.config configmap

2018-04-06 Thread GitBox
rabbah commented on a change in pull request #179: put db_host and db_port in db.config configmap URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/179#discussion_r179831057 ## File path: kubernetes/couchdb/README.md ## @@ -9,7 +9,7 @@ The db.auth