[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

[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

[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

[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] 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] 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

[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

[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

[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=h1) Report > Merging

[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

[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

[GitHub] cbickel commented on a change in pull request #3418: support cleanup of cloudant/kafka trigger dbs

2018-03-13 Thread GitBox
cbickel commented on a change in pull request #3418: support cleanup of cloudant/kafka trigger dbs URL: https://github.com/apache/incubator-openwhisk/pull/3418#discussion_r174033580 ## File path: tests/src/test/scala/whisk/core/database/test/CleanUpWhisksDbSkriptTests.scala

[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-372572682 @jasonpet @abaruni @csantanapr Could you please also have a look, to not delete valid documents in the trigger databases?

[GitHub] Param-S commented on issue #24: Make AdoptOpenJDK Eclipse OpenJ9 the Java runtime for Java actions

2018-03-13 Thread GitBox
Param-S commented on issue #24: Make AdoptOpenJDK Eclipse OpenJ9 the Java runtime for Java actions URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/24#issuecomment-372559986 actionContainers.JavaActionContainerTests > Java action should handle unicode in source, input

[GitHub] cbickel commented on a change in pull request #3418: support cleanup of cloudant/kafka trigger dbs

2018-03-13 Thread GitBox
cbickel commented on a change in pull request #3418: support cleanup of cloudant/kafka trigger dbs URL: https://github.com/apache/incubator-openwhisk/pull/3418#discussion_r174033207 ## File path: tests/src/test/scala/whisk/core/database/test/CleanUpWhisksDbSkriptTests.scala

[GitHub] steffenrost commented on a change in pull request #3418: support cleanup of cloudant/kafka trigger dbs

2018-03-13 Thread GitBox
steffenrost commented on a change in pull request #3418: support cleanup of cloudant/kafka trigger dbs URL: https://github.com/apache/incubator-openwhisk/pull/3418#discussion_r174035748 ## File path: tests/src/test/scala/whisk/core/database/test/CleanUpWhisksDbSkriptTests.scala

[GitHub] daisy-ycguo opened a new pull request #62: [For technical review] Generate release note

2018-03-13 Thread GitBox
daisy-ycguo opened a new pull request #62: [For technical review] Generate release note URL: https://github.com/apache/incubator-openwhisk-release/pull/62 This PR is to generate a release note, for a technical review. Future work includes: - a design of a release template -

[GitHub] Dalfons0 commented on issue #162: Key not found: "couchdb", while starting the Controller.

2018-03-13 Thread GitBox
Dalfons0 commented on issue #162: Key not found: "couchdb", while starting the Controller. URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/162#issuecomment-372579500 Thanks for the quick solve @dgrove-oss, worked like a charm and I finally got the cluster fully

[GitHub] cbickel commented on a change in pull request #3418: support cleanup of cloudant/kafka trigger dbs

2018-03-13 Thread GitBox
cbickel commented on a change in pull request #3418: support cleanup of cloudant/kafka trigger dbs URL: https://github.com/apache/incubator-openwhisk/pull/3418#discussion_r174032955 ## File path: tools/db/cleanUpWhisks.py ## @@ -95,56 +98,108 @@ def deleteWhisk(dbWhisks,

[GitHub] csantanapr commented on issue #105: Adapt the DB config to match with pureconfig switch

2018-03-13 Thread GitBox
csantanapr commented on issue #105: Adapt the DB config to match with pureconfig switch URL: https://github.com/apache/incubator-openwhisk-devtools/pull/105#issuecomment-372629050 Yes go ahead and do the work to collect anything you need into a tgz then later I can hook it up with an

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

2018-03-13 Thread GitBox
csantanapr commented on issue #3426: CLI Builds not found in expected location URL: https://github.com/apache/incubator-openwhisk/issues/3426#issuecomment-372610480 Run the teardown.yml playbook This is an automated message

[GitHub] chetanmeh commented on issue #105: Adapt the DB config to match with pureconfig switch

2018-03-13 Thread GitBox
chetanmeh commented on issue #105: Adapt the DB config to match with pureconfig switch URL: https://github.com/apache/incubator-openwhisk-devtools/pull/105#issuecomment-372626795 The build is getting stuck in controller startup which fails with following error ``` Exception in

[GitHub] chetanmeh commented on issue #105: Adapt the DB config to match with pureconfig switch

2018-03-13 Thread GitBox
chetanmeh commented on issue #105: Adapt the DB config to match with pureconfig switch URL: https://github.com/apache/incubator-openwhisk-devtools/pull/105#issuecomment-372634609 The build now passed with switch to cluster mode. Locally also it works fine. Would look into the log file

[GitHub] csantanapr closed pull request #105: Adapt the DB config to match with pureconfig switch

2018-03-13 Thread GitBox
csantanapr closed pull request #105: Adapt the DB config to match with pureconfig switch URL: https://github.com/apache/incubator-openwhisk-devtools/pull/105 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] lornajane commented on issue #3426: CLI Builds not found in expected location

2018-03-13 Thread GitBox
lornajane commented on issue #3426: CLI Builds not found in expected location URL: https://github.com/apache/incubator-openwhisk/issues/3426#issuecomment-372607064 This fails but I think because the playbook isn't designed to be run more than once. Is there a way of cleaning up what it

[GitHub] rabbah commented on a change in pull request #24: Make AdoptOpenJDK Eclipse OpenJ9 the Java runtime for Java actions

2018-03-13 Thread GitBox
rabbah commented on a change in pull request #24: Make AdoptOpenJDK Eclipse OpenJ9 the Java runtime for Java actions URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/24#discussion_r174081006 ## File path: core/javaAction/Dockerfile ## @@ -1,38 +1,11

[GitHub] rabbah commented on a change in pull request #24: Make AdoptOpenJDK Eclipse OpenJ9 the Java runtime for Java actions

2018-03-13 Thread GitBox
rabbah commented on a change in pull request #24: Make AdoptOpenJDK Eclipse OpenJ9 the Java runtime for Java actions URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/24#discussion_r174080740 ## File path: core/javaAction/Dockerfile ## @@ -1,38 +1,11

[GitHub] chetanmeh commented on issue #105: Adapt the DB config to match with pureconfig switch

2018-03-13 Thread GitBox
chetanmeh commented on issue #105: Adapt the DB config to match with pureconfig switch URL: https://github.com/apache/incubator-openwhisk-devtools/pull/105#issuecomment-372626795 The build is getting stuck in controller startup which fails with following error ``` Exception in

[GitHub] Param-S commented on a change in pull request #24: Make AdoptOpenJDK Eclipse OpenJ9 the Java runtime for Java actions

2018-03-13 Thread GitBox
Param-S commented on a change in pull request #24: Make AdoptOpenJDK Eclipse OpenJ9 the Java runtime for Java actions URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/24#discussion_r174102144 ## File path: core/javaAction/Dockerfile ## @@ -1,38 +1,11

[GitHub] vvraskin opened a new pull request #3430: Bump ansible version

2018-03-13 Thread GitBox
vvraskin opened a new pull request #3430: Bump ansible version URL: https://github.com/apache/incubator-openwhisk/pull/3430 Bump ansible version in the documentation and vagrant setup. ## My changes affect the following components - [x ] Deployment - [x ] Documentation

[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-372612412 @cbickel I think you?re already hinting at the concern I have with this script. Why should it assume anything about the

[GitHub] csantanapr commented on a change in pull request #24: Make AdoptOpenJDK Eclipse OpenJ9 the Java runtime for Java actions

2018-03-13 Thread GitBox
csantanapr commented on a change in pull request #24: Make AdoptOpenJDK Eclipse OpenJ9 the Java runtime for Java actions URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/24#discussion_r174081577 ## File path: core/javaAction/Dockerfile ## @@ -1,38

[GitHub] chetanmeh commented on issue #105: Adapt the DB config to match with pureconfig switch

2018-03-13 Thread GitBox
chetanmeh commented on issue #105: Adapt the DB config to match with pureconfig switch URL: https://github.com/apache/incubator-openwhisk-devtools/pull/105#issuecomment-372626795 The build is getting stuck in controller startup which fails with following error ``` Exception in

[GitHub] csantanapr commented on issue #24: Make AdoptOpenJDK Eclipse OpenJ9 the Java runtime for Java actions

2018-03-13 Thread GitBox
csantanapr commented on issue #24: Make AdoptOpenJDK Eclipse OpenJ9 the Java runtime for Java actions URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/24#issuecomment-372612609 It could be related to locale and utf need to be set in env in the runtime

[GitHub] markusthoemmes closed pull request #3423: Measure kafka queue in consumer with better exactitude

2018-03-13 Thread GitBox
markusthoemmes closed pull request #3423: Measure kafka queue in consumer with better exactitude URL: https://github.com/apache/incubator-openwhisk/pull/3423 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] chetanmeh commented on issue #105: Adapt the DB config to match with pureconfig switch

2018-03-13 Thread GitBox
chetanmeh commented on issue #105: Adapt the DB config to match with pureconfig switch URL: https://github.com/apache/incubator-openwhisk-devtools/pull/105#issuecomment-372634609 The build now passed with switch to cluster mode. Locally also it works fine

[GitHub] jonpspri commented on issue #3430: Bump ansible version

2018-03-13 Thread GitBox
jonpspri commented on issue #3430: Bump ansible version URL: https://github.com/apache/incubator-openwhisk/pull/3430#issuecomment-372642364 Also need to ensure that we capture not to install `docker-py` but to do `pip install 'docker<3.0'`.

[GitHub] Param-S commented on a change in pull request #24: Make AdoptOpenJDK Eclipse OpenJ9 the Java runtime for Java actions

2018-03-13 Thread GitBox
Param-S commented on a change in pull request #24: Make AdoptOpenJDK Eclipse OpenJ9 the Java runtime for Java actions URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/24#discussion_r174123978 ## File path: core/javaAction/Dockerfile ## @@ -1,38 +1,11

[GitHub] cbickel closed pull request #3424: Send active-ack in any case of a parseable message.

2018-03-13 Thread GitBox
cbickel closed pull request #3424: Send active-ack in any case of a parseable message. URL: https://github.com/apache/incubator-openwhisk/pull/3424 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] 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-372652546 PG4/1469 ? This is an automated message from

[GitHub] markusthoemmes commented on issue #3430: Bump ansible version

2018-03-13 Thread GitBox
markusthoemmes commented on issue #3430: Bump ansible version URL: https://github.com/apache/incubator-openwhisk/pull/3430#issuecomment-372643883 @jonpspri can you elaborate? This is an automated message from the Apache Git

[GitHub] Param-S commented on a change in pull request #24: Make AdoptOpenJDK Eclipse OpenJ9 the Java runtime for Java actions

2018-03-13 Thread GitBox
Param-S commented on a change in pull request #24: Make AdoptOpenJDK Eclipse OpenJ9 the Java runtime for Java actions URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/24#discussion_r174123978 ## File path: core/javaAction/Dockerfile ## @@ -1,38 +1,11

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

2018-03-13 Thread GitBox
mrutkows commented on issue #789: Add assets annotation tags when assets exported URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/789#issuecomment-372685417 @kpavel @pritidesai why would we filter out managed annotations? If they are removed on export, then their

[GitHub] kpavel opened a new issue #795: Rules become inactive

2018-03-13 Thread GitBox
kpavel opened a new issue #795: Rules become inactive URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/795 Consecutive deployments of manifest containing trigger with feed action (and rule) result in inactive rule. The rule seems to become inactive automatically when

[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] 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

[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

[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 { //

[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

[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

[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

[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

[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=h1) Report > Merging

[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

[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_r174159633 ## File path: tools/download_source_code.sh ## @@ -33,7 +33,7 @@ function

[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

[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] 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] 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

[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] 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

[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 #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 #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] 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

[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 @@

[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

[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

[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

[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?

[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] 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] 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 @@

[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

[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 @@

[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] 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 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 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

[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

[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

[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-372000528 TODOs - [x] Update Docs This is an

[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

[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] 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

[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] 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

[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

[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

[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] 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

[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

[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

[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=h1) Report > Merging

[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

[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

[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] 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

  1   2   >