[GitHub] tz70s opened a new pull request #3469: Ensure clean up testspace under wskadmin test.

2018-03-20 Thread GitBox
tz70s opened a new pull request #3469: Ensure clean up testspace under wskadmin test. URL: https://github.com/apache/incubator-openwhisk/pull/3469 Fix up testsuite `block and unblock a user respectively` fails when `testspace` contains pre-existing users. Signed-off-by: Tzu-Chiao

[GitHub] dubee commented on a change in pull request #244: Add --web-secure option to action create/update

2018-03-20 Thread GitBox
dubee commented on a change in pull request #244: Add --web-secure option to action create/update URL: https://github.com/apache/incubator-openwhisk-cli/pull/244#discussion_r175848990 ## File path: commands/action.go ## @@ -442,14 +457,109 @@ func parseAction(cmd *cobra.C

[GitHub] dubee commented on a change in pull request #244: Add --web-secure option to action create/update

2018-03-20 Thread GitBox
dubee commented on a change in pull request #244: Add --web-secure option to action create/update URL: https://github.com/apache/incubator-openwhisk-cli/pull/244#discussion_r175840729 ## File path: commands/action.go ## @@ -442,14 +457,109 @@ func parseAction(cmd *cobra.C

[GitHub] dubee commented on a change in pull request #244: Add --web-secure option to action create/update

2018-03-20 Thread GitBox
dubee commented on a change in pull request #244: Add --web-secure option to action create/update URL: https://github.com/apache/incubator-openwhisk-cli/pull/244#discussion_r175841358 ## File path: commands/action.go ## @@ -442,14 +457,109 @@ func parseAction(cmd *cobra.C

[GitHub] dubee commented on a change in pull request #244: Add --web-secure option to action create/update

2018-03-20 Thread GitBox
dubee commented on a change in pull request #244: Add --web-secure option to action create/update URL: https://github.com/apache/incubator-openwhisk-cli/pull/244#discussion_r175849353 ## File path: commands/action.go ## @@ -442,14 +457,109 @@ func parseAction(cmd *cobra.C

[GitHub] dubee commented on a change in pull request #244: Add --web-secure option to action create/update

2018-03-20 Thread GitBox
dubee commented on a change in pull request #244: Add --web-secure option to action create/update URL: https://github.com/apache/incubator-openwhisk-cli/pull/244#discussion_r175849808 ## File path: commands/action.go ## @@ -442,14 +457,109 @@ func parseAction(cmd *cobra.C

[GitHub] dubee commented on a change in pull request #244: Add --web-secure option to action create/update

2018-03-20 Thread GitBox
dubee commented on a change in pull request #244: Add --web-secure option to action create/update URL: https://github.com/apache/incubator-openwhisk-cli/pull/244#discussion_r175850403 ## File path: tests/src/test/scala/whisk/core/cli/test/WskBasicUsageTests.scala ## @@ -4

[GitHub] dubee commented on a change in pull request #3464: Add wsk action get to CLI docs

2018-03-20 Thread GitBox
dubee commented on a change in pull request #3464: Add wsk action get to CLI docs URL: https://github.com/apache/incubator-openwhisk/pull/3464#discussion_r175865776 ## File path: docs/actions.md ## @@ -1219,6 +1184,71 @@ This command starts a polling loop that continuousl

[GitHub] dubee commented on a change in pull request #3464: Add wsk action get to CLI docs

2018-03-20 Thread GitBox
dubee commented on a change in pull request #3464: Add wsk action get to CLI docs URL: https://github.com/apache/incubator-openwhisk/pull/3464#discussion_r175865776 ## File path: docs/actions.md ## @@ -1219,6 +1184,71 @@ This command starts a polling loop that continuousl

[GitHub] dubee commented on a change in pull request #3464: Add wsk action get to CLI docs

2018-03-20 Thread GitBox
dubee commented on a change in pull request #3464: Add wsk action get to CLI docs URL: https://github.com/apache/incubator-openwhisk/pull/3464#discussion_r175873830 ## File path: docs/actions.md ## @@ -1219,6 +1184,71 @@ This command starts a polling loop that continuousl

[GitHub] dubee commented on a change in pull request #3464: Add wsk action get to CLI docs

2018-03-20 Thread GitBox
dubee commented on a change in pull request #3464: Add wsk action get to CLI docs URL: https://github.com/apache/incubator-openwhisk/pull/3464#discussion_r175876068 ## File path: docs/actions.md ## @@ -1219,6 +1184,71 @@ This command starts a polling loop that continuousl

[GitHub] dubee commented on a change in pull request #3464: Add wsk action get to CLI docs

2018-03-20 Thread GitBox
dubee commented on a change in pull request #3464: Add wsk action get to CLI docs URL: https://github.com/apache/incubator-openwhisk/pull/3464#discussion_r175876068 ## File path: docs/actions.md ## @@ -1219,6 +1184,71 @@ This command starts a polling loop that continuousl

[GitHub] dubee commented on a change in pull request #3464: Add wsk action get to CLI docs

2018-03-20 Thread GitBox
dubee commented on a change in pull request #3464: Add wsk action get to CLI docs URL: https://github.com/apache/incubator-openwhisk/pull/3464#discussion_r175876601 ## File path: docs/actions.md ## @@ -31,6 +31,7 @@ Learn how to create, invoke, and debug actions in your

[GitHub] dubee commented on a change in pull request #3464: Add wsk action get to CLI docs

2018-03-20 Thread GitBox
dubee commented on a change in pull request #3464: Add wsk action get to CLI docs URL: https://github.com/apache/incubator-openwhisk/pull/3464#discussion_r175545924 ## File path: docs/actions.md ## @@ -1219,6 +1184,71 @@ This command starts a polling loop that continuousl

[GitHub] dubee commented on issue #3464: Add wsk action get to CLI docs

2018-03-20 Thread GitBox
dubee commented on issue #3464: Add wsk action get to CLI docs URL: https://github.com/apache/incubator-openwhisk/pull/3464#issuecomment-374713064 @rabbah, @mdeuser, anything else? This is an automated message from the Apache

[GitHub] dubee commented on a change in pull request #3464: Add wsk action get to CLI docs

2018-03-20 Thread GitBox
dubee commented on a change in pull request #3464: Add wsk action get to CLI docs URL: https://github.com/apache/incubator-openwhisk/pull/3464#discussion_r175882446 ## File path: docs/actions.md ## @@ -1219,6 +1184,71 @@ This command starts a polling loop that continuousl

[GitHub] dubee commented on a change in pull request #3464: Add wsk action get to CLI docs

2018-03-20 Thread GitBox
dubee commented on a change in pull request #3464: Add wsk action get to CLI docs URL: https://github.com/apache/incubator-openwhisk/pull/3464#discussion_r175886022 ## File path: docs/actions.md ## @@ -1219,6 +1184,71 @@ This command starts a polling loop that continuousl

[GitHub] bwmcadams opened a new issue #3470: Change ContainerProxy `unusedTimeout` and `pauseGrace` from hardcoded defaults to config driven

2018-03-20 Thread GitBox
bwmcadams opened a new issue #3470: Change ContainerProxy `unusedTimeout` and `pauseGrace` from hardcoded defaults to config driven URL: https://github.com/apache/incubator-openwhisk/issues/3470 ## Steps to reproduce the issue: 1. Currently, `unusedTimeout` and `pauseGrace` h

[GitHub] bwmcadams opened a new pull request #3471: Fixes #3470 - Changed hardcoded timeouts for idle containers and pause grace to config entries

2018-03-20 Thread GitBox
bwmcadams opened a new pull request #3471: Fixes #3470 - Changed hardcoded timeouts for idle containers and pause grace to config entries URL: https://github.com/apache/incubator-openwhisk/pull/3471 ## Description Previously, the `idleTimeout` and `pauseGrace` settings fo

[GitHub] mrutkows closed issue #758: Deprecate interactive mode in wskdeploy

2018-03-20 Thread GitBox
mrutkows closed issue #758: Deprecate interactive mode in wskdeploy URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/758 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] mrutkows closed issue #693: whiskclient_test.go: "TestNewWhiskConfigInteractiveMode" incomplete/commented out

2018-03-20 Thread GitBox
mrutkows closed issue #693: whiskclient_test.go: "TestNewWhiskConfigInteractiveMode" incomplete/commented out URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/693 This is an automated message from the Apac

[GitHub] mrutkows commented on issue #807: Deprecating Interactive Mode and introducing Preview

2018-03-20 Thread GitBox
mrutkows commented on issue #807: Deprecating Interactive Mode and introducing Preview URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/807#issuecomment-374729320 @csantanapr today! This is an automated mess

[GitHub] mrutkows closed pull request #807: Deprecating Interactive Mode and introducing Preview

2018-03-20 Thread GitBox
mrutkows closed pull request #807: Deprecating Interactive Mode and introducing Preview URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/807 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 pro

[GitHub] mdeuser opened a new pull request #246: Fix Go regex for path parameter parsing

2018-03-20 Thread GitBox
mdeuser opened a new pull request #246: Fix Go regex for path parameter parsing URL: https://github.com/apache/incubator-openwhisk-cli/pull/246 Update Go regex code used to extract path parameters Update test to have a back-to-back path parameter Fixes #237 --

[GitHub] mdeuser commented on issue #246: Fix Go regex for path parameter parsing

2018-03-20 Thread GitBox
mdeuser commented on issue #246: Fix Go regex for path parameter parsing URL: https://github.com/apache/incubator-openwhisk-cli/pull/246#issuecomment-374731987 @drcariel -- would you mind reviewing This is an automated messag

[GitHub] mdeuser commented on a change in pull request #244: Add --web-secure option to action create/update

2018-03-20 Thread GitBox
mdeuser commented on a change in pull request #244: Add --web-secure option to action create/update URL: https://github.com/apache/incubator-openwhisk-cli/pull/244#discussion_r175903104 ## File path: commands/action.go ## @@ -442,14 +457,109 @@ func parseAction(cmd *cobra

[GitHub] mrutkows commented on issue #811: Deprecating WithinOpenWhisk flag

2018-03-20 Thread GitBox
mrutkows commented on issue #811: Deprecating WithinOpenWhisk flag URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/811#issuecomment-374736355 @beemarie Can you confirm that you no longer need to pass in arguments via a JSON structure when running 'wskdeploy' as an action.

[GitHub] mrutkows commented on issue #791: Crash when running with __OW_API_KEY and __OW_API_HOST set

2018-03-20 Thread GitBox
mrutkows commented on issue #791: Crash when running with __OW_API_KEY and __OW_API_HOST set URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/791#issuecomment-374736872 @pritidesai it seems that package-deploy does not use them, but did ask for Belinda to affirm herself

[GitHub] mdeuser commented on a change in pull request #244: Add --web-secure option to action create/update

2018-03-20 Thread GitBox
mdeuser commented on a change in pull request #244: Add --web-secure option to action create/update URL: https://github.com/apache/incubator-openwhisk-cli/pull/244#discussion_r175904374 ## File path: commands/action.go ## @@ -442,14 +457,109 @@ func parseAction(cmd *cobra

[GitHub] dubee opened a new pull request #247: Fix intermittent test failure

2018-03-20 Thread GitBox
dubee opened a new pull request #247: Fix intermittent test failure URL: https://github.com/apache/incubator-openwhisk-cli/pull/247 `create, and get a trigger summary` fails intermittently as the returned annotations array is not always in the expected order. --

[GitHub] mdeuser commented on a change in pull request #244: Add --web-secure option to action create/update

2018-03-20 Thread GitBox
mdeuser commented on a change in pull request #244: Add --web-secure option to action create/update URL: https://github.com/apache/incubator-openwhisk-cli/pull/244#discussion_r175903104 ## File path: commands/action.go ## @@ -442,14 +457,109 @@ func parseAction(cmd *cobra

[GitHub] mdeuser commented on a change in pull request #3464: Add wsk action get to CLI docs

2018-03-20 Thread GitBox
mdeuser commented on a change in pull request #3464: Add wsk action get to CLI docs URL: https://github.com/apache/incubator-openwhisk/pull/3464#discussion_r175907988 ## File path: docs/actions.md ## @@ -1219,6 +1184,71 @@ This command starts a polling loop that continuou

[GitHub] mdeuser commented on a change in pull request #3464: Add wsk action get to CLI docs

2018-03-20 Thread GitBox
mdeuser commented on a change in pull request #3464: Add wsk action get to CLI docs URL: https://github.com/apache/incubator-openwhisk/pull/3464#discussion_r175907988 ## File path: docs/actions.md ## @@ -1219,6 +1184,71 @@ This command starts a polling loop that continuou

[GitHub] mdeuser commented on a change in pull request #3464: Add wsk action get to CLI docs

2018-03-20 Thread GitBox
mdeuser commented on a change in pull request #3464: Add wsk action get to CLI docs URL: https://github.com/apache/incubator-openwhisk/pull/3464#discussion_r175910182 ## File path: docs/actions.md ## @@ -31,6 +31,7 @@ Learn how to create, invoke, and debug actions in your

[GitHub] dubee opened a new pull request #248: Fix test to not compare namespace to subject

2018-03-20 Thread GitBox
dubee opened a new pull request #248: Fix test to not compare namespace to subject URL: https://github.com/apache/incubator-openwhisk-cli/pull/248 `create a trigger, and fire a trigger to get its individual fields from an activation` directly compares a namespace to a subject which will re

[GitHub] pritidesai commented on issue #811: Deprecating WithinOpenWhisk flag

2018-03-20 Thread GitBox
pritidesai commented on issue #811: Deprecating WithinOpenWhisk flag URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/811#issuecomment-374748897 @mrutkows based on the `exec` command from [package deploy](https://github.com/apache/incubator-openwhisk-package-deploy/blob/7e

[GitHub] pritidesai commented on issue #811: Deprecating WithinOpenWhisk flag

2018-03-20 Thread GitBox
pritidesai commented on issue #811: Deprecating WithinOpenWhisk flag URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/811#issuecomment-374748897 @mrutkows based on the `exec` command from [package deploy](https://github.com/apache/incubator-openwhisk-package-deploy/blob/7e

[GitHub] dubee opened a new pull request #249: Use proper license headers

2018-03-20 Thread GitBox
dubee opened a new pull request #249: Use proper license headers URL: https://github.com/apache/incubator-openwhisk-cli/pull/249 The wrong license headers were being used in a couple of files. This is an automated message from

[GitHub] mdeuser commented on a change in pull request #247: Fix intermittent test failure

2018-03-20 Thread GitBox
mdeuser commented on a change in pull request #247: Fix intermittent test failure URL: https://github.com/apache/incubator-openwhisk-cli/pull/247#discussion_r175916324 ## File path: tests/src/test/scala/system/basic/WskBasicTests.scala ## @@ -549,13 +549,18 @@ class WskBa

[GitHub] dubee opened a new pull request #250: Remove email addresses

2018-03-20 Thread GitBox
dubee opened a new pull request #250: Remove email addresses URL: https://github.com/apache/incubator-openwhisk-cli/pull/250 Scrub email address from a couple of files. This is an automated message from the Apache Git Service.

[GitHub] dubee opened a new pull request #3472: Remove email address

2018-03-20 Thread GitBox
dubee opened a new pull request #3472: Remove email address URL: https://github.com/apache/incubator-openwhisk/pull/3472 Removes an email address from API GW tests. ## Description ## Related issue and scope - [ ] I opened an issue to propose and discuss this

[GitHub] houshengbo commented on issue #63: Verify API Gateway legal NOTICE / CREDITS files ack. licenses/contribs.

2018-03-20 Thread GitBox
houshengbo commented on issue #63: Verify API Gateway legal NOTICE / CREDITS files ack. licenses/contribs. URL: https://github.com/apache/incubator-openwhisk-release/issues/63#issuecomment-374753437 closes: https://github.com/apache/incubator-openwhisk-release/issues/9 ---

[GitHub] houshengbo commented on issue #63: Verify API Gateway legal NOTICE / CREDITS files ack. licenses/contribs.

2018-03-20 Thread GitBox
houshengbo commented on issue #63: Verify API Gateway legal NOTICE / CREDITS files ack. licenses/contribs. URL: https://github.com/apache/incubator-openwhisk-release/issues/63#issuecomment-374753437 closes: https://github.com/apache/incubator-openwhisk-release/issues/9 ---

[GitHub] houshengbo opened a new pull request #251: Add the support to specify the version of the binaries

2018-03-20 Thread GitBox
houshengbo opened a new pull request #251: Add the support to specify the version of the binaries URL: https://github.com/apache/incubator-openwhisk-cli/pull/251 This PR changes the task name from release to releaseBinaries in order to make it consistent in the release process. It also a

[GitHub] beemarie commented on issue #811: Deprecating WithinOpenWhisk flag

2018-03-20 Thread GitBox
beemarie commented on issue #811: Deprecating WithinOpenWhisk flag URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/811#issuecomment-374756424 @pritidesai @mrutkows The code priti posted is how wskdeploy is being called from the deploy package. The environment data (cloud

[GitHub] dgrove-oss commented on issue #167: Add webhook for slackbot notification of TravisCI jobs

2018-03-20 Thread GitBox
dgrove-oss commented on issue #167: Add webhook for slackbot notification of TravisCI jobs URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/167#issuecomment-374759296 changed as requested; ready to merge.

[GitHub] dubee closed pull request #248: Fix test to not compare namespace to subject

2018-03-20 Thread GitBox
dubee closed pull request #248: Fix test to not compare namespace to subject URL: https://github.com/apache/incubator-openwhisk-cli/pull/248 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] houshengbo commented on issue #251: Add the support to specify the version of the binaries

2018-03-20 Thread GitBox
houshengbo commented on issue #251: Add the support to specify the version of the binaries URL: https://github.com/apache/incubator-openwhisk-cli/pull/251#issuecomment-374760632 @jonpspri Please help me review this PR, thanks. --

[GitHub] dubee closed pull request #249: Use proper license headers

2018-03-20 Thread GitBox
dubee closed pull request #249: Use proper license headers URL: https://github.com/apache/incubator-openwhisk-cli/pull/249 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] dubee commented on a change in pull request #247: Fix intermittent test failure

2018-03-20 Thread GitBox
dubee commented on a change in pull request #247: Fix intermittent test failure URL: https://github.com/apache/incubator-openwhisk-cli/pull/247#discussion_r175927899 ## File path: tests/src/test/scala/system/basic/WskBasicTests.scala ## @@ -549,13 +549,18 @@ class WskBasic

[GitHub] pritidesai commented on issue #811: Deprecating WithinOpenWhisk flag

2018-03-20 Thread GitBox
pritidesai commented on issue #811: Deprecating WithinOpenWhisk flag URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/811#issuecomment-374763837 @beemarie `exec` deals with the set of env. variables specified in execOptions, its nothing to do with `wskdeploy`. Its more abo

[GitHub] mdeuser commented on issue #3464: Add wsk action get to CLI docs

2018-03-20 Thread GitBox
mdeuser commented on issue #3464: Add wsk action get to CLI docs URL: https://github.com/apache/incubator-openwhisk/pull/3464#issuecomment-374765536 @dubee changes look ok... although i think this section probably needs additional rework to better integrate the web action aspects (web acti

[GitHub] dubee commented on issue #250: Remove email addresses

2018-03-20 Thread GitBox
dubee commented on issue #250: Remove email addresses URL: https://github.com/apache/incubator-openwhisk-cli/pull/250#issuecomment-374769905 Travis failed because of intermittent test failure to be fixed in this PR https://github.com/apache/incubator-openwhisk-cli/pull/247. --

[GitHub] dubee closed pull request #250: Remove email addresses

2018-03-20 Thread GitBox
dubee closed pull request #250: Remove email addresses URL: https://github.com/apache/incubator-openwhisk-cli/pull/250 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 requ

[GitHub] dubee closed pull request #3472: Remove email address

2018-03-20 Thread GitBox
dubee closed pull request #3472: Remove email address URL: https://github.com/apache/incubator-openwhisk/pull/3472 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 request

[GitHub] csantanapr closed issue #30: Remove Swift 4.0, only support 4.1 and forward

2018-03-20 Thread GitBox
csantanapr closed issue #30: Remove Swift 4.0, only support 4.1 and forward URL: https://github.com/apache/incubator-openwhisk-runtime-swift/issues/30 This is an automated message from the Apache Git Service. To respond to th

[GitHub] csantanapr closed pull request #34: Remove swift:4.0, only swift:4.1

2018-03-20 Thread GitBox
csantanapr closed pull request #34: Remove swift:4.0, only swift:4.1 URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/34 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] mrutkows closed issue #791: Crash when running with __OW_API_KEY and __OW_API_HOST set

2018-03-20 Thread GitBox
mrutkows closed issue #791: Crash when running with __OW_API_KEY and __OW_API_HOST set URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/791 This is an automated message from the Apache Git Service. To resp

[GitHub] mrutkows closed pull request #811: Deprecating WithinOpenWhisk flag

2018-03-20 Thread GitBox
mrutkows closed pull request #811: Deprecating WithinOpenWhisk flag URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/811 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] tz70s commented on issue #3469: Ensure clean up testspace under wskadmin test.

2018-03-20 Thread GitBox
tz70s commented on issue #3469: Ensure clean up testspace under wskadmin test. URL: https://github.com/apache/incubator-openwhisk/pull/3469#issuecomment-374837104 Reopen this pr later, b.c. I need to figure out why the test got failed and avoid disturbing upstream travis build. --

[GitHub] tz70s closed pull request #3469: Ensure clean up testspace under wskadmin test.

2018-03-20 Thread GitBox
tz70s closed pull request #3469: Ensure clean up testspace under wskadmin test. URL: https://github.com/apache/incubator-openwhisk/pull/3469 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] jiangpengcheng commented on issue #3459: Wrap producer.send with Future

2018-03-20 Thread GitBox
jiangpengcheng commented on issue #3459: Wrap producer.send with Future URL: https://github.com/apache/incubator-openwhisk/pull/3459#issuecomment-374841982 @rabbah you mean like this?: ```scala Future { blocking( producer.send(record) ) } andThen { case Success(future

[GitHub] jiangpengcheng commented on issue #3459: Wrap producer.send with Future

2018-03-20 Thread GitBox
jiangpengcheng commented on issue #3459: Wrap producer.send with Future URL: https://github.com/apache/incubator-openwhisk/pull/3459#issuecomment-374841982 @rabbah you mean like this?: ```scala Future { blocking( producer.send(record) ) } andThen { case Success(future

[GitHub] tz70s opened a new pull request #3473: Support string based status code from actions.

2018-03-20 Thread GitBox
tz70s opened a new pull request #3473: Support string based status code from actions. URL: https://github.com/apache/incubator-openwhisk/pull/3473 Add an additional option to parse status code from string to int, throws out illegal code if parsing failed. Also, add a relevant test code.

[GitHub] tz70s opened a new pull request #3474: Ensure clean up testspace under wskadmin test.

2018-03-21 Thread GitBox
tz70s opened a new pull request #3474: Ensure clean up testspace under wskadmin test. URL: https://github.com/apache/incubator-openwhisk/pull/3474 Fix up testsuite `block and unblock a user respectively` fails when `testspace` contains pre-existing users. Signed-off-by: Tzu-Chiao

[GitHub] tz70s closed pull request #3474: Ensure clean up testspace under wskadmin test.

2018-03-21 Thread GitBox
tz70s closed pull request #3474: Ensure clean up testspace under wskadmin test. URL: https://github.com/apache/incubator-openwhisk/pull/3474 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] tz70s opened a new pull request #3474: Ensure clean up testspace under wskadmin test.

2018-03-21 Thread GitBox
tz70s opened a new pull request #3474: Ensure clean up testspace under wskadmin test. URL: https://github.com/apache/incubator-openwhisk/pull/3474 Fix up testsuite `block and unblock a user respectively` fails when `testspace` contains pre-existing users. Signed-off-by: Tzu-Chiao

[GitHub] cbickel opened a new pull request #3475: Move performance repository to core repository.

2018-03-21 Thread GitBox
cbickel opened a new pull request #3475: Move performance repository to core repository. URL: https://github.com/apache/incubator-openwhisk/pull/3475 As discussed on the Apache Mailing list, this PR moves the content of the performance repository to the core repository, as performance is o

[GitHub] z14572 opened a new issue #3476: issue about "SSL routines:SSL3_READ_BYTES:sslv3 alert bad"

2018-03-21 Thread GitBox
z14572 opened a new issue #3476: issue about "SSL routines:SSL3_READ_BYTES:sslv3 alert bad" URL: https://github.com/apache/incubator-openwhisk/issues/3476 when I run the CLI "ansible-playbook -i environments/local openwhisk.yml", there is an error as blow: fatal: [controller0]:

[GitHub] chetanmeh commented on issue #3476: issue about "SSL routines:SSL3_READ_BYTES:sslv3 alert bad"

2018-03-21 Thread GitBox
chetanmeh commented on issue #3476: issue about "SSL routines:SSL3_READ_BYTES:sslv3 alert bad" URL: https://github.com/apache/incubator-openwhisk/issues/3476#issuecomment-374880559 Issue looks similar to #3431. Try updating the ansible version 2.4.2.0 (see #3430)

[GitHub] cbickel commented on issue #3451: Fix several loadbalancer bugs.

2018-03-21 Thread GitBox
cbickel commented on issue #3451: Fix several loadbalancer bugs. URL: https://github.com/apache/incubator-openwhisk/pull/3451#issuecomment-374887995 PG5#185 is 🔵 . This is an automated message from the Apache Git Service. To

[GitHub] mdeknowis commented on issue #1824: Allow package deletion for non-empty packages.

2018-03-21 Thread GitBox
mdeknowis commented on issue #1824: Allow package deletion for non-empty packages. URL: https://github.com/apache/incubator-openwhisk/issues/1824#issuecomment-374888227 That would be very helpful. Currently we have to do several get actions request and delete action by action, befo

[GitHub] rabbah commented on issue #3451: Fix several loadbalancer bugs.

2018-03-21 Thread GitBox
rabbah commented on issue #3451: Fix several loadbalancer bugs. URL: https://github.com/apache/incubator-openwhisk/pull/3451#issuecomment-374905167 @cbickel I don’t see you in the co-authored list. Do you need to be? I only see Markus.

[GitHub] rabbah commented on issue #1824: Allow package deletion for non-empty packages.

2018-03-21 Thread GitBox
rabbah commented on issue #1824: Allow package deletion for non-empty packages. URL: https://github.com/apache/incubator-openwhisk/issues/1824#issuecomment-374905329 Do you use the API or a client (wsk cli)? This is an automa

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

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

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

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

[GitHub] rabbah commented on issue #3459: Wrap producer.send with Future

2018-03-21 Thread GitBox
rabbah commented on issue #3459: Wrap producer.send with Future URL: https://github.com/apache/incubator-openwhisk/pull/3459#issuecomment-374906130 Ah. Never mind then. Thanks for trying it out. This is an automated message

[GitHub] mdeknowis commented on issue #1824: Allow package deletion for non-empty packages.

2018-03-21 Thread GitBox
mdeknowis commented on issue #1824: Allow package deletion for non-empty packages. URL: https://github.com/apache/incubator-openwhisk/issues/1824#issuecomment-374918546 I used the REST API, as we have automated processes, and it's part of our software to interact with OpenWhisk -

[GitHub] z14572 commented on issue #3476: issue about "SSL routines:SSL3_READ_BYTES:sslv3 alert bad"

2018-03-21 Thread GitBox
z14572 commented on issue #3476: issue about "SSL routines:SSL3_READ_BYTES:sslv3 alert bad" URL: https://github.com/apache/incubator-openwhisk/issues/3476#issuecomment-374918919 thanks,update the ansible version to 2.4.2.0, the issue has been fixed

[GitHub] z14572 closed issue #3476: issue about "SSL routines:SSL3_READ_BYTES:sslv3 alert bad"

2018-03-21 Thread GitBox
z14572 closed issue #3476: issue about "SSL routines:SSL3_READ_BYTES:sslv3 alert bad" URL: https://github.com/apache/incubator-openwhisk/issues/3476 This is an automated message from the Apache Git Service. To respond to the

[GitHub] cbickel commented on issue #3451: Fix several loadbalancer bugs.

2018-03-21 Thread GitBox
cbickel commented on issue #3451: Fix several loadbalancer bugs. URL: https://github.com/apache/incubator-openwhisk/pull/3451#issuecomment-374919580 @rabbah Do I have to be in the co-authored list? Because my name and image are at the commits anyway, because I'm the author. --

[GitHub] jonpspri commented on a change in pull request #251: Add the support to specify the version of the binaries

2018-03-21 Thread GitBox
jonpspri commented on a change in pull request #251: Add the support to specify the version of the binaries URL: https://github.com/apache/incubator-openwhisk-cli/pull/251#discussion_r176064175 ## File path: build.gradle ## @@ -121,6 +122,10 @@ OpenWhiskPlatform.zipFileNa

[GitHub] rabbah closed pull request #3451: Fix several loadbalancer bugs.

2018-03-21 Thread GitBox
rabbah closed pull request #3451: Fix several loadbalancer bugs. URL: https://github.com/apache/incubator-openwhisk/pull/3451 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 pu

[GitHub] jonpspri commented on a change in pull request #251: Add the support to specify the version of the binaries

2018-03-21 Thread GitBox
jonpspri commented on a change in pull request #251: Add the support to specify the version of the binaries URL: https://github.com/apache/incubator-openwhisk-cli/pull/251#discussion_r176063713 ## File path: build.gradle ## @@ -61,6 +61,7 @@ class OpenWhiskPlatform {

[GitHub] rabbah commented on issue #3454: Control Container Capabilities?

2018-03-21 Thread GitBox
rabbah commented on issue #3454: Control Container Capabilities? URL: https://github.com/apache/incubator-openwhisk/issues/3454#issuecomment-374921872 It's easier to make this a deployment time parameter for the system vs per action. Did you mean per action? --

[GitHub] rabbah commented on issue #3474: Ensure clean up testspace under wskadmin test.

2018-03-21 Thread GitBox
rabbah commented on issue #3474: Ensure clean up testspace under wskadmin test. URL: https://github.com/apache/incubator-openwhisk/pull/3474#issuecomment-374922804 @dubee @cbickel this will need a pg I think. This is an autom

[GitHub] z14572 opened a new issue #168: issue about couchdb container (Error syncing pod)

2018-03-21 Thread GitBox
z14572 opened a new issue #168: issue about couchdb container (Error syncing pod) URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/168 when I deploy openwhisk for kube , run the script file "build.sh", Failed to finish deploying CouchDB. kubectl get pods -n

[GitHub] rabbah commented on a change in pull request #3474: Ensure clean up testspace under wskadmin test.

2018-03-21 Thread GitBox
rabbah commented on a change in pull request #3474: Ensure clean up testspace under wskadmin test. URL: https://github.com/apache/incubator-openwhisk/pull/3474#discussion_r176067405 ## File path: tests/src/test/scala/whisk/core/admin/WskAdminTests.scala ## @@ -33,7 +34,26

[GitHub] rabbah commented on a change in pull request #3473: Support string based status code from actions.

2018-03-21 Thread GitBox
rabbah commented on a change in pull request #3473: Support string based status code from actions. URL: https://github.com/apache/incubator-openwhisk/pull/3473#discussion_r176068333 ## File path: core/controller/src/main/scala/whisk/core/controller/WebActions.scala ## @@

[GitHub] rabbah commented on a change in pull request #3473: Support string based status code from actions.

2018-03-21 Thread GitBox
rabbah commented on a change in pull request #3473: Support string based status code from actions. URL: https://github.com/apache/incubator-openwhisk/pull/3473#discussion_r176068502 ## File path: tests/src/test/scala/whisk/core/controller/test/WebActionsApiTests.scala ##

[GitHub] rabbah commented on issue #3466: Support alternative path for python in whiskprops

2018-03-21 Thread GitBox
rabbah commented on issue #3466: Support alternative path for python in whiskprops URL: https://github.com/apache/incubator-openwhisk/pull/3466#issuecomment-374925652 This change is ok but I'm wondering if we actually need this still. I a few scripts are still in python (wskdamin, wskdev)

[GitHub] chetanmeh commented on issue #3453: Introduce a AttachmentStore SPI

2018-03-21 Thread GitBox
chetanmeh commented on issue #3453: Introduce a AttachmentStore SPI URL: https://github.com/apache/incubator-openwhisk/pull/3453#issuecomment-374931027 As a next step there is now an implementation for S3 aka S3AttachementStore (see [here][1] for the diff). Once this PR is merged would cre

[GitHub] jonpspri commented on issue #3467: WIP: Remove actionControllers tests from incubator-openwhisk repo

2018-03-21 Thread GitBox
jonpspri commented on issue #3467: WIP: Remove actionControllers tests from incubator-openwhisk repo URL: https://github.com/apache/incubator-openwhisk/pull/3467#issuecomment-374932385 Marking as WIP because apparently, the what’s in the runtime repo and what’s in the core repo was even m

[GitHub] tz70s commented on a change in pull request #3474: Ensure clean up testspace under wskadmin test.

2018-03-21 Thread GitBox
tz70s commented on a change in pull request #3474: Ensure clean up testspace under wskadmin test. URL: https://github.com/apache/incubator-openwhisk/pull/3474#discussion_r176076960 ## File path: tests/src/test/scala/whisk/core/admin/WskAdminTests.scala ## @@ -33,7 +34,26

[GitHub] tz70s commented on a change in pull request #3474: Ensure clean up testspace under wskadmin test.

2018-03-21 Thread GitBox
tz70s commented on a change in pull request #3474: Ensure clean up testspace under wskadmin test. URL: https://github.com/apache/incubator-openwhisk/pull/3474#discussion_r176076960 ## File path: tests/src/test/scala/whisk/core/admin/WskAdminTests.scala ## @@ -33,7 +34,26

[GitHub] tz70s commented on issue #3466: Support alternative path for python in whiskprops

2018-03-21 Thread GitBox
tz70s commented on issue #3466: Support alternative path for python in whiskprops URL: https://github.com/apache/incubator-openwhisk/pull/3466#issuecomment-374935244 In my experience, `wskadmin` is not python 3 compatible. For example, `httplib` is particular a python2.7 name, `http.clien

[GitHub] tz70s commented on a change in pull request #3473: Support string based status code from actions.

2018-03-21 Thread GitBox
tz70s commented on a change in pull request #3473: Support string based status code from actions. URL: https://github.com/apache/incubator-openwhisk/pull/3473#discussion_r176079078 ## File path: tests/src/test/scala/whisk/core/controller/test/WebActionsApiTests.scala ##

[GitHub] joshuabuildsthings commented on issue #3454: Control Container Capabilities?

2018-03-21 Thread GitBox
joshuabuildsthings commented on issue #3454: Control Container Capabilities? URL: https://github.com/apache/incubator-openwhisk/issues/3454#issuecomment-374937405 @rabbah - Please bear with me, I'd like to clarify terminology & make sure I'm using it correctly! I'm currently assumin

[GitHub] jonpspri commented on issue #24: Make AdoptOpenJDK Eclipse OpenJ9 the Java runtime for Java actions

2018-03-21 Thread GitBox
jonpspri commented on issue #24: Make AdoptOpenJDK Eclipse OpenJ9 the Java runtime for Java actions URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/24#issuecomment-374938212 I _think_ I found my problem buried in a circular dependency on `incubator-openwhisk` for some

[GitHub] rabbah commented on issue #3454: Control Container Capabilities?

2018-03-21 Thread GitBox
rabbah commented on issue #3454: Control Container Capabilities? URL: https://github.com/apache/incubator-openwhisk/issues/3454#issuecomment-374951200 > I'm currently assuming that what you're saying is that it would be easier to add this feature so it's system-wide & affects all actions e

<    1   2   3   4   5   6   7   8   9   10   >