[GitHub] cbickel closed pull request #2665: Use all controllers round robin

2017-08-31 Thread git
cbickel closed pull request #2665: Use all controllers round robin URL: https://github.com/apache/incubator-openwhisk/pull/2665 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] feliu opened a new issue #2687: Non-blocking web action

2017-08-31 Thread git
feliu opened a new issue #2687: Non-blocking web action URL: https://github.com/apache/incubator-openwhisk/issues/2687 Is there any way of using a web action using an HTTP request in a non-blocking manner? I understand that the result should be the activation code in this case. Thanks!

[GitHub] csantanapr closed pull request #2524: Docker for mac support

2017-08-31 Thread git
csantanapr closed pull request #2524: Docker for mac support URL: https://github.com/apache/incubator-openwhisk/pull/2524 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] csantanapr commented on issue #2524: Docker for mac support

2017-08-31 Thread git
csantanapr commented on issue #2524: Docker for mac support URL: https://github.com/apache/incubator-openwhisk/pull/2524#issuecomment-326482391 continue work in #2686 This is an automated message from the Apache Git

[GitHub] csantanapr opened a new pull request #2686: Docker for mac take 2

2017-08-31 Thread git
csantanapr opened a new pull request #2686: Docker for mac take 2 URL: https://github.com/apache/incubator-openwhisk/pull/2686 This change is a continuation of PR #2440, PR #2452, PR #2684 #2524 To give it a try ``` sudo ifconfig lo0 alias 172.17.0.1/24 cd ansible

[GitHub] rabbah opened a new pull request #2684: Decorate throttled error response with count/limit.

2017-08-31 Thread git
rabbah opened a new pull request #2684: Decorate throttled error response with count/limit. URL: https://github.com/apache/incubator-openwhisk/pull/2684 Fixes #1385. Reports error messages as: `Too many requests in the last minute (count: 120, allowed: 60). (code ...)` `Too

[GitHub] lzbj commented on issue #425: Run some lint command before submit PR.

2017-08-31 Thread git
lzbj commented on issue #425: Run some lint command before submit PR. URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/425#issuecomment-326474853 close is as I think everybody knows it. This is an

[GitHub] lzbj closed issue #425: Run some lint command before submit PR.

2017-08-31 Thread git
lzbj closed issue #425: Run some lint command before submit PR. URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/425 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] mrutkows commented on a change in pull request #262: Adjust the code to reflect latest API gateway version.

2017-08-31 Thread git
mrutkows commented on a change in pull request #262: Adjust the code to reflect latest API gateway version. URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/262#discussion_r136487512 ## File path: tests/src/integration/apigateway/manifest.yml ## @@ -0,0

[GitHub] mrutkows commented on issue #427: Should move towards function for creating Dependent Packages?

2017-08-31 Thread git
mrutkows commented on issue #427: Should move towards function for creating Dependent Packages? URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/427#issuecomment-326467353 Will leave this open so we can discuss in the future; do not want to make any changes besides the

[GitHub] rabbah commented on a change in pull request #2591: User-Agent CLI/version header

2017-08-31 Thread git
rabbah commented on a change in pull request #2591: User-Agent CLI/version header URL: https://github.com/apache/incubator-openwhisk/pull/2591#discussion_r136485499 ## File path: tests/src/test/scala/whisk/core/cli/test/WskBasicUsageTests.scala ## @@ -119,6 +119,10 @@

[GitHub] rabbah closed issue #2682: Binary web action responses are not handled correctly

2017-08-31 Thread git
rabbah closed issue #2682: Binary web action responses are not handled correctly URL: https://github.com/apache/incubator-openwhisk/issues/2682 This is an automated message from the Apache Git Service. To respond to the

[GitHub] rabbah closed pull request #2683: Fix Binary Content-Type Responses for Web Actions

2017-08-31 Thread git
rabbah closed pull request #2683: Fix Binary Content-Type Responses for Web Actions URL: https://github.com/apache/incubator-openwhisk/pull/2683 This is an automated message from the Apache Git Service. To respond to the

[GitHub] lzbj closed issue #359: Integration Test - webhook

2017-08-31 Thread git
lzbj closed issue #359: Integration Test - webhook URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/359 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] dubeejw commented on issue #2683: Fix Binary Content-Type Responses for Web Actions

2017-08-31 Thread git
dubeejw commented on issue #2683: Fix Binary Content-Type Responses for Web Actions URL: https://github.com/apache/incubator-openwhisk/pull/2683#issuecomment-326458736 PG2 2028 ? This is an automated message from the

[GitHub] rabbah commented on issue #2493: API gw test cleanup

2017-08-31 Thread git
rabbah commented on issue #2493: API gw test cleanup URL: https://github.com/apache/incubator-openwhisk/pull/2493#issuecomment-326387467 pg1/2004 ? This is an automated message from the Apache Git Service. To respond to

[GitHub] rabbah commented on issue #2493: API gw test cleanup

2017-08-31 Thread git
rabbah commented on issue #2493: API gw test cleanup URL: https://github.com/apache/incubator-openwhisk/pull/2493#issuecomment-326458325 @mdeuser can you sanity check the throttle changes, please. This is an automated

[GitHub] daisy-ycguo commented on issue #399: Error out when env. variables from manifest/deployment are not defined

2017-08-31 Thread git
daisy-ycguo commented on issue #399: Error out when env. variables from manifest/deployment are not defined URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/399#issuecomment-326453696 Fixed and closed.

[GitHub] daisy-ycguo closed issue #399: Error out when env. variables from manifest/deployment are not defined

2017-08-31 Thread git
daisy-ycguo closed issue #399: Error out when env. variables from manifest/deployment are not defined URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/399 This is an automated message from the Apache Git

[GitHub] pritidesai commented on issue #356: Integration Test - hello world

2017-08-31 Thread git
pritidesai commented on issue #356: Integration Test - hello world URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/356#issuecomment-326445649 I am working on this and soon create a PR. Thanks! This is an

[GitHub] rabbah commented on a change in pull request #2683: Fix Binary Content-Type Responses for Web Actions

2017-08-31 Thread git
rabbah commented on a change in pull request #2683: Fix Binary Content-Type Responses for Web Actions URL: https://github.com/apache/incubator-openwhisk/pull/2683#discussion_r136469938 ## File path: tests/src/test/scala/whisk/core/controller/test/WebActionsApiTests.scala

[GitHub] rabbah commented on issue #429: runtime swift vs swift:2 vs swift:3.1.1

2017-08-31 Thread git
rabbah commented on issue #429: runtime swift vs swift:2 vs swift:3.1.1 URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/429#issuecomment-326443937 Not anymore. This is an automated message from the

[GitHub] pritidesai opened a new issue #429: runtime swift vs swift:2 vs swift:3.1.1

2017-08-31 Thread git
pritidesai opened a new issue #429: runtime swift vs swift:2 vs swift:3.1.1 URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/429 I am writing an action and have set runtime to `swift` or `swift:2` which fails: ``` 2017/08/31 14:09:46 Got error creating action

[GitHub] rabbah opened a new issue #2682: Binary web action responses are not handled correctly

2017-08-31 Thread git
rabbah opened a new issue #2682: Binary web action responses are not handled correctly URL: https://github.com/apache/incubator-openwhisk/issues/2682 A web action which returns a binary response is not handled correctly. Surprisingly we are missing a test for this case so it was

[GitHub] rabbah commented on issue #2677: Add blocking indicator for activation request.

2017-08-31 Thread git
rabbah commented on issue #2677: Add blocking indicator for activation request. URL: https://github.com/apache/incubator-openwhisk/pull/2677#issuecomment-326428829 PG2/2027 ? This is an automated message from the Apache

[GitHub] rabbah commented on issue #2677: Add blocking indicator for activation request.

2017-08-31 Thread git
rabbah commented on issue #2677: Add blocking indicator for activation request. URL: https://github.com/apache/incubator-openwhisk/pull/2677#issuecomment-326401034 PG2/2027 ? This is an automated message from the Apache

[GitHub] rabbah commented on issue #2493: API gw test cleanup

2017-08-31 Thread git
rabbah commented on issue #2493: API gw test cleanup URL: https://github.com/apache/incubator-openwhisk/pull/2493#issuecomment-326387467 pg1/2004 ? This is an automated message from the Apache Git Service. To respond to

[GitHub] csantanapr closed pull request #2671: Ignore IntelliJ '.iml' project file

2017-08-31 Thread git
csantanapr closed pull request #2671: Ignore IntelliJ '.iml' project file URL: https://github.com/apache/incubator-openwhisk/pull/2671 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] csantanapr commented on issue #2671: Ignore IntelliJ '.iml' project file

2017-08-31 Thread git
csantanapr commented on issue #2671: Ignore IntelliJ '.iml' project file URL: https://github.com/apache/incubator-openwhisk/pull/2671#issuecomment-326421619 thanks for the contrib ? This is an automated message from the

[GitHub] drcariel commented on issue #2591: User-Agent CLI/version header

2017-08-31 Thread git
drcariel commented on issue #2591: User-Agent CLI/version header URL: https://github.com/apache/incubator-openwhisk/pull/2591#issuecomment-326420416 I could not find any logs pertaining to "Illegal header" in PG3~1051. Not sure if I looked in the right places though

[GitHub] drcariel commented on issue #2591: User-Agent CLI/version header

2017-08-31 Thread git
drcariel commented on issue #2591: User-Agent CLI/version header URL: https://github.com/apache/incubator-openwhisk/pull/2591#issuecomment-326420416 I could not find any logs pertaining to "Illegal header" in PG3~1051. This

[GitHub] narinearraul opened a new issue #88: Activation size mismatch

2017-08-31 Thread git
narinearraul opened a new issue #88: Activation size mismatch URL: https://github.com/apache/incubator-openwhisk-package-alarms/issues/88 **Applies to:** ypAlarmsHealth-London **Test Failling:** AlarmsHealthFeedTests

[GitHub] mrutkows closed pull request #420: Add the error handling mechanism

2017-08-31 Thread git
mrutkows closed pull request #420: Add the error handling mechanism URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/420 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] rabbah commented on issue #2671: Ignore IntelliJ '.iml' project file

2017-08-31 Thread git
rabbah commented on issue #2671: Ignore IntelliJ '.iml' project file URL: https://github.com/apache/incubator-openwhisk/pull/2671#issuecomment-326401158 Thanks for the first contribution! This is an automated message from

[GitHub] rabbah commented on issue #2677: Add blocking indicator for activation request.

2017-08-31 Thread git
rabbah commented on issue #2677: Add blocking indicator for activation request. URL: https://github.com/apache/incubator-openwhisk/pull/2677#issuecomment-326401034 PG2/2027 ? This is an automated message from the Apache

[GitHub] drcariel commented on issue #2591: User-Agent CLI/version header

2017-08-31 Thread git
drcariel commented on issue #2591: User-Agent CLI/version header URL: https://github.com/apache/incubator-openwhisk/pull/2591#issuecomment-326397543 Where would I see that log? In Kibana? This is an automated message from

[GitHub] pritidesai commented on issue #359: Integration Test - webhook

2017-08-31 Thread git
pritidesai commented on issue #359: Integration Test - webhook URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/359#issuecomment-326396231 PR #428 This is an automated message from the Apache Git

[GitHub] mrutkows commented on issue #427: Should move towards function for creating Dependent Packages?

2017-08-31 Thread git
mrutkows commented on issue #427: Should move towards function for creating Dependent Packages? URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/427#issuecomment-326391542 @priti location is meant to be URI/URL in all places, we made a conscious decision to only change

[GitHub] pritidesai opened a new pull request #428: Fix for 359

2017-08-31 Thread git
pritidesai opened a new pull request #428: Fix for 359 URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/428 - Deleting webhook from use cases as its outdated and copy pasted from openwhisk catalog

[GitHub] mrutkows commented on issue #427: Should move towards function for creating Dependent Packages?

2017-08-31 Thread git
mrutkows commented on issue #427: Should move towards function for creating Dependent Packages? URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/427#issuecomment-326391542 @priti location is meant to be URI/URL in all places, we made a conscious decision to only change

[GitHub] qabbasi commented on issue #2671: Ignore IntelliJ '.iml' project file

2017-08-31 Thread git
qabbasi commented on issue #2671: Ignore IntelliJ '.iml' project file URL: https://github.com/apache/incubator-openwhisk/pull/2671#issuecomment-326388907 Obviously my first commit to an Apache project :) I just submitted my signed ACL to Apache...

[GitHub] rabbah commented on issue #2493: API gw test cleanup

2017-08-31 Thread git
rabbah commented on issue #2493: API gw test cleanup URL: https://github.com/apache/incubator-openwhisk/pull/2493#issuecomment-326387467 pg1/2003 ? This is an automated message from the Apache Git Service. To respond to

[GitHub] pritidesai closed issue #383: Delete openwhisk credentials from manifest and deployment files

2017-08-31 Thread git
pritidesai closed issue #383: Delete openwhisk credentials from manifest and deployment files URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/383 This is an automated message from the Apache Git

[GitHub] pritidesai commented on issue #383: Delete openwhisk credentials from manifest and deployment files

2017-08-31 Thread git
pritidesai commented on issue #383: Delete openwhisk credentials from manifest and deployment files URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/383#issuecomment-326372968 Confirming that we no longer have any hardcoded credential in manifest and deployment file

[GitHub] pritidesai commented on issue #415: dependent packages requires manifest.yml and doesn't support manifest.yaml

2017-08-31 Thread git
pritidesai commented on issue #415: dependent packages requires manifest.yml and doesn't support manifest.yaml URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/415#issuecomment-326369055 addressing `function` vs `location` in issue

[GitHub] pritidesai opened a new issue #427: Should move towards `function` for creating Dependent Packages?

2017-08-31 Thread git
pritidesai opened a new issue #427: Should move towards `function` for creating Dependent Packages? URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/427 we recently standardized on `function` deprecating `location` for specifying source location of action code/files.

[GitHub] mdeuser commented on issue #2591: User-Agent CLI/version header

2017-08-31 Thread git
mdeuser commented on issue #2591: User-Agent CLI/version header URL: https://github.com/apache/incubator-openwhisk/pull/2591#issuecomment-326364504 @drcariel - is that error log message now gone? This is an automated

[GitHub] houshengbo closed issue #409: Undeploy doesn't work with manifest.yml

2017-08-31 Thread git
houshengbo closed issue #409: Undeploy doesn't work with manifest.yml URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/409 This is an automated message from the Apache Git Service. To respond to the

[GitHub] houshengbo closed pull request #426: Fixing undeployment with yml

2017-08-31 Thread git
houshengbo closed pull request #426: Fixing undeployment with yml URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/426 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] houshengbo closed issue #415: dependent packages requires manifest.yml and doesn't support manifest.yaml

2017-08-31 Thread git
houshengbo closed issue #415: dependent packages requires manifest.yml and doesn't support manifest.yaml URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/415 This is an automated message from the Apache

[GitHub] dgrove-oss commented on issue #2680: do not use InstanceId.int within TransactionCounter

2017-08-31 Thread git
dgrove-oss commented on issue #2680: do not use InstanceId.int within TransactionCounter URL: https://github.com/apache/incubator-openwhisk/pull/2680#issuecomment-326358311 to expand on what Rodric said: invoker0 would generate tids 0,0,0,... invoker1 would generate 1,2,3,...

[GitHub] rabbah commented on issue #2680: do not use InstanceId.int within TransactionCounter

2017-08-31 Thread git
rabbah commented on issue #2680: do not use InstanceId.int within TransactionCounter URL: https://github.com/apache/incubator-openwhisk/pull/2680#issuecomment-326356312 The patch here addressed a bug: the invoker tids would be an infinite sequence of 0s. The invoker however does not

[GitHub] pritidesai opened a new pull request #426: Fixing undeployment with yml

2017-08-31 Thread git
pritidesai opened a new pull request #426: Fixing undeployment with yml URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/426 This is an automated message from the Apache Git Service. To respond to the

[GitHub] markusthoemmes commented on issue #2680: do not use InstanceId.int within TransactionCounter

2017-08-31 Thread git
markusthoemmes commented on issue #2680: do not use InstanceId.int within TransactionCounter URL: https://github.com/apache/incubator-openwhisk/pull/2680#issuecomment-326352557 What's the desired goal here? Note that the current loadbalancer is strictly based numbering the instances from

[GitHub] drcariel opened a new issue #2681: CLI 'CRD a subject' WskAdminTest Should Not Print DB Password to Log

2017-08-31 Thread git
drcariel opened a new issue #2681: CLI 'CRD a subject' WskAdminTest Should Not Print DB Password to Log URL: https://github.com/apache/incubator-openwhisk/issues/2681 See mainOpenwhisk build #11390 for an example. See

[GitHub] dgrove-oss opened a new pull request #2680: do not use InstanceId.int within TransactionCounter

2017-08-31 Thread git
dgrove-oss opened a new pull request #2680: do not use InstanceId.int within TransactionCounter URL: https://github.com/apache/incubator-openwhisk/pull/2680 abstract away the assumption that InstanceId must be an integer in the range 0..numInstances-1 in TransactionCounter. Small

[GitHub] pritidesai commented on issue #415: dependent packages requires manifest.yml and doesn't support manifest.yaml

2017-08-31 Thread git
pritidesai commented on issue #415: dependent packages requires manifest.yml and doesn't support manifest.yaml URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/415#issuecomment-326343951 ``` /deployers/whiskclient.go:MANIDESTFILE = "manifest.yml"

[GitHub] pritidesai opened a new issue #415: dependent packages requires manifest.yml and doesn't support manifest.yaml

2017-08-31 Thread git
pritidesai opened a new issue #415: dependent packages requires manifest.yml and doesn't support manifest.yaml URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/415 wskdeploy does not support dependent packages which has manifest.yaml file. It only deploys dependent

[GitHub] pritidesai commented on issue #415: dependent packages requires manifest.yml and doesn't support manifest.yaml

2017-08-31 Thread git
pritidesai commented on issue #415: dependent packages requires manifest.yml and doesn't support manifest.yaml URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/415#issuecomment-326337747 [update] verified hardcoding of manifest and deployment is **not** gone with PR

[GitHub] pritidesai commented on issue #415: dependent packages requires manifest.yml and doesn't support manifest.yaml

2017-08-31 Thread git
pritidesai commented on issue #415: dependent packages requires manifest.yml and doesn't support manifest.yaml URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/415#issuecomment-326337747 verified hardcoding of manifest and deployment is gone with PR #424

[GitHub] pritidesai closed issue #415: dependent packages requires manifest.yml and doesn't support manifest.yaml

2017-08-31 Thread git
pritidesai closed issue #415: dependent packages requires manifest.yml and doesn't support manifest.yaml URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/415 This is an automated message from the Apache

[GitHub] houshengbo commented on a change in pull request #262: Adjust the code to reflect latest API gateway version.

2017-08-31 Thread git
houshengbo commented on a change in pull request #262: Adjust the code to reflect latest API gateway version. URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/262#discussion_r136365156 ## File path: deployers/whiskclient.go ## @@ -25,175 +25,175 @@

[GitHub] houshengbo commented on a change in pull request #262: Adjust the code to reflect latest API gateway version.

2017-08-31 Thread git
houshengbo commented on a change in pull request #262: Adjust the code to reflect latest API gateway version. URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/262#discussion_r136365132 ## File path: deployers/whiskclient.go ## @@ -25,175 +25,175 @@

[GitHub] houshengbo commented on a change in pull request #262: Adjust the code to reflect latest API gateway version.

2017-08-31 Thread git
houshengbo commented on a change in pull request #262: Adjust the code to reflect latest API gateway version. URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/262#discussion_r136365081 ## File path: deployers/whiskclient.go ## @@ -25,175 +25,175 @@

[GitHub] drcariel commented on issue #2591: User-Agent CLI/version header

2017-08-31 Thread git
drcariel commented on issue #2591: User-Agent CLI/version header URL: https://github.com/apache/incubator-openwhisk/pull/2591#issuecomment-326321030 @dubeejw and @mdeuser can you show me a desired example? In my head, I am going to update this PR to show the User Agent like so: ```

[GitHub] mrutkows closed pull request #419: Print warning msg when env var is empty

2017-08-31 Thread git
mrutkows closed pull request #419: Print warning msg when env var is empty URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/419 This is an automated message from the Apache Git Service. To respond to the

[GitHub] mrutkows closed pull request #424: Use const value to replace hard coded manifest yaml file name.

2017-08-31 Thread git
mrutkows closed pull request #424: Use const value to replace hard coded manifest yaml file name. URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/424 This is an automated message from the Apache Git

[GitHub] rabbah closed pull request #2679: Fix compile error because of subsequent merges.

2017-08-31 Thread git
rabbah closed pull request #2679: Fix compile error because of subsequent merges. URL: https://github.com/apache/incubator-openwhisk/pull/2679 This is an automated message from the Apache Git Service. To respond to the

[GitHub] markusthoemmes commented on a change in pull request #2676: Change response when entity is empty from 200 to 204.

2017-08-31 Thread git
markusthoemmes commented on a change in pull request #2676: Change response when entity is empty from 200 to 204. URL: https://github.com/apache/incubator-openwhisk/pull/2676#discussion_r136340753 ## File path: core/controller/src/main/scala/whisk/core/controller/WebActions.scala

[GitHub] markusthoemmes opened a new pull request #2679: Fix compile error because of subsequent merges.

2017-08-31 Thread git
markusthoemmes opened a new pull request #2679: Fix compile error because of subsequent merges. URL: https://github.com/apache/incubator-openwhisk/pull/2679 We trusted git because there where no conflicts. Meh. This is an

[GitHub] rabbah commented on issue #2671: Ignore IntelliJ '.iml' project file

2017-08-31 Thread git
rabbah commented on issue #2671: Ignore IntelliJ '.iml' project file URL: https://github.com/apache/incubator-openwhisk/pull/2671#issuecomment-326298666 Thanks for the patch. Do you have an Apache CLA on file? This is an

[GitHub] rabbah closed pull request #2663: Simplify Spi implementations.

2017-08-31 Thread git
rabbah closed pull request #2663: Simplify Spi implementations. URL: https://github.com/apache/incubator-openwhisk/pull/2663 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] rabbah closed pull request #2624: Add cache invalidation between controllers

2017-08-31 Thread git
rabbah closed pull request #2624: Add cache invalidation between controllers URL: https://github.com/apache/incubator-openwhisk/pull/2624 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] mdeuser commented on issue #2591: User-Agent CLI/version header

2017-08-31 Thread git
mdeuser commented on issue #2591: User-Agent CLI/version header URL: https://github.com/apache/incubator-openwhisk/pull/2591#issuecomment-326288681 @dubeejw - good catch. there has been talk of adding semantic versioning to the various components, so perhaps this build timestamp is good

[GitHub] rabbah commented on a change in pull request #2676: Change response when entity is empty from 200 to 204.

2017-08-31 Thread git
rabbah commented on a change in pull request #2676: Change response when entity is empty from 200 to 204. URL: https://github.com/apache/incubator-openwhisk/pull/2676#discussion_r136314625 ## File path: core/controller/src/main/scala/whisk/core/controller/WebActions.scala

[GitHub] rabbah commented on a change in pull request #2676: Change response when entity is empty from 200 to 204.

2017-08-31 Thread git
rabbah commented on a change in pull request #2676: Change response when entity is empty from 200 to 204. URL: https://github.com/apache/incubator-openwhisk/pull/2676#discussion_r136314625 ## File path: core/controller/src/main/scala/whisk/core/controller/WebActions.scala

[GitHub] rabbah commented on a change in pull request #2676: Change response when entity is empty from 200 to 204.

2017-08-31 Thread git
rabbah commented on a change in pull request #2676: Change response when entity is empty from 200 to 204. URL: https://github.com/apache/incubator-openwhisk/pull/2676#discussion_r136314625 ## File path: core/controller/src/main/scala/whisk/core/controller/WebActions.scala

[GitHub] rabbah commented on a change in pull request #2676: Change response when entity is empty from 200 to 204.

2017-08-31 Thread git
rabbah commented on a change in pull request #2676: Change response when entity is empty from 200 to 204. URL: https://github.com/apache/incubator-openwhisk/pull/2676#discussion_r136314625 ## File path: core/controller/src/main/scala/whisk/core/controller/WebActions.scala

[GitHub] rabbah commented on a change in pull request #2624: Add cache invalidation between controllers

2017-08-31 Thread git
rabbah commented on a change in pull request #2624: Add cache invalidation between controllers URL: https://github.com/apache/incubator-openwhisk/pull/2624#discussion_r136307644 ## File path: common/scala/src/main/scala/whisk/core/entity/CacheKey.scala ## @@ -0,0 +1,55

[GitHub] rabbah commented on a change in pull request #2624: Add cache invalidation between controllers

2017-08-31 Thread git
rabbah commented on a change in pull request #2624: Add cache invalidation between controllers URL: https://github.com/apache/incubator-openwhisk/pull/2624#discussion_r136112686 ## File path: common/scala/src/main/scala/whisk/core/database/MultipleReadersSingleWriterCache.scala

[GitHub] rabbah commented on a change in pull request #2624: Add cache invalidation between controllers

2017-08-31 Thread git
rabbah commented on a change in pull request #2624: Add cache invalidation between controllers URL: https://github.com/apache/incubator-openwhisk/pull/2624#discussion_r136307876 ## File path: common/scala/src/main/scala/whisk/core/database/RemoteCacheInvalidation.scala

[GitHub] rabbah commented on a change in pull request #2676: Change response when entity is empty from 200 to 204.

2017-08-31 Thread git
rabbah commented on a change in pull request #2676: Change response when entity is empty from 200 to 204. URL: https://github.com/apache/incubator-openwhisk/pull/2676#discussion_r136306610 ## File path: tests/dat/actions/multipleHeaders.js ## @@ -3,6 +3,6 @@ function

[GitHub] rabbah commented on a change in pull request #2676: Change response when entity is empty from 200 to 204.

2017-08-31 Thread git
rabbah commented on a change in pull request #2676: Change response when entity is empty from 200 to 204. URL: https://github.com/apache/incubator-openwhisk/pull/2676#discussion_r136306493 ## File path: core/controller/src/main/scala/whisk/core/controller/WebActions.scala

[GitHub] markusthoemmes commented on a change in pull request #2676: Change response when entity is empty from 200 to 204.

2017-08-31 Thread git
markusthoemmes commented on a change in pull request #2676: Change response when entity is empty from 200 to 204. URL: https://github.com/apache/incubator-openwhisk/pull/2676#discussion_r136289207 ## File path: core/controller/src/main/scala/whisk/core/controller/WebActions.scala

[GitHub] markusthoemmes commented on a change in pull request #2676: Change response when entity is empty from 200 to 204.

2017-08-31 Thread git
markusthoemmes commented on a change in pull request #2676: Change response when entity is empty from 200 to 204. URL: https://github.com/apache/incubator-openwhisk/pull/2676#discussion_r136289207 ## File path: core/controller/src/main/scala/whisk/core/controller/WebActions.scala

[GitHub] markusthoemmes commented on a change in pull request #2676: Change response when entity is empty from 200 to 204.

2017-08-31 Thread git
markusthoemmes commented on a change in pull request #2676: Change response when entity is empty from 200 to 204. URL: https://github.com/apache/incubator-openwhisk/pull/2676#discussion_r136289207 ## File path: core/controller/src/main/scala/whisk/core/controller/WebActions.scala

[GitHub] markusthoemmes commented on a change in pull request #2676: Change response when entity is empty from 200 to 204.

2017-08-31 Thread git
markusthoemmes commented on a change in pull request #2676: Change response when entity is empty from 200 to 204. URL: https://github.com/apache/incubator-openwhisk/pull/2676#discussion_r136288101 ## File path: core/controller/src/main/scala/whisk/core/controller/WebActions.scala

[GitHub] lzbj commented on issue #262: Adjust the code to reflect latest API gateway version.

2017-08-31 Thread git
lzbj commented on issue #262: Adjust the code to reflect latest API gateway version. URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/262#issuecomment-326241021 let's discuss tomorrow's meeting, thanks.

[GitHub] lzbj commented on issue #262: Adjust the code to reflect latest API gateway version.

2017-08-31 Thread git
lzbj commented on issue #262: Adjust the code to reflect latest API gateway version. URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/262#issuecomment-326240891 @mrutkows , please review the latest api gateway yaml, I just want confirm, with this api definition, we're

[GitHub] lzbj commented on issue #425: Run some lint command before submit PR.

2017-08-31 Thread git
lzbj commented on issue #425: Run some lint command before submit PR. URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/425#issuecomment-326239950 let's close this one after our tomorrow's meeting. This is

[GitHub] markusthoemmes commented on issue #2678: How to upgrade invoker gracefully

2017-08-31 Thread git
markusthoemmes commented on issue #2678: How to upgrade invoker gracefully URL: https://github.com/apache/incubator-openwhisk/issues/2678#issuecomment-326234383 The invoker does already have a shutdown hook which is supposed to clean up. That hook needs to wait until all the user load is

[GitHub] ningyougang opened a new issue #2678: How to upgrade invoker gracefully

2017-08-31 Thread git
ningyougang opened a new issue #2678: How to upgrade invoker gracefully URL: https://github.com/apache/incubator-openwhisk/issues/2678 If i want to upgrade the `invoker` component using new image. When use `-e mode=clean` to remove invoker, the relative action containers will be

[GitHub] markusthoemmes commented on a change in pull request #2677: Add blocking indicator for activation request.

2017-08-31 Thread git
markusthoemmes commented on a change in pull request #2677: Add blocking indicator for activation request. URL: https://github.com/apache/incubator-openwhisk/pull/2677#discussion_r136259460 ## File path: common/scala/src/main/scala/whisk/core/entity/WhiskActivation.scala

[GitHub] akrabat commented on a change in pull request #2650: Apply standard scala formatting.

2017-08-31 Thread git
akrabat commented on a change in pull request #2650: Apply standard scala formatting. URL: https://github.com/apache/incubator-openwhisk/pull/2650#discussion_r136255353 ## File path: common/scala/src/main/scala/whisk/common/Config.scala ## @@ -42,113 +42,116 @@ import