[GitHub] duynguyen commented on a change in pull request #25: Use prebuilt openwhisk images for docker-compose

2017-05-08 Thread git
duynguyen commented on a change in pull request #25: Use prebuilt openwhisk images for docker-compose URL: https://github.com/apache/incubator-openwhisk-devtools/pull/25#discussion_r115194736 ## File path: docker-compose/docker-compose.yml ## @@ -51,7 +51,7 @@ services:

[GitHub] duynguyen commented on a change in pull request #25: Use prebuilt openwhisk images for docker-compose

2017-05-08 Thread git
duynguyen commented on a change in pull request #25: Use prebuilt openwhisk images for docker-compose URL: https://github.com/apache/incubator-openwhisk-devtools/pull/25#discussion_r115194736 ## File path: docker-compose/docker-compose.yml ## @@ -51,7 +51,7 @@ services:

notifications from github to dev list

2017-05-08 Thread Carlos Santana
I was informed on Slack that dev list is receiving emails from github with the recent move of repos to the apache org. For some reason I don't get them on my mail clients including Google Inbox. but I can see them in the archives https://lists.apache.org/list.html?dev@openwhisk.apache.org I opene

Re: PassportJS with OpenWhisk

2017-05-08 Thread Matt Rutkowski
This is such a great thought and piece of work. Plan to try this myself perhaps after ApacheCon... would like to make it a Sample (new repo.) "app" and add support for wskdeploy. Kind regards, Matt From: Dascalita Dragos To: "dev@openwhisk.apache.org" Date: 05/07/2017 05:04 PM S

[GitHub] houshengbo opened a new pull request #11: Update with openwhisk

2017-05-08 Thread git
houshengbo opened a new pull request #11: Update with openwhisk URL: https://github.com/apache/incubator-openwhisk-client-go/pull/11 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] houshengbo closed pull request #11: Update with openwhisk

2017-05-08 Thread git
houshengbo closed pull request #11: Update with openwhisk URL: https://github.com/apache/incubator-openwhisk-client-go/pull/11 This is an automated message from the Apache Git Service. To respond to the message, please log o

Re: PassportJS with OpenWhisk

2017-05-08 Thread James Thomas
This looks amazing! Great work - will have to try it out. On 7 May 2017 at 23:03, Dascalita Dragos wrote: > Recently I’ve been trying to see if we can reuse PassportJS with OpenWhisk > in order to create a User Authentication experience in a Serverless > fashion. PassportJS is a Node library tha

[GitHub] houshengbo opened a new pull request #39: Update openwhisk-cli to incubator-openwhisk-cli

2017-05-08 Thread git
houshengbo opened a new pull request #39: Update openwhisk-cli to incubator-openwhisk-cli URL: https://github.com/apache/incubator-openwhisk-cli/pull/39 This is an automated message from the Apache Git Service. To respond t

[GitHub] alexsong93 commented on a change in pull request #198: Dao

2017-05-08 Thread git
alexsong93 commented on a change in pull request #198: Dao URL: https://github.com/apache/incubator-openwhisk-apigateway/pull/198#discussion_r115308284 ## File path: scripts/lua/oauth/google.lua ## @@ -23,46 +23,44 @@ local cjson = require "cjson" local http = require "re

[GitHub] alexsong93 commented on a change in pull request #198: Dao

2017-05-08 Thread git
alexsong93 commented on a change in pull request #198: Dao URL: https://github.com/apache/incubator-openwhisk-apigateway/pull/198#discussion_r115307204 ## File path: scripts/lua/management/lib/apis.lua ## @@ -35,10 +35,10 @@ MANAGEDURL_PORT = (MANAGEDURL_PORT ~= nil and M

[GitHub] alexsong93 commented on a change in pull request #198: Dao

2017-05-08 Thread git
alexsong93 commented on a change in pull request #198: Dao URL: https://github.com/apache/incubator-openwhisk-apigateway/pull/198#discussion_r115309170 ## File path: scripts/lua/lib/redis.lua ## @@ -56,6 +61,9 @@ local _M = {} -- @param password redis password (nil if no

Re: PassportJS with OpenWhisk

2017-05-08 Thread Carlos Santana
This is awesome Dragos +1 I think everybody by now realize that with web actions and having access to the request and response, there is a lot of modules in ExpressJS that cab be leverage in OpenWhisk without re-inventing the wheel (web server) :-) Don't know if you saw last post from Lionel [1]

[GitHub] csantanapr closed pull request #39: Update openwhisk-cli to incubator-openwhisk-cli

2017-05-08 Thread git
csantanapr closed pull request #39: Update openwhisk-cli to incubator-openwhisk-cli URL: https://github.com/apache/incubator-openwhisk-cli/pull/39 This is an automated message from the Apache Git Service. To respond to the

[GitHub] csantanapr opened a new issue #40: add travis status button to readme

2017-05-08 Thread git
csantanapr opened a new issue #40: add travis status button to readme URL: https://github.com/apache/incubator-openwhisk-cli/issues/40 This is an automated message from the Apache Git Service. To respond to the message, plea

[GitHub] csantanapr opened a new pull request #41: add travis button to readme

2017-05-08 Thread git
csantanapr opened a new pull request #41: add travis button to readme URL: https://github.com/apache/incubator-openwhisk-cli/pull/41 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] taylorking commented on issue #183: Create a DAO adapter layer for calls to the datastore

2017-05-08 Thread git
taylorking commented on issue #183: Create a DAO adapter layer for calls to the datastore URL: https://github.com/apache/incubator-openwhisk-apigateway/issues/183#issuecomment-300010355 for some reason I can't tick the boxes on your issue - [x] Add a config flag defining which driv

[GitHub] taylorking commented on issue #183: Create a DAO adapter layer for calls to the datastore

2017-05-08 Thread git
taylorking commented on issue #183: Create a DAO adapter layer for calls to the datastore URL: https://github.com/apache/incubator-openwhisk-apigateway/issues/183#issuecomment-300010355 for some reason I can't tick the boxes on your issue - [x] Add a config flag defining which driv

[GitHub] mrutkows commented on issue #27: Top-level README should list included projects by subdir.

2017-05-08 Thread git
mrutkows commented on issue #27: Top-level README should list included projects by subdir. URL: https://github.com/apache/incubator-openwhisk-devtools/issues/27#issuecomment-300017240 The PR I referenced above is here: https://github.com/apache/incubator-openwhisk-devtools/pull/26/files

[GitHub] mrutkows commented on issue #27: Top-level README should list included projects by subdir.

2017-05-08 Thread git
mrutkows commented on issue #27: Top-level README should list included projects by subdir. URL: https://github.com/apache/incubator-openwhisk-devtools/issues/27#issuecomment-300017240 The PR I referenced above is here: https://github.com/apache/incubator-openwhisk-devtools/pull/26/files

[GitHub] mrutkows opened a new issue #27: Top-level README should list included projects by subdir.

2017-05-08 Thread git
mrutkows opened a new issue #27: Top-level README should list included projects by subdir. URL: https://github.com/apache/incubator-openwhisk-devtools/issues/27 As this repo. now houses 2 independent "deployments" (i.e., Compose and Kube) and now we have a PR for node-test subdir. we need

Re: PassportJS with OpenWhisk

2017-05-08 Thread Dascalita Dragos
Thanks everyone for the feedback. > "...we currently don't have passport installed in nodejs6action runtime, to use your repo as is today I need to create a zip first, can't use the .js as your doc show right?..." Actually no. I've used browserify to combine the dependencies into a single JS. Tha

[GitHub] ddragosd commented on issue #26: A local test script for node.js functions

2017-05-08 Thread git
ddragosd commented on issue #26: A local test script for node.js functions URL: https://github.com/apache/incubator-openwhisk-devtools/pull/26#issuecomment-300023097 @trieloff wanna include the suggestion made by @mrutkows in this PR or separate ?

Re: PassportJS with OpenWhisk

2017-05-08 Thread Matt Rutkowski
Good news though... wskdeploy has added ZIP support... so we can do this for you if you create a manifest. If it doesn't work for some reason or another open an issue and I am sure Daisy or David can make it work. In fact, would like this to be featured in wskdeploy as a use case in the spec

[GitHub] csantanapr commented on issue #41: add travis button to readme

2017-05-08 Thread git
csantanapr commented on issue #41: add travis button to readme URL: https://github.com/apache/incubator-openwhisk-cli/pull/41#issuecomment-300048228 @csantanapr Do we want to add golint to the scanCode.py for all .go files? sounds like a good idea --

[GitHub] csantanapr closed pull request #41: add travis button to readme

2017-05-08 Thread git
csantanapr closed pull request #41: add travis button to readme URL: https://github.com/apache/incubator-openwhisk-cli/pull/41 This is an automated message from the Apache Git Service. To respond to the message, please log o

[GitHub] csantanapr opened a new issue #59: update travis based on new apache location

2017-05-08 Thread git
csantanapr opened a new issue #59: update travis based on new apache location URL: https://github.com/apache/incubator-openwhisk-package-alarms/issues/59 This is an automated message from the Apache Git Service. To respond t

[GitHub] csantanapr opened a new pull request #60: Issue 59 update travis for dockerhub integration

2017-05-08 Thread git
csantanapr opened a new pull request #60: Issue 59 update travis for dockerhub integration URL: https://github.com/apache/incubator-openwhisk-package-alarms/pull/60 Closes #59 This is an automated message from the Apache Gi