[GitHub] dubee commented on a change in pull request #3421: Add ElasticSearch Log Store

2018-03-27 Thread GitBox
dubee commented on a change in pull request #3421: Add ElasticSearch Log Store URL: https://github.com/apache/incubator-openwhisk/pull/3421#discussion_r177653596 ## File path: tests/src/test/scala/whisk/core/containerpool/logging/ElasticSearchRestClientTests.scala ## @@ -

[GitHub] dubee commented on a change in pull request #3421: Add ElasticSearch Log Store

2018-03-27 Thread GitBox
dubee commented on a change in pull request #3421: Add ElasticSearch Log Store URL: https://github.com/apache/incubator-openwhisk/pull/3421#discussion_r177653610 ## File path: tests/src/test/scala/whisk/core/containerpool/logging/ElasticSearchRestClientTests.scala ## @@ -

[GitHub] dubee commented on a change in pull request #3421: Add ElasticSearch Log Store

2018-03-27 Thread GitBox
dubee commented on a change in pull request #3421: Add ElasticSearch Log Store URL: https://github.com/apache/incubator-openwhisk/pull/3421#discussion_r177653586 ## File path: tests/src/test/scala/whisk/core/containerpool/logging/ElasticSearchRestClientTests.scala ## @@ -

[GitHub] dubee commented on a change in pull request #3421: Add ElasticSearch Log Store

2018-03-27 Thread GitBox
dubee commented on a change in pull request #3421: Add ElasticSearch Log Store URL: https://github.com/apache/incubator-openwhisk/pull/3421#discussion_r177653148 ## File path: common/scala/src/main/scala/whisk/core/containerpool/logging/ElasticSearchLogStore.scala ## @@ -

[GitHub] dubee commented on a change in pull request #3421: Add ElasticSearch Log Store

2018-03-27 Thread GitBox
dubee commented on a change in pull request #3421: Add ElasticSearch Log Store URL: https://github.com/apache/incubator-openwhisk/pull/3421#discussion_r177652864 ## File path: common/scala/src/main/scala/whisk/core/containerpool/logging/ElasticSearchLogStore.scala ## @@ -

[GitHub] dubee commented on a change in pull request #3421: Add ElasticSearch Log Store

2018-03-27 Thread GitBox
dubee commented on a change in pull request #3421: Add ElasticSearch Log Store URL: https://github.com/apache/incubator-openwhisk/pull/3421#discussion_r177645635 ## File path: common/scala/src/main/scala/whisk/core/containerpool/logging/ElasticSearchRestClient.scala ## @@

[GitHub] dubee commented on a change in pull request #3421: Add ElasticSearch Log Store

2018-03-27 Thread GitBox
dubee commented on a change in pull request #3421: Add ElasticSearch Log Store URL: https://github.com/apache/incubator-openwhisk/pull/3421#discussion_r177645635 ## File path: common/scala/src/main/scala/whisk/core/containerpool/logging/ElasticSearchRestClient.scala ## @@

[GitHub] dubee commented on a change in pull request #3421: Add ElasticSearch Log Store

2018-03-27 Thread GitBox
dubee commented on a change in pull request #3421: Add ElasticSearch Log Store URL: https://github.com/apache/incubator-openwhisk/pull/3421#discussion_r177645635 ## File path: common/scala/src/main/scala/whisk/core/containerpool/logging/ElasticSearchRestClient.scala ## @@

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

2018-03-27 Thread GitBox
dubee commented on issue #3421: Add ElasticSearch Log Store URL: https://github.com/apache/incubator-openwhisk/pull/3421#issuecomment-376767073 @markusthoemmes, I think I addressed all of your comments. Can you take another look? ---

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

2018-03-27 Thread GitBox
dubee commented on issue #3421: Add ElasticSearch Log Store URL: https://github.com/apache/incubator-openwhisk/pull/3421#issuecomment-376767073 @markusthoemmes, I think I address all of your comments. Can you take another look? -

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

2018-03-27 Thread GitBox
dubee commented on issue #3421: Add ElasticSearch Log Store URL: https://github.com/apache/incubator-openwhisk/pull/3421#issuecomment-376727282 PG4 1533 šŸ”µ This is an automated message from the Apache Git Service. To respond

[GitHub] dubee commented on issue #3416: Unit test PoolingRestClient

2018-03-27 Thread GitBox
dubee commented on issue #3416: Unit test PoolingRestClient URL: https://github.com/apache/incubator-openwhisk/pull/3416#issuecomment-376756180 @markusthoemmes, thanks for the comments. Simplified the testFlow code. Will address other comments later. --

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

2018-03-27 Thread GitBox
dubee commented on issue #3421: Add ElasticSearch Log Store URL: https://github.com/apache/incubator-openwhisk/pull/3421#issuecomment-376727282 PG3 1533 šŸ”µ This is an automated message from the Apache Git Service. To respond

[GitHub] tysonnorris opened a new pull request #3497: allow singleton (active/passive) and replicated pools in invoker

2018-03-27 Thread GitBox
tysonnorris opened a new pull request #3497: allow singleton (active/passive) and replicated pools in invoker URL: https://github.com/apache/incubator-openwhisk/pull/3497 To support a cluster-aware deployment, the ContainerPool in invoker needs to a) be able to operate as a singleton w

[GitHub] csantanapr closed pull request #31: update npm openwhisk to 3.14.0

2018-03-27 Thread GitBox
csantanapr closed pull request #31: update npm openwhisk to 3.14.0 URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/31 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

[GitHub] jonpspri closed pull request #30: Correct package for tests to 'runtime.ActionContainers'

2018-03-27 Thread GitBox
jonpspri closed pull request #30: Correct package for tests to 'runtime.ActionContainers' URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/30 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake

[GitHub] jonpspri commented on issue #30: Correct package for tests to 'runtime.ActionContainers'

2018-03-27 Thread GitBox
jonpspri commented on issue #30: Correct package for tests to 'runtime.ActionContainers' URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/30#issuecomment-376741290 Closing as the same change is now included in #31

[GitHub] tysonnorris commented on issue #3211: implement ContainerFactory.cpuShare to fix #3110

2018-03-27 Thread GitBox
tysonnorris commented on issue #3211: implement ContainerFactory.cpuShare to fix #3110 URL: https://github.com/apache/incubator-openwhisk/pull/3211#issuecomment-376739735 @dubee @markusthoemmes any issues left on this? This

[GitHub] tysonnorris commented on a change in pull request #3211: implement ContainerFactory.cpuShare to fix #3110

2018-03-27 Thread GitBox
tysonnorris commented on a change in pull request #3211: implement ContainerFactory.cpuShare to fix #3110 URL: https://github.com/apache/incubator-openwhisk/pull/3211#discussion_r177624400 ## File path: tests/src/test/scala/whisk/core/containerpool/test/ContainerArgsConfigTest.sca

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

2018-03-27 Thread GitBox
dubee commented on issue #3421: Add ElasticSearch Log Store URL: https://github.com/apache/incubator-openwhisk/pull/3421#issuecomment-376727282 PG3 1532 šŸ”µ This is an automated message from the Apache Git Service. To respond

[GitHub] ningyougang closed pull request #3432: Introduce jmx for kafka

2018-03-27 Thread GitBox
ningyougang closed pull request #3432: Introduce jmx for kafka URL: https://github.com/apache/incubator-openwhisk/pull/3432 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 foreign pull

[GitHub] ningyougang commented on issue #3432: Introduce jmx for kafka

2018-03-27 Thread GitBox
ningyougang commented on issue #3432: Introduce jmx for kafka URL: https://github.com/apache/incubator-openwhisk/pull/3432#issuecomment-37672 Closed This is an automated message from the Apache Git Service. To respond to

[GitHub] jonpspri opened a new pull request #13: Multi-architecture build for Python

2018-03-27 Thread GitBox
jonpspri opened a new pull request #13: Multi-architecture build for Python URL: https://github.com/apache/incubator-openwhisk-runtime-python/pull/13 Creates a multi-architecture python image. Also: - Unifies the Dockerfile and *.py files for the image to allow for better synchron

[GitHub] pritidesai opened a new pull request #823: Adding documentation on Sync

2018-03-27 Thread GitBox
pritidesai opened a new pull request #823: Adding documentation on Sync URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/823 This is an automated message from the Apache Git Service. To respond to the message

[GitHub] dubee commented on a change in pull request #5: The go proxy

2018-03-27 Thread GitBox
dubee commented on a change in pull request #5: The go proxy URL: https://github.com/apache/incubator-openwhisk-runtime-go/pull/5#discussion_r177605214 ## File path: openwhisk/runHandler.go ## @@ -0,0 +1,125 @@ +/* + * Licensed to the Apache Software Foundation (ASF) unde

[GitHub] dubee commented on a change in pull request #5: The go proxy

2018-03-27 Thread GitBox
dubee commented on a change in pull request #5: The go proxy URL: https://github.com/apache/incubator-openwhisk-runtime-go/pull/5#discussion_r177603881 ## File path: openwhisk/executor_test.go ## @@ -0,0 +1,121 @@ +/* + * Licensed to the Apache Software Foundation (ASF) u

[GitHub] dubee commented on a change in pull request #5: The go proxy

2018-03-27 Thread GitBox
dubee commented on a change in pull request #5: The go proxy URL: https://github.com/apache/incubator-openwhisk-runtime-go/pull/5#discussion_r177603336 ## File path: openwhisk/actionProxy_test.go ## @@ -0,0 +1,44 @@ +package openwhisk + +import ( + "os" + "tes

[GitHub] dubee commented on a change in pull request #5: The go proxy

2018-03-27 Thread GitBox
dubee commented on a change in pull request #5: The go proxy URL: https://github.com/apache/incubator-openwhisk-runtime-go/pull/5#discussion_r177603962 ## File path: openwhisk/executor_test.go ## @@ -0,0 +1,121 @@ +/* + * Licensed to the Apache Software Foundation (ASF) u

[GitHub] pritidesai commented on a change in pull request #74: Update the document to explain the key "publish_stage" in config.json

2018-03-27 Thread GitBox
pritidesai commented on a change in pull request #74: Update the document to explain the key "publish_stage" in config.json URL: https://github.com/apache/incubator-openwhisk-release/pull/74#discussion_r177595164 ## File path: docs/pick_up_source_code.md ## @@ -56,6 +57,9

[GitHub] houshengbo opened a new pull request #74: Update the document to explain the key "publish_stage" in config.json

2018-03-27 Thread GitBox
houshengbo opened a new pull request #74: Update the document to explain the key "publish_stage" in config.json URL: https://github.com/apache/incubator-openwhisk-release/pull/74 This is an automated message from the Apache

[GitHub] rabbah closed pull request #256: add webhook for slack notification of PR test results

2018-03-27 Thread GitBox
rabbah closed pull request #256: add webhook for slack notification of PR test results URL: https://github.com/apache/incubator-openwhisk-cli/pull/256 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] dgrove-oss commented on issue #256: add webhook for slack notification of PR test results

2018-03-27 Thread GitBox
dgrove-oss commented on issue #256: add webhook for slack notification of PR test results URL: https://github.com/apache/incubator-openwhisk-cli/pull/256#issuecomment-376684793 hook verified. ![screen shot 2018-03-27 at 5 39 08 pm](https://user-images.githubusercontent.com/5099100/3799

[GitHub] houshengbo commented on issue #18: Step 3: Deploy OpenWhisk based on the source code

2018-03-27 Thread GitBox
houshengbo commented on issue #18: Step 3: Deploy OpenWhisk based on the source code URL: https://github.com/apache/incubator-openwhisk-release/issues/18#issuecomment-376681740 So far, we decided to tentatively rely on the Travis build of each openwhisk repository to make sure openwhisk i

[GitHub] houshengbo commented on issue #19: Step 4. Test each OpenWhisk component

2018-03-27 Thread GitBox
houshengbo commented on issue #19: Step 4. Test each OpenWhisk component URL: https://github.com/apache/incubator-openwhisk-release/issues/19#issuecomment-376681521 So far, we decided to tentatively rely on the Travis build of each openwhisk repository to make sure all the tests have passe

[GitHub] mrutkows closed pull request #813: Provides support to export projects with dependencies

2018-03-27 Thread GitBox
mrutkows closed pull request #813: Provides support to export projects with dependencies URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/813 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 pr

[GitHub] mrutkows commented on issue #813: Provides support to export projects with dependencies

2018-03-27 Thread GitBox
mrutkows commented on issue #813: Provides support to export projects with dependencies URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/813#issuecomment-376681052 Closing in favor of agreed-upon impl. as merged in: https://github.com/apache/incubator-openwhisk-wskdeploy/

[GitHub] mrutkows closed pull request #817: add notification webhook for slack notification of PR results.

2018-03-27 Thread GitBox
mrutkows closed pull request #817: add notification webhook for slack notification of PR results. URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/817 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the s

[GitHub] rabbah closed pull request #23: update URL in slack webhook

2018-03-27 Thread GitBox
rabbah closed pull request #23: update URL in slack webhook URL: https://github.com/apache/incubator-openwhisk-runtime-docker/pull/23 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 fo

[GitHub] dgrove-oss commented on issue #23: update URL in slack webhook

2018-03-27 Thread GitBox
dgrove-oss commented on issue #23: update URL in slack webhook URL: https://github.com/apache/incubator-openwhisk-runtime-docker/pull/23#issuecomment-376677473 Hook verified. ![screen shot 2018-03-27 at 5 12 59 pm](https://user-images.githubusercontent.com/5099100/37995362-2b4563ce-31e

[GitHub] jonpspri opened a new pull request #24: Multiarch Docker runtime

2018-03-27 Thread GitBox
jonpspri opened a new pull request #24: Multiarch Docker runtime URL: https://github.com/apache/incubator-openwhisk-runtime-docker/pull/24 Multiarch build of Docker runtime. Bump to the Python version used and to use Alpine 3.7 image (python defaults to Alpine 3.4, which has no multiarch

[GitHub] mrutkows closed pull request #816: Multiple Project Management, adding sync command, and undeploy just using Project Name

2018-03-27 Thread GitBox
mrutkows closed pull request #816: Multiple Project Management, adding sync command, and undeploy just using Project Name URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/816 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is

[GitHub] mrutkows closed issue #730: Multiple projects management

2018-03-27 Thread GitBox
mrutkows closed issue #730: Multiple projects management URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/730 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] mrutkows closed issue #778: consider renaming _OW_** annotations

2018-03-27 Thread GitBox
mrutkows closed issue #778: consider renaming _OW_** annotations URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/778 This is an automated message from the Apache Git Service. To respond to the message, ple

[GitHub] mrutkows closed issue #780: Rename --managed to --sync

2018-03-27 Thread GitBox
mrutkows closed issue #780: Rename --managed to --sync URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/780 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] csantanapr commented on issue #31: update npm openwhisk to 3.14.0

2018-03-27 Thread GitBox
csantanapr commented on issue #31: update npm openwhisk to 3.14.0 URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/31#issuecomment-376665775 @rabbah went ahead and cleaned up the unit tests and remove duplicate classes that are in common-tests in core repo --

[GitHub] csantanapr closed pull request #40: Update swift41 to 03 26 a

2018-03-27 Thread GitBox
csantanapr closed pull request #40: Update swift41 to 03 26 a URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/40 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 f

[GitHub] pritidesai closed issue #9: Package the binaries of each repo in need

2018-03-27 Thread GitBox
pritidesai closed issue #9: Package the binaries of each repo in need URL: https://github.com/apache/incubator-openwhisk-release/issues/9 This is an automated message from the Apache Git Service. To respond to the message, pl

[GitHub] pritidesai commented on issue #819: wsk deploy not picking up new target

2018-03-27 Thread GitBox
pritidesai commented on issue #819: wsk deploy not picking up new target URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/819#issuecomment-376656879 thanks @ScottChapman This is an automated message from t

[GitHub] pritidesai commented on a change in pull request #816: Multiple Project Management, adding sync command, and undeploy just using Project Name

2018-03-27 Thread GitBox
pritidesai commented on a change in pull request #816: Multiple Project Management, adding sync command, and undeploy just using Project Name URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/816#discussion_r177554544 ## File path: deployers/servicedeployer.go ##

[GitHub] pritidesai opened a new issue #822: Support reading of packages from different namespace than the one specified in .wskprops

2018-03-27 Thread GitBox
pritidesai opened a new issue #822: Support reading of packages from different namespace than the one specified in .wskprops URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/822 Running Get method from [Package](https://github.com/apache/incubator-openwhisk-client-go/blo

[GitHub] pritidesai commented on a change in pull request #816: Multiple Project Management, adding sync command, and undeploy just using Project Name

2018-03-27 Thread GitBox
pritidesai commented on a change in pull request #816: Multiple Project Management, adding sync command, and undeploy just using Project Name URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/816#discussion_r177552150 ## File path: deployers/servicedeployer.go ##

[GitHub] pritidesai commented on a change in pull request #816: Multiple Project Management, adding sync command, and undeploy just using Project Name

2018-03-27 Thread GitBox
pritidesai commented on a change in pull request #816: Multiple Project Management, adding sync command, and undeploy just using Project Name URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/816#discussion_r177552041 ## File path: deployers/manifestreader.go ###

[GitHub] pritidesai commented on a change in pull request #816: Multiple Project Management, adding sync command, and undeploy just using Project Name

2018-03-27 Thread GitBox
pritidesai commented on a change in pull request #816: Multiple Project Management, adding sync command, and undeploy just using Project Name URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/816#discussion_r177551977 ## File path: cmd/sync.go ## @@ -0,0

[GitHub] csantanapr opened a new pull request #31: update npm openwhisk to 3.14.0

2018-03-27 Thread GitBox
csantanapr opened a new pull request #31: update npm openwhisk to 3.14.0 URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/31 This is an automated message from the Apache Git Service. To respond to the me

[GitHub] dgrove-oss closed pull request #39: split write of wait command insert in runner.py

2018-03-27 Thread GitBox
dgrove-oss closed pull request #39: split write of wait command insert in runner.py URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/39 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 prov

[GitHub] mrutkows commented on a change in pull request #816: Multiple Project Management, adding sync command, and undeploy just using Project Name

2018-03-27 Thread GitBox
mrutkows commented on a change in pull request #816: Multiple Project Management, adding sync command, and undeploy just using Project Name URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/816#discussion_r177542878 ## File path: deployers/servicedeployer.go

[GitHub] mrutkows commented on a change in pull request #816: Multiple Project Management, adding sync command, and undeploy just using Project Name

2018-03-27 Thread GitBox
mrutkows commented on a change in pull request #816: Multiple Project Management, adding sync command, and undeploy just using Project Name URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/816#discussion_r177541197 ## File path: cmd/sync.go ## @@ -0,0 +

[GitHub] mrutkows commented on a change in pull request #816: Multiple Project Management, adding sync command, and undeploy just using Project Name

2018-03-27 Thread GitBox
mrutkows commented on a change in pull request #816: Multiple Project Management, adding sync command, and undeploy just using Project Name URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/816#discussion_r177540228 ## File path: cmd/root.go ## @@ -239,6

[GitHub] mrutkows commented on a change in pull request #816: Multiple Project Management, adding sync command, and undeploy just using Project Name

2018-03-27 Thread GitBox
mrutkows commented on a change in pull request #816: Multiple Project Management, adding sync command, and undeploy just using Project Name URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/816#discussion_r177542359 ## File path: deployers/manifestreader.go #

[GitHub] mrutkows commented on a change in pull request #816: Multiple Project Management, adding sync command, and undeploy just using Project Name

2018-03-27 Thread GitBox
mrutkows commented on a change in pull request #816: Multiple Project Management, adding sync command, and undeploy just using Project Name URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/816#discussion_r177543317 ## File path: tests/src/integration/managed-depl

[GitHub] mrutkows commented on a change in pull request #816: Multiple Project Management, adding sync command, and undeploy just using Project Name

2018-03-27 Thread GitBox
mrutkows commented on a change in pull request #816: Multiple Project Management, adding sync command, and undeploy just using Project Name URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/816#discussion_r177543108 ## File path: deployers/servicedeployer.go

[GitHub] jonpspri opened a new pull request #11: Set up PHP 7.1 runtime for multiarch build

2018-03-27 Thread GitBox
jonpspri opened a new pull request #11: Set up PHP 7.1 runtime for multiarch build URL: https://github.com/apache/incubator-openwhisk-runtime-php/pull/11 Changes for multi-architecture build of PHP 7.1. *Big change:* The image is being built from Jessie rather than from Alpine becau

[GitHub] ScottChapman commented on issue #819: wsk deploy not picking up new target

2018-03-27 Thread GitBox
ScottChapman commented on issue #819: wsk deploy not picking up new target URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/819#issuecomment-376642528 @pritidesai - yea, I think what was going on was that the `bx target ...` command was not updating the .wskprops. I had

[GitHub] csantanapr opened a new pull request #40: Update swift41 to 03 26 a

2018-03-27 Thread GitBox
csantanapr opened a new pull request #40: Update swift41 to 03 26 a URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/40 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] ScottChapman commented on issue #819: wsk deploy not picking up new target

2018-03-27 Thread GitBox
ScottChapman commented on issue #819: wsk deploy not picking up new target URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/819#issuecomment-376642528 @pritidesai - yea, I think what was going on was that the `bx target ...` command was not updating the .wskproperties. I

[GitHub] ScottChapman closed issue #819: wsk deploy not picking up new target

2018-03-27 Thread GitBox
ScottChapman closed issue #819: wsk deploy not picking up new target URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/819 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] csantanapr opened a new pull request #40: Update swift41 to 03 26 a

2018-03-27 Thread GitBox
csantanapr opened a new pull request #40: Update swift41 to 03 26 a URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/40 Depends on #39 This is an automated message from the Apache Git Service. To respond

[GitHub] csantanapr closed pull request #40: Update swift41 to 03 26 a

2018-03-27 Thread GitBox
csantanapr closed pull request #40: Update swift41 to 03 26 a URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/40 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 f

[GitHub] pritidesai commented on issue #819: wsk deploy not picking up new target

2018-03-27 Thread GitBox
pritidesai commented on issue #819: wsk deploy not picking up new target URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/819#issuecomment-376640310 Hi @ScottChapman can you please check what was the namespace in `.wskprops` after you run `bx target -s `? `wskdeploy` pi

[GitHub] pritidesai opened a new issue #821: Provide a way to make undeployment of deps optional

2018-03-27 Thread GitBox
pritidesai opened a new issue #821: Provide a way to make undeployment of deps optional URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/821 As part of a recent enhancement with `wskdeploy undeploy --projectname AAA`, if a project **AAA** has a list of dependencies speci

[GitHub] pritidesai opened a new issue #820: API Gateway - Add managed annotations

2018-03-27 Thread GitBox
pritidesai opened a new issue #820: API Gateway - Add managed annotations URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/820 Please add managed annotations to api gateway and also add support to undeploy it just using project name. -

[GitHub] pritidesai commented on issue #818: FYI: Changing latest wskdeploy download URL broke some travis jobs

2018-03-27 Thread GitBox
pritidesai commented on issue #818: FYI: Changing latest wskdeploy download URL broke some travis jobs URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/818#issuecomment-376627086 sorry @krook for this, @houshengbo are we expecting any more changes as part of streamlinin

[GitHub] csantanapr closed pull request #73: Update the document by changing the vote policy for incubating

2018-03-27 Thread GitBox
csantanapr closed pull request #73: Update the document by changing the vote policy for incubating URL: https://github.com/apache/incubator-openwhisk-release/pull/73 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sak

[GitHub] csantanapr closed issue #72: Release tutorial/process needs to include 2nd vote for incubating.

2018-03-27 Thread GitBox
csantanapr closed issue #72: Release tutorial/process needs to include 2nd vote for incubating. URL: https://github.com/apache/incubator-openwhisk-release/issues/72 This is an automated message from the Apache Git Service. T

[GitHub] jthomas closed issue #111: 3.14.0 release

2018-03-27 Thread GitBox
jthomas closed issue #111: 3.14.0 release URL: https://github.com/apache/incubator-openwhisk-client-js/issues/111 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and u

[GitHub] jthomas closed pull request #112: update package.json and d.ts for 3.14

2018-03-27 Thread GitBox
jthomas closed pull request #112: update package.json and d.ts for 3.14 URL: https://github.com/apache/incubator-openwhisk-client-js/pull/112 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 thi

[GitHub] houshengbo opened a new pull request #73: Update the document by changing the vote policy for incubating

2018-03-27 Thread GitBox
houshengbo opened a new pull request #73: Update the document by changing the vote policy for incubating URL: https://github.com/apache/incubator-openwhisk-release/pull/73 Closes: #72 This is an automated message from the Apa

[GitHub] moritzraho opened a new issue #106: Build for docker-compose fails

2018-03-27 Thread GitBox
moritzraho opened a new issue #106: Build for docker-compose fails URL: https://github.com/apache/incubator-openwhisk-devtools/issues/106 steps: Cloned repo then `cd docker-compose` then `make quick-start` error same as in: https://travis-ci.org/apache/incubator-openwhisk-devtools/jo

[GitHub] csantanapr commented on issue #3264: A list of source files miss Apache license headers

2018-03-27 Thread GitBox
csantanapr commented on issue #3264: A list of source files miss Apache license headers URL: https://github.com/apache/incubator-openwhisk/issues/3264#issuecomment-376599093 @daisy-ycguo What are the next action items for this issue? - To add the files to the list of excludes. -

[GitHub] csantanapr commented on issue #5: Swift 4 web proxy in swift

2018-03-27 Thread GitBox
csantanapr commented on issue #5: Swift 4 web proxy in swift URL: https://github.com/apache/incubator-openwhisk-runtime-swift/issues/5#issuecomment-376597348 We might use in the interim a go proxy which seems to be faster than the python flask that we use today. --

[GitHub] csantanapr commented on issue #2: Swift 4 support with Codable

2018-03-27 Thread GitBox
csantanapr commented on issue #2: Swift 4 support with Codable URL: https://github.com/apache/incubator-openwhisk-runtime-swift/issues/2#issuecomment-376597085 This is already done This is an automated message from the Apache

[GitHub] csantanapr closed issue #2: Swift 4 support with Codable

2018-03-27 Thread GitBox
csantanapr closed issue #2: Swift 4 support with Codable URL: https://github.com/apache/incubator-openwhisk-runtime-swift/issues/2 This is an automated message from the Apache Git Service. To respond to the message, please lo

[GitHub] csantanapr opened a new pull request #39: split write of wait command insert in runner.py

2018-03-27 Thread GitBox
csantanapr opened a new pull request #39: split write of wait command insert in runner.py URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/39 This is an automated message from the Apache Git Service. To

[GitHub] csantanapr opened a new issue #72: Release tutorial/process needs to include 2nd vote for incubating.

2018-03-27 Thread GitBox
csantanapr opened a new issue #72: Release tutorial/process needs to include 2nd vote for incubating. URL: https://github.com/apache/incubator-openwhisk-release/issues/72 The tutorial/process should include additional steps to include a 2nd vote on the gene...@incubator.apache.org list to

[GitHub] csantanapr commented on issue #61: tag repos with version after first release with openwhisk@0.1.0-incubating

2018-03-27 Thread GitBox
csantanapr commented on issue #61: tag repos with version after first release with openwhisk@0.1.0-incubating URL: https://github.com/apache/incubator-openwhisk-release/issues/61#issuecomment-376585333 @houshengbo The can be compose using semver 2.0 MAJOR.MINOR.PATCH-PRERELEASE

[GitHub] csantanapr commented on issue #61: tag repos with version after first release with openwhisk@0.1.0-incubating

2018-03-27 Thread GitBox
csantanapr commented on issue #61: tag repos with version after first release with openwhisk@0.1.0-incubating URL: https://github.com/apache/incubator-openwhisk-release/issues/61#issuecomment-376585333 @houshengbo The can be compose using semver 2.0 MAJOR.MINOR.PATCH-PRERELEASE

[GitHub] jonpspri commented on issue #28: Create a multi-architecture Docker image for the runtime

2018-03-27 Thread GitBox
jonpspri commented on issue #28: Create a multi-architecture Docker image for the runtime URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/28#issuecomment-376509612 @csantanapr @rabbah Could you give this a review? Here's the follow-on items list: - Refactor the

[GitHub] jonpspri commented on issue #28: Create a multi-architecture Docker image for the runtime

2018-03-27 Thread GitBox
jonpspri commented on issue #28: Create a multi-architecture Docker image for the runtime URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/28#issuecomment-376509612 @csantanapr @rabbah Could you give this a review? Here's the follow-on items list: - Refactor the

[GitHub] jonpspri commented on issue #28: Create a multi-architecture Docker image for the runtime

2018-03-27 Thread GitBox
jonpspri commented on issue #28: Create a multi-architecture Docker image for the runtime URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/28#issuecomment-376509612 @csantanapr @rabbah Could you give this a review? Here's the follow-on items list: - Factor docume

[GitHub] rabbah closed pull request #38: try again; fix travis2slack URL

2018-03-27 Thread GitBox
rabbah closed pull request #38: try again; fix travis2slack URL URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/38 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 pull request #28: update URL in travis2slack webhook

2018-03-27 Thread GitBox
rabbah closed pull request #28: update URL in travis2slack webhook URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/28 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

[GitHub] dgrove-oss commented on issue #28: update URL in travis2slack webhook

2018-03-27 Thread GitBox
dgrove-oss commented on issue #28: update URL in travis2slack webhook URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/28#issuecomment-376578648 Hook verified. ![screen shot 2018-03-27 at 11 57 05 am](https://user-images.githubusercontent.com/5099100/37979241-1b1fe6b

[GitHub] dgrove-oss commented on issue #38: try again; fix travis2slack URL

2018-03-27 Thread GitBox
dgrove-oss commented on issue #38: try again; fix travis2slack URL URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/38#issuecomment-376577298 Hook verified. ![screen shot 2018-03-27 at 11 53 42 am](https://user-images.githubusercontent.com/5099100/37979048-91c55a1e-

[GitHub] csantanapr commented on a change in pull request #37: update epilogue with correct @escaping for Codable

2018-03-27 Thread GitBox
csantanapr commented on a change in pull request #37: update epilogue with correct @escaping for Codable URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/37#discussion_r177477374 ## File path: core/swift41Action/swift4runner.py ## @@ -66,7 +66,7 @@ de

[GitHub] markusthoemmes commented on issue #3495: Update akka to 2.5.11 and akka-http to 10.1.0.

2018-03-27 Thread GitBox
markusthoemmes commented on issue #3495: Update akka to 2.5.11 and akka-http to 10.1.0. URL: https://github.com/apache/incubator-openwhisk/pull/3495#issuecomment-376574594 @dubee Happy to do that (you can also revive your PR if you want to). Felt like it's easier to do it without the vari

[GitHub] jonpspri opened a new pull request #3496: Add multi-arch runtimes doc

2018-03-27 Thread GitBox
jonpspri opened a new pull request #3496: Add multi-arch runtimes doc URL: https://github.com/apache/incubator-openwhisk/pull/3496 ## Description To avoid duplication among the various runtimes repository, Iā€™ve compiled the more technical aspects of building and working with a multi-a

[GitHub] dgrove-oss opened a new pull request #23: update URL in slack webhook

2018-03-27 Thread GitBox
dgrove-oss opened a new pull request #23: update URL in slack webhook URL: https://github.com/apache/incubator-openwhisk-runtime-docker/pull/23 This is an automated message from the Apache Git Service. To respond to the messa

[GitHub] rabbah closed pull request #10: add webhook for slack notification of PR results

2018-03-27 Thread GitBox
rabbah closed pull request #10: add webhook for slack notification of PR results URL: https://github.com/apache/incubator-openwhisk-runtime-php/pull/10 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] rabbah closed pull request #29: webhook for slack notification

2018-03-27 Thread GitBox
rabbah closed pull request #29: webhook for slack notification URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/29 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] dgrove-oss opened a new pull request #28: update URL in travis2slack webhook

2018-03-27 Thread GitBox
dgrove-oss opened a new pull request #28: update URL in travis2slack webhook URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/28 This is an automated message from the Apache Git Service. To respond to the

  1   2   >