[GitHub] [openwhisk] chetanmeh commented on a change in pull request #4724: Implement an ElasticSearchActivationStore

chetanmeh commented on a change in pull request #4724: Implement an ElasticSearchActivationStore URL: https://github.com/apache/openwhisk/pull/4724#discussion_r346159261 ## File path: tools/travis/setupPrereq.sh ## @@ -32,5 +32,13 @@ $ANSIBLE_CMD couchdb.yml $ANSIBLE_CMD

[GitHub] [openwhisk] jiangpengcheng commented on a change in pull request #4724: Implement an ElasticSearchActivationStore

jiangpengcheng commented on a change in pull request #4724: Implement an ElasticSearchActivationStore URL: https://github.com/apache/openwhisk/pull/4724#discussion_r346149314 ## File path: common/scala/src/main/scala/org/apache/openwhisk/core/database/elasticsearch/ElasticSearchAct

[GitHub] [openwhisk] jiangpengcheng commented on a change in pull request #4724: Implement an ElasticSearchActivationStore

jiangpengcheng commented on a change in pull request #4724: Implement an ElasticSearchActivationStore URL: https://github.com/apache/openwhisk/pull/4724#discussion_r346147098 ## File path: common/scala/src/main/scala/org/apache/openwhisk/core/database/elasticsearch/ElasticSearchAct

[GitHub] [openwhisk] jiangpengcheng commented on a change in pull request #4724: Implement an ElasticSearchActivationStore

jiangpengcheng commented on a change in pull request #4724: Implement an ElasticSearchActivationStore URL: https://github.com/apache/openwhisk/pull/4724#discussion_r346146586 ## File path: tools/travis/setupPrereq.sh ## @@ -32,5 +32,13 @@ $ANSIBLE_CMD couchdb.yml $ANSIBLE

[GitHub] [openwhisk] jiangpengcheng commented on a change in pull request #4724: Implement an ElasticSearchActivationStore

jiangpengcheng commented on a change in pull request #4724: Implement an ElasticSearchActivationStore URL: https://github.com/apache/openwhisk/pull/4724#discussion_r346145994 ## File path: tests/src/test/scala/org/apache/openwhisk/core/database/elasticsearch/ElasticSearchActivation

[GitHub] [openwhisk] jiangpengcheng commented on a change in pull request #4724: Implement an ElasticSearchActivationStore

jiangpengcheng commented on a change in pull request #4724: Implement an ElasticSearchActivationStore URL: https://github.com/apache/openwhisk/pull/4724#discussion_r346145994 ## File path: tests/src/test/scala/org/apache/openwhisk/core/database/elasticsearch/ElasticSearchActivation

[GitHub] [openwhisk] style95 commented on a change in pull request #4724: Implement an ElasticSearchActivationStore

style95 commented on a change in pull request #4724: Implement an ElasticSearchActivationStore URL: https://github.com/apache/openwhisk/pull/4724#discussion_r346143396 ## File path: ansible/elasticsearch.yml ## @@ -0,0 +1,20 @@ +# Licensed to the Apache Software Foundation

[GitHub] [openwhisk] style95 commented on a change in pull request #4724: Implement an ElasticSearchActivationStore

style95 commented on a change in pull request #4724: Implement an ElasticSearchActivationStore URL: https://github.com/apache/openwhisk/pull/4724#discussion_r346142148 ## File path: ansible/group_vars/all ## @@ -271,6 +271,27 @@ db: invoker: user: "{{ db_invoke

[GitHub] [openwhisk] style95 commented on issue #4724: Implement an ElasticSearchActivationStore

style95 commented on issue #4724: Implement an ElasticSearchActivationStore URL: https://github.com/apache/openwhisk/pull/4724#issuecomment-553729333 FTR, with this change, we can take advantage of ElasticSearch. So we can do full-text searching against Activation including logs, drawing

[GitHub] [openwhisk] jiangpengcheng opened a new pull request #4724: Implement an ElasticSearchActivationStore

jiangpengcheng opened a new pull request #4724: Implement an ElasticSearchActivationStore URL: https://github.com/apache/openwhisk/pull/4724 Implement an ElasticSearchActivationStore ## Description Although currently we can use `ArtifactWithFileStorageActivationStore` to store ac

[GitHub] [openwhisk] tysonnorris commented on issue #4705: SPI for ContainerProxy

tysonnorris commented on issue #4705: SPI for ContainerProxy URL: https://github.com/apache/openwhisk/issues/4705#issuecomment-553709624 Hi - Can you elaborate why this should not be an aspect of the runtime image? ContainerProxy manages the state only from perspective of the ContainerP

[GitHub] [openwhisk-deploy-kube] BillZong commented on issue #545: No invokers available

BillZong commented on issue #545: No invokers available URL: https://github.com/apache/openwhisk-deploy-kube/issues/545#issuecomment-553701166 That's weird. I got DNS resolved as `10.xx...` too. What about other services instead of OpenWhisk in your K8S cluster?

[GitHub] [openwhisk-deploy-kube] starsmall-xiaoqq edited a comment on issue #545: No invokers available

starsmall-xiaoqq edited a comment on issue #545: No invokers available URL: https://github.com/apache/openwhisk-deploy-kube/issues/545#issuecomment-553696516 I have traced the problem further by checking logs of each pod. - invoker pod:i checked the logs of invoker, they are norm

[GitHub] [openwhisk-deploy-kube] starsmall-xiaoqq edited a comment on issue #545: No invokers available

starsmall-xiaoqq edited a comment on issue #545: No invokers available URL: https://github.com/apache/openwhisk-deploy-kube/issues/545#issuecomment-553696516 I have traced the problem further by checking logs of each pod. - invoker pod:i checked the logs of invoker, they are norm

[GitHub] [openwhisk-deploy-kube] starsmall-xiaoqq commented on issue #545: No invokers available

starsmall-xiaoqq commented on issue #545: No invokers available URL: https://github.com/apache/openwhisk-deploy-kube/issues/545#issuecomment-553696516 I have traced the problem further by checking logs of each pod. - invoker pod:i checked the logs of invoker, they are normal.

[GitHub] [openwhisk-deploy-kube] starsmall-xiaoqq edited a comment on issue #545: No invokers available

starsmall-xiaoqq edited a comment on issue #545: No invokers available URL: https://github.com/apache/openwhisk-deploy-kube/issues/545#issuecomment-553185632 Hi, dgrove-oss here is the output of kubectl describe pods. It is a little longthy. ・・ Tolerations: node.kubern

[GitHub] [openwhisk] codecov-io edited a comment on issue #4593: Change ContainerPool buffer processing

codecov-io edited a comment on issue #4593: Change ContainerPool buffer processing URL: https://github.com/apache/openwhisk/pull/4593#issuecomment-525526438 # [Codecov](https://codecov.io/gh/apache/openwhisk/pull/4593?src=pr&el=h1) Report > Merging [#4593](https://codecov.io/gh/apache

[GitHub] [openwhisk] upgle commented on issue #4646: Serialize `updated` value of entity document in response

upgle commented on issue #4646: Serialize `updated` value of entity document in response URL: https://github.com/apache/openwhisk/pull/4646#issuecomment-553686435 One test case has been broken by refactoring. I just fixed it.

[GitHub] [openwhisk-composer] tardieu merged pull request #53: Support setting all limits on conductor and composed actions

tardieu merged pull request #53: Support setting all limits on conductor and composed actions URL: https://github.com/apache/openwhisk-composer/pull/53 This is an automated message from the Apache Git Service. To respond to

[GitHub] [openwhisk-composer] tardieu closed issue #44: Support setting memory limits

tardieu closed issue #44: Support setting memory limits URL: https://github.com/apache/openwhisk-composer/issues/44 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [openwhisk-composer] tardieu opened a new pull request #54: Avoid loading uuid when not needed

tardieu opened a new pull request #54: Avoid loading uuid when not needed URL: https://github.com/apache/openwhisk-composer/pull/54 `uuid` is only used to synthesize a `barrierId`. This is an automated message from the Apache

[GitHub] [openwhisk-composer] tardieu commented on a change in pull request #53: Support setting all limits on conductor and composed actions

tardieu commented on a change in pull request #53: Support setting all limits on conductor and composed actions URL: https://github.com/apache/openwhisk-composer/pull/53#discussion_r346046393 ## File path: docs/COMMANDS.md ## @@ -103,7 +105,9 @@ definitions. More precisely

[GitHub] [openwhisk-composer] dgrove-oss commented on a change in pull request #53: Support setting all limits on conductor and composed actions

dgrove-oss commented on a change in pull request #53: Support setting all limits on conductor and composed actions URL: https://github.com/apache/openwhisk-composer/pull/53#discussion_r346010241 ## File path: docs/COMMANDS.md ## @@ -103,7 +105,9 @@ definitions. More precis

[GitHub] [openwhisk-composer] dgrove-oss merged pull request #52: Update COMMANDS.md

dgrove-oss merged pull request #52: Update COMMANDS.md URL: https://github.com/apache/openwhisk-composer/pull/52 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [openwhisk-composer] dgrove-oss closed issue #42: Typo in documentation of debug flag

dgrove-oss closed issue #42: Typo in documentation of debug flag URL: https://github.com/apache/openwhisk-composer/issues/42 This is an automated message from the Apache Git Service. To respond to the message, please log on t

[GitHub] [openwhisk-composer] dgrove-oss merged pull request #51: Only require redis in conductor code when running parallel constructs

dgrove-oss merged pull request #51: Only require redis in conductor code when running parallel constructs URL: https://github.com/apache/openwhisk-composer/pull/51 This is an automated message from the Apache Git Service. To

[GitHub] [openwhisk-composer] dgrove-oss closed issue #43: Optional Redis dependency

dgrove-oss closed issue #43: Optional Redis dependency URL: https://github.com/apache/openwhisk-composer/issues/43 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub a

[GitHub] [openwhisk-composer] tardieu opened a new pull request #53: Support setting all limits on conductor and composed actions

tardieu opened a new pull request #53: Support setting all limits on conductor and composed actions URL: https://github.com/apache/openwhisk-composer/pull/53 Fixes #44 This is an automated message from the Apache Git Service.

[GitHub] [openwhisk-composer] tardieu opened a new pull request #52: Update COMMANDS.md

tardieu opened a new pull request #52: Update COMMANDS.md URL: https://github.com/apache/openwhisk-composer/pull/52 Fixes #42 This is an automated message from the Apache Git Service. To respond to the message, please log on t

[GitHub] [openwhisk-composer] tardieu opened a new pull request #51: Only require redis in conductor code when running parallel constructs

tardieu opened a new pull request #51: Only require redis in conductor code when running parallel constructs URL: https://github.com/apache/openwhisk-composer/pull/51 Fixes #43 This is an automated message from the Apache Gi

[GitHub] [openwhisk-deploy-kube] dgrove-oss commented on issue #545: No invokers available

dgrove-oss commented on issue #545: No invokers available URL: https://github.com/apache/openwhisk-deploy-kube/issues/545#issuecomment-553560512 Thanks for posting the `kubectl describe pods` output. Looking at the invokers, we can see that the two init containers on the invoker pod

[GitHub] [openwhisk-wskdeploy] mrutkows edited a comment on issue #1078: WIP: Add the ability to specify api via swagger in manifest

mrutkows edited a comment on issue #1078: WIP: Add the ability to specify api via swagger in manifest URL: https://github.com/apache/openwhisk-wskdeploy/pull/1078#issuecomment-553534559 @pwplusnick we should add as a follow-on PR the i18n error/warning messages/strings specific to swagger

[GitHub] [openwhisk-wskdeploy] mrutkows commented on issue #1078: WIP: Add the ability to specify api via swagger in manifest

mrutkows commented on issue #1078: WIP: Add the ability to specify api via swagger in manifest URL: https://github.com/apache/openwhisk-wskdeploy/pull/1078#issuecomment-553534559 @we should add as a follow-on PR the i18n error/warning messages/strings specific to swagger (config) document

[GitHub] [openwhisk] tysonnorris commented on a change in pull request #4593: Change ContainerPool buffer processing

tysonnorris commented on a change in pull request #4593: Change ContainerPool buffer processing URL: https://github.com/apache/openwhisk/pull/4593#discussion_r345862956 ## File path: core/invoker/src/main/scala/org/apache/openwhisk/core/containerpool/ContainerProxy.scala

[GitHub] [openwhisk] tysonnorris merged pull request #4638: Remove legacy view code from subjects index.

tysonnorris merged pull request #4638: Remove legacy view code from subjects index. URL: https://github.com/apache/openwhisk/pull/4638 This is an automated message from the Apache Git Service. To respond to the message, plea

[GitHub] [openwhisk] tysonnorris commented on issue #4646: Serialize `updated` value of entity document in response

tysonnorris commented on issue #4646: Serialize `updated` value of entity document in response URL: https://github.com/apache/openwhisk/pull/4646#issuecomment-553468313 Looks like a failure in RulesApiTests This is an automat

[GitHub] [openwhisk] mcdan commented on issue #4706: Move to Java 11 for building and running

mcdan commented on issue #4706: Move to Java 11 for building and running URL: https://github.com/apache/openwhisk/pull/4706#issuecomment-553447170 @dubee - > Are all of the `JAVA_OPTS` in the Ansible deployment roles still valid with the newer version of Java? I looked there and n

[GitHub] [openwhisk] style95 opened a new pull request #4723: Add a config to run tests in parallel

style95 opened a new pull request #4723: Add a config to run tests in parallel URL: https://github.com/apache/openwhisk/pull/4723 test This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [openwhisk] style95 closed pull request #4723: Add a config to run tests in parallel

style95 closed pull request #4723: Add a config to run tests in parallel URL: https://github.com/apache/openwhisk/pull/4723 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [openwhisk-runtime-swift] sciabarracom removed a comment on issue #106: Swift51 support

sciabarracom removed a comment on issue #106: Swift51 support URL: https://github.com/apache/openwhisk-runtime-swift/pull/106#issuecomment-553411634 added in the swift 51 pr This is an automated message from the Apache Git Se

[GitHub] [openwhisk-runtime-swift] sciabarracom opened a new pull request #106: Swift51 support

sciabarracom opened a new pull request #106: Swift51 support URL: https://github.com/apache/openwhisk-runtime-swift/pull/106 This is an automated message from the Apache Git Service. To respond to the message, please log on t

[GitHub] [openwhisk-runtime-swift] sciabarracom closed pull request #106: Swift51 support

sciabarracom closed pull request #106: Swift51 support URL: https://github.com/apache/openwhisk-runtime-swift/pull/106 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitH

[GitHub] [openwhisk-runtime-swift] sciabarracom commented on issue #106: Swift51 support

sciabarracom commented on issue #106: Swift51 support URL: https://github.com/apache/openwhisk-runtime-swift/pull/106#issuecomment-553411634 added in the swift 51 pr This is an automated message from the Apache Git Service. T

[GitHub] [openwhisk] style95 commented on a change in pull request #4698: Add healthcheck for Invoker -> Action Container

style95 commented on a change in pull request #4698: Add healthcheck for Invoker -> Action Container URL: https://github.com/apache/openwhisk/pull/4698#discussion_r345633815 ## File path: core/invoker/src/main/scala/org/apache/openwhisk/core/containerpool/ContainerProxy.scala ###

[GitHub] [openwhisk] style95 commented on a change in pull request #4593: Change ContainerPool buffer processing

style95 commented on a change in pull request #4593: Change ContainerPool buffer processing URL: https://github.com/apache/openwhisk/pull/4593#discussion_r345627173 ## File path: core/invoker/src/main/scala/org/apache/openwhisk/core/containerpool/ContainerPool.scala ## @

[GitHub] [openwhisk] style95 commented on a change in pull request #4593: Change ContainerPool buffer processing

style95 commented on a change in pull request #4593: Change ContainerPool buffer processing URL: https://github.com/apache/openwhisk/pull/4593#discussion_r345616146 ## File path: core/invoker/src/main/scala/org/apache/openwhisk/core/containerpool/ContainerProxy.scala ##

[GitHub] [openwhisk] style95 commented on a change in pull request #4593: Change ContainerPool buffer processing

style95 commented on a change in pull request #4593: Change ContainerPool buffer processing URL: https://github.com/apache/openwhisk/pull/4593#discussion_r345625514 ## File path: core/invoker/src/main/scala/org/apache/openwhisk/core/containerpool/ContainerProxy.scala ##

[GitHub] [openwhisk] upgle commented on a change in pull request #4646: Serialize `updated` value of entity document in response

upgle commented on a change in pull request #4646: Serialize `updated` value of entity document in response URL: https://github.com/apache/openwhisk/pull/4646#discussion_r345624217 ## File path: common/scala/src/main/scala/org/apache/openwhisk/core/entity/WhiskEntity.scala ###