[GitHub] ningyougang commented on issue #3432: Introduce jmx for kafka

2018-03-13 Thread GitBox
ningyougang commented on issue #3432: Introduce jmx for kafka URL: https://github.com/apache/incubator-openwhisk/pull/3432#issuecomment-372915971 Any suggestion? This is an automated message from the Apache Git Service. To re

[GitHub] ningyougang commented on issue #3432: Introduce jmx for kafka

2018-03-13 Thread GitBox
ningyougang commented on issue #3432: Introduce jmx for kafka URL: https://github.com/apache/incubator-openwhisk/pull/3432#issuecomment-372915814 How strange, in my local building env, don't export below error ``` /root/jmxremote.password\nError: Password file not readable: /root/jmx

[GitHub] ningyougang commented on issue #3432: Introduce jmx for kafka

2018-03-13 Thread GitBox
ningyougang commented on issue #3432: Introduce jmx for kafka URL: https://github.com/apache/incubator-openwhisk/pull/3432#issuecomment-372915814 How strange, in my local building env, don't export below error ``` /root/jmxremote.password\nError: Password file not readable: /root/jmx

[GitHub] chetanmeh commented on issue #3306: Script to generate IntelliJ Run Configuration for Controller and Invoker

2018-03-13 Thread GitBox
chetanmeh commented on issue #3306: Script to generate IntelliJ Run Configuration for Controller and Invoker URL: https://github.com/apache/incubator-openwhisk/pull/3306#issuecomment-372904173 @rabbah Can you review this Thi

[GitHub] chetanmeh commented on issue #3431: Vagrant provisioning results in SSLV3_ALERT_BAD_CERTIFICATE and failure to ping https://172.17.0.1:10001/ping

2018-03-13 Thread GitBox
chetanmeh commented on issue #3431: Vagrant provisioning results in SSLV3_ALERT_BAD_CERTIFICATE and failure to ping https://172.17.0.1:10001/ping URL: https://github.com/apache/incubator-openwhisk/issues/3431#issuecomment-372903541 @develpr This may be due to vagrant setup failing in betwe

[GitHub] dubeejw commented on issue #3421: Add ElasticSearch Log Store

2018-03-13 Thread GitBox
dubeejw commented on issue #3421: Add ElasticSearch Log Store URL: https://github.com/apache/incubator-openwhisk/pull/3421#issuecomment-372867292 PG2 2918 👍 This is an automated message from the Apache Git Service. To respon

[GitHub] csantanapr opened a new pull request #26: update to pickup openwhisk@3.13.0

2018-03-13 Thread GitBox
csantanapr opened a new pull request #26: update to pickup openwhisk@3.13.0 URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/26 This is an automated message from the Apache Git Service. To respond to the

[GitHub] rabbah closed pull request #104: v3.13.0 NPM version bump

2018-03-13 Thread GitBox
rabbah closed pull request #104: v3.13.0 NPM version bump URL: https://github.com/apache/incubator-openwhisk-client-js/pull/104 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

[GitHub] rabbah closed issue #103: 3.13.0 release

2018-03-13 Thread GitBox
rabbah closed issue #103: 3.13.0 release URL: https://github.com/apache/incubator-openwhisk-client-js/issues/103 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and us

[GitHub] codecov-io commented on issue #104: v3.13.0 NPM version bump

2018-03-13 Thread GitBox
codecov-io commented on issue #104: v3.13.0 NPM version bump URL: https://github.com/apache/incubator-openwhisk-client-js/pull/104#issuecomment-372890484 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk-client-js/pull/104?src=pr&el=h1) Report > Merging [#104](https://codec

[GitHub] csantanapr opened a new pull request #104: v3.13.0 NPM version bump

2018-03-13 Thread GitBox
csantanapr opened a new pull request #104: v3.13.0 NPM version bump URL: https://github.com/apache/incubator-openwhisk-client-js/pull/104 Closes #103 This is an automated message from the Apache Git Service. To respond to the

[GitHub] csantanapr opened a new issue #103: 3.13.0 release

2018-03-13 Thread GitBox
csantanapr opened a new issue #103: 3.13.0 release URL: https://github.com/apache/incubator-openwhisk-client-js/issues/103 Minor release, rather than patch, as #99 is a new feature. ## release notes ### 3.13.0 (2018.03.14) - Adds an optional query parameter for action f

[GitHub] csantanapr closed pull request #99: Add Code Query Parameter

2018-03-13 Thread GitBox
csantanapr closed pull request #99: Add Code Query Parameter URL: https://github.com/apache/incubator-openwhisk-client-js/pull/99 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 foreig

[GitHub] csantanapr closed pull request #98: ShortDesc name field is optional

2018-03-13 Thread GitBox
csantanapr closed pull request #98: ShortDesc name field is optional URL: https://github.com/apache/incubator-openwhisk-client-js/pull/98 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 commented on issue #98: ShortDesc name field is optional

2018-03-13 Thread GitBox
csantanapr commented on issue #98: ShortDesc name field is optional URL: https://github.com/apache/incubator-openwhisk-client-js/pull/98#issuecomment-372887567 rebased. This is an automated message from the Apache Git Service

[GitHub] csantanapr opened a new issue #3433: Remove all runtimes unit tests

2018-03-13 Thread GitBox
csantanapr opened a new issue #3433: Remove all runtimes unit tests URL: https://github.com/apache/incubator-openwhisk/issues/3433 Runtimes unit tests are already moved to their corresponding repos. Also the CLI have moved to it's own repo. The unit tests under `actionContainers` shoul

[GitHub] pritidesai commented on issue #790: Add support to export binary (zip and jar) actions

2018-03-13 Thread GitBox
pritidesai commented on issue #790: Add support to export binary (zip and jar) actions URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/790#issuecomment-372887027 we have two new IDs in `wski18`, thanks @kpavel

[GitHub] pritidesai commented on issue #790: Add support to export binary (zip and jar) actions

2018-03-13 Thread GitBox
pritidesai commented on issue #790: Add support to export binary (zip and jar) actions URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/790#issuecomment-372887027 we have two new IDs in `wski18` This is an a

[GitHub] codecov-io commented on issue #98: ShortDesc name field is optional

2018-03-13 Thread GitBox
codecov-io commented on issue #98: ShortDesc name field is optional URL: https://github.com/apache/incubator-openwhisk-client-js/pull/98#issuecomment-372886770 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk-client-js/pull/98?src=pr&el=h1) Report > Merging [#98](https://c

[GitHub] pritidesai closed pull request #800: Display error message for each missing wskprop value.

2018-03-13 Thread GitBox
pritidesai closed pull request #800: Display error message for each missing wskprop value. URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/800 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] ningyougang opened a new pull request #3432: Introduce jmx for kafka

2018-03-13 Thread GitBox
ningyougang opened a new pull request #3432: Introduce jmx for kafka URL: https://github.com/apache/incubator-openwhisk/pull/3432 - jmx is introduced into controller and invoker, in the actual scene, we not only collect kafka producer and consumer metrics but also collect kafka ser

[GitHub] rabbah commented on a change in pull request #3420: Add swift:4.1 with Codable to default set of runtimes

2018-03-13 Thread GitBox
rabbah commented on a change in pull request #3420: Add swift:4.1 with Codable to default set of runtimes URL: https://github.com/apache/incubator-openwhisk/pull/3420#discussion_r174333706 ## File path: docs/reference.md ## @@ -378,13 +378,19 @@ Python 2 actions are execu

[GitHub] rabbah commented on a change in pull request #3420: Add swift:4.1 with Codable to default set of runtimes

2018-03-13 Thread GitBox
rabbah commented on a change in pull request #3420: Add swift:4.1 with Codable to default set of runtimes URL: https://github.com/apache/incubator-openwhisk/pull/3420#discussion_r174333493 ## File path: docs/actions.md ## @@ -685,17 +686,45 @@ func main(args: [String:Any]

[GitHub] rabbah commented on a change in pull request #3420: Add swift:4.1 with Codable to default set of runtimes

2018-03-13 Thread GitBox
rabbah commented on a change in pull request #3420: Add swift:4.1 with Codable to default set of runtimes URL: https://github.com/apache/incubator-openwhisk/pull/3420#discussion_r174333476 ## File path: docs/actions.md ## @@ -685,17 +686,45 @@ func main(args: [String:Any]

[GitHub] daisy-ycguo commented on issue #62: [For technical review] Generate release note

2018-03-13 Thread GitBox
daisy-ycguo commented on issue #62: [For technical review] Generate release note URL: https://github.com/apache/incubator-openwhisk-release/pull/62#issuecomment-372874379 @houshengbo `rm -rf .git` will remove all the git information and `git log` will report error. I do need to keep `.git`

[GitHub] pritidesai closed pull request #789: Add assets annotation tags when assets exported

2018-03-13 Thread GitBox
pritidesai closed pull request #789: Add assets annotation tags when assets exported URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/789 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 proven

[GitHub] codecov-io commented on issue #99: Add Code Query Parameter

2018-03-13 Thread GitBox
codecov-io commented on issue #99: Add Code Query Parameter URL: https://github.com/apache/incubator-openwhisk-client-js/pull/99#issuecomment-372808959 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk-client-js/pull/99?src=pr&el=h1) Report > Merging [#99](https://codecov.i

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

2018-03-13 Thread GitBox
ddragosd commented on issue #762: wskdeploy only supports /whisk.system bindings or github.com packages URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/762#issuecomment-372870448 🎉 💯 many thanks ! Th

[GitHub] dubeejw commented on issue #3421: Add ElasticSearch Log Store

2018-03-13 Thread GitBox
dubeejw commented on issue #3421: Add ElasticSearch Log Store URL: https://github.com/apache/incubator-openwhisk/pull/3421#issuecomment-372867292 PG2 2918 ⏳ This is an automated message from the Apache Git Service. To respon

[GitHub] dubeejw commented on issue #3421: Add ElasticSearch Log Store

2018-03-13 Thread GitBox
dubeejw commented on issue #3421: Add ElasticSearch Log Store URL: https://github.com/apache/incubator-openwhisk/pull/3421#issuecomment-372867292 PG2 2981 ⏳ This is an automated message from the Apache Git Service. To respon

[GitHub] pritidesai opened a new pull request #801: WIP - Adding Docker and Native support

2018-03-13 Thread GitBox
pritidesai opened a new pull request #801: WIP - Adding Docker and Native support URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/801 Closes #623 This is an automated message from the Apache Git Service. To

[GitHub] rabbah closed pull request #242: Use HTTP instead of HTTPS for controller

2018-03-13 Thread GitBox
rabbah closed pull request #242: Use HTTP instead of HTTPS for controller URL: https://github.com/apache/incubator-openwhisk-cli/pull/242 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] rabbah closed issue #243: Travis is currently failing

2018-03-13 Thread GitBox
rabbah closed issue #243: Travis is currently failing URL: https://github.com/apache/incubator-openwhisk-cli/issues/243 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub

[GitHub] dubeejw closed pull request #241: WIP: test latest cli code

2018-03-13 Thread GitBox
dubeejw closed pull request #241: WIP: test latest cli code URL: https://github.com/apache/incubator-openwhisk-cli/pull/241 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

[GitHub] dubeejw closed pull request #240: adding something to test latest openwhisk on master - do not merge

2018-03-13 Thread GitBox
dubeejw closed pull request #240: adding something to test latest openwhisk on master - do not merge URL: https://github.com/apache/incubator-openwhisk-cli/pull/240 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake

[GitHub] dubeejw commented on issue #242: Use HTTP instead of HTTPS for controller

2018-03-13 Thread GitBox
dubeejw commented on issue #242: Use HTTP instead of HTTPS for controller URL: https://github.com/apache/incubator-openwhisk-cli/pull/242#issuecomment-372863727 @csantanapr, @houshengbo, @pritidesai, @mdeuser, whoever gets to this first, please merge. -

[GitHub] dubeejw commented on a change in pull request #3421: Add ElasticSearch Log Store

2018-03-13 Thread GitBox
dubeejw commented on a change in pull request #3421: Add ElasticSearch Log Store URL: https://github.com/apache/incubator-openwhisk/pull/3421#discussion_r174313574 ## File path: core/controller/src/main/scala/whisk/core/controller/ApiUtils.scala ## @@ -184,7 +184,11 @@ tra

[GitHub] rabbah commented on a change in pull request #3421: Add ElasticSearch Log Store

2018-03-13 Thread GitBox
rabbah commented on a change in pull request #3421: Add ElasticSearch Log Store URL: https://github.com/apache/incubator-openwhisk/pull/3421#discussion_r173699456 ## File path: common/scala/src/main/scala/whisk/core/containerpool/logging/ElasticSearchLogStore.scala ## @@

[GitHub] dubeejw commented on a change in pull request #3421: Add ElasticSearch Log Store

2018-03-13 Thread GitBox
dubeejw commented on a change in pull request #3421: Add ElasticSearch Log Store URL: https://github.com/apache/incubator-openwhisk/pull/3421#discussion_r174153522 ## File path: common/scala/src/main/scala/whisk/core/containerpool/logging/ElasticSearchLogStore.scala ## @@

[GitHub] tysonnorris commented on issue #2833: MesosContainerFactory

2018-03-13 Thread GitBox
tysonnorris commented on issue #2833: MesosContainerFactory URL: https://github.com/apache/incubator-openwhisk/pull/2833#issuecomment-372843744 I think this should work now; please try again when possible. This is an automate

[GitHub] dubeejw opened a new issue #243: Travis is currently failing

2018-03-13 Thread GitBox
dubeejw opened a new issue #243: Travis is currently failing URL: https://github.com/apache/incubator-openwhisk-cli/issues/243 https://github.com/apache/incubator-openwhisk-cli/pull/240 shows that Travis is failing with the error below. ``` whisk.core.cli.test.WskConfigTests > Wsk CLI

[GitHub] dubeejw opened a new pull request #242: Get build version using HTTPS

2018-03-13 Thread GitBox
dubeejw opened a new pull request #242: Get build version using HTTPS URL: https://github.com/apache/incubator-openwhisk-cli/pull/242 Fixes Travis test failure. ``` whisk.core.cli.test.WskConfigTests > Wsk CLI config should show api build using http apihost FAILED org.scalatest

[GitHub] csantanapr commented on issue #3420: Add swift:4.1 with Codable to default set of runtimes

2018-03-13 Thread GitBox
csantanapr commented on issue #3420: Add swift:4.1 with Codable to default set of runtimes URL: https://github.com/apache/incubator-openwhisk/pull/3420#issuecomment-372000671 ## Swift 4.x support Some examples of using Codable In and Out ### Codable style function signature

[GitHub] csantanapr commented on issue #3420: Add swift:4.1 with Codable to default set of runtimes

2018-03-13 Thread GitBox
csantanapr commented on issue #3420: Add swift:4.1 with Codable to default set of runtimes URL: https://github.com/apache/incubator-openwhisk/pull/3420#issuecomment-372836494 added initial stab at docs This is an automated m

[GitHub] csantanapr commented on issue #3420: Add swift:4.1 with Codable to default set of runtimes

2018-03-13 Thread GitBox
csantanapr commented on issue #3420: Add swift:4.1 with Codable to default set of runtimes URL: https://github.com/apache/incubator-openwhisk/pull/3420#issuecomment-372000528 TODOs - [x] Update Docs This is an automat

[GitHub] dubeejw commented on issue #3426: CLI Builds not found in expected location

2018-03-13 Thread GitBox
dubeejw commented on issue #3426: CLI Builds not found in expected location URL: https://github.com/apache/incubator-openwhisk/issues/3426#issuecomment-372815217 To first clean the environment: `ansible-playbook -i environments/ openwhisk.yml -e mode=clean`. --

[GitHub] dubeejw commented on issue #3426: CLI Builds not found in expected location

2018-03-13 Thread GitBox
dubeejw commented on issue #3426: CLI Builds not found in expected location URL: https://github.com/apache/incubator-openwhisk/issues/3426#issuecomment-372815082 @jonpspri, FYI. This is an automated message from the Apache Gi

[GitHub] dubeejw commented on issue #99: Add Code Query Parameter

2018-03-13 Thread GitBox
dubeejw commented on issue #99: Add Code Query Parameter URL: https://github.com/apache/incubator-openwhisk-client-js/pull/99#issuecomment-372811662 @jthomas, Travis is passing for this one now. This is an automated message f

[GitHub] dubeejw closed pull request #260: Update MessageHubMultiWorkersTest to use ExtendedCouchDbRestClient

2018-03-13 Thread GitBox
dubeejw closed pull request #260: Update MessageHubMultiWorkersTest to use ExtendedCouchDbRestClient URL: https://github.com/apache/incubator-openwhisk-package-kafka/pull/260 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below f

[GitHub] codecov-io commented on issue #99: Add Code Query Parameter

2018-03-13 Thread GitBox
codecov-io commented on issue #99: Add Code Query Parameter URL: https://github.com/apache/incubator-openwhisk-client-js/pull/99#issuecomment-372808959 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk-client-js/pull/99?src=pr&el=h1) Report > Merging [#99](https://codecov.i

[GitHub] dubeejw commented on a change in pull request #99: Add Code Query Parameter

2018-03-13 Thread GitBox
dubeejw commented on a change in pull request #99: Add Code Query Parameter URL: https://github.com/apache/incubator-openwhisk-client-js/pull/99#discussion_r174272933 ## File path: test/unit/actions.test.js ## @@ -60,7 +79,7 @@ test('should retrieve action from string iden

[GitHub] develpr closed issue #3431: Vagrant provisioning results in SSLV3_ALERT_BAD_CERTIFICATE and failure to ping https://172.17.0.1:10001/ping

2018-03-13 Thread GitBox
develpr closed issue #3431: Vagrant provisioning results in SSLV3_ALERT_BAD_CERTIFICATE and failure to ping https://172.17.0.1:10001/ping URL: https://github.com/apache/incubator-openwhisk/issues/3431 This is an automated me

[GitHub] develpr commented on issue #3431: Vagrant provisioning results in SSLV3_ALERT_BAD_CERTIFICATE and failure to ping https://172.17.0.1:10001/ping

2018-03-13 Thread GitBox
develpr commented on issue #3431: Vagrant provisioning results in SSLV3_ALERT_BAD_CERTIFICATE and failure to ping https://172.17.0.1:10001/ping URL: https://github.com/apache/incubator-openwhisk/issues/3431#issuecomment-372805522 @chetanmeh sorry you're right, and I just took a look at the

[GitHub] dubeejw closed pull request #102: Fixes #100

2018-03-13 Thread GitBox
dubeejw closed pull request #102: Fixes #100 URL: https://github.com/apache/incubator-openwhisk-client-js/pull/102 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] dubeejw closed issue #100: Travis Test Breakage

2018-03-13 Thread GitBox
dubeejw closed issue #100: Travis Test Breakage URL: https://github.com/apache/incubator-openwhisk-client-js/issues/100 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub

[GitHub] abaruni opened a new pull request #260: Update MessageHubMultiWorkersTest to use ExtendedCouchDbRestClient

2018-03-13 Thread GitBox
abaruni opened a new pull request #260: Update MessageHubMultiWorkersTest to use ExtendedCouchDbRestClient URL: https://github.com/apache/incubator-openwhisk-package-kafka/pull/260 This is an automated message from the Apac

[GitHub] ScottChapman commented on issue #101: please add ignore_certs env var

2018-03-13 Thread GitBox
ScottChapman commented on issue #101: please add ignore_certs env var URL: https://github.com/apache/incubator-openwhisk-client-js/issues/101#issuecomment-372778594 > This is specific to openwhisk actions making an openwhisk api call. @rabbah - right, exactly; that's what all the cli

[GitHub] ScottChapman commented on issue #101: please add ignore_certs env var

2018-03-13 Thread GitBox
ScottChapman commented on issue #101: please add ignore_certs env var URL: https://github.com/apache/incubator-openwhisk-client-js/issues/101#issuecomment-372778594 > This is specific to openwhisk actions making an openwhisk api call. @rabbah - right, exactly; that's what all the cli

[GitHub] rabbah commented on issue #101: please add ignore_certs env var

2018-03-13 Thread GitBox
rabbah commented on issue #101: please add ignore_certs env var URL: https://github.com/apache/incubator-openwhisk-client-js/issues/101#issuecomment-372762702 > I would think you would have the same issue with any of the client libraries you are making available for action development

[GitHub] rabbah commented on issue #101: please add ignore_certs env var

2018-03-13 Thread GitBox
rabbah commented on issue #101: please add ignore_certs env var URL: https://github.com/apache/incubator-openwhisk-client-js/issues/101#issuecomment-372762237 We don?t have such a setting today - so we?d need to think about how to add it, thread it through the invoker, and into the runtime

[GitHub] aviweit commented on issue #3431: Vagrant provisioning results in SSLV3_ALERT_BAD_CERTIFICATE and failure to ping https://172.17.0.1:10001/ping

2018-03-13 Thread GitBox
aviweit commented on issue #3431: Vagrant provisioning results in SSLV3_ALERT_BAD_CERTIFICATE and failure to ping https://172.17.0.1:10001/ping URL: https://github.com/apache/incubator-openwhisk/issues/3431#issuecomment-372761399 @develpr , I did not use Vagrant but a regular install via A

[GitHub] ScottChapman commented on issue #101: please add ignore_certs env var

2018-03-13 Thread GitBox
ScottChapman commented on issue #101: please add ignore_certs env var URL: https://github.com/apache/incubator-openwhisk-client-js/issues/101#issuecomment-372761362 @rabbah - shouldn't that be defined as part of the configuration of the deployment? I would think you would have the same iss

[GitHub] chetanmeh commented on issue #3431: Vagrant provisioning results in SSLV3_ALERT_BAD_CERTIFICATE and failure to ping https://172.17.0.1:10001/ping

2018-03-13 Thread GitBox
chetanmeh commented on issue #3431: Vagrant provisioning results in SSLV3_ALERT_BAD_CERTIFICATE and failure to ping https://172.17.0.1:10001/ping URL: https://github.com/apache/incubator-openwhisk/issues/3431#issuecomment-372757312 @develpr not sure on that. With PR #3430 ansible version w

[GitHub] develpr commented on issue #3431: Vagrant provisioning results in SSLV3_ALERT_BAD_CERTIFICATE and failure to ping https://172.17.0.1:10001/ping

2018-03-13 Thread GitBox
develpr commented on issue #3431: Vagrant provisioning results in SSLV3_ALERT_BAD_CERTIFICATE and failure to ping https://172.17.0.1:10001/ping URL: https://github.com/apache/incubator-openwhisk/issues/3431#issuecomment-372755922 Thanks @chetanmeh - do you happen to know if this also reso

[GitHub] chetanmeh commented on issue #3431: Vagrant provisioning results in SSLV3_ALERT_BAD_CERTIFICATE and failure to ping https://172.17.0.1:10001/ping

2018-03-13 Thread GitBox
chetanmeh commented on issue #3431: Vagrant provisioning results in SSLV3_ALERT_BAD_CERTIFICATE and failure to ping https://172.17.0.1:10001/ping URL: https://github.com/apache/incubator-openwhisk/issues/3431#issuecomment-372755014 This should get fixed with #3430

[GitHub] rabbah commented on issue #101: please add ignore_certs env var

2018-03-13 Thread GitBox
rabbah commented on issue #101: please add ignore_certs env var URL: https://github.com/apache/incubator-openwhisk-client-js/issues/101#issuecomment-372753829 @jthomas this would be ok if running locally with node. We do not know in a deployment if the certificate is trusted or not. This i

[GitHub] develpr commented on issue #3431: Vagrant provisioning results in SSLV3_ALERT_BAD_CERTIFICATE and failure to ping https://172.17.0.1:10001/ping

2018-03-13 Thread GitBox
develpr commented on issue #3431: Vagrant provisioning results in SSLV3_ALERT_BAD_CERTIFICATE and failure to ping https://172.17.0.1:10001/ping URL: https://github.com/apache/incubator-openwhisk/issues/3431#issuecomment-372753015 @aviweit out of curiosity, did you modify the `Vagrantfile`

[GitHub] aviweit commented on issue #3431: Vagrant provisioning results in SSLV3_ALERT_BAD_CERTIFICATE and failure to ping https://172.17.0.1:10001/ping

2018-03-13 Thread GitBox
aviweit commented on issue #3431: Vagrant provisioning results in SSLV3_ALERT_BAD_CERTIFICATE and failure to ping https://172.17.0.1:10001/ping URL: https://github.com/apache/incubator-openwhisk/issues/3431#issuecomment-372742409 I just got a similar certificate issue when installing all-i

[GitHub] duynguyen commented on issue #2: Add DC/OS universe for OpenWhisk and orchestration scripts

2018-03-13 Thread GitBox
duynguyen commented on issue #2: Add DC/OS universe for OpenWhisk and orchestration scripts URL: https://github.com/apache/incubator-openwhisk-deploy-mesos/pull/2#issuecomment-372735755 thanks @ddragosd , yes that makes sense. Added the status line. ---

[GitHub] duynguyen commented on a change in pull request #2: Add DC/OS universe for OpenWhisk and orchestration scripts

2018-03-13 Thread GitBox
duynguyen commented on a change in pull request #2: Add DC/OS universe for OpenWhisk and orchestration scripts URL: https://github.com/apache/incubator-openwhisk-deploy-mesos/pull/2#discussion_r174203845 ## File path: dcos-deploy-script/Makefile ## @@ -0,0 +1,127 @@ +DCOS

[GitHub] duynguyen commented on a change in pull request #2: Add DC/OS universe for OpenWhisk and orchestration scripts

2018-03-13 Thread GitBox
duynguyen commented on a change in pull request #2: Add DC/OS universe for OpenWhisk and orchestration scripts URL: https://github.com/apache/incubator-openwhisk-deploy-mesos/pull/2#discussion_r174203845 ## File path: dcos-deploy-script/Makefile ## @@ -0,0 +1,127 @@ +DCOS

[GitHub] mdeuser commented on issue #3418: support cleanup of cloudant/kafka trigger dbs

2018-03-13 Thread GitBox
mdeuser commented on issue #3418: support cleanup of cloudant/kafka trigger dbs URL: https://github.com/apache/incubator-openwhisk/pull/3418#issuecomment-372734774 is this part of a broader "clean up" effort to rid the environment of all entities (i.e. actions, triggers, rules, apis, etc)

[GitHub] jthomas commented on issue #98: ShortDesc name field is optional

2018-03-13 Thread GitBox
jthomas commented on issue #98: ShortDesc name field is optional URL: https://github.com/apache/incubator-openwhisk-client-js/pull/98#issuecomment-372730574 Can you re-base this PR once #102 is merged to resolve the build issue? -

[GitHub] jthomas commented on issue #99: Add Code Query Parameter

2018-03-13 Thread GitBox
jthomas commented on issue #99: Add Code Query Parameter URL: https://github.com/apache/incubator-openwhisk-client-js/pull/99#issuecomment-372730542 Can you re-base this PR once #102 is merged to resolve the build issue? This

[GitHub] jthomas commented on issue #98: ShortDesc name field is optional

2018-03-13 Thread GitBox
jthomas commented on issue #98: ShortDesc name field is optional URL: https://github.com/apache/incubator-openwhisk-client-js/pull/98#issuecomment-372730574 Can you re-base this PR to resolve the build issue now I've fixed the trigger failure? -

[GitHub] jthomas commented on issue #99: Add Code Query Parameter

2018-03-13 Thread GitBox
jthomas commented on issue #99: Add Code Query Parameter URL: https://github.com/apache/incubator-openwhisk-client-js/pull/99#issuecomment-372730542 Can you re-base this PR to resolve the build issue now I've fixed the trigger failure? -

[GitHub] jthomas commented on issue #101: please add ignore_certs env var

2018-03-13 Thread GitBox
jthomas commented on issue #101: please add ignore_certs env var URL: https://github.com/apache/incubator-openwhisk-client-js/issues/101#issuecomment-372729726 I think having `ignore_certs` as an env param is a valid suggestion. It'll be consistent with setting other properties using env p

[GitHub] rabbah commented on issue #2008: Support Default Parameters for Sequences

2018-03-13 Thread GitBox
rabbah commented on issue #2008: Support Default Parameters for Sequences URL: https://github.com/apache/incubator-openwhisk/issues/2008#issuecomment-372726303 help wanted :) This is an automated message from the Apache Git S

[GitHub] jthomas commented on a change in pull request #99: Add Code Query Parameter

2018-03-13 Thread GitBox
jthomas commented on a change in pull request #99: Add Code Query Parameter URL: https://github.com/apache/incubator-openwhisk-client-js/pull/99#discussion_r174194664 ## File path: test/unit/actions.test.js ## @@ -60,7 +79,7 @@ test('should retrieve action from string iden

[GitHub] jthomas commented on issue #100: Travis Test Breakage

2018-03-13 Thread GitBox
jthomas commented on issue #100: Travis Test Breakage URL: https://github.com/apache/incubator-openwhisk-client-js/issues/100#issuecomment-372724018 @csantanapr is correct. Fixed. This is an automated message from the Apache

[GitHub] rabbah commented on issue #2220: Ansible error while deploying to distributed environment

2018-03-13 Thread GitBox
rabbah commented on issue #2220: Ansible error while deploying to distributed environment URL: https://github.com/apache/incubator-openwhisk/issues/2220#issuecomment-372719236 @davglass would you consider a kube based deployment? i'm afraid these playbooks are not supported and brea

[GitHub] codecov-io commented on issue #102: Fixes #100

2018-03-13 Thread GitBox
codecov-io commented on issue #102: Fixes #100 URL: https://github.com/apache/incubator-openwhisk-client-js/pull/102#issuecomment-372718809 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk-client-js/pull/102?src=pr&el=h1) Report > Merging [#102](https://codecov.io/gh/apach

[GitHub] rabbah commented on issue #3249: #3243 - Use gradle scoverage to collect test coverage stats

2018-03-13 Thread GitBox
rabbah commented on issue #3249: #3243 - Use gradle scoverage to collect test coverage stats URL: https://github.com/apache/incubator-openwhisk/pull/3249#issuecomment-372717356 @markusthoemmes @csantanapr bump. This is an au

[GitHub] rabbah commented on issue #3418: support cleanup of cloudant/kafka trigger dbs

2018-03-13 Thread GitBox
rabbah commented on issue #3418: support cleanup of cloudant/kafka trigger dbs URL: https://github.com/apache/incubator-openwhisk/pull/3418#issuecomment-372714019 Thanks @cbickel for the followup - Really my issue with this method https://github.com/apache/incubator-openwhisk/pull/3

[GitHub] cbickel commented on issue #3418: support cleanup of cloudant/kafka trigger dbs

2018-03-13 Thread GitBox
cbickel commented on issue #3418: support cleanup of cloudant/kafka trigger dbs URL: https://github.com/apache/incubator-openwhisk/pull/3418#issuecomment-372708265 @rabbah We discussed again, and we will move the script somewhere else. --

[GitHub] jthomas opened a new pull request #102: Fixes #100

2018-03-13 Thread GitBox
jthomas opened a new pull request #102: Fixes #100 URL: https://github.com/apache/incubator-openwhisk-client-js/pull/102 Hook up test trigger to action with rule to ensure there is an activation record. This is an automated m

[GitHub] kpavel commented on issue #790: Add support to export binary (zip and jar) actions

2018-03-13 Thread GitBox
kpavel commented on issue #790: Add support to export binary (zip and jar) actions URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/790#issuecomment-372698812 @mrutkows done This is an automated message from

[GitHub] mrutkows opened a new pull request #800: WIP: Display error message for each missing wskprop value.

2018-03-13 Thread GitBox
mrutkows opened a new pull request #800: WIP: Display error message for each missing wskprop value. URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/800 This is an automated message from the Apache Git Servi

[GitHub] mrutkows opened a new issue #799: Add i18n err msg in servicedeployer.go, DeployDependencies()

2018-03-13 Thread GitBox
mrutkows opened a new issue #799: Add i18n err msg in servicedeployer.go, DeployDependencies() URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/799 Need an error string added to i18n translated messages/IDs: ``` errMessage :=

[GitHub] mrutkows opened a new issue #798: Add i18n string/ID in manifestreader.go, SetSequences()

2018-03-13 Thread GitBox
mrutkows opened a new issue #798: Add i18n string/ID in manifestreader.go, SetSequences() URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/798 hardcoded error string/message needs to be added to translations: ``` if exists == true { // TO

[GitHub] mrutkows opened a new issue #797: Add i18n strings for warning messages in InterpolateStringWithEnvVar() function

2018-03-13 Thread GitBox
mrutkows opened a new issue #797: Add i18n strings for warning messages in InterpolateStringWithEnvVar() function URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/797 In environment.go, function InterpolateStringWithEnvVar(): add i18n strings/IDs for: ```

[GitHub] houshengbo commented on a change in pull request #62: [For technical review] Generate release note

2018-03-13 Thread GitBox
houshengbo commented on a change in pull request #62: [For technical review] Generate release note URL: https://github.com/apache/incubator-openwhisk-release/pull/62#discussion_r174159776 ## File path: tools/package_source_code.sh ## @@ -33,5 +33,7 @@ rsync -rtp --exclude

[GitHub] houshengbo commented on a change in pull request #62: [For technical review] Generate release note

2018-03-13 Thread GitBox
houshengbo commented on a change in pull request #62: [For technical review] Generate release note URL: https://github.com/apache/incubator-openwhisk-release/pull/62#discussion_r174159633 ## File path: tools/download_source_code.sh ## @@ -33,7 +33,7 @@ function git_clone_

[GitHub] develpr opened a new issue #3431: Vagrant provisioning results in SSLV3_ALERT_BAD_CERTIFICATE and failure to ping https://172.17.0.1:10001/ping

2018-03-13 Thread GitBox
develpr opened a new issue #3431: Vagrant provisioning results in SSLV3_ALERT_BAD_CERTIFICATE and failure to ping https://172.17.0.1:10001/ping URL: https://github.com/apache/incubator-openwhisk/issues/3431 ## Environment details: *Vagrant install quick start method used*

[GitHub] kpavel commented on issue #789: Add assets annotation tags when assets exported

2018-03-13 Thread GitBox
kpavel commented on issue #789: Add assets annotation tags when assets exported URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/789#issuecomment-372691395 @mrutkows the idea is to export all the annotations asset already has, excluding the "MANAGED" annotation. We should

[GitHub] mrutkows closed issue #795: Rules become inactive

2018-03-13 Thread GitBox
mrutkows closed issue #795: Rules become inactive URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/795 This is an automated message from the Apache Git Service. To respond to the message, please log on GitH

[GitHub] mrutkows closed pull request #796: Resolves #795

2018-03-13 Thread GitBox
mrutkows closed pull request #796: Resolves #795 URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/796 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 requ

[GitHub] dubeejw commented on a change in pull request #3421: Add ElasticSearch Log Store

2018-03-13 Thread GitBox
dubeejw commented on a change in pull request #3421: Add ElasticSearch Log Store URL: https://github.com/apache/incubator-openwhisk/pull/3421#discussion_r174153522 ## File path: common/scala/src/main/scala/whisk/core/containerpool/logging/ElasticSearchLogStore.scala ## @@

[GitHub] dubeejw commented on a change in pull request #3421: Add ElasticSearch Log Store

2018-03-13 Thread GitBox
dubeejw commented on a change in pull request #3421: Add ElasticSearch Log Store URL: https://github.com/apache/incubator-openwhisk/pull/3421#discussion_r174153522 ## File path: common/scala/src/main/scala/whisk/core/containerpool/logging/ElasticSearchLogStore.scala ## @@

[GitHub] kpavel opened a new pull request #796: Resolves #795

2018-03-13 Thread GitBox
kpavel opened a new pull request #796: Resolves #795 URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/796 This is an automated message from the Apache Git Service. To respond to the message, please log on Git

  1   2   >