[GitHub] codecov-io edited a comment on issue #3760: Add API gateway to redo.

codecov-io edited a comment on issue #3760: Add API gateway to redo. URL: https://github.com/apache/incubator-openwhisk/pull/3760#issuecomment-397050317 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3760?src=pr&el=h1) Report > Merging [#3760](https://codecov.io/gh/

[GitHub] dgrove-oss opened a new pull request #256: reorganize ingress documentation to group by ingress instead of cloud…

dgrove-oss opened a new pull request #256: reorganize ingress documentation to group by ingress instead of cloud… URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/256 … provider. This is an automated messag

[GitHub] markusthoemmes commented on a change in pull request #3878: Validate the Controller Swagger spec matches the impl

markusthoemmes commented on a change in pull request #3878: Validate the Controller Swagger spec matches the impl URL: https://github.com/apache/incubator-openwhisk/pull/3878#discussion_r202484736 ## File path: tests/src/test/scala/common/rest/SwaggerValidator.scala ## @@

[GitHub] markusthoemmes commented on a change in pull request #3878: Validate the Controller Swagger spec matches the impl

markusthoemmes commented on a change in pull request #3878: Validate the Controller Swagger spec matches the impl URL: https://github.com/apache/incubator-openwhisk/pull/3878#discussion_r202479910 ## File path: tests/src/test/scala/common/rest/WskRestOperations.scala ## @

[GitHub] markusthoemmes commented on a change in pull request #3878: Validate the Controller Swagger spec matches the impl

markusthoemmes commented on a change in pull request #3878: Validate the Controller Swagger spec matches the impl URL: https://github.com/apache/incubator-openwhisk/pull/3878#discussion_r202484697 ## File path: tests/src/test/scala/common/rest/SwaggerValidator.scala ## @@

[GitHub] dgrove-oss closed pull request #255: Missing documentation about ingress in previous pull request

dgrove-oss closed pull request #255: Missing documentation about ingress in previous pull request URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/255 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] dgrove-oss commented on issue #255: Missing documentation about ingress in previous pull request

dgrove-oss commented on issue #255: Missing documentation about ingress in previous pull request URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/255#issuecomment-404964736 Thanks! This is an automated mes

[GitHub] codecov-io commented on issue #3878: Validate the Controller Swagger spec matches the impl

codecov-io commented on issue #3878: Validate the Controller Swagger spec matches the impl URL: https://github.com/apache/incubator-openwhisk/pull/3878#issuecomment-404963809 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3878?src=pr&el=h1) Report > Merging [#3878]

[GitHub] cduque89 opened a new pull request #255: Missing documentation about ingress in previous pull request

cduque89 opened a new pull request #255: Missing documentation about ingress in previous pull request URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/255 @dgrove-oss just realised I the docs/ingress.md updates in my previous pull requests explaining on how to configure

[GitHub] rabbah commented on issue #3878: Validate the Controller Swagger spec matches the impl

rabbah commented on issue #3878: Validate the Controller Swagger spec matches the impl URL: https://github.com/apache/incubator-openwhisk/pull/3878#issuecomment-404961949 This is great! This is an automated message from the

[GitHub] beemarie closed issue #21: Defect: Only delete the tmp directory, not preInstalled directory.

beemarie closed issue #21: Defect: Only delete the tmp directory, not preInstalled directory. URL: https://github.com/apache/incubator-openwhisk-package-deploy/issues/21 This is an automated message from the Apache Git Servi

[GitHub] bbrowning commented on issue #2466: [Feature request] OpenWhisk Java Client sdk

bbrowning commented on issue #2466: [Feature request] OpenWhisk Java Client sdk URL: https://github.com/apache/incubator-openwhisk/issues/2466#issuecomment-404959985 There isn't any official Java client yet, but as a followup to updating the Swagger spec in #3878 I'm hoping we can get an a

[GitHub] beemarie opened a new issue #30: Should also accept manifest.yml as a valid file name.

beemarie opened a new issue #30: Should also accept manifest.yml as a valid file name. URL: https://github.com/apache/incubator-openwhisk-package-deploy/issues/30 Currently only accepts manifest.yaml. This is an automated m

[GitHub] beemarie commented on issue #21: Defect: Only delete the tmp directory, not preInstalled directory.

beemarie commented on issue #21: Defect: Only delete the tmp directory, not preInstalled directory. URL: https://github.com/apache/incubator-openwhisk-package-deploy/issues/21#issuecomment-404958554 Closed by: https://github.com/apache/incubator-openwhisk-package-deploy/pull/20 -

[GitHub] bbrowning opened a new pull request #3878: Validate the Controller Swagger spec matches the impl

bbrowning opened a new pull request #3878: Validate the Controller Swagger spec matches the impl URL: https://github.com/apache/incubator-openwhisk/pull/3878 ## Description This change wires up all existing API tests that use WskRestOperations to validate the HTT

[GitHub] cduque89 commented on issue #254: Improvement in default chart values and make #251 work by just pulling the chart

cduque89 commented on issue #254: Improvement in default chart values and make #251 work by just pulling the chart URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/254#issuecomment-404954327 Cool hope this helps someone in the future 👍 -

[GitHub] dgrove-oss closed pull request #254: Improvement in default chart values and make #251 work by just pulling the chart

dgrove-oss closed pull request #254: Improvement in default chart values and make #251 work by just pulling the chart URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/254 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is dis

[GitHub] dgrove-oss commented on issue #253: No whisk-system Namespace after installation

dgrove-oss commented on issue #253: No whisk-system Namespace after installation URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/253#issuecomment-404941602 Thinking about how to narrow the problem. A couple of things to look at. 1. Did the installCatalog job succee

[GitHub] dgrove-oss commented on issue #254: Improvement in default chart values and make #251 work by just pulling the chart

dgrove-oss commented on issue #254: Improvement in default chart values and make #251 work by just pulling the chart URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/254#issuecomment-404929962 Thanks @cduque89, that's all we need. I will merge today. --

[GitHub] dgrove-oss closed pull request #3866: Fix invokerId assignment path in zookeeper

dgrove-oss closed pull request #3866: Fix invokerId assignment path in zookeeper URL: https://github.com/apache/incubator-openwhisk/pull/3866 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this

[GitHub] dgrove-oss closed issue #3856: Invoker id assignment path in zookeeper uses wrong id format

dgrove-oss closed issue #3856: Invoker id assignment path in zookeeper uses wrong id format URL: https://github.com/apache/incubator-openwhisk/issues/3856 This is an automated message from the Apache Git Service. To respond

[GitHub] dgrove-oss commented on issue #3866: Fix invokerId assignment path in zookeeper

dgrove-oss commented on issue #3866: Fix invokerId assignment path in zookeeper URL: https://github.com/apache/incubator-openwhisk/pull/3866#issuecomment-404913923 PG3 2526 👍 This is an automated message from the Apache Git

[GitHub] codecov-io edited a comment on issue #3866: Fix invokerId assignment path in zookeeper

codecov-io edited a comment on issue #3866: Fix invokerId assignment path in zookeeper URL: https://github.com/apache/incubator-openwhisk/pull/3866#issuecomment-403773688 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3866?src=pr&el=h1) Report > Merging [#3866](htt

[GitHub] dgrove-oss commented on issue #249: refactor of kube-deploy docker images

dgrove-oss commented on issue #249: refactor of kube-deploy docker images URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/249#issuecomment-404876350 @bwmcadams - would you be willing to review this & merge if it looks ok to you? Also wondering if the refactor to use ge

[GitHub] dgrove-oss commented on issue #3866: Fix invokerId assignment path in zookeeper

dgrove-oss commented on issue #3866: Fix invokerId assignment path in zookeeper URL: https://github.com/apache/incubator-openwhisk/pull/3866#issuecomment-404875690 PG3 2526 🏃 This is an automated message from the Apache Git

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

ddragosd 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-404875560 @lornajane is the `DOCKER_HOST_IP` issue still present ? --

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

ddragosd 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-404875339 @satwikk `docker-compose` is listed as a prereq in https://github.com/apache/incubator-openwhisk-devtools/blob/

[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-404874144 @edwardstudy these 2 errors shouldn't be related. Would you be able to share the Invoker [logs](ht

[GitHub] chetanmeh commented on issue #3866: Fix invokerId assignment path in zookeeper

chetanmeh commented on issue #3866: Fix invokerId assignment path in zookeeper URL: https://github.com/apache/incubator-openwhisk/pull/3866#issuecomment-404865939 Rebased to resolve conflicts This is an automated message from

[GitHub] markusthoemmes commented on issue #3866: Fix invokerId assignment path in zookeeper

markusthoemmes commented on issue #3866: Fix invokerId assignment path in zookeeper URL: https://github.com/apache/incubator-openwhisk/pull/3866#issuecomment-404859958 Sorry this conflicted with @vvraskin's change, which I merged this morning 😢 ---

[GitHub] dgrove-oss commented on issue #3866: Fix invokerId assignment path in zookeeper

dgrove-oss commented on issue #3866: Fix invokerId assignment path in zookeeper URL: https://github.com/apache/incubator-openwhisk/pull/3866#issuecomment-404855199 PG3 2525 🏃 This is an automated message from the Apache Git

[GitHub] rabbah commented on issue #3725: Add Ruby2.5 runtime support

rabbah commented on issue #3725: Add Ruby2.5 runtime support URL: https://github.com/apache/incubator-openwhisk/pull/3725#issuecomment-404821383 Repository requested: https://issues.apache.org/jira/browse/INFRA-16764 This is

[GitHub] codecov-io edited a comment on issue #3725: Add Ruby2.5 runtime support

codecov-io edited a comment on issue #3725: Add Ruby2.5 runtime support URL: https://github.com/apache/incubator-openwhisk/pull/3725#issuecomment-397487604 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3725?src=pr&el=h1) Report > Merging [#3725](https://codecov.io/

[GitHub] rabbah commented on a change in pull request #3871: Document OPENWHISK_TMP_DIR usage with Docker for Mac

rabbah commented on a change in pull request #3871: Document OPENWHISK_TMP_DIR usage with Docker for Mac URL: https://github.com/apache/incubator-openwhisk/pull/3871#discussion_r202327917 ## File path: ansible/README.md ## @@ -60,6 +60,15 @@ An expedient workaround is to

[GitHub] rabbah closed pull request #3874: Fix warnings in python scripts.

rabbah closed pull request #3874: Fix warnings in python scripts. URL: https://github.com/apache/incubator-openwhisk/pull/3874 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 p

[GitHub] rabbah commented on a change in pull request #3874: Fix warnings in python scripts.

rabbah commented on a change in pull request #3874: Fix warnings in python scripts. URL: https://github.com/apache/incubator-openwhisk/pull/3874#discussion_r202327369 ## File path: tools/build/redo ## @@ -278,12 +278,12 @@ Components = [ 'build a runtim

[GitHub] codecov-io edited a comment on issue #3725: Add Ruby2.5 runtime support

codecov-io edited a comment on issue #3725: Add Ruby2.5 runtime support URL: https://github.com/apache/incubator-openwhisk/pull/3725#issuecomment-397487604 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3725?src=pr&el=h1) Report > Merging [#3725](https://codecov.io/

[GitHub] markusthoemmes closed pull request #3877: Add missing license header for wrk lua file

markusthoemmes closed pull request #3877: Add missing license header for wrk lua file URL: https://github.com/apache/incubator-openwhisk/pull/3877 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: A

[GitHub] markusthoemmes commented on issue #3875: Throttle the system based on active-ack timeouts.

markusthoemmes commented on issue #3875: Throttle the system based on active-ack timeouts. URL: https://github.com/apache/incubator-openwhisk/pull/3875#issuecomment-404807967 @rabbah, in theory, yes! to 1.: If a network partition causes no active-acks, this network partition will a

[GitHub] mhenke1 opened a new pull request #49: Include wrk tests in scancode checks

mhenke1 opened a new pull request #49: Include wrk tests in scancode checks URL: https://github.com/apache/incubator-openwhisk-utilities/pull/49 Re-includes the wrk test directory after having added the correct license header to post.lua

[GitHub] remore commented on issue #3725: Add Ruby2.5 runtime support

remore commented on issue #3725: Add Ruby2.5 runtime support URL: https://github.com/apache/incubator-openwhisk/pull/3725#issuecomment-404807918 FYI: I have just squashed as well to make both git tree view and log messages cleaner. (I have just realized that my commit messages were not fo

[GitHub] remore commented on issue #3725: Add Ruby2.5 runtime support

remore commented on issue #3725: Add Ruby2.5 runtime support URL: https://github.com/apache/incubator-openwhisk/pull/3725#issuecomment-404805251 @rabbah the conflict has been fixed. I will take a look at those issues. This is

[GitHub] rabbah edited a comment on issue #3875: Throttle the system based on active-ack timeouts.

rabbah edited a comment on issue #3875: Throttle the system based on active-ack timeouts. URL: https://github.com/apache/incubator-openwhisk/pull/3875#issuecomment-404801453 I haven’t looked at the changes yet- based on the description: - is this heuristic affected by a network part

[GitHub] rabbah commented on issue #3875: Throttle the system based on active-ack timeouts.

rabbah commented on issue #3875: Throttle the system based on active-ack timeouts. URL: https://github.com/apache/incubator-openwhisk/pull/3875#issuecomment-404801453 I haven’t looked at the changes yet- based on the description: - is this heuristic affected by a network partition w

[GitHub] rabbah commented on issue #3725: Add Ruby2.5 runtime support

rabbah commented on issue #3725: Add Ruby2.5 runtime support URL: https://github.com/apache/incubator-openwhisk/pull/3725#issuecomment-404799846 @remore I opened 3 issues against the ruby repo that we’ll need to also address. --

[GitHub] rabbah commented on issue #3725: Add Ruby2.5 runtime support

rabbah commented on issue #3725: Add Ruby2.5 runtime support URL: https://github.com/apache/incubator-openwhisk/pull/3725#issuecomment-404797957 There some conflicts to resolve on the PR. This is an automated message from the

[GitHub] vvraskin commented on issue #3853: Set erlang magic cookie for couchdb

vvraskin commented on issue #3853: Set erlang magic cookie for couchdb URL: https://github.com/apache/incubator-openwhisk/pull/3853#issuecomment-404790918 Should the cookie generation task run for the first cluster node only? e.g. `when: (inventory_hostname == coordinator) and (db.instan

[GitHub] codecov-io edited a comment on issue #3829: Introduce SPI to swap authentication directives

codecov-io edited a comment on issue #3829: Introduce SPI to swap authentication directives URL: https://github.com/apache/incubator-openwhisk/pull/3829#issuecomment-402040451 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3829?src=pr&el=h1) Report > Merging [#3829

[GitHub] markusthoemmes closed pull request #3855: Add invoker container name to health protocol

markusthoemmes closed pull request #3855: Add invoker container name to health protocol URL: https://github.com/apache/incubator-openwhisk/pull/3855 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] codecov-io commented on issue #3877: Add missing license header for wrk lua file

codecov-io commented on issue #3877: Add missing license header for wrk lua file URL: https://github.com/apache/incubator-openwhisk/pull/3877#issuecomment-404778039 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3877?src=pr&el=h1) Report > Merging [#3877](https://co

[GitHub] codecov-io edited a comment on issue #3829: Introduce SPI to swap authentication directives

codecov-io edited a comment on issue #3829: Introduce SPI to swap authentication directives URL: https://github.com/apache/incubator-openwhisk/pull/3829#issuecomment-402040451 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3829?src=pr&el=h1) Report > Merging [#3829

[GitHub] mhenke1 opened a new pull request #3877: Add missing license header for wrk lua file

mhenke1 opened a new pull request #3877: Add missing license header for wrk lua file URL: https://github.com/apache/incubator-openwhisk/pull/3877 ## Description ## Related issue and scope - [ ] I opened an issue to propose and discuss this change (#)

[GitHub] remore commented on issue #3725: Add Ruby2.5 runtime support

remore commented on issue #3725: Add Ruby2.5 runtime support URL: https://github.com/apache/incubator-openwhisk/pull/3725#issuecomment-404754612 > are you able to do this small change and we can get it merged? Yes for sure, I have just made a few changes to get everything organized i

[GitHub] cduque89 commented on issue #254: Improvement in default chart values and make #251 work by just pulling the chart

cduque89 commented on issue #254: Improvement in default chart values and make #251 work by just pulling the chart URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/254#issuecomment-404751238 Hi @dgrove-oss just got the confirmation e-mail. The relevant part is:

[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_r202264038 ## File path: docs/reference.md ## @@ -434,6 +434,14 @@ The following Composer packages are also available: -

[GitHub] codecov-io commented on issue #3876: Use separate db users for deployed components.

codecov-io commented on issue #3876: Use separate db users for deployed components. URL: https://github.com/apache/incubator-openwhisk/pull/3876#issuecomment-404749269 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3876?src=pr&el=h1) Report > Merging [#3876](https:

[GitHub] remore opened a new pull request #352: Add new file extension tentatively for Ruby runtime

remore opened a new pull request #352: Add new file extension tentatively for Ruby runtime URL: https://github.com/apache/incubator-openwhisk-cli/pull/352 This PR will add add new file extension tentatively for ruby runtime. Hopefully in future this should be removed by #286 though.

[GitHub] vvraskin commented on issue #3855: Add invoker container name to health protocol

vvraskin commented on issue #3855: Add invoker container name to health protocol URL: https://github.com/apache/incubator-openwhisk/pull/3855#issuecomment-404746061 PG2/3369 is :large_blue_circle: This is an automated message