[GitHub] tz70s commented on a change in pull request #3722: Scala based admin tooling

tz70s commented on a change in pull request #3722: Scala based admin tooling URL: https://github.com/apache/incubator-openwhisk/pull/3722#discussion_r193958701 ## File path: core/admin/src/main/scala/whisk/core/cli/Main.scala ## @@ -0,0 +1,161 @@ +/* + * Licensed to the

[GitHub] ningyougang opened a new pull request #3735: Adjust offset topic replication factor

ningyougang opened a new pull request #3735: Adjust offset topic replication factor URL: https://github.com/apache/incubator-openwhisk/pull/3735 Current, default.replication.factor and offsets.topic.replication.factor also uses kafka.replicationFactor, it is better to adjust

[GitHub] pritidesai commented on issue #898: wskdeploy export doesn't support multiple packages in a single project

pritidesai commented on issue #898: wskdeploy export doesn't support multiple packages in a single project URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/898#issuecomment-395652768 Closing this as its fixed now.

[GitHub] pritidesai closed issue #898: wskdeploy export doesn't support multiple packages in a single project

pritidesai closed issue #898: wskdeploy export doesn't support multiple packages in a single project URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/898 This is an automated message from the Apache Git

[GitHub] pritidesai commented on issue #899: wskdeploy export fails to export the `main` attribute for Java actions

pritidesai commented on issue #899: wskdeploy export fails to export the `main` attribute for Java actions URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/899#issuecomment-395652519 @brunogirin please try again with the latest build, its working for me.

[GitHub] chetanmeh closed pull request #3709: Inlined attachments

chetanmeh closed pull request #3709: Inlined attachments URL: https://github.com/apache/incubator-openwhisk/pull/3709 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] pritidesai closed pull request #950: Continue spec. conversion to markdown and add navigation

pritidesai closed pull request #950: Continue spec. conversion to markdown and add navigation URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/950 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] chetanmeh commented on a change in pull request #3722: Scala based admin tooling

chetanmeh commented on a change in pull request #3722: Scala based admin tooling URL: https://github.com/apache/incubator-openwhisk/pull/3722#discussion_r193949023 ## File path: core/admin/src/main/scala/whisk/core/cli/Main.scala ## @@ -0,0 +1,161 @@ +/* + * Licensed to

[GitHub] chetanmeh commented on a change in pull request #3722: Scala based admin tooling

chetanmeh commented on a change in pull request #3722: Scala based admin tooling URL: https://github.com/apache/incubator-openwhisk/pull/3722#discussion_r193949023 ## File path: core/admin/src/main/scala/whisk/core/cli/Main.scala ## @@ -0,0 +1,161 @@ +/* + * Licensed to

[GitHub] codecov-io commented on issue #3734: Correct text in webaction doc.

codecov-io commented on issue #3734: Correct text in webaction doc. URL: https://github.com/apache/incubator-openwhisk/pull/3734#issuecomment-395640397 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3734?src=pr=h1) Report > Merging

[GitHub] tz70s opened a new pull request #3734: Correct text in webaction doc.

tz70s opened a new pull request #3734: Correct text in webaction doc. URL: https://github.com/apache/incubator-openwhisk/pull/3734 Mainly remove the redundant `A`. Signed-off-by: Tzu-Chiao Yeh ## Description ## Related issue and scope - [ ] I

[GitHub] tz70s closed pull request #3734: Correct text in webaction doc.

tz70s closed pull request #3734: Correct text in webaction doc. URL: https://github.com/apache/incubator-openwhisk/pull/3734 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] tz70s commented on a change in pull request #3722: Scala based admin tooling

tz70s commented on a change in pull request #3722: Scala based admin tooling URL: https://github.com/apache/incubator-openwhisk/pull/3722#discussion_r193944971 ## File path: core/admin/src/main/scala/whisk/core/cli/Main.scala ## @@ -0,0 +1,161 @@ +/* + * Licensed to the

[GitHub] codecov-io commented on issue #3704: Invoker graceful shutdown and drain mode

codecov-io commented on issue #3704: Invoker graceful shutdown and drain mode URL: https://github.com/apache/incubator-openwhisk/pull/3704#issuecomment-391821150 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3704?src=pr=h1) Report > Merging

[GitHub] tz70s opened a new pull request #3734: Correct text in webaction doc.

tz70s opened a new pull request #3734: Correct text in webaction doc. URL: https://github.com/apache/incubator-openwhisk/pull/3734 Mainly remove the redundant `A`. Signed-off-by: Tzu-Chiao Yeh ## Description ## Related issue and scope - [ ] I

[GitHub] tz70s commented on issue #3530: factor out max code blob size to a deployment configuration

tz70s commented on issue #3530: factor out max code blob size to a deployment configuration URL: https://github.com/apache/incubator-openwhisk/issues/3530#issuecomment-395623297 @rabbah , I'm working on this. Is the limit on code size configurable during creating/updating actions or just

[GitHub] csantanapr commented on issue #3709: Inlined attachments

csantanapr commented on issue #3709: Inlined attachments URL: https://github.com/apache/incubator-openwhisk/pull/3709#issuecomment-395620572 PG1 2996  This is an automated message from the Apache Git Service. To respond to

[GitHub] rabbah closed pull request #3728: Add information on using Composer with the PHP runtime

rabbah closed pull request #3728: Add information on using Composer with the PHP runtime URL: https://github.com/apache/incubator-openwhisk/pull/3728 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] codecov-io commented on issue #3728: Add information on using Composer with the PHP runtime

codecov-io commented on issue #3728: Add information on using Composer with the PHP runtime URL: https://github.com/apache/incubator-openwhisk/pull/3728#issuecomment-395614861 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3728?src=pr=h1) Report > Merging

[GitHub] tysonnorris commented on a change in pull request #2795: enable concurrent activation processing

tysonnorris commented on a change in pull request #2795: enable concurrent activation processing URL: https://github.com/apache/incubator-openwhisk/pull/2795#discussion_r193926390 ## File path: core/invoker/src/main/scala/whisk/core/containerpool/ContainerProxy.scala ##

[GitHub] codecov-io commented on issue #3707: enable concurrent runs on ActionContainer test utility

codecov-io commented on issue #3707: enable concurrent runs on ActionContainer test utility URL: https://github.com/apache/incubator-openwhisk/pull/3707#issuecomment-392093713 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3707?src=pr=h1) Report > :exclamation: No

[GitHub] codecov-io commented on issue #3707: enable concurrent runs on ActionContainer test utility

codecov-io commented on issue #3707: enable concurrent runs on ActionContainer test utility URL: https://github.com/apache/incubator-openwhisk/pull/3707#issuecomment-392093713 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3707?src=pr=h1) Report > :exclamation: No

[GitHub] rabbah closed pull request #3729: Add brief Kubernetes Setup section to main README.md

rabbah closed pull request #3729: Add brief Kubernetes Setup section to main README.md URL: https://github.com/apache/incubator-openwhisk/pull/3729 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance:

[GitHub] tysonnorris commented on a change in pull request #2795: enable concurrent activation processing

tysonnorris commented on a change in pull request #2795: enable concurrent activation processing URL: https://github.com/apache/incubator-openwhisk/pull/2795#discussion_r193925034 ## File path: core/invoker/src/main/scala/whisk/core/containerpool/ContainerProxy.scala ##

[GitHub] tysonnorris commented on a change in pull request #2795: enable concurrent activation processing

tysonnorris commented on a change in pull request #2795: enable concurrent activation processing URL: https://github.com/apache/incubator-openwhisk/pull/2795#discussion_r193924904 ## File path: core/invoker/src/main/scala/whisk/core/containerpool/ContainerProxy.scala ##

[GitHub] codecov-io commented on issue #3704: Provide graceful shutdown for invoker

codecov-io commented on issue #3704: Provide graceful shutdown for invoker URL: https://github.com/apache/incubator-openwhisk/pull/3704#issuecomment-391821150 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3704?src=pr=h1) Report > Merging

[GitHub] remore commented on a change in pull request #3725: Add Ruby2.5 runtime support

remore commented on a change in pull request #3725: Add Ruby2.5 runtime support URL: https://github.com/apache/incubator-openwhisk/pull/3725#discussion_r193923095 ## File path: docs/parameters.md ## @@ -25,7 +25,7 @@ This page outlines how to configure parameters when

[GitHub] codecov-io commented on issue #2795: enable concurrent activation processing

codecov-io commented on issue #2795: enable concurrent activation processing URL: https://github.com/apache/incubator-openwhisk/pull/2795#issuecomment-386889684 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/2795?src=pr=h1) Report > Merging

[GitHub] tysonnorris commented on a change in pull request #3707: enable concurrent runs on ActionContainer test utility

tysonnorris commented on a change in pull request #3707: enable concurrent runs on ActionContainer test utility URL: https://github.com/apache/incubator-openwhisk/pull/3707#discussion_r193920614 ## File path: tests/src/test/scala/actionContainers/ActionContainer.scala ##

[GitHub] akharitonov opened a new issue #3733: Distributed deployment: export | unreachable

akharitonov opened a new issue #3733: Distributed deployment: export | unreachable URL: https://github.com/apache/incubator-openwhisk/issues/3733 I'm attempting to deploy a distributed OpenWhisk according to [this

[GitHub] rabbah commented on a change in pull request #3707: enable concurrent runs on ActionContainer test utility

rabbah commented on a change in pull request #3707: enable concurrent runs on ActionContainer test utility URL: https://github.com/apache/incubator-openwhisk/pull/3707#discussion_r193919343 ## File path: tests/src/test/scala/actionContainers/ActionContainer.scala ## @@

[GitHub] tysonnorris commented on a change in pull request #3707: enable concurrent runs on ActionContainer test utility

tysonnorris commented on a change in pull request #3707: enable concurrent runs on ActionContainer test utility URL: https://github.com/apache/incubator-openwhisk/pull/3707#discussion_r193897689 ## File path: tests/src/test/scala/actionContainers/ActionContainer.scala ##

[GitHub] tysonnorris commented on a change in pull request #3707: enable concurrent runs on ActionContainer test utility

tysonnorris commented on a change in pull request #3707: enable concurrent runs on ActionContainer test utility URL: https://github.com/apache/incubator-openwhisk/pull/3707#discussion_r193916422 ## File path: tests/src/test/scala/actionContainers/ActionContainer.scala ##

[GitHub] dubee closed pull request #3732: Switch to use adoptopenjdk/openjdk8:jdk8u172-b11-alpine for scala

dubee closed pull request #3732: Switch to use adoptopenjdk/openjdk8:jdk8u172-b11-alpine for scala URL: https://github.com/apache/incubator-openwhisk/pull/3732 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

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

codecov-io commented on issue #3467: Remove actionControllers tests from incubator-openwhisk repo URL: https://github.com/apache/incubator-openwhisk/pull/3467#issuecomment-386808429 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3467?src=pr=h1) Report > Merging

[GitHub] apercova opened a new pull request #132: Fix to fallback when LOCAL_IP cannot be obtained

apercova opened a new pull request #132: Fix to fallback when LOCAL_IP cannot be obtained URL: https://github.com/apache/incubator-openwhisk-devtools/pull/132 I've tried to do make quick-start from CentOS 7 I noticed that LOCAL_IP cannot be obtained within CentOS 7 the way Makefile

[GitHub] rabbah commented on a change in pull request #3507: allow use of string for controller id

rabbah commented on a change in pull request #3507: allow use of string for controller id URL: https://github.com/apache/incubator-openwhisk/pull/3507#discussion_r193903326 ## File path: common/scala/src/main/scala/whisk/core/entity/InvokerInstanceId.scala ## @@ -0,0

[GitHub] lornajane commented on issue #108: make quick-start fails when DOCKER_HOST isn't set

lornajane commented on issue #108: make quick-start fails when DOCKER_HOST isn't set URL: https://github.com/apache/incubator-openwhisk-devtools/issues/108#issuecomment-395583513 Finally got time to retry with my linux machine and I had a similar but different problem. I now need to set

[GitHub] lornajane commented on issue #129: make quick-start hangs on setup phase

lornajane commented on issue #129: make quick-start hangs on setup phase URL: https://github.com/apache/incubator-openwhisk-devtools/issues/129#issuecomment-395582959 I have just had a similar problem on Linux and fixed it with `export DOCKER_HOST_IP=172.17.0.1` - maybe this will help you

[GitHub] rabbah commented on issue #3507: allow use of string for controller id

rabbah commented on issue #3507: allow use of string for controller id URL: https://github.com/apache/incubator-openwhisk/pull/3507#issuecomment-395581898 should get a PG. This is an automated message from the Apache Git

[GitHub] rabbah commented on a change in pull request #3507: allow use of string for controller id

rabbah commented on a change in pull request #3507: allow use of string for controller id URL: https://github.com/apache/incubator-openwhisk/pull/3507#discussion_r193903226 ## File path: common/scala/src/main/scala/whisk/core/entity/InvokerInstanceId.scala ## @@ -0,0

[GitHub] rabbah commented on a change in pull request #3507: allow use of string for controller id

rabbah commented on a change in pull request #3507: allow use of string for controller id URL: https://github.com/apache/incubator-openwhisk/pull/3507#discussion_r193903495 ## File path: core/controller/src/main/scala/whisk/core/controller/Controller.scala ## @@ -202,7

[GitHub] rabbah commented on a change in pull request #3507: allow use of string for controller id

rabbah commented on a change in pull request #3507: allow use of string for controller id URL: https://github.com/apache/incubator-openwhisk/pull/3507#discussion_r193903178 ## File path: common/scala/src/main/scala/whisk/core/entity/InvokerInstanceId.scala ## @@ -0,0

[GitHub] rabbah commented on a change in pull request #3507: allow use of string for controller id

rabbah commented on a change in pull request #3507: allow use of string for controller id URL: https://github.com/apache/incubator-openwhisk/pull/3507#discussion_r193903004 ## File path: common/scala/src/main/scala/whisk/core/entity/InvokerInstanceId.scala ## @@ -0,0

[GitHub] rabbah commented on a change in pull request #3507: allow use of string for controller id

rabbah commented on a change in pull request #3507: allow use of string for controller id URL: https://github.com/apache/incubator-openwhisk/pull/3507#discussion_r193252805 ## File path: tests/src/test/scala/whisk/core/controller/test/WebActionsApiTests.scala ## @@

[GitHub] rabbah commented on a change in pull request #3507: allow use of string for controller id

rabbah commented on a change in pull request #3507: allow use of string for controller id URL: https://github.com/apache/incubator-openwhisk/pull/3507#discussion_r193903815 ## File path: tests/src/test/scala/whisk/core/entity/test/ControllerInstanceIdTests.scala ## @@

[GitHub] rabbah commented on a change in pull request #3507: allow use of string for controller id

rabbah commented on a change in pull request #3507: allow use of string for controller id URL: https://github.com/apache/incubator-openwhisk/pull/3507#discussion_r193903140 ## File path: common/scala/src/main/scala/whisk/core/entity/InvokerInstanceId.scala ## @@ -0,0

[GitHub] csantanapr commented on issue #3732: Switch to use adoptopenjdk/openjdk8:jdk8u172-b11-alpine for scala

csantanapr commented on issue #3732: Switch to use adoptopenjdk/openjdk8:jdk8u172-b11-alpine for scala URL: https://github.com/apache/incubator-openwhisk/pull/3732#issuecomment-395581479 @tysonnorris @houshengbo was able to find an `adoptopenjdk` that is hotspot and not openj9. So

[GitHub] rabbah commented on a change in pull request #3507: allow use of string for controller id

rabbah commented on a change in pull request #3507: allow use of string for controller id URL: https://github.com/apache/incubator-openwhisk/pull/3507#discussion_r193902709 ## File path: common/scala/src/main/scala/whisk/core/entity/InvokerInstanceId.scala ## @@ -0,0

[GitHub] rabbah opened a new pull request #315: Introduce some utility methods in triggers.go to reduce copy/paste

rabbah opened a new pull request #315: Introduce some utility methods in triggers.go to reduce copy/paste URL: https://github.com/apache/incubator-openwhisk-cli/pull/315 Add a method for working with parameters and annotations, and creating the feed life cycle event. This is intended

[GitHub] rabbah opened a new issue #314: refactor CLI to reduce bloat

rabbah opened a new issue #314: refactor CLI to reduce bloat URL: https://github.com/apache/incubator-openwhisk-cli/issues/314 The CLI code is quite bloated with a lot of copy/paste, little sharing of code for CRUD operations, redundancy between CREATE and UPDATE, and unnecessary JSON

[GitHub] rabbah opened a new issue #313: cannot wsk trigger update

rabbah opened a new issue #313: cannot wsk trigger update URL: https://github.com/apache/incubator-openwhisk-cli/issues/313 `wsk trigger update x` for any new trigger `x` will fail because the code tries to fetch the trigger first and exists if it's not found. This is not consistent with

[GitHub] mdeuser commented on a change in pull request #3694: Add retries to CLI test framework for network errors

mdeuser commented on a change in pull request #3694: Add retries to CLI test framework for network errors URL: https://github.com/apache/incubator-openwhisk/pull/3694#discussion_r193895451 ## File path: tests/src/test/scala/common/Wsk.scala ## @@ -1036,13 +1036,28 @@

[GitHub] tysonnorris commented on a change in pull request #3707: enable concurrent runs on ActionContainer test utility

tysonnorris commented on a change in pull request #3707: enable concurrent runs on ActionContainer test utility URL: https://github.com/apache/incubator-openwhisk/pull/3707#discussion_r193895111 ## File path: common/scala/src/main/scala/whisk/core/containerpool/HttpUtils.scala

[GitHub] dgrove-oss opened a new pull request #214: Add Notice file

dgrove-oss opened a new pull request #214: Add Notice file URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/214 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] tysonnorris commented on issue #3732: Switch to use adoptopenjdk/openjdk8:jdk8u172-b11-alpine for scala

tysonnorris commented on issue #3732: Switch to use adoptopenjdk/openjdk8:jdk8u172-b11-alpine for scala URL: https://github.com/apache/incubator-openwhisk/pull/3732#issuecomment-395570940 @dubee fine by me - I only left as `openjdk` instead of `adoptopenjdk` per carlos request in #3504

[GitHub] tysonnorris commented on issue #3507: allow use of string for controller id

tysonnorris commented on issue #3507: allow use of string for controller id URL: https://github.com/apache/incubator-openwhisk/pull/3507#issuecomment-395568367 classes renamed, let me know if you have other comments? This is

[GitHub] mrutkows closed issue #931: schema encoding of the runtime manifest schema uses uint16

mrutkows closed issue #931: schema encoding of the runtime manifest schema uses uint16 URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/931 This is an automated message from the Apache Git Service. To

[GitHub] mrutkows closed issue #930: wskdeploy should fail if host info does not parse or cannot be retrieved

mrutkows closed issue #930: wskdeploy should fail if host info does not parse or cannot be retrieved URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/930 This is an automated message from the Apache Git

[GitHub] mrutkows closed issue #929: if manifest read fails, wskdeploy should not read details from hardcoded manifest

mrutkows closed issue #929: if manifest read fails, wskdeploy should not read details from hardcoded manifest URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/929 This is an automated message from the

[GitHub] mrutkows closed issue #928: hardcoded protocol does not allow for http to be used

mrutkows closed issue #928: hardcoded protocol does not allow for http to be used URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/928 This is an automated message from the Apache Git Service. To respond

[GitHub] mrutkows closed issue #927: action deployed when runtime is deprecated

mrutkows closed issue #927: action deployed when runtime is deprecated URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/927 This is an automated message from the Apache Git Service. To respond to the

[GitHub] dubee commented on a change in pull request #3704: Provide graceful shutdown for invoker

dubee commented on a change in pull request #3704: Provide graceful shutdown for invoker URL: https://github.com/apache/incubator-openwhisk/pull/3704#discussion_r193886385 ## File path: common/scala/src/main/scala/whisk/core/connector/MessageConsumer.scala ## @@ -163,6

[GitHub] dubee commented on a change in pull request #3704: Provide graceful shutdown for invoker

dubee commented on a change in pull request #3704: Provide graceful shutdown for invoker URL: https://github.com/apache/incubator-openwhisk/pull/3704#discussion_r193886385 ## File path: common/scala/src/main/scala/whisk/core/connector/MessageConsumer.scala ## @@ -163,6

[GitHub] dgrove-oss closed pull request #213: Add the DISCLAIMER file for Apache incubator project

dgrove-oss closed pull request #213: Add the DISCLAIMER file for Apache incubator project URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/213 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] maximann opened a new pull request #270: Fix binary encoding

maximann opened a new pull request #270: Fix binary encoding URL: https://github.com/apache/incubator-openwhisk-package-kafka/pull/270 This addresses #269 Note: I'm not a python coder, so I'm sure there might be better ways to achieve this. When encoding binary data (ascii

[GitHub] maximann commented on issue #269: Binary Encoding causes Corruption

maximann commented on issue #269: Binary Encoding causes Corruption URL: https://github.com/apache/incubator-openwhisk-package-kafka/issues/269#issuecomment-395519353 I will submit a PR shortly. This is an automated message

[GitHub] codecov-io commented on issue #3694: Add retries to CLI test framework for network errors

codecov-io commented on issue #3694: Add retries to CLI test framework for network errors URL: https://github.com/apache/incubator-openwhisk/pull/3694#issuecomment-391475522 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3694?src=pr=h1) Report > Merging

[GitHub] codecov-io commented on issue #3729: Add brief Kubernetes Setup section to main README.md

codecov-io commented on issue #3729: Add brief Kubernetes Setup section to main README.md URL: https://github.com/apache/incubator-openwhisk/pull/3729#issuecomment-395538299 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3729?src=pr=h1) Report > Merging

[GitHub] rabbah commented on a change in pull request #3467: Remove actionControllers tests from incubator-openwhisk repo

rabbah commented on a change in pull request #3467: Remove actionControllers tests from incubator-openwhisk repo URL: https://github.com/apache/incubator-openwhisk/pull/3467#discussion_r193863012 ## File path: tests/src/test/scala/actionContainers/ActionContainer.scala ##

[GitHub] codecov-io commented on issue #3704: Provide graceful shutdown for invoker

codecov-io commented on issue #3704: Provide graceful shutdown for invoker URL: https://github.com/apache/incubator-openwhisk/pull/3704#issuecomment-391821150 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3704?src=pr=h1) Report > Merging

[GitHub] dubee closed pull request #3731: Relax connection retry in invoker

dubee closed pull request #3731: Relax connection retry in invoker URL: https://github.com/apache/incubator-openwhisk/pull/3731 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 commented on a change in pull request #3467: Remove actionControllers tests from incubator-openwhisk repo

csantanapr commented on a change in pull request #3467: Remove actionControllers tests from incubator-openwhisk repo URL: https://github.com/apache/incubator-openwhisk/pull/3467#discussion_r193860177 ## File path: tests/src/test/scala/actionContainers/ActionContainer.scala

[GitHub] codecov-io commented on issue #3732: Switch to use adoptopenjdk/openjdk8:jdk8u172-b11-alpine for scala

codecov-io commented on issue #3732: Switch to use adoptopenjdk/openjdk8:jdk8u172-b11-alpine for scala URL: https://github.com/apache/incubator-openwhisk/pull/3732#issuecomment-395532604 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3732?src=pr=h1) Report >

[GitHub] dubee commented on a change in pull request #3694: Add retries to CLI test framework for network errors

dubee commented on a change in pull request #3694: Add retries to CLI test framework for network errors URL: https://github.com/apache/incubator-openwhisk/pull/3694#discussion_r193858792 ## File path: tests/src/test/scala/common/Wsk.scala ## @@ -1036,13 +1036,28 @@ trait

[GitHub] csantanapr commented on issue #3709: Inlined attachments

csantanapr commented on issue #3709: Inlined attachments URL: https://github.com/apache/incubator-openwhisk/pull/3709#issuecomment-395531440 PG1 2996 ⌛️ This is an automated message from the Apache Git Service. To respond

[GitHub] codecov-io commented on issue #3731: Relax connection retry in invoker

codecov-io commented on issue #3731: Relax connection retry in invoker URL: https://github.com/apache/incubator-openwhisk/pull/3731#issuecomment-395528400 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3731?src=pr=h1) Report > Merging

[GitHub] rabbah commented on a change in pull request #3704: Provide graceful shutdown for invoker

rabbah commented on a change in pull request #3704: Provide graceful shutdown for invoker URL: https://github.com/apache/incubator-openwhisk/pull/3704#discussion_r193853597 ## File path: common/scala/src/main/scala/whisk/core/connector/MessageConsumer.scala ## @@ -163,6

[GitHub] rabbah commented on issue #269: Binary Encoding causes Corruption

rabbah commented on issue #269: Binary Encoding causes Corruption URL: https://github.com/apache/incubator-openwhisk-package-kafka/issues/269#issuecomment-395519569 @maximann fork the repo, then you can create branches in your own fork, and submit a pull request.

[GitHub] maximann commented on issue #269: Binary Encoding causes Corruption

maximann commented on issue #269: Binary Encoding causes Corruption URL: https://github.com/apache/incubator-openwhisk-package-kafka/issues/269#issuecomment-395519353 I don't have permissions to create a branch so I'm attaching a patch here.

[GitHub] rabbah commented on issue #3731: Relax connection retry in invoker

rabbah commented on issue #3731: Relax connection retry in invoker URL: https://github.com/apache/incubator-openwhisk/pull/3731#issuecomment-395516765 travis not enough? This is an automated message from the Apache Git

[GitHub] rabbah commented on issue #3731: Relax connection retry in invoker

rabbah commented on issue #3731: Relax connection retry in invoker URL: https://github.com/apache/incubator-openwhisk/pull/3731#issuecomment-395516765 travis not enugh? This is an automated message from the Apache Git

[GitHub] sven-lange-last commented on issue #3731: Relax connection retry in invoker

sven-lange-last commented on issue #3731: Relax connection retry in invoker URL: https://github.com/apache/incubator-openwhisk/pull/3731#issuecomment-395515577 PG 1 / 2995  This is an automated message from the Apache Git

[GitHub] houshengbo opened a new pull request #3732: Try another openjdk to see it fits

houshengbo opened a new pull request #3732: Try another openjdk to see it fits URL: https://github.com/apache/incubator-openwhisk/pull/3732 ## Description ## Related issue and scope - [ ] I opened an issue to propose and discuss this change (#) ## My

[GitHub] maximann opened a new issue #269: Binary Encoding causes Corruption

maximann opened a new issue #269: Binary Encoding causes Corruption URL: https://github.com/apache/incubator-openwhisk-package-kafka/issues/269 It looks to me like

[GitHub] mdeuser commented on a change in pull request #3694: Add retries to CLI test framework for network errors

mdeuser commented on a change in pull request #3694: Add retries to CLI test framework for network errors URL: https://github.com/apache/incubator-openwhisk/pull/3694#discussion_r193823978 ## File path: tests/src/test/scala/common/Wsk.scala ## @@ -1036,13 +1036,28 @@

[GitHub] sven-lange-last opened a new pull request #3731: Relax connection retry in invoker

sven-lange-last opened a new pull request #3731: Relax connection retry in invoker URL: https://github.com/apache/incubator-openwhisk/pull/3731 ## Description * In the past, sleep time between connection retries has been 100 ms. This has been lowered to 10 ms with #3710. On a

[GitHub] idibidiart opened a new issue #3730: Is latest NodeJS 10.2.x supported as a runtime?

idibidiart opened a new issue #3730: Is latest NodeJS 10.2.x supported as a runtime? URL: https://github.com/apache/incubator-openwhisk/issues/3730 Is latest NodeJS 10.2.x supported as a runtime? This is an automated message

[GitHub] dgrove-oss opened a new pull request #213: Add the DISCLAIMER file for Apache incubator project

dgrove-oss opened a new pull request #213: Add the DISCLAIMER file for Apache incubator project URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/213 This is an automated message from the Apache Git

[GitHub] pritidesai commented on issue #724: API Gateway - Add support to specify response type in manifest file

pritidesai commented on issue #724: API Gateway - Add support to specify response type in manifest file URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/724#issuecomment-395464144 yup definitely, please refer to wip PR #951, expecting it to be completed by this week.

[GitHub] akrabat commented on issue #11: Set up PHP 7.1 runtime for multiarch build

akrabat commented on issue #11: Set up PHP 7.1 runtime for multiarch build URL: https://github.com/apache/incubator-openwhisk-runtime-php/pull/11#issuecomment-395458557 What does using Jesse do to the size of the image and does that have implications for cold-start time?

[GitHub] rabbah commented on issue #27: Update to PHP 7.1.18, Uuid 3.7.3 & Guzzle 6.3.3

rabbah commented on issue #27: Update to PHP 7.1.18, Uuid 3.7.3 & Guzzle 6.3.3 URL: https://github.com/apache/incubator-openwhisk-runtime-php/pull/27#issuecomment-395455618 not necessary should be able to rebase and merge as there are no conflicts.

[GitHub] akrabat commented on issue #27: Update to PHP 7.1.18, Uuid 3.7.3 & Guzzle 6.3.3

akrabat commented on issue #27: Update to PHP 7.1.18, Uuid 3.7.3 & Guzzle 6.3.3 URL: https://github.com/apache/incubator-openwhisk-runtime-php/pull/27#issuecomment-395455325 Fixed that version number @rabbah. Do you want me to rebase it into the previous commit?

[GitHub] akrabat commented on a change in pull request #27: Update to PHP 7.1.18, Uuid 3.7.3 & Guzzle 6.3.3

akrabat commented on a change in pull request #27: Update to PHP 7.1.18, Uuid 3.7.3 & Guzzle 6.3.3 URL: https://github.com/apache/incubator-openwhisk-runtime-php/pull/27#discussion_r193781999 ## File path: core/php7.1Action/CHANGELOG.md ## @@ -0,0 +1,44 @@ + + +## 1.0.1

[GitHub] codecov-io commented on issue #3507: allow use of string for controller id

codecov-io commented on issue #3507: allow use of string for controller id URL: https://github.com/apache/incubator-openwhisk/pull/3507#issuecomment-390280747 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3507?src=pr=h1) Report > Merging

[GitHub] codecov-io commented on issue #3507: allow use of string for controller id

codecov-io commented on issue #3507: allow use of string for controller id URL: https://github.com/apache/incubator-openwhisk/pull/3507#issuecomment-390280747 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3507?src=pr=h1) Report > Merging

[GitHub] dgrove-oss opened a new pull request #3729: Add brief Kubernetes Setup section to main README.md

dgrove-oss opened a new pull request #3729: Add brief Kubernetes Setup section to main README.md URL: https://github.com/apache/incubator-openwhisk/pull/3729 ## Description With the recent improvements to the Helm-based deployment of OpenWhisk on Kubernetes, I

[GitHub] rabbah closed pull request #26: Add the DISCLAIMER file for Apache incubator project

rabbah closed pull request #26: Add the DISCLAIMER file for Apache incubator project URL: https://github.com/apache/incubator-openwhisk-runtime-php/pull/26 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] rabbah commented on a change in pull request #27: Update to PHP 7.1.18, Uuid 3.7.3 & Guzzle 6.3.3

rabbah commented on a change in pull request #27: Update to PHP 7.1.18, Uuid 3.7.3 & Guzzle 6.3.3 URL: https://github.com/apache/incubator-openwhisk-runtime-php/pull/27#discussion_r193777978 ## File path: core/php7.1Action/CHANGELOG.md ## @@ -0,0 +1,44 @@ + + +## 1.0.1 +

[GitHub] rabbah commented on a change in pull request #27: Update to PHP 7.1.18, Uuid 3.7.3 & Guzzle 6.3.3

rabbah commented on a change in pull request #27: Update to PHP 7.1.18, Uuid 3.7.3 & Guzzle 6.3.3 URL: https://github.com/apache/incubator-openwhisk-runtime-php/pull/27#discussion_r193777696 ## File path: core/php7.1Action/CHANGELOG.md ## @@ -0,0 +1,44 @@ + + +## 1.0.1 +

  1   2   >