[GitHub] dubeejw commented on issue #2898: Use proper host for Wsk REST tests

2017-10-25 Thread GitBox
dubeejw commented on issue #2898: Use proper host for Wsk REST tests URL: https://github.com/apache/incubator-openwhisk/pull/2898#issuecomment-339514680 PG3 1256 ? This is an automated message from the Apache Git Service.

[GitHub] houshengbo closed pull request #2898: Use proper host for Wsk REST tests

2017-10-25 Thread GitBox
houshengbo closed pull request #2898: Use proper host for Wsk REST tests URL: https://github.com/apache/incubator-openwhisk/pull/2898 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

[GitHub] rabbah closed issue #2901: vagrant error in cryptography setup command

2017-10-25 Thread GitBox
rabbah closed issue #2901: vagrant error in cryptography setup command URL: https://github.com/apache/incubator-openwhisk/issues/2901 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] rabbah closed pull request #2903: fix ansible install errors in vagrant

2017-10-25 Thread GitBox
rabbah closed pull request #2903: fix ansible install errors in vagrant URL: https://github.com/apache/incubator-openwhisk/pull/2903 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

[GitHub] pritidesai commented on issue #609: Annotations on Actions only work if "web export" exists

2017-10-25 Thread GitBox
pritidesai commented on issue #609: Annotations on Actions only work if "web export" exists URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/609#issuecomment-339517654 This WIP, priti is working on it ...

[GitHub] pritidesai opened a new pull request #628: WIP: Adding action annotations while creating that action

2017-10-25 Thread GitBox
pritidesai opened a new pull request #628: WIP: Adding action annotations while creating that action URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/628 Closes #609 This is an automated message from the

[GitHub] dubeejw commented on issue #2898: Use proper host for Wsk REST tests

2017-10-25 Thread GitBox
dubeejw commented on issue #2898: Use proper host for Wsk REST tests URL: https://github.com/apache/incubator-openwhisk/pull/2898#issuecomment-339514680 PG3 1256 ? This is an automated message from the Apache Git Service.

[GitHub] dubeejw commented on issue #2898: Use proper host for Wsk REST tests

2017-10-25 Thread GitBox
dubeejw commented on issue #2898: Use proper host for Wsk REST tests URL: https://github.com/apache/incubator-openwhisk/pull/2898#issuecomment-339514680 PG3 1255 ? This is an automated message from the Apache Git Service.

[GitHub] dubeejw commented on issue #2898: Use proper host for Wsk REST tests

2017-10-25 Thread GitBox
dubeejw commented on issue #2898: Use proper host for Wsk REST tests URL: https://github.com/apache/incubator-openwhisk/pull/2898#issuecomment-339514408 PG3 1254 ? This is an automated message from the Apache Git Service.

[GitHub] tysonnorris commented on issue #2902: make INVOKER_NAME optional

2017-10-25 Thread GitBox
tysonnorris commented on issue #2902: make INVOKER_NAME optional URL: https://github.com/apache/incubator-openwhisk/pull/2902#issuecomment-339509568 LGTM This is an automated message from the Apache Git Service. To respond

[GitHub] ddragosd commented on a change in pull request #2902: make INVOKER_NAME optional

2017-10-25 Thread GitBox
ddragosd commented on a change in pull request #2902: make INVOKER_NAME optional URL: https://github.com/apache/incubator-openwhisk/pull/2902#discussion_r147017958 ## File path: core/invoker/src/main/scala/whisk/core/invoker/Invoker.scala ## @@ -60,7 +60,7 @@ object

[GitHub] tysonnorris commented on a change in pull request #2902: make INVOKER_NAME optional

2017-10-25 Thread GitBox
tysonnorris commented on a change in pull request #2902: make INVOKER_NAME optional URL: https://github.com/apache/incubator-openwhisk/pull/2902#discussion_r147017228 ## File path: core/invoker/src/main/scala/whisk/core/invoker/Invoker.scala ## @@ -60,7 +60,7 @@ object

[GitHub] dubeejw closed issue #2899: Pick up nodejs runtime with new openwhisk client-js 3.9

2017-10-25 Thread GitBox
dubeejw closed issue #2899: Pick up nodejs runtime with new openwhisk client-js 3.9 URL: https://github.com/apache/incubator-openwhisk/issues/2899 This is an automated message from the Apache Git Service. To respond to the

[GitHub] dubeejw closed pull request #2900: update nodejs runtime to pickup openwhisk client-js 3.9.0

2017-10-25 Thread GitBox
dubeejw closed pull request #2900: update nodejs runtime to pickup openwhisk client-js 3.9.0 URL: https://github.com/apache/incubator-openwhisk/pull/2900 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 opened a new pull request #2903: fix ansible install errors in vagrant

2017-10-25 Thread GitBox
csantanapr opened a new pull request #2903: fix ansible install errors in vagrant URL: https://github.com/apache/incubator-openwhisk/pull/2903 Fixes #2901 Related to #2891 Fix to ansible error with crypto is `sudo pip install --upgrade setuptools` We have mismatch

[GitHub] ddragosd commented on a change in pull request #62: added redis host for invoker

2017-10-25 Thread GitBox
ddragosd commented on a change in pull request #62: added redis host for invoker URL: https://github.com/apache/incubator-openwhisk-devtools/pull/62#discussion_r147012475 ## File path: docker-compose/docker-compose.yml ## @@ -81,12 +82,15 @@ services: -

[GitHub] ddragosd opened a new pull request #2902: make INVOKER_NAME optional

2017-10-25 Thread GitBox
ddragosd opened a new pull request #2902: make INVOKER_NAME optional URL: https://github.com/apache/incubator-openwhisk/pull/2902 This slipped in with the Redis changes and I don't think it's needed b/c the invoker already starts with an `id`. Relates to: #2872 and

[GitHub] csantanapr opened a new issue #2901: vagrant errors

2017-10-25 Thread GitBox
csantanapr opened a new issue #2901: vagrant errors URL: https://github.com/apache/incubator-openwhisk/issues/2901 ## Environment details: * vagrant * MacOS ## Steps to reproduce the issue: 1. Follow steps in README

[GitHub] dubeejw commented on issue #2898: Use proper host for Wsk REST tests

2017-10-25 Thread GitBox
dubeejw commented on issue #2898: Use proper host for Wsk REST tests URL: https://github.com/apache/incubator-openwhisk/pull/2898#issuecomment-339479631 PG1 2191 ? This is an automated message from the Apache Git Service.

[GitHub] csantanapr commented on issue #2900: update nodejs runtime to pickup openwhisk client-js 3.9.0

2017-10-25 Thread GitBox
csantanapr commented on issue #2900: update nodejs runtime to pickup openwhisk client-js 3.9.0 URL: https://github.com/apache/incubator-openwhisk/pull/2900#issuecomment-339493695 ? PG2 2216 This is an automated message from

[GitHub] dgrove-oss commented on a change in pull request #62: added redis host for invoker

2017-10-25 Thread GitBox
dgrove-oss commented on a change in pull request #62: added redis host for invoker URL: https://github.com/apache/incubator-openwhisk-devtools/pull/62#discussion_r147004387 ## File path: docker-compose/docker-compose.yml ## @@ -81,12 +82,15 @@ services: -

[GitHub] csantanapr commented on a change in pull request #2891: Vagrant base image update to latest Ubuntu LTS

2017-10-25 Thread GitBox
csantanapr commented on a change in pull request #2891: Vagrant base image update to latest Ubuntu LTS URL: https://github.com/apache/incubator-openwhisk/pull/2891#discussion_r147003096 ## File path: tools/ubuntu-setup/scala.sh ## @@ -2,7 +2,8 @@ set -e set -x -wget

[GitHub] dubeejw commented on issue #2898: Use proper host for Wsk REST tests

2017-10-25 Thread GitBox
dubeejw commented on issue #2898: Use proper host for Wsk REST tests URL: https://github.com/apache/incubator-openwhisk/pull/2898#issuecomment-339479631 PG1 2191 ? This is an automated message from the Apache Git Service.

[GitHub] tysonnorris commented on a change in pull request #62: added redis host for invoker

2017-10-25 Thread GitBox
tysonnorris commented on a change in pull request #62: added redis host for invoker URL: https://github.com/apache/incubator-openwhisk-devtools/pull/62#discussion_r146999639 ## File path: docker-compose/docker-compose.yml ## @@ -81,12 +82,15 @@ services: -

[GitHub] houshengbo commented on issue #2898: Use proper host for Wsk REST tests

2017-10-25 Thread GitBox
houshengbo commented on issue #2898: Use proper host for Wsk REST tests URL: https://github.com/apache/incubator-openwhisk/pull/2898#issuecomment-339484160 Let's wait for a green build of travis. This is an automated message

[GitHub] houshengbo commented on issue #2898: Use proper host for Wsk REST tests

2017-10-25 Thread GitBox
houshengbo commented on issue #2898: Use proper host for Wsk REST tests URL: https://github.com/apache/incubator-openwhisk/pull/2898#issuecomment-339484160 Let's wait for a green build of travis. @dubeejw This is an

[GitHub] ddragosd commented on a change in pull request #62: added redis host for invoker

2017-10-25 Thread GitBox
ddragosd commented on a change in pull request #62: added redis host for invoker URL: https://github.com/apache/incubator-openwhisk-devtools/pull/62#discussion_r146998430 ## File path: docker-compose/docker-compose.yml ## @@ -81,12 +82,15 @@ services: -

[GitHub] ddragosd commented on a change in pull request #62: added redis host for invoker

2017-10-25 Thread GitBox
ddragosd commented on a change in pull request #62: added redis host for invoker URL: https://github.com/apache/incubator-openwhisk-devtools/pull/62#discussion_r146998430 ## File path: docker-compose/docker-compose.yml ## @@ -81,12 +82,15 @@ services: -

[GitHub] dubeejw commented on issue #2898: Use proper host for Wsk REST tests

2017-10-25 Thread GitBox
dubeejw commented on issue #2898: Use proper host for Wsk REST tests URL: https://github.com/apache/incubator-openwhisk/pull/2898#issuecomment-339480952 @houshengbo, can you review? This is an automated message from the

[GitHub] csantanapr commented on a change in pull request #2891: Vagrant base image update to latest Ubuntu LTS

2017-10-25 Thread GitBox
csantanapr commented on a change in pull request #2891: Vagrant base image update to latest Ubuntu LTS URL: https://github.com/apache/incubator-openwhisk/pull/2891#discussion_r146995630 ## File path: tools/ubuntu-setup/docker.sh ## @@ -3,28 +3,32 @@ set -e set -x

[GitHub] csantanapr commented on a change in pull request #2891: Vagrant base image update to latest Ubuntu LTS

2017-10-25 Thread GitBox
csantanapr commented on a change in pull request #2891: Vagrant base image update to latest Ubuntu LTS URL: https://github.com/apache/incubator-openwhisk/pull/2891#discussion_r146995415 ## File path: tools/ubuntu-setup/docker.sh ## @@ -3,28 +3,32 @@ set -e set -x

[GitHub] dubeejw commented on issue #2898: Use proper host for rest tests

2017-10-25 Thread GitBox
dubeejw commented on issue #2898: Use proper host for rest tests URL: https://github.com/apache/incubator-openwhisk/pull/2898#issuecomment-339479631 PG1 2190 ? This is an automated message from the Apache Git Service. To

[GitHub] pritidesai commented on issue #609: Annotations on Actions only work if "web export" exists

2017-10-25 Thread GitBox
pritidesai commented on issue #609: Annotations on Actions only work if "web export" exists URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/609#issuecomment-339479376 I am running some sample tests to determine `wskdeploy` behavior for annotations on actions and found

[GitHub] csantanapr commented on issue #2891: Vagrant base image update to latest Ubuntu LTS

2017-10-25 Thread GitBox
csantanapr commented on issue #2891: Vagrant base image update to latest Ubuntu LTS URL: https://github.com/apache/incubator-openwhisk/pull/2891#issuecomment-339478102 Can we make a new set of instructions for Xenial, maybe under `tools/setup/ubuntu-setup/xenial/*` ? We currently

[GitHub] abaruni commented on issue #2893: WIP Support retrieving status and configuration of feed triggers in CLI

2017-10-25 Thread GitBox
abaruni commented on issue #2893: WIP Support retrieving status and configuration of feed triggers in CLI URL: https://github.com/apache/incubator-openwhisk/pull/2893#issuecomment-339467766 ? PG3 1252 This is an automated

[GitHub] spattewar commented on issue #2894: What is the purpose of each component in openwhisk??

2017-10-25 Thread GitBox
spattewar commented on issue #2894: What is the purpose of each component in openwhisk?? URL: https://github.com/apache/incubator-openwhisk/issues/2894#issuecomment-339460597 Hi @markusthoemmes thank you for your response and the information document. It is nice. But it does

[GitHub] csantanapr commented on issue #101: Trigger management

2017-10-25 Thread GitBox
csantanapr commented on issue #101: Trigger management URL: https://github.com/apache/incubator-openwhisk-package-alarms/pull/101#issuecomment-339454967 Need to handle how to deal when an old trigger has a different format for dateChange,

[GitHub] csantanapr commented on a change in pull request #101: Trigger management

2017-10-25 Thread GitBox
csantanapr commented on a change in pull request #101: Trigger management URL: https://github.com/apache/incubator-openwhisk-package-alarms/pull/101#discussion_r146971392 ## File path: action/alarmWebAction.js ## @@ -72,6 +72,34 @@ function main(params) { });

[GitHub] mdeuser commented on issue #2893: WIP Support retrieving status and configuration of feed triggers in CLI

2017-10-25 Thread GitBox
mdeuser commented on issue #2893: WIP Support retrieving status and configuration of feed triggers in CLI URL: https://github.com/apache/incubator-openwhisk/pull/2893#issuecomment-339453979 @abaruni - when ready for review, please cc: @jessealva @drcariel @dubeejw and me. thanks!

[GitHub] jonpspri closed pull request #243: Adding provisions for building s390x (Linux on Z) images of openwhisk-apigateway

2017-10-25 Thread GitBox
jonpspri closed pull request #243: Adding provisions for building s390x (Linux on Z) images of openwhisk-apigateway URL: https://github.com/apache/incubator-openwhisk-apigateway/pull/243 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is

[GitHub] tysonnorris commented on issue #62: added redis host for invoker

2017-10-25 Thread GitBox
tysonnorris commented on issue #62: added redis host for invoker URL: https://github.com/apache/incubator-openwhisk-devtools/pull/62#issuecomment-339425901 Is this PR required at all now? This is an automated message from

[GitHub] tysonnorris commented on a change in pull request #62: added redis host for invoker

2017-10-25 Thread GitBox
tysonnorris commented on a change in pull request #62: added redis host for invoker URL: https://github.com/apache/incubator-openwhisk-devtools/pull/62#discussion_r146945859 ## File path: docker-compose/docker-compose.yml ## @@ -81,12 +82,15 @@ services: -

[GitHub] pritidesai opened a new pull request #627: Adding support for project

2017-10-25 Thread GitBox
pritidesai opened a new pull request #627: Adding support for project URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/627 Closes #617 Sorry for a huge PR, didn't realize the number of files changed until I created this PR :(

[GitHub] csantanapr opened a new pull request #2900: update nodejs runtime to pickup openwhisk client-js 3.9.0

2017-10-25 Thread GitBox
csantanapr opened a new pull request #2900: update nodejs runtime to pickup openwhisk client-js 3.9.0 URL: https://github.com/apache/incubator-openwhisk/pull/2900 This is an automated message from the Apache Git Service. To

[GitHub] csantanapr opened a new issue #2899: Pick up nodejs runtime with new openwhisk client-js 3.9

2017-10-25 Thread GitBox
csantanapr opened a new issue #2899: Pick up nodejs runtime with new openwhisk client-js 3.9 URL: https://github.com/apache/incubator-openwhisk/issues/2899 New image contains openwhisk client-js 3.9 This is an automated

[GitHub] dubeejw opened a new pull request #2898: Use proper host for rest tests

2017-10-25 Thread GitBox
dubeejw opened a new pull request #2898: Use proper host for rest tests URL: https://github.com/apache/incubator-openwhisk/pull/2898 `WhiskProperties.getEdgeHost` should be used instead of `WhiskProperties.getBaseControllerAddress` for WskRest.

[GitHub] csantanapr commented on issue #2897: Deprecate swift:3.0 kind, direct users to use swift:3.1.1 kind instead

2017-10-25 Thread GitBox
csantanapr commented on issue #2897: Deprecate swift:3.0 kind, direct users to use swift:3.1.1 kind instead URL: https://github.com/apache/incubator-openwhisk/issues/2897#issuecomment-339399226 >After a period (maybe 4 weeks) update manifest in invoker to mark it deprecated, meaning user

[GitHub] csantanapr commented on issue #2897: Deprecate swift:3.0 kind, direct users to use swift:3.1.1 kind instead

2017-10-25 Thread GitBox
csantanapr commented on issue #2897: Deprecate swift:3.0 kind, direct users to use swift:3.1.1 kind instead URL: https://github.com/apache/incubator-openwhisk/issues/2897#issuecomment-339398911 @rabbah >Since deprecation is a deployment based toggle should master just deprecate

[GitHub] csantanapr commented on issue #2897: Deprecate swift:3.0 kind, direct users to use swift:3.1.1 kind instead

2017-10-25 Thread GitBox
csantanapr commented on issue #2897: Deprecate swift:3.0 kind, direct users to use swift:3.1.1 kind instead URL: https://github.com/apache/incubator-openwhisk/issues/2897#issuecomment-339398911 @rabbah >Since deprecation is a deployment based toggle should master just deprecate

[GitHub] rabbah commented on issue #2897: Deprecate swift:3.0 kind, direct users to use swift:3.1.1 kind instead

2017-10-25 Thread GitBox
rabbah commented on issue #2897: Deprecate swift:3.0 kind, direct users to use swift:3.1.1 kind instead URL: https://github.com/apache/incubator-openwhisk/issues/2897#issuecomment-339397050 Since deprecation is a deployment based toggle should master just deprecate immediately?

[GitHub] csantanapr opened a new issue #2897: Deprecate swift:3.0 kind, direct users to use swift:3.1.1 kind instead

2017-10-25 Thread GitBox
csantanapr opened a new issue #2897: Deprecate swift:3.0 kind, direct users to use swift:3.1.1 kind instead URL: https://github.com/apache/incubator-openwhisk/issues/2897 I'm proposing that we deprecate swift:3.0 and get everyone using swift:3.1.1 kind The default kind for swift is

[GitHub] jeremiaswerner closed pull request #2870: Remove broken containers if docker run fails

2017-10-25 Thread GitBox
jeremiaswerner closed pull request #2870: Remove broken containers if docker run fails URL: https://github.com/apache/incubator-openwhisk/pull/2870 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 #2: Switch to version 3.9.0 of the openwhisk SDK

2017-10-25 Thread GitBox
csantanapr commented on issue #2: Switch to version 3.9.0 of the openwhisk SDK URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/2#issuecomment-339324509 @gbdrt Don't know what's going on with Travis and docker build. I restarted the build and it worked.

[GitHub] abaruni opened a new pull request #80: WIP Support retrieving status and configuration of feed triggers

2017-10-25 Thread GitBox
abaruni opened a new pull request #80: WIP Support retrieving status and configuration of feed triggers URL: https://github.com/apache/incubator-openwhisk-client-js/pull/80 This PR is to add support for retrieving the status and configuration of a feed trigger Support has been

[GitHub] cfjedimaster opened a new issue #2896: Provide a hint that more data exists when fetching resources

2017-10-25 Thread GitBox
cfjedimaster opened a new issue #2896: Provide a hint that more data exists when fetching resources URL: https://github.com/apache/incubator-openwhisk/issues/2896 Currently when you `wsk action list` or trigger, rule, package, etc, you are not given any clue that more data exists. Yes the

[GitHub] xingzhou commented on issue #45: Install Route Management from the script

2017-10-25 Thread GitBox
xingzhou commented on issue #45: Install Route Management from the script URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/45#issuecomment-339257838 So user shall manually install this package after deployment is finished? or if there is any check to validate whether

[GitHub] markusthoemmes commented on a change in pull request #2895: Update the test cases about rules with REST implementation

2017-10-25 Thread GitBox
markusthoemmes commented on a change in pull request #2895: Update the test cases about rules with REST implementation URL: https://github.com/apache/incubator-openwhisk/pull/2895#discussion_r146778352 ## File path: tests/src/test/scala/system/basic/WskRestRuleTests.scala

[GitHub] markusthoemmes commented on issue #2894: What is the purpose of each component in openwhisk??

2017-10-25 Thread GitBox
markusthoemmes commented on issue #2894: What is the purpose of each component in openwhisk?? URL: https://github.com/apache/incubator-openwhisk/issues/2894#issuecomment-339236268 Hi, for a starter you might want to read [my article on the general

[GitHub] akrabat commented on issue #2891: Vagrant base image update to latest Ubuntu LTS

2017-10-25 Thread GitBox
akrabat commented on issue #2891: Vagrant base image update to latest Ubuntu LTS URL: https://github.com/apache/incubator-openwhisk/pull/2891#issuecomment-339224265 As @bbrowning has noted, the current Vagrant box does not successfully provision. I've tested this PR and it works.