[GitHub] rahulqelfo opened a new pull request #3956: Ow karate

2018-08-09 Thread GitBox
rahulqelfo opened a new pull request #3956: Ow karate URL: https://github.com/apache/incubator-openwhisk/pull/3956 This PR consists of functional test cases using Karate BDD Framework. ## Description This PR intends to complement the existing functional tests cases using the Karat

[GitHub] codecov-io commented on issue #3955: Add an variable to set the timezone for all of components

2018-08-09 Thread GitBox
codecov-io commented on issue #3955: Add an variable to set the timezone for all of components URL: https://github.com/apache/incubator-openwhisk/pull/3955#issuecomment-411667536 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3955?src=pr&el=h1) Report > Merging [#3

[GitHub] markusthoemmes commented on a change in pull request #3955: Add an variable to set the timezone for all of components

2018-08-09 Thread GitBox
markusthoemmes commented on a change in pull request #3955: Add an variable to set the timezone for all of components URL: https://github.com/apache/incubator-openwhisk/pull/3955#discussion_r208838373 ## File path: ansible/roles/apigateway/tasks/deploy.yml ## @@ -28,3 +29

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

2018-08-09 Thread GitBox
chetanmeh commented on issue #3798: Allow for activation store to accept user and request information URL: https://github.com/apache/incubator-openwhisk/pull/3798#issuecomment-411672562 > where do you think the UserContext case class should live Adding it to `ActivationStore` looks

[GitHub] fmaschler commented on issue #2926: Missing document for wskadmin

2018-08-09 Thread GitBox
fmaschler commented on issue #2926: Missing document for wskadmin URL: https://github.com/apache/incubator-openwhisk/issues/2926#issuecomment-411676875 I came across this issue using the docker-compose quick-start. I guess it's deprecated with the new tool `wskadmin-next`? Since neither of

[GitHub] chetanmeh commented on issue #2926: Missing document for wskadmin

2018-08-09 Thread GitBox
chetanmeh commented on issue #2926: Missing document for wskadmin URL: https://github.com/apache/incubator-openwhisk/issues/2926#issuecomment-411678781 `wskadmin-next` is not deprecated. You need to have required properties in `whisk.properties` file. Can you share output of `wskadmin -v u

[GitHub] jiangpengcheng commented on a change in pull request #3955: Add an variable to set the timezone for all of components

2018-08-09 Thread GitBox
jiangpengcheng commented on a change in pull request #3955: Add an variable to set the timezone for all of components URL: https://github.com/apache/incubator-openwhisk/pull/3955#discussion_r208845167 ## File path: ansible/roles/apigateway/tasks/deploy.yml ## @@ -28,3 +29

[GitHub] codecov-io edited a comment on issue #3814: wskadmin scala db command to export and import

2018-08-09 Thread GitBox
codecov-io edited a comment on issue #3814: wskadmin scala db command to export and import URL: https://github.com/apache/incubator-openwhisk/pull/3814#issuecomment-400666862 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3814?src=pr&el=h1) Report > Merging [#3814]

[GitHub] fmaschler opened a new issue #3957: wskadmin-next configuration error

2018-08-09 Thread GitBox
fmaschler opened a new issue #3957: wskadmin-next configuration error URL: https://github.com/apache/incubator-openwhisk/issues/3957 The tool `wskadmin-next` failed to list namespaces when following the steps in the [README](https://github.com/apache/incubator-openwhisk/blob/master/tools/a

[GitHub] fmaschler commented on issue #2926: Missing document for wskadmin

2018-08-09 Thread GitBox
fmaschler commented on issue #2926: Missing document for wskadmin URL: https://github.com/apache/incubator-openwhisk/issues/2926#issuecomment-411684133 Of course `wskadmin-next` ist not deprecated, I referred to `wskadmin`. The verbose flag did help to find the mistake in the request

[GitHub] codecov-io commented on issue #3950: Extend system testsuite

2018-08-09 Thread GitBox
codecov-io commented on issue #3950: Extend system testsuite URL: https://github.com/apache/incubator-openwhisk/pull/3950#issuecomment-411691678 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3950?src=pr&el=h1) Report > Merging [#3950](https://codecov.io/gh/apache/i

[GitHub] codecov-io edited a comment on issue #3950: Extend system testsuite

2018-08-09 Thread GitBox
codecov-io edited a comment on issue #3950: Extend system testsuite URL: https://github.com/apache/incubator-openwhisk/pull/3950#issuecomment-411691678 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3950?src=pr&el=h1) Report > Merging [#3950](https://codecov.io/gh/a

[GitHub] chetanmeh commented on issue #2926: Missing document for wskadmin

2018-08-09 Thread GitBox
chetanmeh commented on issue #2926: Missing document for wskadmin URL: https://github.com/apache/incubator-openwhisk/issues/2926#issuecomment-411695146 My mistake ... I mean `wskadmin` is yet not deprecated and should work for your usecase -

[GitHub] codecov-io commented on issue #3951: Docs fixes for action invocations

2018-08-09 Thread GitBox
codecov-io commented on issue #3951: Docs fixes for action invocations URL: https://github.com/apache/incubator-openwhisk/pull/3951#issuecomment-411695432 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3951?src=pr&el=h1) Report > Merging [#3951](https://codecov.io/g

[GitHub] jvdgeest opened a new pull request #271: Correctly writing custom OpenWhisk authentication keys (for the guest…

2018-08-09 Thread GitBox
jvdgeest opened a new pull request #271: Correctly writing custom OpenWhisk authentication keys (for the guest… URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/271 … and whisk.system users) into the CouchDB database. With this fix, custom auth keys that are define

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

2018-08-09 Thread GitBox
markusthoemmes commented on a change in pull request #3950: Extend system testsuite URL: https://github.com/apache/incubator-openwhisk/pull/3950#discussion_r208862427 ## File path: tests/src/test/scala/whisk/core/cli/test/WskEntitlementTests.scala ## @@ -160,6 +160,46 @@

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

2018-08-09 Thread GitBox
markusthoemmes commented on a change in pull request #3950: Extend system testsuite URL: https://github.com/apache/incubator-openwhisk/pull/3950#discussion_r208863182 ## File path: tests/src/test/scala/whisk/core/controller/test/ActivationsApiTests.scala ## @@ -462,6 +46

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

2018-08-09 Thread GitBox
markusthoemmes commented on a change in pull request #3950: Extend system testsuite URL: https://github.com/apache/incubator-openwhisk/pull/3950#discussion_r208861393 ## File path: tests/src/test/scala/system/basic/WskActivationTests.scala ## @@ -21,6 +21,7 @@ import comm

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

2018-08-09 Thread GitBox
markusthoemmes commented on a change in pull request #3950: Extend system testsuite URL: https://github.com/apache/incubator-openwhisk/pull/3950#discussion_r208862658 ## File path: tests/src/test/scala/whisk/core/controller/test/ActivationsApiTests.scala ## @@ -462,6 +46

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

2018-08-09 Thread GitBox
markusthoemmes commented on a change in pull request #3950: Extend system testsuite URL: https://github.com/apache/incubator-openwhisk/pull/3950#discussion_r208862176 ## File path: tests/src/test/scala/system/basic/WskRestBasicTests.scala ## @@ -320,6 +320,40 @@ class Wsk

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

2018-08-09 Thread GitBox
markusthoemmes commented on a change in pull request #3950: Extend system testsuite URL: https://github.com/apache/incubator-openwhisk/pull/3950#discussion_r208861341 ## 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

2018-08-09 Thread GitBox
markusthoemmes commented on a change in pull request #3950: Extend system testsuite URL: https://github.com/apache/incubator-openwhisk/pull/3950#discussion_r208863005 ## File path: tests/src/test/scala/whisk/core/controller/test/ActivationsApiTests.scala ## @@ -462,6 +46

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

2018-08-09 Thread GitBox
markusthoemmes commented on a change in pull request #3950: Extend system testsuite URL: https://github.com/apache/incubator-openwhisk/pull/3950#discussion_r208860665 ## File path: tests/src/test/scala/common/rest/WskRestOperations.scala ## @@ -733,13 +738,23 @@ class Res

[GitHub] chetanmeh opened a new pull request #3958: Add tracing component name to wskadmin-next

2018-08-09 Thread GitBox
chetanmeh opened a new pull request #3958: Add tracing component name to wskadmin-next URL: https://github.com/apache/incubator-openwhisk/pull/3958 `wskadmin-next` fails to start with configuration exception. This config is required for each independent component which depends on common mo

[GitHub] chetanmeh commented on issue #3957: wskadmin-next configuration error

2018-08-09 Thread GitBox
chetanmeh commented on issue #3957: wskadmin-next configuration error URL: https://github.com/apache/incubator-openwhisk/issues/3957#issuecomment-411724078 I had currently [fixed it][1] as part of work in #3814. Now created a separate PR #3958 to fix this specifically [1]: https://gith

[GitHub] chetanmeh commented on issue #3958: Add tracing component name to wskadmin-next

2018-08-09 Thread GitBox
chetanmeh commented on issue #3958: Add tracing component name to wskadmin-next URL: https://github.com/apache/incubator-openwhisk/pull/3958#issuecomment-411724410 > Could this be included in the generated conf file? Can be but its more of a constant config so better part of embedded

[GitHub] chetanmeh edited a comment on issue #3957: wskadmin-next configuration error

2018-08-09 Thread GitBox
chetanmeh edited a comment on issue #3957: wskadmin-next configuration error URL: https://github.com/apache/incubator-openwhisk/issues/3957#issuecomment-411724078 I had currently [fixed it][1] as part of work in #3814. Now created a separate PR #3958 to fix this specifically [1]: h

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

2018-08-09 Thread GitBox
mgencur commented on a change in pull request #3950: Extend system testsuite URL: https://github.com/apache/incubator-openwhisk/pull/3950#discussion_r208896992 ## File path: tests/src/test/scala/whisk/core/controller/test/ActivationsApiTests.scala ## @@ -462,6 +462,58 @@

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

2018-08-09 Thread GitBox
markusthoemmes commented on a change in pull request #3950: Extend system testsuite URL: https://github.com/apache/incubator-openwhisk/pull/3950#discussion_r208900516 ## File path: tests/src/test/scala/whisk/core/controller/test/ActivationsApiTests.scala ## @@ -462,6 +46

[GitHub] markusthoemmes commented on issue #3958: Add tracing component name to wskadmin-next

2018-08-09 Thread GitBox
markusthoemmes commented on issue #3958: Add tracing component name to wskadmin-next URL: https://github.com/apache/incubator-openwhisk/pull/3958#issuecomment-411732911 Why is this needed if tracing is disabled? Do we have a bug somewhere else?

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

2018-08-09 Thread GitBox
mgencur commented on a change in pull request #3950: Extend system testsuite URL: https://github.com/apache/incubator-openwhisk/pull/3950#discussion_r208905795 ## File path: tests/src/test/scala/whisk/core/controller/test/ActivationsApiTests.scala ## @@ -462,6 +462,58 @@

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

2018-08-09 Thread GitBox
markusthoemmes commented on a change in pull request #3950: Extend system testsuite URL: https://github.com/apache/incubator-openwhisk/pull/3950#discussion_r208909320 ## File path: tests/src/test/scala/whisk/core/controller/test/ActivationsApiTests.scala ## @@ -462,6 +46

[GitHub] rabbah commented on a change in pull request #3955: Add an variable to set the timezone for all of components

2018-08-09 Thread GitBox
rabbah commented on a change in pull request #3955: Add an variable to set the timezone for all of components URL: https://github.com/apache/incubator-openwhisk/pull/3955#discussion_r208915212 ## File path: ansible/README.md ## @@ -359,3 +359,11 @@ limits: - The `limits.

[GitHub] rabbah commented on a change in pull request #3955: Add an variable to set the timezone for all of components

2018-08-09 Thread GitBox
rabbah commented on a change in pull request #3955: Add an variable to set the timezone for all of components URL: https://github.com/apache/incubator-openwhisk/pull/3955#discussion_r208915359 ## File path: ansible/README.md ## @@ -359,3 +359,11 @@ limits: - The `limits.

[GitHub] csantanapr commented on a change in pull request #3955: Add an variable to set the timezone for all of components

2018-08-09 Thread GitBox
csantanapr commented on a change in pull request #3955: Add an variable to set the timezone for all of components URL: https://github.com/apache/incubator-openwhisk/pull/3955#discussion_r208926951 ## File path: ansible/roles/apigateway/tasks/deploy.yml ## @@ -28,3 +29,6 @

[GitHub] mgencur opened a new issue #3959: Validate API-level tests against Swagger spec

2018-08-09 Thread GitBox
mgencur opened a new issue #3959: Validate API-level tests against Swagger spec URL: https://github.com/apache/incubator-openwhisk/issues/3959 The goal is to include validation of requests and responses (maybe only responses) in *ApiTests which are mostly located under whisk/core/co

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

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

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

2018-08-09 Thread GitBox
rabbah commented on a change in pull request #3951: Docs fixes for action invocations URL: https://github.com/apache/incubator-openwhisk/pull/3951#discussion_r208928955 ## 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

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

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

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

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

2018-08-09 Thread GitBox
rabbah commented on a change in pull request #3951: Docs fixes for action invocations URL: https://github.com/apache/incubator-openwhisk/pull/3951#discussion_r208927943 ## File path: docs/actions.md ## @@ -233,8 +240,11 @@ Each action invocation results in an activation r

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

2018-08-09 Thread GitBox
rabbah commented on a change in pull request #3951: Docs fixes for action invocations URL: https://github.com/apache/incubator-openwhisk/pull/3951#discussion_r208928501 ## File path: docs/actions.md ## @@ -233,8 +240,11 @@ Each action invocation results in an activation r

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

2018-08-09 Thread GitBox
rabbah commented on issue #3951: Docs fixes for action invocations URL: https://github.com/apache/incubator-openwhisk/pull/3951#issuecomment-411758033 Thanks for improving the docs - this is important and appreciated. This is

[GitHub] rabbah closed pull request #3173: incubator-openwhisk-2882 Delete printlns

2018-08-09 Thread GitBox
rabbah closed pull request #3173: incubator-openwhisk-2882 Delete printlns URL: https://github.com/apache/incubator-openwhisk/pull/3173 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] chetanmeh commented on issue #3958: Add tracing component name to wskadmin-next

2018-08-09 Thread GitBox
chetanmeh commented on issue #3958: Add tracing component name to wskadmin-next URL: https://github.com/apache/incubator-openwhisk/pull/3958#issuecomment-411763212 @markusthoemmes This is due to how currently tracing is configured. It requires a component name which is provided via config.

[GitHub] rabbah closed issue #2882: redact information in test log message

2018-08-09 Thread GitBox
rabbah closed issue #2882: redact information in test log message URL: https://github.com/apache/incubator-openwhisk/issues/2882 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] rabbah commented on issue #2882: redact information in test log message

2018-08-09 Thread GitBox
rabbah commented on issue #2882: redact information in test log message URL: https://github.com/apache/incubator-openwhisk/issues/2882#issuecomment-411763485 Thanks for the contribution and fix. This is an automated message f

[GitHub] rabbah commented on issue #3173: incubator-openwhisk-2882 Delete printlns

2018-08-09 Thread GitBox
rabbah commented on issue #3173: incubator-openwhisk-2882 Delete printlns URL: https://github.com/apache/incubator-openwhisk/pull/3173#issuecomment-411764105 @vooolll do you have an Apache CLA? This is an automated messag

[GitHub] rabbah edited a comment on issue #3173: incubator-openwhisk-2882 Delete printlns

2018-08-09 Thread GitBox
rabbah edited a comment on issue #3173: incubator-openwhisk-2882 Delete printlns URL: https://github.com/apache/incubator-openwhisk/pull/3173#issuecomment-411764105 @vooolll do you have an Apache CLA on file? This is an

[GitHub] dgrove-oss commented on issue #271: Correctly writing custom OpenWhisk authentication keys (for the guest…

2018-08-09 Thread GitBox
dgrove-oss commented on issue #271: Correctly writing custom OpenWhisk authentication keys (for the guest… URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/271#issuecomment-411770579 Thanks; change makes sense to me. Could you please file an Apache CLA as describe

[GitHub] mdeuser closed pull request #3760: Separate routemgmt.yml out from openwhisk.yml playbook

2018-08-09 Thread GitBox
mdeuser closed pull request #3760: Separate routemgmt.yml out from openwhisk.yml playbook URL: https://github.com/apache/incubator-openwhisk/pull/3760 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] fmaschler opened a new pull request #147: Use 0.9.0 release as default sources

2018-08-09 Thread GitBox
fmaschler opened a new pull request #147: Use 0.9.0 release as default sources URL: https://github.com/apache/incubator-openwhisk-devtools/pull/147 Now with the first official release one could use that to download sources in the quick-start deployment.

[GitHub] rabbah edited a comment on issue #3173: incubator-openwhisk-2882 Delete printlns

2018-08-09 Thread GitBox
rabbah edited a comment on issue #3173: incubator-openwhisk-2882 Delete printlns URL: https://github.com/apache/incubator-openwhisk/pull/3173#issuecomment-411764105 @vooolll do you have an Apache CLA on file? Could you please file an Apache CLA as described here: https://github.com/apache

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

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

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

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

[GitHub] csantanapr opened a new pull request #7: remove hardcoded env variables for run handler

2018-08-09 Thread GitBox
csantanapr opened a new pull request #7: remove hardcoded env variables for run handler URL: https://github.com/apache/incubator-openwhisk-runtime-ballerina/pull/7 This allows for a customized invoker with an SPI to send more variables in the run handler --

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

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

[GitHub] csantanapr commented on issue #7: remove hardcoded env variables for run handler

2018-08-09 Thread GitBox
csantanapr commented on issue #7: remove hardcoded env variables for run handler URL: https://github.com/apache/incubator-openwhisk-runtime-ballerina/pull/7#issuecomment-411775479 @eweiter could you review this PR. This is an

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

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

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

2018-08-09 Thread GitBox
mgencur commented on a change in pull request #3951: Docs fixes for action invocations URL: https://github.com/apache/incubator-openwhisk/pull/3951#discussion_r208949514 ## File path: docs/actions.md ## @@ -233,8 +240,11 @@ Each action invocation results in an activation

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

2018-08-09 Thread GitBox
mgencur commented on a change in pull request #3951: Docs fixes for action invocations URL: https://github.com/apache/incubator-openwhisk/pull/3951#discussion_r208949464 ## File path: docs/actions.md ## @@ -233,8 +240,11 @@ Each action invocation results in an activation

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

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

[GitHub] codecov-io edited a comment on issue #3919: Allow raw JSON string as response for webaction without base64 encoding.

2018-08-09 Thread GitBox
codecov-io edited a comment on issue #3919: Allow raw JSON string as response for webaction without base64 encoding. URL: https://github.com/apache/incubator-openwhisk/pull/3919#issuecomment-409779533 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3919?src=pr&el=h1) Re

[GitHub] csantanapr opened a new pull request #10: remove hardcoded env variables for run handler

2018-08-09 Thread GitBox
csantanapr opened a new pull request #10: remove hardcoded env variables for run handler URL: https://github.com/apache/incubator-openwhisk-runtime-ruby/pull/10 This allows for the customized invoker with an SPI to send more variables in the run handler @eweiter could you review this PR

[GitHub] csantanapr opened a new pull request #67: remove hardcoded env variables for run handler

2018-08-09 Thread GitBox
csantanapr opened a new pull request #67: remove hardcoded env variables for run handler URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/67 This allows for the customized invoker with an SPI to send more variables in the run handler @eweiter could you review this PR

[GitHub] pritidesai closed pull request #291: ADjust colors for yaml and bash; add bash highlighting

2018-08-09 Thread GitBox
pritidesai closed pull request #291: ADjust colors for yaml and bash; add bash highlighting URL: https://github.com/apache/incubator-openwhisk-website/pull/291 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 p

[GitHub] csantanapr opened a new pull request #55: remove hardcoded env variables for run handler

2018-08-09 Thread GitBox
csantanapr opened a new pull request #55: remove hardcoded env variables for run handler URL: https://github.com/apache/incubator-openwhisk-runtime-docker/pull/55 This allows for the customized invoker with an SPI to send more variables in the run handler @eweiter could you review this

[GitHub] csantanapr opened a new pull request #78: remove hardcoded env variables for run handler

2018-08-09 Thread GitBox
csantanapr opened a new pull request #78: remove hardcoded env variables for run handler URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/78 This allows for the customized invoker with an SPI to send more variables in the run handler @eweiter could you review this

[GitHub] codecov-io edited a comment on issue #3919: Allow raw JSON string as response for webaction without base64 encoding.

2018-08-09 Thread GitBox
codecov-io edited a comment on issue #3919: Allow raw JSON string as response for webaction without base64 encoding. URL: https://github.com/apache/incubator-openwhisk/pull/3919#issuecomment-409779533 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3919?src=pr&el=h1) Re

[GitHub] junoyoon commented on issue #3880: Modify that web action in the bound package can be accessed.

2018-08-09 Thread GitBox
junoyoon commented on issue #3880: Modify that web action in the bound package can be accessed. URL: https://github.com/apache/incubator-openwhisk/pull/3880#issuecomment-411795589 @upgle is it done? This is an automated mess

[GitHub] rabbah commented on issue #3880: Modify that web action in the bound package can be accessed.

2018-08-09 Thread GitBox
rabbah commented on issue #3880: Modify that web action in the bound package can be accessed. URL: https://github.com/apache/incubator-openwhisk/pull/3880#issuecomment-411796869 The last commit looks a lot better now - I'll review more thoroughly. -

[GitHub] dubee closed pull request #3919: Allow raw JSON string as response for webaction without base64 encoding.

2018-08-09 Thread GitBox
dubee closed pull request #3919: Allow raw JSON string as response for webaction without base64 encoding. URL: https://github.com/apache/incubator-openwhisk/pull/3919 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sa

[GitHub] dubee commented on issue #3173: incubator-openwhisk-2882 Delete printlns

2018-08-09 Thread GitBox
dubee commented on issue #3173: incubator-openwhisk-2882 Delete printlns URL: https://github.com/apache/incubator-openwhisk/pull/3173#issuecomment-411853578 This could have used a PG. Looks like upstream dependancies are broken as a result of these changes. ---

[GitHub] mrutkows closed issue #284: Add code highlighting

2018-08-09 Thread GitBox
mrutkows closed issue #284: Add code highlighting URL: https://github.com/apache/incubator-openwhisk-website/issues/284 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub

[GitHub] mrutkows commented on issue #284: Add code highlighting

2018-08-09 Thread GitBox
mrutkows commented on issue #284: Add code highlighting URL: https://github.com/apache/incubator-openwhisk-website/issues/284#issuecomment-411858817 We added syntax highlighting using Jekyll Rouge and Pygmentation based CSS See PR: https://github.com/apache/incubator-openwhisk-website/pu

[GitHub] mrutkows commented on issue #274: Need to provide links/method to point to releases

2018-08-09 Thread GitBox
mrutkows commented on issue #274: Need to provide links/method to point to releases URL: https://github.com/apache/incubator-openwhisk-website/issues/274#issuecomment-411860127 IMO, we should do a "dropdown" from the "Downloads" menu on the home page like shown here: https://kubernetes.io

[GitHub] mrutkows commented on issue #270: Create README Templates for Deployment methods

2018-08-09 Thread GitBox
mrutkows commented on issue #270: Create README Templates for Deployment methods URL: https://github.com/apache/incubator-openwhisk-website/issues/270#issuecomment-411861632 We should reach out to Dave Grove, Brendan/Ben and Dragos when developing the template (perhaps not all at once but

[GitHub] mrutkows commented on issue #268: Add Quotes (somewhere) and remove old pages

2018-08-09 Thread GitBox
mrutkows commented on issue #268: Add Quotes (somewhere) and remove old pages URL: https://github.com/apache/incubator-openwhisk-website/issues/268#issuecomment-411861876 Perhaps add to the "Community" page? This is an autom

[GitHub] mrutkows commented on issue #268: Add Quotes (somewhere) and remove old pages

2018-08-09 Thread GitBox
mrutkows commented on issue #268: Add Quotes (somewhere) and remove old pages URL: https://github.com/apache/incubator-openwhisk-website/issues/268#issuecomment-411862139 Quotes are problematic since we have no means to keep them current and the ones we have are over 2+ years old and 1/3

[GitHub] mrutkows closed issue #255: WHAT, HOW, WHY Install OpenWhisk on the main page.

2018-08-09 Thread GitBox
mrutkows closed issue #255: WHAT, HOW, WHY Install OpenWhisk on the main page. URL: https://github.com/apache/incubator-openwhisk-website/issues/255 This is an automated message from the Apache Git Service. To respond to the

[GitHub] mrutkows commented on issue #255: WHAT, HOW, WHY Install OpenWhisk on the main page.

2018-08-09 Thread GitBox
mrutkows commented on issue #255: WHAT, HOW, WHY Install OpenWhisk on the main page. URL: https://github.com/apache/incubator-openwhisk-website/issues/255#issuecomment-411865132 With the redesign of website I believe we fixed all these issues. If we missed any, I would suggest opening ind

[GitHub] mrutkows opened a new issue #292: Project Website requires License page (or link to)

2018-08-09 Thread GitBox
mrutkows opened a new issue #292: Project Website requires License page (or link to) URL: https://github.com/apache/incubator-openwhisk-website/issues/292 See Apache Spark (for ideas..): https://spark.apache.org/ With the "Apache Software Foundation" menu and drop-down with "License"

[GitHub] mrutkows commented on issue #292: Project Website requires License page (or link to)

2018-08-09 Thread GitBox
mrutkows commented on issue #292: Project Website requires License page (or link to) URL: https://github.com/apache/incubator-openwhisk-website/issues/292#issuecomment-411868610 We should likely look at using the Footer more, especially since upon graduation the disclaimer goes away and t

[GitHub] dubee commented on issue #3941: to address #3918, reuse a container on applicationError

2018-08-09 Thread GitBox
dubee commented on issue #3941: to address #3918, reuse a container on applicationError URL: https://github.com/apache/incubator-openwhisk/pull/3941#issuecomment-411868731 @csantanapr any comments on this one? This is an aut

[GitHub] mrutkows commented on issue #219: Add IBM logo to supporters web page

2018-08-09 Thread GitBox
mrutkows commented on issue #219: Add IBM logo to supporters web page URL: https://github.com/apache/incubator-openwhisk-website/issues/219#issuecomment-411868859 Added the IBM Logo (with hyperlink to IBM Cloud Functions as part of the redesign dropped Aug. 2018. -

[GitHub] mrutkows closed issue #219: Add IBM logo to supporters web page

2018-08-09 Thread GitBox
mrutkows closed issue #219: Add IBM logo to supporters web page URL: https://github.com/apache/incubator-openwhisk-website/issues/219 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] mrutkows opened a new issue #293: Need list (or link to one) of all project Committers as per ASF reqs.

2018-08-09 Thread GitBox
mrutkows opened a new issue #293: Need list (or link to one) of all project Committers as per ASF reqs. URL: https://github.com/apache/incubator-openwhisk-website/issues/293 This is an automated message from the Apache Git S

[GitHub] mrutkows opened a new issue #294: Need FAQ to adhere to ASF reqs.

2018-08-09 Thread GitBox
mrutkows opened a new issue #294: Need FAQ to adhere to ASF reqs. URL: https://github.com/apache/incubator-openwhisk-website/issues/294 Perhaps provide a link from Documentation page? [] FAQ: frequently asked questions are answered here. the old FAQ (before redesign) was not ve

[GitHub] mrutkows commented on issue #294: Need FAQ to adhere to ASF reqs.

2018-08-09 Thread GitBox
mrutkows commented on issue #294: Need FAQ to adhere to ASF reqs. URL: https://github.com/apache/incubator-openwhisk-website/issues/294#issuecomment-411871034 Note Spark, has a Menuitem "FAQ" (under Documentation menu), perhaps we can simply link on our main Documentation page? https://

[GitHub] mrutkows commented on issue #241: Revisit the web site Incubator graduation checklist, and add issues as needed

2018-08-09 Thread GitBox
mrutkows commented on issue #241: Revisit the web site Incubator graduation checklist, and add issues as needed URL: https://github.com/apache/incubator-openwhisk-website/issues/241#issuecomment-411872137 Added individual issues against the recommended checklist. -

[GitHub] mrutkows closed issue #241: Revisit the web site Incubator graduation checklist, and add issues as needed

2018-08-09 Thread GitBox
mrutkows closed issue #241: Revisit the web site Incubator graduation checklist, and add issues as needed URL: https://github.com/apache/incubator-openwhisk-website/issues/241 This is an automated message from the Apache Git

[GitHub] mrutkows closed issue #222: Verify Ruby installed command not correct for OSX (homebrew)

2018-08-09 Thread GitBox
mrutkows closed issue #222: Verify Ruby installed command not correct for OSX (homebrew) URL: https://github.com/apache/incubator-openwhisk-website/issues/222 This is an automated message from the Apache Git Service. To resp

[GitHub] mrutkows commented on issue #212: Update Apache Incubator logo

2018-08-09 Thread GitBox
mrutkows commented on issue #212: Update Apache Incubator logo URL: https://github.com/apache/incubator-openwhisk-website/issues/212#issuecomment-411872887 With the website redesign, I would recommend placing the "incubator" logo in the Footer (alongside the disclaimer). See http://incuba

[GitHub] mrutkows commented on issue #220: Add dates/links for China OpenWhisk Academy events / videos / slides /transcripts

2018-08-09 Thread GitBox
mrutkows commented on issue #220: Add dates/links for China OpenWhisk Academy events / videos / slides /transcripts URL: https://github.com/apache/incubator-openwhisk-website/issues/220#issuecomment-411873662 We REALLY need a means to submit (email/form requests) for submitting not only N

[GitHub] mrutkows commented on issue #220: Add dates/links for China OpenWhisk Academy events / videos / slides /transcripts

2018-08-09 Thread GitBox
mrutkows commented on issue #220: Add dates/links for China OpenWhisk Academy events / videos / slides /transcripts URL: https://github.com/apache/incubator-openwhisk-website/issues/220#issuecomment-411875210 Perhaps add an invitation at the top of the Community page like: News and

[GitHub] mrutkows edited a comment on issue #220: Add dates/links for China OpenWhisk Academy events / videos / slides /transcripts

2018-08-09 Thread GitBox
mrutkows edited a comment on issue #220: Add dates/links for China OpenWhisk Academy events / videos / slides /transcripts URL: https://github.com/apache/incubator-openwhisk-website/issues/220#issuecomment-411875210 Perhaps add an invitation at the top of the Community page like: **

[GitHub] mrutkows commented on issue #217: Update GitHub link to Apache org. and suggest new page for "code" (repos).

2018-08-09 Thread GitBox
mrutkows commented on issue #217: Update GitHub link to Apache org. and suggest new page for "code" (repos). URL: https://github.com/apache/incubator-openwhisk-website/issues/217#issuecomment-411877331 How do we do something like this: https://cwiki.apache.org/confluence/display/OPENWHISK

[GitHub] mrutkows commented on issue #217: Update GitHub link to Apache org. and suggest new page for "code" (repos).

2018-08-09 Thread GitBox
mrutkows commented on issue #217: Update GitHub link to Apache org. and suggest new page for "code" (repos). URL: https://github.com/apache/incubator-openwhisk-website/issues/217#issuecomment-411877467 link to https://github.com/apache/incubator-openwhisk/blob/master/docs/dev/modules.md

[GitHub] bbrowning commented on issue #3772: Update kubectl version in invoker Dockerfile

2018-08-09 Thread GitBox
bbrowning commented on issue #3772: Update kubectl version in invoker Dockerfile URL: https://github.com/apache/incubator-openwhisk/pull/3772#issuecomment-411899256 Bumping the Kube version makes sense to me. I'll let @bwmcadams do the review and merge dance. Thanks! --

[GitHub] ddragosd commented on issue #147: Use 0.9.0 release as default sources

2018-08-09 Thread GitBox
ddragosd commented on issue #147: Use 0.9.0 release as default sources URL: https://github.com/apache/incubator-openwhisk-devtools/pull/147#issuecomment-411900387 @fmaschler great idea. Can we make the version variable so one can continue to run with the master too ? For Travis in part

  1   2   >