[GitHub] [openwhisk] chetanmeh opened a new pull request #4594: CouchDB support in OpenWhisk Standalone mode

2019-08-28 Thread GitBox
chetanmeh opened a new pull request #4594: CouchDB support in OpenWhisk Standalone mode URL: https://github.com/apache/openwhisk/pull/4594 Enable using CouchDB with standalone mode (#4516). This would be useful for cases where users would like to save there changes upon restarts. ##

[GitHub] [openwhisk] chetanmeh opened a new pull request #4595: Pass an optional action instance to ContainerFactory

2019-08-28 Thread GitBox
chetanmeh opened a new pull request #4595: Pass an optional action instance to ContainerFactory URL: https://github.com/apache/openwhisk/pull/4595 Pass the action instance (for cold start case) as an optional argument to ContainerFactory ## Description For enabling integratio

[GitHub] [openwhisk] chetanmeh merged pull request #4590: update lastUse timestamp when advancing ContainerProxy instances in use

2019-08-28 Thread GitBox
chetanmeh merged pull request #4590: update lastUse timestamp when advancing ContainerProxy instances in use URL: https://github.com/apache/openwhisk/pull/4590 This is an automated message from the Apache Git Service. To res

[GitHub] [openwhisk] chetanmeh commented on a change in pull request #4595: Pass an optional action instance to ContainerFactory

2019-08-28 Thread GitBox
chetanmeh commented on a change in pull request #4595: Pass an optional action instance to ContainerFactory URL: https://github.com/apache/openwhisk/pull/4595#discussion_r318532489 ## File path: common/scala/src/main/scala/org/apache/openwhisk/core/containerpool/ContainerFactory.sc

[GitHub] [openwhisk-package-alarms] style95 commented on issue #198: Make cron triggers fire with random delay with parameter `strict`

2019-08-28 Thread GitBox
style95 commented on issue #198: Make cron triggers fire with random delay with parameter `strict` URL: https://github.com/apache/openwhisk-package-alarms/pull/198#issuecomment-525709692 @machenity Thank you for the contribution. It would be great to share the impact of this change.

[GitHub] [openwhisk] chetanmeh commented on a change in pull request #4584: OpenWhisk User Events

2019-08-28 Thread GitBox
chetanmeh commented on a change in pull request #4584: OpenWhisk User Events URL: https://github.com/apache/openwhisk/pull/4584#discussion_r318535480 ## File path: core/monitoring/user-events/src/main/resources/logback.xml ## @@ -0,0 +1,17 @@ + Review comment: This ca

[GitHub] [openwhisk] chetanmeh commented on a change in pull request #4584: OpenWhisk User Events

2019-08-28 Thread GitBox
chetanmeh commented on a change in pull request #4584: OpenWhisk User Events URL: https://github.com/apache/openwhisk/pull/4584#discussion_r318536475 ## File path: tests/build.gradle ## @@ -46,7 +46,8 @@ def projectsWithCoverage = [ ':core:controller', ':core:invo

[GitHub] [openwhisk] chetanmeh commented on a change in pull request #4584: OpenWhisk User Events

2019-08-28 Thread GitBox
chetanmeh commented on a change in pull request #4584: OpenWhisk User Events URL: https://github.com/apache/openwhisk/pull/4584#discussion_r318535615 ## File path: core/monitoring/user-events/src/main/resources/logging.conf ## @@ -0,0 +1,22 @@ +# Review comment: This

[GitHub] [openwhisk-package-alarms] style95 commented on a change in pull request #198: Make cron triggers fire with random delay with parameter `strict`

2019-08-28 Thread GitBox
style95 commented on a change in pull request #198: Make cron triggers fire with random delay with parameter `strict` URL: https://github.com/apache/openwhisk-package-alarms/pull/198#discussion_r318537578 ## File path: provider/lib/cronAlarm.js ## @@ -85,4 +85,27 @@ modul

[GitHub] [openwhisk-package-alarms] style95 commented on a change in pull request #198: Make cron triggers fire with random delay with parameter `strict`

2019-08-28 Thread GitBox
style95 commented on a change in pull request #198: Make cron triggers fire with random delay with parameter `strict` URL: https://github.com/apache/openwhisk-package-alarms/pull/198#discussion_r318537830 ## File path: action/alarmWebAction.js ## @@ -103,6 +103,7 @@ funct

[GitHub] [openwhisk] codecov-io commented on issue #4595: Pass an optional action instance to ContainerFactory

2019-08-28 Thread GitBox
codecov-io commented on issue #4595: Pass an optional action instance to ContainerFactory URL: https://github.com/apache/openwhisk/pull/4595#issuecomment-525717300 # [Codecov](https://codecov.io/gh/apache/openwhisk/pull/4595?src=pr&el=h1) Report > Merging [#4595](https://codecov.io/gh/

[GitHub] [openwhisk] ruediger-maass opened a new pull request #4596: Allow individual field sets for activation_record and user_log

2019-08-28 Thread GitBox
ruediger-maass opened a new pull request #4596: Allow individual field sets for activation_record and user_log URL: https://github.com/apache/openwhisk/pull/4596 Introduce individual field set for activation records in user logs. ## Description We need to handle activation records

[GitHub] [openwhisk] codecov-io commented on issue #4596: Allow individual field sets for activation_record and user_log

2019-08-28 Thread GitBox
codecov-io commented on issue #4596: Allow individual field sets for activation_record and user_log URL: https://github.com/apache/openwhisk/pull/4596#issuecomment-525742834 # [Codecov](https://codecov.io/gh/apache/openwhisk/pull/4596?src=pr&el=h1) Report > Merging [#4596](https://code

[GitHub] [openwhisk-devtools] cesardraw2 commented on issue #276: make quick start fails - error in db

2019-08-28 Thread GitBox
cesardraw2 commented on issue #276: make quick start fails - error in db URL: https://github.com/apache/openwhisk-devtools/issues/276#issuecomment-525749556 Hello! Could you find the solution? This is an automated message fro

[GitHub] [openwhisk] mhenke1 commented on issue #4415: Remove path projections on web actions.

2019-08-28 Thread GitBox
mhenke1 commented on issue #4415: Remove path projections on web actions. URL: https://github.com/apache/openwhisk/pull/4415#issuecomment-525760816 IBM Functions sees uses of `.text`and `.json`projections. I have started a [discussion on the mailing list](https://lists.apache.org/thread

[GitHub] [openwhisk] mhenke1 edited a comment on issue #4415: Remove path projections on web actions.

2019-08-28 Thread GitBox
mhenke1 edited a comment on issue #4415: Remove path projections on web actions. URL: https://github.com/apache/openwhisk/pull/4415#issuecomment-525760816 IBM Functions sees usage of `.text`and `.json`projections. I have started a [discussion on the mailing list](https://lists.apache.or

[GitHub] [openwhisk] rabbah commented on issue #4596: Allow individual field sets for activation_record and user_log

2019-08-28 Thread GitBox
rabbah commented on issue #4596: Allow individual field sets for activation_record and user_log URL: https://github.com/apache/openwhisk/pull/4596#issuecomment-525765999 @ruediger-maass this looks ok but please open a dev list thread with a pointer to the PR. -

[GitHub] [openwhisk-devtools] pepiparaskev30 commented on issue #276: make quick start fails - error in db

2019-08-28 Thread GitBox
pepiparaskev30 commented on issue #276: make quick start fails - error in db URL: https://github.com/apache/openwhisk-devtools/issues/276#issuecomment-525770099 Yes On Wed, 28 Aug 2019 at 16:39, cesardraw2 wrote: > Hello! Could you find the solution? > > — > You are

[GitHub] [openwhisk-composer] NikolasMcGovern-IBM opened a new issue #50: Unable to deploy to an IAM-based namespace

2019-08-28 Thread GitBox
NikolasMcGovern-IBM opened a new issue #50: Unable to deploy to an IAM-based namespace URL: https://github.com/apache/openwhisk-composer/issues/50 IBM Cloud Functions is transitioning from CF-based namespaces to IAM-based namespaces, but the deploy command only works for CF-based namespace

[GitHub] [openwhisk] ruediger-maass commented on issue #4596: Allow individual field sets for activation_record and user_log

2019-08-28 Thread GitBox
ruediger-maass commented on issue #4596: Allow individual field sets for activation_record and user_log URL: https://github.com/apache/openwhisk/pull/4596#issuecomment-525829345 @rabbah thank you for the hint - done. Next I time I'll do that right away.

[GitHub] [openwhisk] codecov-io commented on issue #4594: CouchDB support in OpenWhisk Standalone mode

2019-08-28 Thread GitBox
codecov-io commented on issue #4594: CouchDB support in OpenWhisk Standalone mode URL: https://github.com/apache/openwhisk/pull/4594#issuecomment-525838761 # [Codecov](https://codecov.io/gh/apache/openwhisk/pull/4594?src=pr&el=h1) Report > Merging [#4594](https://codecov.io/gh/apache/o

[GitHub] [openwhisk] codecov-io edited a comment on issue #4584: OpenWhisk User Events

2019-08-28 Thread GitBox
codecov-io edited a comment on issue #4584: OpenWhisk User Events URL: https://github.com/apache/openwhisk/pull/4584#issuecomment-525009358 # [Codecov](https://codecov.io/gh/apache/openwhisk/pull/4584?src=pr&el=h1) Report > Merging [#4584](https://codecov.io/gh/apache/openwhisk/pull/458

[GitHub] [openwhisk-deploy-kube] ChangZihao opened a new issue #512: helm install .... --name owdev-name-test can not have "-" in it

2019-08-28 Thread GitBox
ChangZihao opened a new issue #512: helm install --name owdev-name-test can not have "-" in it URL: https://github.com/apache/openwhisk-deploy-kube/issues/512 I deploy the openwhisk with helm in minikube on my ubuntu 18.04 server. If I set the --name with '-' like owdev-name-te

[GitHub] [openwhisk-devtools] jiangpengcheng commented on issue #280: Restart controller/invoker by docker-compose tool does not work

2019-08-28 Thread GitBox
jiangpengcheng commented on issue #280: Restart controller/invoker by docker-compose tool does not work URL: https://github.com/apache/openwhisk-devtools/issues/280#issuecomment-526007690 this can be resolved by https://github.com/apache/openwhisk-devtools/pull/262 --

[GitHub] [openwhisk-deploy-kube] ChangZihao opened a new pull request #513: fix: kafka, zookeeper, controller get num in hostname

2019-08-28 Thread GitBox
ChangZihao opened a new pull request #513: fix: kafka, zookeeper, controller get num in hostname URL: https://github.com/apache/openwhisk-deploy-kube/pull/513 #512 This is an automated message from the Apache Git Service. To

[GitHub] [openwhisk] chetanmeh commented on issue #4594: CouchDB support in OpenWhisk Standalone mode

2019-08-28 Thread GitBox
chetanmeh commented on issue #4594: CouchDB support in OpenWhisk Standalone mode URL: https://github.com/apache/openwhisk/pull/4594#issuecomment-526016029 This PR is now ready for review This is an automated message from the A

[GitHub] [openwhisk] chetanmeh commented on a change in pull request #4596: Allow individual field sets for activation_record and user_log

2019-08-28 Thread GitBox
chetanmeh commented on a change in pull request #4596: Allow individual field sets for activation_record and user_log URL: https://github.com/apache/openwhisk/pull/4596#discussion_r318885559 ## File path: common/scala/src/main/scala/org/apache/openwhisk/core/database/ActivationFile

[GitHub] [openwhisk] chetanmeh commented on a change in pull request #4596: Allow individual field sets for activation_record and user_log

2019-08-28 Thread GitBox
chetanmeh commented on a change in pull request #4596: Allow individual field sets for activation_record and user_log URL: https://github.com/apache/openwhisk/pull/4596#discussion_r318885559 ## File path: common/scala/src/main/scala/org/apache/openwhisk/core/database/ActivationFile

[GitHub] [openwhisk-deploy-kube] chetanmeh commented on issue #436: Api Gateway not working on Docker Desktop Windows/Mac

2019-08-28 Thread GitBox
chetanmeh commented on issue #436: Api Gateway not working on Docker Desktop Windows/Mac URL: https://github.com/apache/openwhisk-deploy-kube/issues/436#issuecomment-526025731 > if we should attempt to fix/generalize so that the deployment can override to use the internal api host as the

[GitHub] [openwhisk-deploy-kube] chetanmeh commented on issue #513: fix: kafka, zookeeper, controller get num in hostname

2019-08-28 Thread GitBox
chetanmeh commented on issue #513: fix: kafka, zookeeper, controller get num in hostname URL: https://github.com/apache/openwhisk-deploy-kube/pull/513#issuecomment-526028277 Thanks for the fix. This thing hit us also once when we used `-` in release names. Good to have a proper fix for th

[GitHub] [openwhisk-deploy-kube] BillZong opened a new pull request #514: Remove duplicate configurations already in template

2019-08-28 Thread GitBox
BillZong opened a new pull request #514: Remove duplicate configurations already in template URL: https://github.com/apache/openwhisk-deploy-kube/pull/514 ## Description Remove duplicate configurations already in template, it's in file [`helm/openwhisk/templates/_helpers.tpl`](https

[GitHub] [openwhisk-deploy-kube] BillZong commented on issue #514: Remove duplicate configurations already in template

2019-08-28 Thread GitBox
BillZong commented on issue #514: Remove duplicate configurations already in template URL: https://github.com/apache/openwhisk-deploy-kube/pull/514#issuecomment-526037693 Close this PR for not adding a reviewer. This is an a

[GitHub] [openwhisk-deploy-kube] BillZong closed pull request #514: Remove duplicate configurations already in template

2019-08-28 Thread GitBox
BillZong closed pull request #514: Remove duplicate configurations already in template URL: https://github.com/apache/openwhisk-deploy-kube/pull/514 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [openwhisk-deploy-kube] BillZong opened a new pull request #515: Remove duplicate configurations already in template

2019-08-28 Thread GitBox
BillZong opened a new pull request #515: Remove duplicate configurations already in template URL: https://github.com/apache/openwhisk-deploy-kube/pull/515 ## Description Remove duplicate configurations already in template, it's in file [`helm/openwhisk/templates/_helpers.tpl`](https

[GitHub] [openwhisk-deploy-kube] BillZong closed pull request #515: Remove duplicate configurations already in template

2019-08-28 Thread GitBox
BillZong closed pull request #515: Remove duplicate configurations already in template URL: https://github.com/apache/openwhisk-deploy-kube/pull/515 This is an automated message from the Apache Git Service. To respond to the