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

2018-03-11 Thread GitBox
rabbah commented on issue #3232: Making prewarm kind (and count) configurable URL: https://github.com/apache/incubator-openwhisk/pull/3232#issuecomment-372168758 Yes. We?d like to get this in for sure. This is an automated

[GitHub] jasonpet commented on issue #139: fix failing test and refactor others into appropriate suites

2018-03-11 Thread GitBox
jasonpet commented on issue #139: fix failing test and refactor others into appropriate suites URL: https://github.com/apache/incubator-openwhisk-package-alarms/pull/139#issuecomment-372160930 @dubeejw This is an automated

[GitHub] jasonpet opened a new pull request #139: fix failing test and refactor others into appropriate suites

2018-03-11 Thread GitBox
jasonpet opened a new pull request #139: fix failing test and refactor others into appropriate suites URL: https://github.com/apache/incubator-openwhisk-package-alarms/pull/139 fix for the AlarmsMultiWorkerTests to use whisk properties instead of pure config simply moved some of the

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

2018-03-11 Thread GitBox
jiangpengcheng commented on issue #3232: Making prewarm kind (and count) configurable URL: https://github.com/apache/incubator-openwhisk/pull/3232#issuecomment-372166561 is this pr alive? This is an automated message from

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

2018-03-11 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-372140430 @rabbah - I am building a "Template" for IBM Cloud Functions. Part of that is to implement automated tests that run in a

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

2018-03-11 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-372141502 Yes, I've similarly implemented a work-around by altering my code, and modified my travis. I'm doing it as a package param,

[GitHub] dubeejw closed pull request #139: fix failing test and refactor others into appropriate suites

2018-03-11 Thread GitBox
dubeejw closed pull request #139: fix failing test and refactor others into appropriate suites URL: https://github.com/apache/incubator-openwhisk-package-alarms/pull/139 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for

[GitHub] jasonpet opened a new pull request #160: fix failing test and other test cleanup

2018-03-11 Thread GitBox
jasonpet opened a new pull request #160: fix failing test and other test cleanup URL: https://github.com/apache/incubator-openwhisk-package-cloudant/pull/160 fix for the CloudantMultiWorkerTests to use whisk properties instead of pureconfig test cleanup

[GitHub] jasonpet commented on issue #160: fix failing test and other test cleanup

2018-03-11 Thread GitBox
jasonpet commented on issue #160: fix failing test and other test cleanup URL: https://github.com/apache/incubator-openwhisk-package-cloudant/pull/160#issuecomment-372166177 @dubeejw This is an automated message from the

[GitHub] dubeejw closed pull request #160: fix failing test and other test cleanup

2018-03-11 Thread GitBox
dubeejw closed pull request #160: fix failing test and other test cleanup URL: https://github.com/apache/incubator-openwhisk-package-cloudant/pull/160 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 #101: please add ignore_certs env var

2018-03-11 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-372140846 Thanks for the clarification. So these are actions. The constructors accept the namespace and api host optionally from the

[GitHub] kpavel opened a new pull request #789: Add assets annotation tags when assets exported

2018-03-11 Thread GitBox
kpavel opened a new pull request #789: Add assets annotation tags when assets exported URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/789 Currently exported assets manifest is missing annotations assets have in OpenWhisk.

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

2018-03-11 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-372125855 Why not just set NODE_TLS_REJECT_UNAUTHORIZED This is an

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

2018-03-11 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-372125100 It?s a heuristic that would be prone to error. I use an alias ?openwhisk? as my api host for my local deployment. I?d

[GitHub] dgrove-oss commented on issue #3411: refactor KubernetesClient to separate invokerAgent; add forwarding LogStoreProvider

2018-03-11 Thread GitBox
dgrove-oss commented on issue #3411: refactor KubernetesClient to separate invokerAgent; add forwarding LogStoreProvider URL: https://github.com/apache/incubator-openwhisk/pull/3411#issuecomment-372129332 PG2 / 2909 ? This

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

2018-03-11 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-372128750 Scott can you clarify how you?re developing and running? Are you writing js/node code and running it with node against a

[GitHub] ScottChapman opened a new issue #101: please add ignore_certs env var

2018-03-11 Thread GitBox
ScottChapman opened a new issue #101: please add ignore_certs env var URL: https://github.com/apache/incubator-openwhisk-client-js/issues/101 When running test code for openwhisk actions it is often necessary to ignore certificates since it is running openwhisk in travis. It looks like the

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

2018-03-11 Thread GitBox
csantanapr commented on issue #101: please add ignore_certs env var URL: https://github.com/apache/incubator-openwhisk-client-js/issues/101#issuecomment-372123864 We could improve the library by detecting if apihost its an IP address then ignore ssl if it?s a hostname then do not ignore.

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

2018-03-11 Thread GitBox
starpit commented on issue #101: please add ignore_certs env var URL: https://github.com/apache/incubator-openwhisk-client-js/issues/101#issuecomment-372126570 i do see what scott is after. he doesn't want to have to modify his code for various setups. i certainly wouldn't feel

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

2018-03-11 Thread GitBox
csantanapr commented on issue #101: please add ignore_certs env var URL: https://github.com/apache/incubator-openwhisk-client-js/issues/101#issuecomment-372129389 I just realized I?m an idiot I should have done this for swift4 ssl sdk problem

[GitHub] csantanapr commented on issue #3419: make OW context information available to Action

2018-03-11 Thread GitBox
csantanapr commented on issue #3419: make OW context information available to Action URL: https://github.com/apache/incubator-openwhisk/issues/3419#issuecomment-372110134 Hum that was on the init or run? They should be there on the run On the init not today, but we can think

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

2018-03-11 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-372122094 You can pass a parameter: ignore_certs. Turns off server SSL/TLS certificate verification. This allows the client to be

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

2018-03-11 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-372126177 I?m just pointing out we have at least three ways to achieve the workaround today: parameter, node?s own env var, or use an http

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

2018-03-11 Thread GitBox
csantanapr commented on issue #101: please add ignore_certs env var URL: https://github.com/apache/incubator-openwhisk-client-js/issues/101#issuecomment-372126903 FWIW I had same problem on swift4 sdk testing and use an override baseURL action Param and use controller host and port 10001

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

2018-03-11 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-372128750 Scott can you clarify how you?re developing and running? Are you writing js/node code and running it with node against a

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

2018-03-11 Thread GitBox
starpit commented on issue #101: please add ignore_certs env var URL: https://github.com/apache/incubator-openwhisk-client-js/issues/101#issuecomment-372125938 Maybe users want to be more selective. This is an automated

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

2018-03-11 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-372125994 The wsk prop file doesn?t help. We rejected the pr which records the certificate validity.

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

2018-03-11 Thread GitBox
csantanapr commented on issue #101: please add ignore_certs env var URL: https://github.com/apache/incubator-openwhisk-client-js/issues/101#issuecomment-372129178 >I would rather spend the effort and automate a signed certificate or enable port 80 in the edge host and eschew the problem

[GitHub] csantanapr opened a new issue #61: tag repos with version after first release with openwhisk@1.0.0

2018-03-11 Thread GitBox
csantanapr opened a new issue #61: tag repos with version after first release with openwhisk@1.0.0 URL: https://github.com/apache/incubator-openwhisk-release/issues/61 After is declared done, all the repos included in the release at the respective git hash should be tag with a single

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

2018-03-11 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-372123671 Yes, I am currently doing that in order to run my test code. In my particular case I have an action that is using this

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

2018-03-11 Thread GitBox
starpit commented on issue #101: please add ignore_certs env var URL: https://github.com/apache/incubator-openwhisk-client-js/issues/101#issuecomment-372125490 i think scott is simply suggesting that the openwhisk npm be updated to accept an env var for ignore_certs. it accepts env vars

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

2018-03-11 Thread GitBox
starpit commented on issue #101: please add ignore_certs env var URL: https://github.com/apache/incubator-openwhisk-client-js/issues/101#issuecomment-372126570 i do see what scott is after. he doesn't want to have to modify his code for various setups. i certainly wouldn't feel

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

2018-03-11 Thread GitBox
csantanapr commented on issue #101: please add ignore_certs env var URL: https://github.com/apache/incubator-openwhisk-client-js/issues/101#issuecomment-372126903 FWIW I had same problem on swift4 sdk testing and use an override baseURL and use controller host and port 10001

[GitHub] dubeejw opened a new pull request #3421: Add ElasticSearch Log Store

2018-03-11 Thread GitBox
dubeejw opened a new pull request #3421: Add ElasticSearch Log Store URL: https://github.com/apache/incubator-openwhisk/pull/3421 ## Description ## Related issue and scope - [ ] I opened an issue to propose and discuss this change (#) ## My changes

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

2018-03-11 Thread GitBox
rabbah commented on issue #3421: Add ElasticSearch Log Store URL: https://github.com/apache/incubator-openwhisk/pull/3421#issuecomment-372201237 We should also provide (seperatly) an optional elastic playbook (related to #3390).

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

2018-03-11 Thread GitBox
csantanapr commented on issue #101: please add ignore_certs env var URL: https://github.com/apache/incubator-openwhisk-client-js/issues/101#issuecomment-372188358 @rabbah Just to be sure we want to disable tls only for whisk api but we should not disable for other external url (ie