[incubator-openwhisk-cli] annotated tag latest updated (252f634 -> b9c0e2d)

2017-06-01 Thread csantanapr
This is an automated email from the ASF dual-hosted git repository. csantanapr pushed a change to annotated tag latest in repository https://gitbox.apache.org/repos/asf/incubator-openwhisk-cli.git. *** WARNING: tag latest was modified! *** from 252f634 (tag) to b9c0e2d (tag)

[GitHub] lzbj closed pull request #275: Fixing zip action creation

2017-06-01 Thread git
lzbj closed pull request #275: Fixing zip action creation URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/275 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] pritidesai commented on a change in pull request #275: Fixing zip action creation

2017-06-01 Thread git
pritidesai commented on a change in pull request #275: Fixing zip action creation URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/275#discussion_r119758438 ## File path: parsers/manifest_parser.go ## @@ -262,6 +257,15 @@ func (dm *YAMLParser)

[GitHub] tysonnorris commented on issue #38: added script to node-local testing for running test.js via the action?

2017-06-01 Thread git
tysonnorris commented on issue #38: added script to node-local testing for running test.js via the action? URL: https://github.com/apache/incubator-openwhisk-devtools/pull/38#issuecomment-305654968 ugh, renaming variables...

[GitHub] pritidesai commented on issue #276: Action creation with .jar fails with "The request content was malformed"

2017-06-01 Thread git
pritidesai commented on issue #276: Action creation with .jar fails with "The request content was malformed" URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/276#issuecomment-305654859 As part of fixing this issue, please add integration test for JAR.

[GitHub] tysonnorris commented on issue #38: added script to node-local testing for running test.js via the action?

2017-06-01 Thread git
tysonnorris commented on issue #38: added script to node-local testing for running test.js via the action? URL: https://github.com/apache/incubator-openwhisk-devtools/pull/38#issuecomment-305651625 One more change - I found it did NOT work when using relative paths... fixed now

[GitHub] pritidesai opened a new issue #276: Action creation with .jar fails with "The request content was malformed"

2017-06-01 Thread git
pritidesai opened a new issue #276: Action creation with .jar fails with "The request content was malformed" URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/276 **1: Create a Java file called `Hello.Java` from

[GitHub] ddragosd commented on a change in pull request #38: added script to node-local testing for running test.js via the action?

2017-06-01 Thread git
ddragosd commented on a change in pull request #38: added script to node-local testing for running test.js via the action? URL: https://github.com/apache/incubator-openwhisk-devtools/pull/38#discussion_r119752465 ## File path: node-local/runtest.sh ## @@ -0,0 +1,47 @@

[GitHub] tysonnorris commented on a change in pull request #38: added script to node-local testing for running test.js via the action?

2017-06-01 Thread git
tysonnorris commented on a change in pull request #38: added script to node-local testing for running test.js via the action? URL: https://github.com/apache/incubator-openwhisk-devtools/pull/38#discussion_r119752336 ## File path: node-local/runtest.sh ## @@ -0,0 +1,47 @@

[GitHub] tysonnorris commented on a change in pull request #38: added script to node-local testing for running test.js via the action?

2017-06-01 Thread git
tysonnorris commented on a change in pull request #38: added script to node-local testing for running test.js via the action? URL: https://github.com/apache/incubator-openwhisk-devtools/pull/38#discussion_r119751913 ## File path: node-local/runtest.sh ## @@ -0,0 +1,47 @@

[GitHub] ddragosd commented on a change in pull request #38: added script to node-local testing for running test.js via the action?

2017-06-01 Thread git
ddragosd commented on a change in pull request #38: added script to node-local testing for running test.js via the action? URL: https://github.com/apache/incubator-openwhisk-devtools/pull/38#discussion_r119751508 ## File path: node-local/runtest.sh ## @@ -0,0 +1,47 @@

[GitHub] houshengbo opened a new pull request #52: Add 30 second delay between running API gateway test suites

2017-06-01 Thread git
houshengbo opened a new pull request #52: Add 30 second delay between running API gateway test suites URL: https://github.com/apache/incubator-openwhisk-cli/pull/52 Since openwhisk throttles any given user from issuing too many requests in a rolling 1 minute period, we need to add the

[GitHub] csantanapr commented on issue #2234: OPTIONS verb does not reach web actions

2017-06-01 Thread git
csantanapr commented on issue #2234: OPTIONS verb does not reach web actions URL: https://github.com/apache/incubator-openwhisk/issues/2234#issuecomment-305621954 Ok fine you picky :D let's go with "web-custom-options" I thought I was clear false or not present is the default.

[GitHub] tysonnorris opened a new pull request #38: added script to node-local testing for running test.js via the action?

2017-06-01 Thread git
tysonnorris opened a new pull request #38: added script to node-local testing for running test.js via the action? URL: https://github.com/apache/incubator-openwhisk-devtools/pull/38 ? container itself, so that actions using npm libraries can be tested more simply

[GitHub] pritidesai commented on a change in pull request #275: Fixing zip action creation

2017-06-01 Thread git
pritidesai commented on a change in pull request #275: Fixing zip action creation URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/275#discussion_r119714745 ## File path: parsers/manifest_parser.go ## @@ -262,6 +257,15 @@ func (dm *YAMLParser)

[GitHub] rabbah commented on a change in pull request #2326: (WIP) Alphabetize

2017-06-01 Thread git
rabbah commented on a change in pull request #2326: (WIP) Alphabetize URL: https://github.com/apache/incubator-openwhisk/pull/2326#discussion_r119712723 ## File path: tools/cli/go-whisk-cli/commands/action.go ## @@ -310,7 +311,7 @@ var actionListCmd = { if

[GitHub] rabbah commented on issue #2326: (WIP) Alphabetize

2017-06-01 Thread git
rabbah commented on issue #2326: (WIP) Alphabetize URL: https://github.com/apache/incubator-openwhisk/pull/2326#issuecomment-305601929 Thanks for picking this up. FYI you might want to look at this PR which seems abandoned https://github.com/apache/incubator-openwhisk/pull/1729 (it has

[GitHub] mrutkows commented on issue #2323: add support for email address scan to scanCode.py

2017-06-01 Thread git
mrutkows commented on issue #2323: add support for email address scan to scanCode.py URL: https://github.com/apache/incubator-openwhisk/pull/2323#issuecomment-305601977 @domdom82 can you provide these same file changes to "incubator-openwhisk-utilities" instances of scancode files? I

[GitHub] houshengbo commented on issue #46: Add scancode to travis and fix all scancode errors

2017-06-01 Thread git
houshengbo commented on issue #46: Add scancode to travis and fix all scancode errors URL: https://github.com/apache/incubator-openwhisk-cli/pull/46#issuecomment-305601632 This PR look good to me. This is an automated

[GitHub] houshengbo commented on issue #46: Add scancode to travis and fix all scancode errors

2017-06-01 Thread git
houshengbo commented on issue #46: Add scancode to travis and fix all scancode errors URL: https://github.com/apache/incubator-openwhisk-cli/pull/46#issuecomment-305601632 This PR look good to me. This is an automated

[GitHub] mrutkows commented on issue #2323: add support for email address scan to scanCode.py

2017-06-01 Thread git
mrutkows commented on issue #2323: add support for email address scan to scanCode.py URL: https://github.com/apache/incubator-openwhisk/pull/2323#issuecomment-305601597 @markusthoemmes @domdom82 we are moving to use it out of the "incubator-openwhisk-utilities" repo. and any changes

[GitHub] houshengbo closed pull request #46: Add scancode to travis and fix all scancode errors

2017-06-01 Thread git
houshengbo closed pull request #46: Add scancode to travis and fix all scancode errors URL: https://github.com/apache/incubator-openwhisk-cli/pull/46 This is an automated message from the Apache Git Service. To respond to

[GitHub] underwoodb-sd-ibm opened a new pull request #2326: (WIP) Alphabetize

2017-06-01 Thread git
underwoodb-sd-ibm opened a new pull request #2326: (WIP) Alphabetize URL: https://github.com/apache/incubator-openwhisk/pull/2326 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] csantanapr commented on issue #2234: OPTIONS verb does not reach web actions

2017-06-01 Thread git
csantanapr commented on issue #2234: OPTIONS verb does not reach web actions URL: https://github.com/apache/incubator-openwhisk/issues/2234#issuecomment-305590803 I propose (P3) By default controller response to OPTIONS with web actions ``` Access-Control-Allow-Origin: * (all

[GitHub] taylorking closed pull request #200: provide snapshotting of swagger stored in redis

2017-06-01 Thread git
taylorking closed pull request #200: provide snapshotting of swagger stored in redis URL: https://github.com/apache/incubator-openwhisk-apigateway/pull/200 This is an automated message from the Apache Git Service. To

[GitHub] csantanapr commented on issue #8: avoid merge commits and create a linear git history

2017-06-01 Thread git
csantanapr commented on issue #8: avoid merge commits and create a linear git history URL: https://github.com/apache/incubator-openwhisk-utilities/issues/8#issuecomment-305559277 @rabbah I ping INFRA on chat to see if someone can pick this one up, it's being a week since I open

[GitHub] csantanapr commented on issue #8: avoid merge commits and create a linear git history

2017-06-01 Thread git
csantanapr commented on issue #8: avoid merge commits and create a linear git history URL: https://github.com/apache/incubator-openwhisk-utilities/issues/8#issuecomment-305559158 Yes we can enable the setting. Already opened an INFRA ticket to get all the openwhisk repos configured the

[GitHub] animeshsingh commented on issue #5: Pin docker API version to support minikube

2017-06-01 Thread git
animeshsingh commented on issue #5: Pin docker API version to support minikube URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/5#issuecomment-305557668 Thanks @tobias - LGTM @DanLavine please review it

[GitHub] rabbah commented on issue #2324: Many "Message length exceeded" from a subject will result in lock out (for that subject)

2017-06-01 Thread git
rabbah commented on issue #2324: Many "Message length exceeded" from a subject will result in lock out (for that subject) URL: https://github.com/apache/incubator-openwhisk/issues/2324#issuecomment-305490952 You are right - this will result in a subject being throttled until the

[GitHub] rabbah commented on issue #2324: Many "Message length exceeded" from a subject will result in lock out (for that subject)

2017-06-01 Thread git
rabbah commented on issue #2324: Many "Message length exceeded" from a subject will result in lock out (for that subject) URL: https://github.com/apache/incubator-openwhisk/issues/2324#issuecomment-305530259 I haven't opened the pull request yet (It's part of several inter-related

[GitHub] rabbah commented on issue #2234: OPTIONS verb does not reach web actions

2017-06-01 Thread git
rabbah commented on issue #2234: OPTIONS verb does not reach web actions URL: https://github.com/apache/incubator-openwhisk/issues/2234#issuecomment-305470861 For example we can add annotations: * `default-options`: true or false, and * `accept-methods`: {set of verbs}

[GitHub] rabbah commented on issue #2234: OPTIONS verb does not reach web actions

2017-06-01 Thread git
rabbah commented on issue #2234: OPTIONS verb does not reach web actions URL: https://github.com/apache/incubator-openwhisk/issues/2234#issuecomment-305470861 For example we can add annotations: * `default-options`: true or false, and * `accept-methods`: {set of verbs}

[GitHub] rabbah commented on issue #2234: OPTIONS verb does not reach web actions

2017-06-01 Thread git
rabbah commented on issue #2234: OPTIONS verb does not reach web actions URL: https://github.com/apache/incubator-openwhisk/issues/2234#issuecomment-305470861 For example we can add annotations: * `default-options`: true or false, and * `accept-methods`: {set of verbs}

[GitHub] mrutkows commented on issue #2325: Run scancode utility from the utility repo/project

2017-06-01 Thread git
mrutkows commented on issue #2325: Run scancode utility from the utility repo/project URL: https://github.com/apache/incubator-openwhisk/pull/2325#issuecomment-305526735 @rabbah thanks for the reminder. This is an

[GitHub] rabbah commented on issue #2234: OPTIONS verb does not reach web actions

2017-06-01 Thread git
rabbah commented on issue #2234: OPTIONS verb does not reach web actions URL: https://github.com/apache/incubator-openwhisk/issues/2234#issuecomment-305470861 For example we can add annotations: * `default-options`: true or false, and * `accept-methods`: {set of verbs}

[GitHub] rabbah commented on issue #2234: OPTIONS verb does not reach web actions

2017-06-01 Thread git
rabbah commented on issue #2234: OPTIONS verb does not reach web actions URL: https://github.com/apache/incubator-openwhisk/issues/2234#issuecomment-305470861 For example we can add annotations: * default-options: true or false, and * accept-methods: {set of verbs} Where

[GitHub] jlost commented on issue #2324: Many "Message length exceeded" from a subject will result is lock out (for that subject)

2017-06-01 Thread git
jlost commented on issue #2324: Many "Message length exceeded" from a subject will result is lock out (for that subject) URL: https://github.com/apache/incubator-openwhisk/issues/2324#issuecomment-305521085 Thanks for the info! Is there any reason that patch hasn't been accepted into the

[GitHub] rabbah opened a new issue #8: avoid merge commits and create a linear git history

2017-06-01 Thread git
rabbah opened a new issue #8: avoid merge commits and create a linear git history URL: https://github.com/apache/incubator-openwhisk-utilities/issues/8 @csantanapr can we enable rebase and squash for this repository as the only way to merge to avoid merge commits?

[GitHub] rabbah opened a new issue #7: add unit tests and CI

2017-06-01 Thread git
rabbah opened a new issue #7: add unit tests and CI URL: https://github.com/apache/incubator-openwhisk-utilities/issues/7 scan code is used to verify proper licenses, and common code formats across a number of repositories. this repo should have unit tests that validate that scan

[GitHub] rabbah commented on issue #2325: Run scancode utility from the utility repo/project

2017-06-01 Thread git
rabbah commented on issue #2325: Run scancode utility from the utility repo/project URL: https://github.com/apache/incubator-openwhisk/pull/2325#issuecomment-305519814 Will also need to update the wiki (pre commit hook tip):

[GitHub] mrutkows opened a new pull request #2325: Run scancode utility from the utility repo/project

2017-06-01 Thread git
mrutkows opened a new pull request #2325: Run scancode utility from the utility repo/project URL: https://github.com/apache/incubator-openwhisk/pull/2325 In an effort, to share scancode (and other utils) across project repos. (and not pre-req the entire openewhisk repo. for travis/builds)

[GitHub] rabbah commented on issue #2324: Many "Message length exceeded" locks OpenWhisk (DoS vulnerability?)

2017-06-01 Thread git
rabbah commented on issue #2324: Many "Message length exceeded" locks OpenWhisk (DoS vulnerability?) URL: https://github.com/apache/incubator-openwhisk/issues/2324#issuecomment-305490952 You are right - this will result in a subject being throttled until the accounting is reset. This is

[GitHub] markusthoemmes commented on issue #2323: add support for email address scan to scanCode.py

2017-06-01 Thread git
markusthoemmes commented on issue #2323: add support for email address scan to scanCode.py URL: https://github.com/apache/incubator-openwhisk/pull/2323#issuecomment-305485863 For old times sake I tried to find an indentation error. Could'nt find one :+1: ? @mrutkows are we moving

[GitHub] bjustin-ibm commented on a change in pull request #2321: clean up formatting docs for actions

2017-06-01 Thread git
bjustin-ibm commented on a change in pull request #2321: clean up formatting docs for actions URL: https://github.com/apache/incubator-openwhisk/pull/2321#discussion_r119605354 ## File path: docs/actions.md ## @@ -632,48 +634,61 @@ When you create an OpenWhisk Swift

[GitHub] tobias opened a new pull request #5: Pin docker API version to support minikube

2017-06-01 Thread git
tobias opened a new pull request #5: Pin docker API version to support minikube URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/5 Minikube currently uses Docker 1.23, and Openwhisk uses a newer client. But Openwhisk doesn't currently require anything that isn't

[GitHub] domdom82 opened a new pull request #2323: add support for email address scan to scanCode.py

2017-06-01 Thread git
domdom82 opened a new pull request #2323: add support for email address scan to scanCode.py URL: https://github.com/apache/incubator-openwhisk/pull/2323 Added feature to scan for given email address regex patterns. Useful if you want to avoid internal company or private emails in the

[GitHub] rabbah commented on issue #2234: OPTIONS verb does not reach web actions

2017-06-01 Thread git
rabbah commented on issue #2234: OPTIONS verb does not reach web actions URL: https://github.com/apache/incubator-openwhisk/issues/2234#issuecomment-305470861 For example we can add annotations: * default-options: true or false, and * accept-methods: {set of verbs} Where

[GitHub] rabbah commented on issue #2234: OPTIONS verb does not reach web actions

2017-06-01 Thread git
rabbah commented on issue #2234: OPTIONS verb does not reach web actions URL: https://github.com/apache/incubator-openwhisk/issues/2234#issuecomment-305466994 The action is already fetched - so access to the annotation will not incur additional overhead.

[GitHub] csantanapr commented on issue #2234: OPTIONS verb does not reach web actions

2017-06-01 Thread git
csantanapr commented on issue #2234: OPTIONS verb does not reach web actions URL: https://github.com/apache/incubator-openwhisk/issues/2234#issuecomment-305465903 I'm having seconds thoughts on making the controller do more for every OPTIONS request, it would need to do a DB call to fetch

[GitHub] sven-lange-last commented on issue #2319: Fix activation feed book-keeping in reactive pool (#2285)

2017-06-01 Thread git
sven-lange-last commented on issue #2319: Fix activation feed book-keeping in reactive pool (#2285) URL: https://github.com/apache/incubator-openwhisk/pull/2319#issuecomment-305435724 @markusthoemmes thanks for making sure messages are now sent in a fixed sequence such that corresponding

[GitHub] sven-lange-last commented on issue #2319: Fix activation feed book-keeping in reactive pool (#2285)

2017-06-01 Thread git
sven-lange-last commented on issue #2319: Fix activation feed book-keeping in reactive pool (#2285) URL: https://github.com/apache/incubator-openwhisk/pull/2319#issuecomment-305435724 @markusthoemmes thanks for making sure messages are now sent in a fixed sequence such that corresponding

[incubator-openwhisk] branch master updated: Install/upgrade six module before installing other dependencies (#2322)

2017-06-01 Thread cbickel
This is an automated email from the ASF dual-hosted git repository. cbickel pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-openwhisk.git The following commit(s) were added to refs/heads/master by this push: new 71cec8f Install/upgrade six

[GitHub] markusthoemmes closed pull request #2283: Add tool to delete logs from old activations

2017-06-01 Thread git
markusthoemmes closed pull request #2283: Add tool to delete logs from old activations URL: https://github.com/apache/incubator-openwhisk/pull/2283 This is an automated message from the Apache Git Service. To respond to

[incubator-openwhisk] branch master updated: Add tool to delete logs from old activations (#2283)

2017-06-01 Thread markusthoemmes
This is an automated email from the ASF dual-hosted git repository. markusthoemmes pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-openwhisk.git The following commit(s) were added to refs/heads/master by this push: new ecccb63 Add tool to