[GitHub] jiangpengcheng opened a new pull request #3955: Add an variable to set the timezone for all of components

jiangpengcheng opened a new pull request #3955: Add an variable to set the timezone for all of components URL: https://github.com/apache/incubator-openwhisk/pull/3955 This makes it easy to analyze when errors were happened from logs ## Description All components except CouchDB wil

[GitHub] mgencur commented on issue #3950: Extend system testsuite

mgencur commented on issue #3950: Extend system testsuite URL: https://github.com/apache/incubator-openwhisk/pull/3950#issuecomment-411656682 @markusthoemmes thanks, I can see the API tests now. Will try to move some tests there. ---

[GitHub] mrutkows opened a new pull request #291: ADjust colors for yaml and bash; add bash highlighting

mrutkows opened a new pull request #291: ADjust colors for yaml and bash; add bash highlighting URL: https://github.com/apache/incubator-openwhisk-website/pull/291 This is an automated message from the Apache Git Service. To

[GitHub] pritidesai closed pull request #290: tweak css color for syntax highlite of name constants

pritidesai closed pull request #290: tweak css color for syntax highlite of name constants URL: https://github.com/apache/incubator-openwhisk-website/pull/290 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 opened a new pull request #290: tweak css color for syntax highlite of name constants

mrutkows opened a new pull request #290: tweak css color for syntax highlite of name constants URL: https://github.com/apache/incubator-openwhisk-website/pull/290 This is an automated message from the Apache Git Service. To

[GitHub] pritidesai closed pull request #288: Importing code from scripts instead of hardcoding in html file

pritidesai closed pull request #288: Importing code from scripts instead of hardcoding in html file URL: https://github.com/apache/incubator-openwhisk-website/pull/288 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] mrutkows closed pull request #289: an attempt to download latest css

mrutkows closed pull request #289: an attempt to download latest css URL: https://github.com/apache/incubator-openwhisk-website/pull/289 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 opened a new pull request #289: an attempt to download latest css

pritidesai opened a new pull request #289: an attempt to download latest css URL: https://github.com/apache/incubator-openwhisk-website/pull/289 This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] pritidesai closed pull request #287: Fix terminal style and add code syntax highlighting support

pritidesai closed pull request #287: Fix terminal style and add code syntax highlighting support URL: https://github.com/apache/incubator-openwhisk-website/pull/287 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] pritidesai closed pull request #286: Fix spacing between logo and buttons in header

pritidesai closed pull request #286: Fix spacing between logo and buttons in header URL: https://github.com/apache/incubator-openwhisk-website/pull/286 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 provenanc

[GitHub] pritidesai opened a new pull request #288: Importing code from scripts instead of hardcoding in html file

pritidesai opened a new pull request #288: Importing code from scripts instead of hardcoding in html file URL: https://github.com/apache/incubator-openwhisk-website/pull/288 This is an automated message from the Apache Git S

[GitHub] mrutkows opened a new pull request #287: Fix terminal style and add code syntax highlighting support

mrutkows opened a new pull request #287: Fix terminal style and add code syntax highlighting support URL: https://github.com/apache/incubator-openwhisk-website/pull/287 This is an automated message from the Apache Git Servic

[GitHub] Juice10 opened a new pull request #286: Fix spacing between logo and buttons in header

Juice10 opened a new pull request #286: Fix spacing between logo and buttons in header URL: https://github.com/apache/incubator-openwhisk-website/pull/286 Currently the spacing between the logo on the left and the social icons on the right is not the same. This pull request fixes that and

[GitHub] ddragosd commented on issue #139: Meet minio file access denied when waiting for the controller

ddragosd commented on issue #139: Meet minio file access denied when waiting for the controller URL: https://github.com/apache/incubator-openwhisk-devtools/issues/139#issuecomment-411557644 > the message "Access Denied" persists This is actually OK, as the script waits for docker co

[GitHub] ddragosd commented on issue #2: integrate the openwhisk apigateway docker container

ddragosd commented on issue #2: integrate the openwhisk apigateway docker container URL: https://github.com/apache/incubator-openwhisk-devtools/issues/2#issuecomment-411555984 Yes. And thanks for cleaning the issues @chetanmeh !

[GitHub] ddragosd closed issue #2: integrate the openwhisk apigateway docker container

ddragosd closed issue #2: integrate the openwhisk apigateway docker container URL: https://github.com/apache/incubator-openwhisk-devtools/issues/2 This is an automated message from the Apache Git Service. To respond to the me

[GitHub] dubee commented on issue #3798: Allow for activation store to accept user and request information

dubee commented on issue #3798: Allow for activation store to accept user and request information URL: https://github.com/apache/incubator-openwhisk/pull/3798#issuecomment-411554929 PG4 2057 🔵 This is an automated message f

[GitHub] csantanapr closed pull request #77: update Changelogs for PR #73

csantanapr closed pull request #77: update Changelogs for PR #73 URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/77 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] pritidesai closed pull request #285: adding scripts/manifest

pritidesai closed pull request #285: adding scripts/manifest URL: https://github.com/apache/incubator-openwhisk-website/pull/285 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

[GitHub] csantanapr closed pull request #3954: Add Twitter badge

csantanapr closed pull request #3954: Add Twitter badge URL: https://github.com/apache/incubator-openwhisk/pull/3954 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 reques

[GitHub] codecov-io commented on issue #3954: Add Twitter badge

codecov-io commented on issue #3954: Add Twitter badge URL: https://github.com/apache/incubator-openwhisk/pull/3954#issuecomment-411529587 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3954?src=pr&el=h1) Report > Merging [#3954](https://codecov.io/gh/apache/incubat

[GitHub] dubee commented on a change in pull request #3941: to address #3918, reuse a container on applicationError

dubee commented on a change in pull request #3941: to address #3918, reuse a container on applicationError URL: https://github.com/apache/incubator-openwhisk/pull/3941#discussion_r208698819 ## File path: core/invoker/src/main/scala/whisk/core/containerpool/ContainerProxy.scala ##

[GitHub] dubee commented on a change in pull request #3941: to address #3918, reuse a container on applicationError

dubee commented on a change in pull request #3941: to address #3918, reuse a container on applicationError URL: https://github.com/apache/incubator-openwhisk/pull/3941#discussion_r208698819 ## File path: core/invoker/src/main/scala/whisk/core/containerpool/ContainerProxy.scala ##

[GitHub] dubee commented on a change in pull request #3941: to address #3918, reuse a container on applicationError

dubee commented on a change in pull request #3941: to address #3918, reuse a container on applicationError URL: https://github.com/apache/incubator-openwhisk/pull/3941#discussion_r208698819 ## File path: core/invoker/src/main/scala/whisk/core/containerpool/ContainerProxy.scala ##

[GitHub] eweiter opened a new pull request #77: update Changelogs for PR #73

eweiter opened a new pull request #77: update Changelogs for PR #73 URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/77 This updates the Changelog files to document the change that occurred in pull request #73 -

[GitHub] dubee opened a new pull request #3954: Add Twitter badge

dubee opened a new pull request #3954: Add Twitter badge URL: https://github.com/apache/incubator-openwhisk/pull/3954 ## Description Adds a twitter badge to the README that shows number of follows. When the badge is clicked, the user will be redirect to a follow page.

[GitHub] dubee opened a new issue #3953: Improve README

dubee opened a new issue #3953: Improve README URL: https://github.com/apache/incubator-openwhisk/issues/3953 Here are some ideas to improve the README: - Add demo gifs and/or videos - Add a twitter badge This is an auto

[GitHub] codecov-io edited a comment on issue #3798: Allow for activation store to accept user and request information

codecov-io edited a comment on issue #3798: Allow for activation store to accept user and request information URL: https://github.com/apache/incubator-openwhisk/pull/3798#issuecomment-399226990 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3798?src=pr&el=h1) Report

[GitHub] pritidesai opened a new pull request #285: adding scripts/manifest

pritidesai opened a new pull request #285: adding scripts/manifest URL: https://github.com/apache/incubator-openwhisk-website/pull/285 Partially fixing issue #269 This is an automated message from the Apache Git Service. To r

[GitHub] csantanapr closed pull request #3946: update documentation for nodejs runtime

csantanapr closed pull request #3946: update documentation for nodejs runtime URL: https://github.com/apache/incubator-openwhisk/pull/3946 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] tysonnorris commented on issue #3687: Concurrency limit per action

tysonnorris commented on issue #3687: Concurrency limit per action URL: https://github.com/apache/incubator-openwhisk/pull/3687#issuecomment-411498213 Closing this PR, and rolling these changes into #2795 (since these changes are required in any case, to make concurrency configs useful) -

[GitHub] tysonnorris closed pull request #3687: Concurrency limit per action

tysonnorris closed pull request #3687: Concurrency limit per action URL: https://github.com/apache/incubator-openwhisk/pull/3687 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

[GitHub] asteed commented on a change in pull request #3661: Restrict allowed namespaces when creating action of certain kinds

asteed commented on a change in pull request #3661: Restrict allowed namespaces when creating action of certain kinds URL: https://github.com/apache/incubator-openwhisk/pull/3661#discussion_r208672247 ## File path: core/controller/src/main/scala/whisk/core/controller/Actions.scala

[GitHub] csantanapr closed pull request #73: Don't override runtime npm packages when user provides their own

csantanapr closed pull request #73: Don't override runtime npm packages when user provides their own URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/73 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below fo

[GitHub] dubee commented on issue #3798: Allow for activation store to accept user and request information

dubee commented on issue #3798: Allow for activation store to accept user and request information URL: https://github.com/apache/incubator-openwhisk/pull/3798#issuecomment-411488668 Added UserContext in the third commit. The case class definition should probably be moved elsewhere though.

[GitHub] mdeuser commented on issue #3906: Correct documentation to setup database.

mdeuser commented on issue #3906: Correct documentation to setup database. URL: https://github.com/apache/incubator-openwhisk/pull/3906#issuecomment-411476656 @cbickel - i also believe the cloudant instances will not have the `_users` database defined. i ran into this and had to manually

[GitHub] dubee commented on issue #3798: Allow for activation store to accept user and request information

dubee commented on issue #3798: Allow for activation store to accept user and request information URL: https://github.com/apache/incubator-openwhisk/pull/3798#issuecomment-411457977 @chetanmeh, where do you think the UserContext case class should live?

[GitHub] chetanmeh opened a new issue #3952: Update Scala to 2.12

chetanmeh opened a new issue #3952: Update Scala to 2.12 URL: https://github.com/apache/incubator-openwhisk/issues/3952 Currently OpenWhisk uses Scala 2.11.11. We should switch to [Scala 2.12.6][1] which is current latest stable release. See [here][3] for details on all changes. Thi

[GitHub] codecov-io edited a comment on issue #3698: Make amount of parallel docker runs configurable.

codecov-io edited a comment on issue #3698: Make amount of parallel docker runs configurable. URL: https://github.com/apache/incubator-openwhisk/pull/3698#issuecomment-392048718 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3698?src=pr&el=h1) Report > Merging [#36

[GitHub] cbickel commented on a change in pull request #3747: Memory based loadbalancing

cbickel commented on a change in pull request #3747: Memory based loadbalancing URL: https://github.com/apache/incubator-openwhisk/pull/3747#discussion_r208618430 ## File path: common/scala/src/main/scala/whisk/core/entity/Size.scala ## @@ -69,6 +69,23 @@ case class ByteSi

[GitHub] markusthoemmes commented on issue #3950: Extend system testsuite

markusthoemmes commented on issue #3950: Extend system testsuite URL: https://github.com/apache/incubator-openwhisk/pull/3950#issuecomment-411428155 @mgencur As far as I'm aware, the API level tests (ActionsApiTests.scala for example) are heavily stubbed out through `ControllerTestCommon`.

[GitHub] markusthoemmes closed pull request #3942: make error count and percentage configurable for gatling tests

markusthoemmes closed pull request #3942: make error count and percentage configurable for gatling tests URL: https://github.com/apache/incubator-openwhisk/pull/3942 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] mgencur commented on issue #3950: Extend system testsuite

mgencur commented on issue #3950: Extend system testsuite URL: https://github.com/apache/incubator-openwhisk/pull/3950#issuecomment-411408315 @markusthoemmes thanks for the quick review. I will address most of the concerns. But I'd like to know what you mean by "API level" tests and how th

[GitHub] mgencur commented on a change in pull request #3950: Extend system testsuite

mgencur commented on a change in pull request #3950: Extend system testsuite URL: https://github.com/apache/incubator-openwhisk/pull/3950#discussion_r208580980 ## File path: tests/src/test/scala/system/basic/WskActionTests.scala ## @@ -210,6 +253,32 @@ class WskActionTests

[GitHub] mgencur commented on a change in pull request #3950: Extend system testsuite

mgencur commented on a change in pull request #3950: Extend system testsuite URL: https://github.com/apache/incubator-openwhisk/pull/3950#discussion_r208573813 ## File path: tests/src/test/scala/system/basic/WskActionTests.scala ## @@ -71,6 +71,49 @@ class WskActionTests e

[GitHub] mgencur commented on a change in pull request #3950: Extend system testsuite

mgencur commented on a change in pull request #3950: Extend system testsuite URL: https://github.com/apache/incubator-openwhisk/pull/3950#discussion_r208572609 ## File path: tests/src/test/scala/common/rest/WskRestOperations.scala ## @@ -1202,6 +1220,16 @@ trait RunRestCmd

[GitHub] rabbah edited a comment on issue #3671: Activation id in header

rabbah edited a comment on issue #3671: Activation id in header URL: https://github.com/apache/incubator-openwhisk/pull/3671#issuecomment-411380282 I think I've changed perspective on the activation id in the header response of web actions, after some further consideration (to agree with y

[GitHub] rabbah commented on issue #3671: Activation id in header

rabbah commented on issue #3671: Activation id in header URL: https://github.com/apache/incubator-openwhisk/pull/3671#issuecomment-411380282 I think I've changed perspective on the activation id in the header response of web actions, after some further consideration. I can explain in a lon

[GitHub] style95 commented on issue #3671: Activation id in header

style95 commented on issue #3671: Activation id in header URL: https://github.com/apache/incubator-openwhisk/pull/3671#issuecomment-411377191 @rabbah Regarding exposure of ActivationID of webaction in response header, how about having a configuration to control it by OW operator? In

[GitHub] style95 commented on issue #3671: Activation id in header

style95 commented on issue #3671: Activation id in header URL: https://github.com/apache/incubator-openwhisk/pull/3671#issuecomment-411376673 I think the point is, if we allow activationID generation in multiple components(nginx, controller), we cannot guarantee the activationID is unique

[GitHub] chetanmeh commented on a change in pull request #3886: Proposing Lean OpenWhisk

chetanmeh commented on a change in pull request #3886: Proposing Lean OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3886#discussion_r208539746 ## File path: .travis.yml ## @@ -73,3 +73,19 @@ jobs: - OPENWHISK_HOST="172.17.0.1" USERS="1" REQUEST

[GitHub] chetanmeh commented on a change in pull request #3886: Proposing Lean OpenWhisk

chetanmeh commented on a change in pull request #3886: Proposing Lean OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3886#discussion_r208538634 ## File path: common/scala/src/main/scala/whisk/connector/lean/LeanProducer.scala ## @@ -0,0 +1,53 @@ +/* + *

[GitHub] chetanmeh commented on a change in pull request #3886: Proposing Lean OpenWhisk

chetanmeh commented on a change in pull request #3886: Proposing Lean OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3886#discussion_r208537513 ## File path: common/scala/src/main/scala/whisk/connector/lean/LeanConsumer.scala ## @@ -0,0 +1,44 @@ +/* + *

[GitHub] chetanmeh commented on a change in pull request #3886: Proposing Lean OpenWhisk

chetanmeh commented on a change in pull request #3886: Proposing Lean OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3886#discussion_r208538330 ## File path: common/scala/src/main/scala/whisk/connector/lean/LeanMessagingProvider.scala ## @@ -0,0 +1,63 @

[GitHub] kpavel edited a comment on issue #3886: Proposing Lean OpenWhisk

kpavel edited a comment on issue #3886: Proposing Lean OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3886#issuecomment-411364636 @markusthoemmes I think I've finished with requested changes. Can you please review it again? --

[GitHub] kpavel commented on issue #3886: Proposing Lean OpenWhisk

kpavel commented on issue #3886: Proposing Lean OpenWhisk URL: https://github.com/apache/incubator-openwhisk/pull/3886#issuecomment-411364636 @markusthoemmes I think i finished with requested changes. Can you please review it again?

[GitHub] mgencur commented on a change in pull request #3950: Extend system testsuite

mgencur commented on a change in pull request #3950: Extend system testsuite URL: https://github.com/apache/incubator-openwhisk/pull/3950#discussion_r208519673 ## File path: tests/dat/actions/argsPrint.js ## @@ -1,6 +1,8 @@ // Licensed to the Apache Software Foundation (A

[GitHub] mgencur commented on issue #3951: Docs fixes for action invocations

mgencur commented on issue #3951: Docs fixes for action invocations URL: https://github.com/apache/incubator-openwhisk/pull/3951#issuecomment-411348690 Hey @rabbah @markusthoemmes , what do you think now? I've fixed the description a little bit in separate commits. ---

[GitHub] mgencur commented on a change in pull request #3951: Docs fixes for action invocations

mgencur commented on a change in pull request #3951: Docs fixes for action invocations URL: https://github.com/apache/incubator-openwhisk/pull/3951#discussion_r208511709 ## File path: docs/actions.md ## @@ -216,10 +216,11 @@ ok: invoked /whisk.system/samples/greeting with

[GitHub] markusthoemmes commented on a change in pull request #3951: Docs fixes for action invocations

markusthoemmes commented on a change in pull request #3951: Docs fixes for action invocations URL: https://github.com/apache/incubator-openwhisk/pull/3951#discussion_r208509072 ## File path: docs/actions.md ## @@ -216,10 +216,11 @@ ok: invoked /whisk.system/samples/greeti

[GitHub] mgencur commented on a change in pull request #3951: Docs fixes for action invocations

mgencur commented on a change in pull request #3951: Docs fixes for action invocations URL: https://github.com/apache/incubator-openwhisk/pull/3951#discussion_r208507708 ## File path: docs/actions.md ## @@ -233,7 +234,10 @@ Each action invocation results in an activation

[GitHub] mgencur commented on a change in pull request #3951: Docs fixes for action invocations

mgencur commented on a change in pull request #3951: Docs fixes for action invocations URL: https://github.com/apache/incubator-openwhisk/pull/3951#discussion_r208507380 ## File path: docs/actions.md ## @@ -216,10 +216,11 @@ ok: invoked /whisk.system/samples/greeting with

[GitHub] fmaschler commented on issue #139: Meet minio file access denied when waiting for the controller

fmaschler commented on issue #139: Meet minio file access denied when waiting for the controller URL: https://github.com/apache/incubator-openwhisk-devtools/issues/139#issuecomment-411334058 Indeed `make quick-start` runs through now. Though the message "Access Denied" persists. @edwardst

[GitHub] markusthoemmes commented on issue #3950: Extend system testsuite

markusthoemmes commented on issue #3950: Extend system testsuite URL: https://github.com/apache/incubator-openwhisk/pull/3950#issuecomment-411330491 @mgencur greate someone is looking into our testcoverage, very much appreciated 😊 . I should've created a single review, sorry for the notifi

[GitHub] markusthoemmes commented on a change in pull request #3950: Extend system testsuite

markusthoemmes commented on a change in pull request #3950: Extend system testsuite URL: https://github.com/apache/incubator-openwhisk/pull/3950#discussion_r208498721 ## File path: tests/src/test/scala/system/basic/WskActivationTests.scala ## @@ -74,4 +75,39 @@ class WskA

[GitHub] markusthoemmes commented on a change in pull request #3950: Extend system testsuite

markusthoemmes commented on a change in pull request #3950: Extend system testsuite URL: https://github.com/apache/incubator-openwhisk/pull/3950#discussion_r208497864 ## File path: tests/src/test/scala/system/basic/WskActionTests.scala ## @@ -210,6 +253,32 @@ class WskAct

[GitHub] markusthoemmes commented on a change in pull request #3950: Extend system testsuite

markusthoemmes commented on a change in pull request #3950: Extend system testsuite URL: https://github.com/apache/incubator-openwhisk/pull/3950#discussion_r208497477 ## File path: tests/src/test/scala/system/basic/WskActionTests.scala ## @@ -71,6 +71,49 @@ class WskActio

[GitHub] markusthoemmes commented on a change in pull request #3950: Extend system testsuite

markusthoemmes commented on a change in pull request #3950: Extend system testsuite URL: https://github.com/apache/incubator-openwhisk/pull/3950#discussion_r208496841 ## File path: tests/src/test/scala/common/rest/WskRestOperations.scala ## @@ -1295,6 +1323,11 @@ trait Ru

[GitHub] mgencur commented on a change in pull request #3951: Docs fixes for action invocations

mgencur commented on a change in pull request #3951: Docs fixes for action invocations URL: https://github.com/apache/incubator-openwhisk/pull/3951#discussion_r208496577 ## File path: docs/actions.md ## @@ -340,7 +344,7 @@ is skipped if an action is dispatched to a previo

[GitHub] markusthoemmes commented on a change in pull request #3950: Extend system testsuite

markusthoemmes commented on a change in pull request #3950: Extend system testsuite URL: https://github.com/apache/incubator-openwhisk/pull/3950#discussion_r208495717 ## File path: tests/src/test/scala/common/rest/WskRestOperations.scala ## @@ -1202,6 +1220,16 @@ trait Ru

[GitHub] markusthoemmes commented on a change in pull request #3951: Docs fixes for action invocations

markusthoemmes commented on a change in pull request #3951: Docs fixes for action invocations URL: https://github.com/apache/incubator-openwhisk/pull/3951#discussion_r208494979 ## File path: docs/actions.md ## @@ -340,7 +344,7 @@ is skipped if an action is dispatched to a

[GitHub] rabbah commented on a change in pull request #3950: Extend system testsuite

rabbah commented on a change in pull request #3950: Extend system testsuite URL: https://github.com/apache/incubator-openwhisk/pull/3950#discussion_r208494908 ## File path: tests/dat/actions/argsPrint.js ## @@ -1,6 +1,8 @@ // Licensed to the Apache Software Foundation (AS

[GitHub] markusthoemmes commented on a change in pull request #3951: Docs fixes for action invocations

markusthoemmes commented on a change in pull request #3951: Docs fixes for action invocations URL: https://github.com/apache/incubator-openwhisk/pull/3951#discussion_r208494703 ## File path: docs/actions.md ## @@ -233,7 +234,10 @@ Each action invocation results in an acti

[GitHub] rabbah commented on a change in pull request #3951: Docs fixes for action invocations

rabbah commented on a change in pull request #3951: Docs fixes for action invocations URL: https://github.com/apache/incubator-openwhisk/pull/3951#discussion_r208493745 ## File path: docs/actions.md ## @@ -340,7 +344,7 @@ is skipped if an action is dispatched to a previou

[GitHub] rabbah commented on a change in pull request #3951: Docs fixes for action invocations

rabbah commented on a change in pull request #3951: Docs fixes for action invocations URL: https://github.com/apache/incubator-openwhisk/pull/3951#discussion_r208493541 ## File path: docs/actions.md ## @@ -233,7 +234,10 @@ Each action invocation results in an activation r

[GitHub] rabbah commented on a change in pull request #3951: Docs fixes for action invocations

rabbah commented on a change in pull request #3951: Docs fixes for action invocations URL: https://github.com/apache/incubator-openwhisk/pull/3951#discussion_r208493371 ## File path: docs/actions.md ## @@ -216,10 +216,11 @@ ok: invoked /whisk.system/samples/greeting with

[GitHub] mgencur opened a new pull request #3951: Docs fixes for action invocations

mgencur opened a new pull request #3951: Docs fixes for action invocations URL: https://github.com/apache/incubator-openwhisk/pull/3951 More accurate description of blocking calls and the situation when _application error_ is returned upon invoking an action. ## Description

[GitHub] mgencur opened a new pull request #3950: Extend system testsuite

mgencur opened a new pull request #3950: Extend system testsuite URL: https://github.com/apache/incubator-openwhisk/pull/3950 This PR adds more tests for Actions, Activations, Packages. ## Description This PR adds more tests for Actions, Activations, Packages. Mostly some corner