[GitHub] chetanmeh commented on issue #3562: ArtifactStore implementation for CosmosDB

chetanmeh commented on issue #3562: ArtifactStore implementation for CosmosDB URL: https://github.com/apache/incubator-openwhisk/pull/3562#issuecomment-383820178 ### Test suite run result In a [separate branch][1] travis is configured to run whole OW test suite as being run currentl

[GitHub] cbickel opened a new pull request #77: Handle error code (tid) not as int.

cbickel opened a new pull request #77: Handle error code (tid) not as int. URL: https://github.com/apache/incubator-openwhisk-client-go/pull/77 As the tid will not be handled as int in the future anymore, this PR changes this assumption in the CLI. Related PR in the core repository:

[GitHub] chetanmeh commented on issue #3562: ArtifactStore implementation for CosmosDB

chetanmeh commented on issue #3562: ArtifactStore implementation for CosmosDB URL: https://github.com/apache/incubator-openwhisk/pull/3562#issuecomment-383820178 ### Test suite run result In a [separate branch][1] travis is configured to run whole OW test suite as being run currentl

[GitHub] chetanmeh commented on issue #3562: ArtifactStore implementation for CosmosDB

chetanmeh commented on issue #3562: ArtifactStore implementation for CosmosDB URL: https://github.com/apache/incubator-openwhisk/pull/3562#issuecomment-383820178 ### Test suite run result In a [separate branch][1] travis is configured to run whole OW test suite as being run curr

[GitHub] pritidesai commented on issue #877: Adding annotation conversion

pritidesai commented on issue #877: Adding annotation conversion URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/877#issuecomment-383816446 Failing at `go lint`: https://github.com/golang/lint/issues/397 -

[GitHub] pritidesai commented on issue #869: Formatting annotations to include in HTTP request

pritidesai commented on issue #869: Formatting annotations to include in HTTP request URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/869#issuecomment-383815232 Opened a new PR #877 This is an automated m

[GitHub] pritidesai closed pull request #869: Formatting annotations to include in HTTP request

pritidesai closed pull request #869: Formatting annotations to include in HTTP request URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/869 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 pro

[GitHub] pritidesai opened a new pull request #877: Adding annotation conversion

pritidesai opened a new pull request #877: Adding annotation conversion URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/877 This is an automated message from the Apache Git Service. To respond to the message

[GitHub] markusthoemmes commented on a change in pull request #3568: Add activation result integration test

markusthoemmes commented on a change in pull request #3568: Add activation result integration test URL: https://github.com/apache/incubator-openwhisk/pull/3568#discussion_r183611254 ## File path: tests/src/test/scala/system/basic/WskActivationTests.scala ## @@ -54,4 +57,2

[GitHub] markusthoemmes commented on issue #3567: Activation result integration with log store SPI

markusthoemmes commented on issue #3567: Activation result integration with log store SPI URL: https://github.com/apache/incubator-openwhisk/pull/3567#issuecomment-38387 I'm a bit hesitant to make this part of the LogStore SPI. It's about storing **logs** after all. I know we've overl

[GitHub] markusthoemmes commented on a change in pull request #3567: Activation result integration with log store SPI

markusthoemmes commented on a change in pull request #3567: Activation result integration with log store SPI URL: https://github.com/apache/incubator-openwhisk/pull/3567#discussion_r183610694 ## File path: common/scala/src/main/scala/whisk/core/containerpool/logging/ElasticSearchL

[GitHub] markusthoemmes commented on a change in pull request #3567: Activation result integration with log store SPI

markusthoemmes commented on a change in pull request #3567: Activation result integration with log store SPI URL: https://github.com/apache/incubator-openwhisk/pull/3567#discussion_r183610694 ## File path: common/scala/src/main/scala/whisk/core/containerpool/logging/ElasticSearchL

[GitHub] codecov-io commented on issue #3249: #3243 - Use gradle scoverage to collect test coverage stats

codecov-io commented on issue #3249: #3243 - Use gradle scoverage to collect test coverage stats URL: https://github.com/apache/incubator-openwhisk/pull/3249#issuecomment-365147821 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3249?src=pr&el=h1) Report > :exclamati

[GitHub] chetanmeh commented on issue #3249: #3243 - Use gradle scoverage to collect test coverage stats

chetanmeh commented on issue #3249: #3243 - Use gradle scoverage to collect test coverage stats URL: https://github.com/apache/incubator-openwhisk/pull/3249#issuecomment-383798013 Again rebased to current master to resolve the merge error :sweat:. Hopefully we can get this long pending PR

[GitHub] dubee closed pull request #280: log test var refactor

dubee closed pull request #280: log test var refactor URL: https://github.com/apache/incubator-openwhisk-cli/pull/280 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 reque

[GitHub] drcariel opened a new pull request #280: log test var refactor

drcariel opened a new pull request #280: log test var refactor URL: https://github.com/apache/incubator-openwhisk-cli/pull/280 see https://github.com/apache/incubator-openwhisk/commit/b15f2dda5151d35b4ab2e9073570388159af991d

[GitHub] rabbah commented on issue #3568: Add activation result integration test

rabbah commented on issue #3568: Add activation result integration test URL: https://github.com/apache/incubator-openwhisk/pull/3568#issuecomment-383709098 Ok I had it wrong then. Sorry about that; the code is: (activation: WhiskActivation) => Future.successful(activation.respons

[GitHub] dubee commented on a change in pull request #3568: Add activation result integration test

dubee commented on a change in pull request #3568: Add activation result integration test URL: https://github.com/apache/incubator-openwhisk/pull/3568#discussion_r183523258 ## File path: tests/src/test/scala/system/basic/WskActivationTests.scala ## @@ -54,4 +57,24 @@ abst

[GitHub] dubee commented on a change in pull request #3568: Add activation result integration test

dubee commented on a change in pull request #3568: Add activation result integration test URL: https://github.com/apache/incubator-openwhisk/pull/3568#discussion_r183519909 ## File path: tests/src/test/scala/system/basic/WskActivationTests.scala ## @@ -54,4 +57,24 @@ abst

[GitHub] pritidesai closed pull request #26: Add exclusion for new incubator-openwhisk perf. tests.

pritidesai closed pull request #26: Add exclusion for new incubator-openwhisk perf. tests. URL: https://github.com/apache/incubator-openwhisk-utilities/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 p

[GitHub] mrutkows commented on issue #3569: WIP: Use scanCode from utilities repo. using ASF Release configuration.

mrutkows commented on issue #3569: WIP: Use scanCode from utilities repo. using ASF Release configuration. URL: https://github.com/apache/incubator-openwhisk/pull/3569#issuecomment-383700643 The following PR is a pre-req for this PR: https://github.com/apache/incubator-openwhisk-utiliti

[GitHub] rabbah commented on a change in pull request #3568: Add activation result integration test

rabbah commented on a change in pull request #3568: Add activation result integration test URL: https://github.com/apache/incubator-openwhisk/pull/3568#discussion_r183514157 ## File path: tests/src/test/scala/system/basic/WskActivationTests.scala ## @@ -54,4 +57,24 @@ abs

[GitHub] rabbah commented on a change in pull request #3568: Add activation result integration test

rabbah commented on a change in pull request #3568: Add activation result integration test URL: https://github.com/apache/incubator-openwhisk/pull/3568#discussion_r183514157 ## File path: tests/src/test/scala/system/basic/WskActivationTests.scala ## @@ -54,4 +57,24 @@ abs

[GitHub] mrutkows opened a new pull request #3569: WIP: Use scanCode from utilities repo. using ASF Release configuration.

mrutkows opened a new pull request #3569: WIP: Use scanCode from utilities repo. using ASF Release configuration. URL: https://github.com/apache/incubator-openwhisk/pull/3569 ## Description ## Related issue and scope - [ ] I opened an issue to propose and discu

[GitHub] dubee commented on a change in pull request #3568: Add activation result integration test

dubee commented on a change in pull request #3568: Add activation result integration test URL: https://github.com/apache/incubator-openwhisk/pull/3568#discussion_r183513869 ## File path: tests/src/test/scala/system/basic/WskActivationTests.scala ## @@ -54,4 +57,24 @@ abst

[GitHub] dubee commented on a change in pull request #3567: Activation result integration with log store SPI

dubee commented on a change in pull request #3567: Activation result integration with log store SPI URL: https://github.com/apache/incubator-openwhisk/pull/3567#discussion_r183509259 ## File path: common/scala/src/main/scala/whisk/core/containerpool/logging/ElasticSearchLogStore.s

[GitHub] dubee commented on a change in pull request #3567: Activation result integration with log store SPI

dubee commented on a change in pull request #3567: Activation result integration with log store SPI URL: https://github.com/apache/incubator-openwhisk/pull/3567#discussion_r183509259 ## File path: common/scala/src/main/scala/whisk/core/containerpool/logging/ElasticSearchLogStore.s

[GitHub] rabbah commented on a change in pull request #3568: Add activation result integration test

rabbah commented on a change in pull request #3568: Add activation result integration test URL: https://github.com/apache/incubator-openwhisk/pull/3568#discussion_r183508019 ## File path: tests/src/test/scala/system/basic/WskActivationTests.scala ## @@ -54,4 +57,24 @@ abs

[GitHub] rabbah commented on a change in pull request #3568: Add activation result integration test

rabbah commented on a change in pull request #3568: Add activation result integration test URL: https://github.com/apache/incubator-openwhisk/pull/3568#discussion_r183508251 ## File path: tests/src/test/scala/system/basic/WskActivationTests.scala ## @@ -54,4 +57,24 @@ abs

[GitHub] dubee commented on issue #3568: Add activation result integration test

dubee commented on issue #3568: Add activation result integration test URL: https://github.com/apache/incubator-openwhisk/pull/3568#issuecomment-383689111 @mdeuser, can you review? This is an automated message from the Apache

[GitHub] dubee opened a new pull request #3568: Add activation result integration test

dubee opened a new pull request #3568: Add activation result integration test URL: https://github.com/apache/incubator-openwhisk/pull/3568 Provides an integration test for the activation result API. ## Description ## Related issue and scope - [ ] I opened an

[GitHub] pritidesai closed issue #692: Create common util for matching source code extensions to runtimes

pritidesai closed issue #692: Create common util for matching source code extensions to runtimes URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/692 This is an automated message from the Apache Git Servic

[GitHub] pritidesai commented on issue #692: Create common util for matching source code extensions to runtimes

pritidesai commented on issue #692: Create common util for matching source code extensions to runtimes URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/692#issuecomment-383679007 `FileSystemReader` does not exist anymore.

[GitHub] pritidesai closed issue #660: Message indicates "Deployment removed successfully" when actually it failed...

pritidesai closed issue #660: Message indicates "Deployment removed successfully" when actually it failed... URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/660 This is an automated message from the Apach

[GitHub] pritidesai commented on issue #660: Message indicates "Deployment removed successfully" when actually it failed...

pritidesai commented on issue #660: Message indicates "Deployment removed successfully" when actually it failed... URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/660#issuecomment-383678536 Its fixed now: ``` ./wskdeploy undeploy -m tests/src/integration/trigg

[GitHub] pritidesai closed issue #611: Action flag definitions, where are they used?

pritidesai closed issue #611: Action flag definitions, where are they used? URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/611 This is an automated message from the Apache Git Service. To respond to the m

[GitHub] dubee opened a new pull request #3567: Activation result integration with log store SPI

dubee opened a new pull request #3567: Activation result integration with log store SPI URL: https://github.com/apache/incubator-openwhisk/pull/3567 Allows the Elasticsearch log store to provide activation results through the activation result controller API. ## Description

[GitHub] pritidesai commented on issue #611: Action flag definitions, where are they used?

pritidesai commented on issue #611: Action flag definitions, where are they used? URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/611#issuecomment-383676470 Doesnt exists anymore ... This is an automated

[GitHub] dubee closed pull request #37: deploy a master tag image

dubee closed pull request #37: deploy a master tag image URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/37 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] pritidesai commented on issue #750: Remove or fix incomplete test/use case in deploymentreader_test.go

pritidesai commented on issue #750: Remove or fix incomplete test/use case in deploymentreader_test.go URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/750#issuecomment-383675334 Fixed as part of cleanup.

[GitHub] pritidesai closed issue #750: Remove or fix incomplete test/use case in deploymentreader_test.go

pritidesai closed issue #750: Remove or fix incomplete test/use case in deploymentreader_test.go URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/750 This is an automated message from the Apache Git Servic

[GitHub] pritidesai closed issue #258: Support concatenated variables in the manifest file

pritidesai closed issue #258: Support concatenated variables in the manifest file URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/258 This is an automated message from the Apache Git Service. To respond t

[GitHub] pritidesai commented on issue #258: Support concatenated variables in the manifest file

pritidesai commented on issue #258: Support concatenated variables in the manifest file URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/258#issuecomment-383674087 this is supported with: ``` inputs: CLOUDANT_USERNAME: $CLOUDANT_USERNAME CLOUDANT_PASS

[GitHub] csantanapr opened a new pull request #37: deploy a master tag image

csantanapr opened a new pull request #37: deploy a master tag image URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/37 This would make it easier to try out the latest code merged in to master but not released under a versioned tag. cc @eweiter --

[GitHub] pritidesai closed issue #557: Warning message does not highlight key words from grammar (e.g., 'package')

pritidesai closed issue #557: Warning message does not highlight key words from grammar (e.g., 'package') URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/557 This is an automated message from the Apache G

[GitHub] pritidesai commented on issue #557: Warning message does not highlight key words from grammar (e.g., 'package')

pritidesai commented on issue #557: Warning message does not highlight key words from grammar (e.g., 'package') URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/557#issuecomment-383667567 Warnings are displayed in yellow, info messages displayed in blue, and error messa

[GitHub] pritidesai commented on issue #554: Bad "command" to wsdeploy displays Error message twice (2x)

pritidesai commented on issue #554: Bad "command" to wsdeploy displays Error message twice (2x) URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/554#issuecomment-383666869 As of now, it displays error message once but still need to how `--help` message: ``` ./

[GitHub] pritidesai commented on issue #548: Description field/key is not supported for Rule entity

pritidesai commented on issue #548: Description field/key is not supported for Rule entity URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/548#issuecomment-383666180 Annotations are supported in Rules. Th

[GitHub] pritidesai closed issue #505: Only show full path/filename on error/warning messages if -v (--verbose) set to True

pritidesai closed issue #505: Only show full path/filename on error/warning messages if -v (--verbose) set to True URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/505 This is an automated message from th

[GitHub] pritidesai commented on issue #505: Only show full path/filename on error/warning messages if -v (--verbose) set to True

pritidesai commented on issue #505: Only show full path/filename on error/warning messages if -v (--verbose) set to True URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/505#issuecomment-383664950 Must have been fixed at some point: ``` ./wskdeploy -v Erro

[GitHub] pritidesai closed issue #494: manifest_basic_tar_grammar.yaml also tests feeds and api grammar; split them out

pritidesai closed issue #494: manifest_basic_tar_grammar.yaml also tests feeds and api grammar; split them out URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/494 This is an automated message from the Apa

[GitHub] pritidesai commented on issue #494: manifest_basic_tar_grammar.yaml also tests feeds and api grammar; split them out

pritidesai commented on issue #494: manifest_basic_tar_grammar.yaml also tests feeds and api grammar; split them out URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/494#issuecomment-383663573 this file doesnt exists anymore ... ``` find . -name "manifest_basic

[GitHub] pritidesai commented on issue #378: Overwrite (true) should be a configurable option

pritidesai commented on issue #378: Overwrite (true) should be a configurable option URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/378#issuecomment-383661650 Its still the same ... This is an automated

[GitHub] pritidesai commented on issue #751: Export command subject to List limits

pritidesai commented on issue #751: Export command subject to List limits URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/751#issuecomment-383660576 Making it a low priority and enhancement as haven't found a real use case with more than 200 actions in a package. -

[GitHub] pritidesai opened a new issue #876: Deploy OW entities from Github Repo

pritidesai opened a new issue #876: Deploy OW entities from Github Repo URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/876 This is an automated message from the Apache Git Service. To respond to the messa

[GitHub] tysonnorris closed pull request #3563: Skip log collection if the log-limit is set to "0".

tysonnorris closed pull request #3563: Skip log collection if the log-limit is set to "0". URL: https://github.com/apache/incubator-openwhisk/pull/3563 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] tysonnorris commented on issue #3563: Skip log collection if the log-limit is set to "0".

tysonnorris commented on issue #3563: Skip log collection if the log-limit is set to "0". URL: https://github.com/apache/incubator-openwhisk/pull/3563#issuecomment-383659685 @markusthoemmes got it thanks This is an automate

[GitHub] markusthoemmes commented on issue #3563: Skip log collection if the log-limit is set to "0".

markusthoemmes commented on issue #3563: Skip log collection if the log-limit is set to "0". URL: https://github.com/apache/incubator-openwhisk/pull/3563#issuecomment-383656029 @tysonnorris The idea behind this: 1. Optimization 2. You're able to disable logging for certain actions at

[GitHub] mrutkows commented on issue #93: Document LICENSE file Exclusions for incubator-openwhisk

mrutkows commented on issue #93: Document LICENSE file Exclusions for incubator-openwhisk URL: https://github.com/apache/incubator-openwhisk-release/issues/93#issuecomment-383644928 @daisy-ycguo @houshengbo Added exclusion for "performance" directory for main openwhisk: PR https://git

[GitHub] mrutkows opened a new pull request #26: Add exclusion for new incubator-openwhisk perf. tests.

mrutkows opened a new pull request #26: Add exclusion for new incubator-openwhisk perf. tests. URL: https://github.com/apache/incubator-openwhisk-utilities/pull/26 This is an automated message from the Apache Git Service. To

[GitHub] rabbah commented on issue #41: remove use of state in nodejs app

rabbah commented on issue #41: remove use of state in nodejs app URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/41#issuecomment-383641601 Indeed this is legacy defensive code but it has been observed to still catch some lifecycle management bugs in recent memory. It

[GitHub] rabbah commented on issue #41: remove use of state in nodejs app

rabbah commented on issue #41: remove use of state in nodejs app URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/41#issuecomment-383641601 Indeed this is legacy defensive code but it has been observed to still catch some lifecycle management bugs in recent memory. It

[GitHub] tysonnorris commented on issue #41: remove use of state in nodejs app

tysonnorris commented on issue #41: remove use of state in nodejs app URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/41#issuecomment-383641420 Required for https://github.com/apache/incubator-openwhisk/pull/2795 --

[GitHub] tysonnorris opened a new pull request #41: remove use of state in nodejs app

tysonnorris opened a new pull request #41: remove use of state in nodejs app URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/41 To allow possibility of concurrent activation processing at the invoker, the "state" handling in the nodejs action container needs to be rem

[GitHub] mrutkows commented on issue #93: Document LICENSE file Exclusions for incubator-openwhisk

mrutkows commented on issue #93: Document LICENSE file Exclusions for incubator-openwhisk URL: https://github.com/apache/incubator-openwhisk-release/issues/93#issuecomment-383635661 @houshengbo @daisy-ycguo It appears that a performance test was recently added to main openwhisk repo. (und

[GitHub] csantanapr closed pull request #145: swap active host on sigterm signal if host being stopped is active

csantanapr closed pull request #145: swap active host on sigterm signal if host being stopped is active URL: https://github.com/apache/incubator-openwhisk-package-alarms/pull/145 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed belo

[GitHub] csantanapr closed pull request #169: swap active host on sigterm signal if host being stopped is active

csantanapr closed pull request #169: swap active host on sigterm signal if host being stopped is active URL: https://github.com/apache/incubator-openwhisk-package-cloudant/pull/169 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed be

[GitHub] mrutkows closed pull request #3565: Add license headers to files under ansible folder

mrutkows closed pull request #3565: Add license headers to files under ansible folder URL: https://github.com/apache/incubator-openwhisk/pull/3565 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] mrutkows closed pull request #3566: Add license headers to 67 files

mrutkows closed pull request #3566: Add license headers to 67 files URL: https://github.com/apache/incubator-openwhisk/pull/3566 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] mrutkows commented on a change in pull request #3566: Add license headers to 67 files

mrutkows commented on a change in pull request #3566: Add license headers to 67 files URL: https://github.com/apache/incubator-openwhisk/pull/3566#discussion_r183434425 ## File path: actionRuntimes/nodejs6Action/Dockerfile ## @@ -1,2 +1 @@ FROM openwhisk/nodejs6action:1.

[GitHub] mrutkows commented on a change in pull request #3566: Add license headers to 67 files

mrutkows commented on a change in pull request #3566: Add license headers to 67 files URL: https://github.com/apache/incubator-openwhisk/pull/3566#discussion_r183434661 ## File path: common/scala/Dockerfile ## @@ -75,4 +91,4 @@ COPY transformEnvironment.sh / RUN chmod +x

[GitHub] mrutkows commented on a change in pull request #3566: Add license headers to 67 files

mrutkows commented on a change in pull request #3566: Add license headers to 67 files URL: https://github.com/apache/incubator-openwhisk/pull/3566#discussion_r183434468 ## File path: actionRuntimes/nodejs8Action/Dockerfile ## @@ -1,2 +1 @@ FROM openwhisk/action-nodejs-v8

[GitHub] mrutkows commented on a change in pull request #3566: Add license headers to 67 files

mrutkows commented on a change in pull request #3566: Add license headers to 67 files URL: https://github.com/apache/incubator-openwhisk/pull/3566#discussion_r183434750 ## File path: core/controller/Dockerfile ## @@ -15,4 +31,4 @@ COPY init.sh / RUN chmod +x init.sh E

[GitHub] dgrove-oss commented on issue #3563: Skip log collection if the log-limit is set to "0".

dgrove-oss commented on issue #3563: Skip log collection if the log-limit is set to "0". URL: https://github.com/apache/incubator-openwhisk/pull/3563#issuecomment-383601857 don't see any obvious kube problems; it should smoothly pick up the default from application.conf. I've manually sk

[GitHub] dgrove-oss commented on issue #3563: Skip log collection if the log-limit is set to "0".

dgrove-oss commented on issue #3563: Skip log collection if the log-limit is set to "0". URL: https://github.com/apache/incubator-openwhisk/pull/3563#issuecomment-383601857 don't see any obvious kube problems; it should smoothly pick up the default from application.conf.

[GitHub] daisy-ycguo commented on issue #93: Document LICENSE file Exclusions for incubator-openwhisk

daisy-ycguo commented on issue #93: Document LICENSE file Exclusions for incubator-openwhisk URL: https://github.com/apache/incubator-openwhisk-release/issues/93#issuecomment-383565543 We should add below files the exclusion list configurations: ``` openwhisk/ansible/group_v

[GitHub] daisy-ycguo opened a new issue #125: Update RAT config to exclude some files in OpenWhisk repo

daisy-ycguo opened a new issue #125: Update RAT config to exclude some files in OpenWhisk repo URL: https://github.com/apache/incubator-openwhisk-release/issues/125 We should edit pom.xml to exclude below files, which are reported in issue #93 . We may need to update scan tool also, to

[GitHub] daisy-ycguo commented on issue #93: Document LICENSE file Exclusions for incubator-openwhisk

daisy-ycguo commented on issue #93: Document LICENSE file Exclusions for incubator-openwhisk URL: https://github.com/apache/incubator-openwhisk-release/issues/93#issuecomment-383595462 Two PRs are created: https://github.com/apache/incubator-openwhisk/pull/3566 https://github.com/apa

[GitHub] daisy-ycguo opened a new pull request #3566: Add license headers to 67 files

daisy-ycguo opened a new pull request #3566: Add license headers to 67 files URL: https://github.com/apache/incubator-openwhisk/pull/3566 Partially resolved issue #3264 . This is an automated message from the Apache Git Servic

[GitHub] daisy-ycguo commented on issue #93: Document LICENSE file Exclusions for incubator-openwhisk

daisy-ycguo commented on issue #93: Document LICENSE file Exclusions for incubator-openwhisk URL: https://github.com/apache/incubator-openwhisk-release/issues/93#issuecomment-383565543 We should add below files the exclusion list configurations: ``` openwhisk/ansible/group_v

[GitHub] daisy-ycguo opened a new pull request #3565: Add license headers to files under ansible folder

daisy-ycguo opened a new pull request #3565: Add license headers to files under ansible folder URL: https://github.com/apache/incubator-openwhisk/pull/3565 To add apache license headers to files under `ansible` folder. Partially resolved issue #3264 . ---

[GitHub] daisy-ycguo commented on issue #93: Document LICENSE file Exclusions for incubator-openwhisk

daisy-ycguo commented on issue #93: Document LICENSE file Exclusions for incubator-openwhisk URL: https://github.com/apache/incubator-openwhisk-release/issues/93#issuecomment-383565543 We should add `*/licenses/*` to the exclusion list, and also add below files because they are configurat

[GitHub] daisy-ycguo commented on issue #93: Document LICENSE file Exclusions for incubator-openwhisk

daisy-ycguo commented on issue #93: Document LICENSE file Exclusions for incubator-openwhisk URL: https://github.com/apache/incubator-openwhisk-release/issues/93#issuecomment-383565543 We should add `*/licenses/*` in the exclusion list.

[GitHub] markusthoemmes commented on issue #2175: Provide a performance test-harness

markusthoemmes commented on issue #2175: Provide a performance test-harness URL: https://github.com/apache/incubator-openwhisk/issues/2175#issuecomment-383551341 Done. This is an automated message from the Apache Git Service.

[GitHub] markusthoemmes closed issue #1625: Turn off Kafka topic-auto-create

markusthoemmes closed issue #1625: Turn off Kafka topic-auto-create URL: https://github.com/apache/incubator-openwhisk/issues/1625 This is an automated message from the Apache Git Service. To respond to the message, please lo

[GitHub] markusthoemmes closed issue #2175: Provide a performance test-harness

markusthoemmes closed issue #2175: Provide a performance test-harness URL: https://github.com/apache/incubator-openwhisk/issues/2175 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] markusthoemmes commented on issue #1625: Turn off Kafka topic-auto-create

markusthoemmes commented on issue #1625: Turn off Kafka topic-auto-create URL: https://github.com/apache/incubator-openwhisk/issues/1625#issuecomment-383551271 This done I believe. This is an automated message from the Apache

[GitHub] chetanmeh commented on issue #3525: Re-enable invokers docker_container module

chetanmeh commented on issue #3525: Re-enable invokers docker_container module URL: https://github.com/apache/incubator-openwhisk/pull/3525#issuecomment-383550367 @kpavel For some of my work I need support for #3477 and was looking for way to extend the environment variables passed. I beli

[GitHub] markusthoemmes commented on issue #3563: Skip log collection if the log-limit is set to "0".

markusthoemmes commented on issue #3563: Skip log collection if the log-limit is set to "0". URL: https://github.com/apache/incubator-openwhisk/pull/3563#issuecomment-383489648 PG1 2805 🔵 This is an automated message from t