[GitHub] rabbah commented on issue #3671: Activation id in header

rabbah commented on issue #3671: Activation id in header URL: https://github.com/apache/incubator-openwhisk/pull/3671#issuecomment-392007532 For the first parts: For a trigger, the request id is the triggers activation id. Every activated action generates a new activation id as we

[GitHub] rabbah commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file."

rabbah commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file." URL: https://github.com/apache/incubator-openwhisk/pull/3705#issuecomment-392015966 @cbickel can you explain the reason for reverting - ie how are the comment/license headers an issue?

[GitHub] style95 commented on a change in pull request #3671: Activation id in header

style95 commented on a change in pull request #3671: Activation id in header URL: https://github.com/apache/incubator-openwhisk/pull/3671#discussion_r190836887 ## File path: core/controller/src/main/scala/whisk/core/controller/Entities.scala ## @@ -65,14 +65,25 @@

[GitHub] mhenke1 commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file."

mhenke1 commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file." URL: https://github.com/apache/incubator-openwhisk/pull/3705#issuecomment-392050223 https://stackoverflow.com/a/1196485

[GitHub] rabbah commented on issue #3704: Provide invoker maintenance mode

rabbah commented on issue #3704: Provide invoker maintenance mode URL: https://github.com/apache/incubator-openwhisk/pull/3704#issuecomment-392049947 +1 @chetanmeh The administrative interfaces should have authentication in place. While at it, we should apply the same to the

[GitHub] rabbah commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file."

rabbah commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file." URL: https://github.com/apache/incubator-openwhisk/pull/3705#issuecomment-392049524 did you try having `` as the first line and comment for license to follow --- just a guess.

[GitHub] jeremiaswerner closed pull request #3705: Revert "Add ASF license to optional XML files and last gradle file."

jeremiaswerner closed pull request #3705: Revert "Add ASF license to optional XML files and last gradle file." URL: https://github.com/apache/incubator-openwhisk/pull/3705 This is an automated message from the Apache Git

[GitHub] rabbah closed pull request #3502: Immutable attachments via putAndAttach

rabbah closed pull request #3502: Immutable attachments via putAndAttach URL: https://github.com/apache/incubator-openwhisk/pull/3502 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[GitHub] vvraskin commented on issue #3706: Limit cipher suites used for controller https

vvraskin commented on issue #3706: Limit cipher suites used for controller https URL: https://github.com/apache/incubator-openwhisk/pull/3706#issuecomment-392043561 PG3 2297 is on its way This is an automated message from

[GitHub] jthomas commented on issue #118: add User-Agent to request header

jthomas commented on issue #118: add User-Agent to request header URL: https://github.com/apache/incubator-openwhisk-client-js/pull/118#issuecomment-391981750 :+1: This is an automated message from the Apache Git

[GitHub] cbickel opened a new pull request #3705: Revert "Add ASF license to optional XML files and last gradle file."

cbickel opened a new pull request #3705: Revert "Add ASF license to optional XML files and last gradle file." URL: https://github.com/apache/incubator-openwhisk/pull/3705 Reverts apache/incubator-openwhisk#3702 This is an

[GitHub] vvraskin commented on issue #3706: Limit cipher suites used for controller https

vvraskin commented on issue #3706: Limit cipher suites used for controller https URL: https://github.com/apache/incubator-openwhisk/pull/3706#issuecomment-392043561 PG3 2297 is on its way This is an automated message from

[GitHub] jthomas closed issue #117: add User-Agent to request header

jthomas closed issue #117: add User-Agent to request header URL: https://github.com/apache/incubator-openwhisk-client-js/issues/117 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] jthomas closed pull request #118: add User-Agent to request header

jthomas closed pull request #118: add User-Agent to request header URL: https://github.com/apache/incubator-openwhisk-client-js/pull/118 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

[GitHub] csantanapr commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file."

csantanapr commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file." URL: https://github.com/apache/incubator-openwhisk/pull/3705#issuecomment-392022657 That’s odd how come adding comments at the top of an xml brakes functionality?

[GitHub] csantanapr commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file."

csantanapr commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file." URL: https://github.com/apache/incubator-openwhisk/pull/3705#issuecomment-392022849 What’s the root cause? How do we fix instead of doing a drastic revert with no other options being

[GitHub] sven-lange-last commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file."

sven-lange-last commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file." URL: https://github.com/apache/incubator-openwhisk/pull/3705#issuecomment-392049152 This is the exception occurring with the XML comment: ``` Failed to auto configure

[GitHub] style95 commented on a change in pull request #3671: Activation id in header

style95 commented on a change in pull request #3671: Activation id in header URL: https://github.com/apache/incubator-openwhisk/pull/3671#discussion_r190837409 ## File path: tests/src/test/scala/whisk/core/controller/test/ControllerTestCommon.scala ## @@ -91,6 +91,8 @@

[GitHub] codecov-io commented on issue #3671: Activation id in header

codecov-io commented on issue #3671: Activation id in header URL: https://github.com/apache/incubator-openwhisk/pull/3671#issuecomment-392002725 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3671?src=pr=h1) Report > Merging

[GitHub] cbickel commented on issue #3671: Activation id in header

cbickel commented on issue #3671: Activation id in header URL: https://github.com/apache/incubator-openwhisk/pull/3671#issuecomment-392002543 @rabbah @ddragosd I also thought about making tid and activationid the same, but I see the following problems: - If a trigger is invoked,

[GitHub] cmonkey commented on issue #3590: Collect test reports as part of log collection

cmonkey commented on issue #3590: Collect test reports as part of log collection URL: https://github.com/apache/incubator-openwhisk/pull/3590#issuecomment-391949397 b!不b c But t e a a. . Ya jajtjj t. T W Carlos Santana 于 2018年5月3日周四 05:36写道：

[GitHub] cbickel commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file."

cbickel commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file." URL: https://github.com/apache/incubator-openwhisk/pull/3705#issuecomment-392017818 @rabbah Sorry, I updated the description.

[GitHub] codecov-io commented on issue #3698: Make amount of parallel docker runs configurable.

codecov-io commented on issue #3698: Make amount of parallel docker runs configurable. URL: https://github.com/apache/incubator-openwhisk/pull/3698#issuecomment-392048718 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3698?src=pr=h1) Report > Merging

[GitHub] codecov-io commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file."

codecov-io commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file." URL: https://github.com/apache/incubator-openwhisk/pull/3705#issuecomment-392021825 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3705?src=pr=h1) Report >

[GitHub] csantanapr commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file."

csantanapr commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file." URL: https://github.com/apache/incubator-openwhisk/pull/3705#issuecomment-392023366 For the most part all files should have an ASF header, if someone can proposed how can we get the

[GitHub] rabbah commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file."

rabbah commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file." URL: https://github.com/apache/incubator-openwhisk/pull/3705#issuecomment-392049524 did you try having `` as the first line --- just a guess.

[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_r190926261 ## File path: common/scala/src/main/scala/whisk/core/containerpool/HttpUtils.scala

[GitHub] chetanmeh commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file."

chetanmeh commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file." URL: https://github.com/apache/incubator-openwhisk/pull/3705#issuecomment-392106682 What @rabbah mentioned should work as we use similar approach at

[GitHub] tysonnorris opened a new pull request #3707: enable concurrent runs on ActionContainer test utility

tysonnorris opened a new pull request #3707: enable concurrent runs on ActionContainer test utility URL: https://github.com/apache/incubator-openwhisk/pull/3707 support for tests in https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/41 ## Description

[GitHub] mrutkows opened a new pull request #42: Update repo. files to latest ASF license scanning standards to match OW project policies

mrutkows opened a new pull request #42: Update repo. files to latest ASF license scanning standards to match OW project policies URL: https://github.com/apache/incubator-openwhisk-utilities/pull/42 This is an automated

[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-392063629 Yes. This is an automated message from the Apache Git

[GitHub] mhenke1 commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file."

mhenke1 commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file." URL: https://github.com/apache/incubator-openwhisk/pull/3705#issuecomment-392050223 ^^^That is the definitive solution https://stackoverflow.com/a/1196485

[GitHub] brunogirin opened a new issue #940: API section doesn't support path parameters

brunogirin opened a new issue #940: API section doesn't support path parameters URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/940 When deploying a manifest that includes an API endpoint with path parameters, `wskdeploy` throws an error saying that the response shoud

[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_r190930105 ## File path: common/scala/src/main/scala/whisk/core/containerpool/HttpUtils.scala

[GitHub] codecov-io commented on issue #3680: Adjust invoker playbook to pull blackbox docker images when a prefix is specified.

codecov-io commented on issue #3680: Adjust invoker playbook to pull blackbox docker images when a prefix is specified. URL: https://github.com/apache/incubator-openwhisk/pull/3680#issuecomment-390292471 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3680?src=pr=h1)

[GitHub] codecov-io commented on issue #3699: Add gatling-throughput test for cold invocations.

codecov-io commented on issue #3699: Add gatling-throughput test for cold invocations. URL: https://github.com/apache/incubator-openwhisk/pull/3699#issuecomment-391705629 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3699?src=pr=h1) Report > Merging

[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-392098039 @cbickel @rabbah @dubee any comments on this? This is an

[GitHub] codecov-io commented on issue #3680: Adjust invoker playbook to pull blackbox docker images when a prefix is specified.

codecov-io commented on issue #3680: Adjust invoker playbook to pull blackbox docker images when a prefix is specified. URL: https://github.com/apache/incubator-openwhisk/pull/3680#issuecomment-390292471 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3680?src=pr=h1)

[GitHub] codecov-io commented on issue #3708: Single build.gradle file missing ASF license header

codecov-io commented on issue #3708: Single build.gradle file missing ASF license header URL: https://github.com/apache/incubator-openwhisk/pull/3708#issuecomment-392096299 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3708?src=pr=h1) Report > Merging

[GitHub] codecov-io commented on issue #3680: Adjust invoker playbook to pull blackbox docker images when a prefix is specified.

codecov-io commented on issue #3680: Adjust invoker playbook to pull blackbox docker images when a prefix is specified. URL: https://github.com/apache/incubator-openwhisk/pull/3680#issuecomment-390292471 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3680?src=pr=h1)

[GitHub] codecov-io commented on issue #3699: Add gatling-throughput test for cold invocations.

codecov-io commented on issue #3699: Add gatling-throughput test for cold invocations. URL: https://github.com/apache/incubator-openwhisk/pull/3699#issuecomment-391705629 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3699?src=pr=h1) Report > Merging

[GitHub] mrutkows closed pull request #941: commenting out TestExportHelloWorld as its unstable in Travis

mrutkows closed pull request #941: commenting out TestExportHelloWorld as its unstable in Travis URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/941 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] mhamann commented on issue #304: API Gateway Path Parameters For Query Params?

mhamann commented on issue #304: API Gateway Path Parameters For Query Params? URL: https://github.com/apache/incubator-openwhisk-apigateway/issues/304#issuecomment-392163031 Query parameters are, by definition, not required. I have yet to come across a well-designed API that required

[GitHub] rabbah commented on issue #304: API Gateway Path Parameters For Query Params?

rabbah commented on issue #304: API Gateway Path Parameters For Query Params? URL: https://github.com/apache/incubator-openwhisk-apigateway/issues/304#issuecomment-392166750 Per the URI RFC https://tools.ietf.org/html/rfc3986#section-3.4 query parameters contain non-hierarchical

[GitHub] codecov-io commented on issue #3703: Add the license and notice for ConcurrentMapBackedCache.scala

codecov-io commented on issue #3703: Add the license and notice for ConcurrentMapBackedCache.scala URL: https://github.com/apache/incubator-openwhisk/pull/3703#issuecomment-391831090 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3703?src=pr=h1) Report > Merging

[GitHub] codecov-io commented on issue #3703: Add the license and notice for ConcurrentMapBackedCache.scala

codecov-io commented on issue #3703: Add the license and notice for ConcurrentMapBackedCache.scala URL: https://github.com/apache/incubator-openwhisk/pull/3703#issuecomment-391831090 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3703?src=pr=h1) Report > Merging

[GitHub] codecov-io commented on issue #3703: Add the license and notice for ConcurrentMapBackedCache.scala

codecov-io commented on issue #3703: Add the license and notice for ConcurrentMapBackedCache.scala URL: https://github.com/apache/incubator-openwhisk/pull/3703#issuecomment-391831090 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3703?src=pr=h1) Report > Merging

[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-392062854 @rabbah you mean to separate the ActionContainer changes into its own PR?

[GitHub] codecov-io commented on issue #3680: Adjust invoker playbook to pull blackbox docker images when a prefix is specified.

codecov-io commented on issue #3680: Adjust invoker playbook to pull blackbox docker images when a prefix is specified. URL: https://github.com/apache/incubator-openwhisk/pull/3680#issuecomment-390292471 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3680?src=pr=h1)

[GitHub] codecov-io commented on issue #3697: Add equals method to ensure that ByteSize instances satisfy Comparable contract

codecov-io commented on issue #3697: Add equals method to ensure that ByteSize instances satisfy Comparable contract URL: https://github.com/apache/incubator-openwhisk/pull/3697#issuecomment-391501095 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3697?src=pr=h1)

[GitHub] codecov-io commented on issue #3669: Make stemcells configurable by deployment

codecov-io commented on issue #3669: Make stemcells configurable by deployment URL: https://github.com/apache/incubator-openwhisk/pull/3669#issuecomment-389936897 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3669?src=pr=h1) Report > Merging

[GitHub] mrutkows opened a new pull request #3708: Single build.gradle file missing ASF license header

mrutkows opened a new pull request #3708: Single build.gradle file missing ASF license header URL: https://github.com/apache/incubator-openwhisk/pull/3708 ## Description ## Related issue and scope - [ ] I opened an issue to propose and discuss this change

[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 > Merging

[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 > Merging

[GitHub] csantanapr commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file."

csantanapr commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file." URL: https://github.com/apache/incubator-openwhisk/pull/3705#issuecomment-392118219 @mrutkows ^^ This is an automated

[GitHub] csantanapr commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file."

csantanapr commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file." URL: https://github.com/apache/incubator-openwhisk/pull/3705#issuecomment-392119885 Yeah I see Travis is  but I see the failure

[GitHub] pritidesai opened a new pull request #941: commenting out TestExportHelloWorld as its unstable in Travis

pritidesai opened a new pull request #941: commenting out TestExportHelloWorld as its unstable in Travis URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/941 This is an automated message from the Apache Git

[GitHub] csantanapr commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file."

csantanapr commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file." URL: https://github.com/apache/incubator-openwhisk/pull/3705#issuecomment-392118167 yeah it looks having the `` at the top is the solution and adjust the code scan to assume that xml

[GitHub] csantanapr commented on issue #204: ignore mycluster.yaml

csantanapr commented on issue #204: ignore mycluster.yaml URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/204#issuecomment-392127587 WTF? Travis failed ``` Reading configuration file [scancode/ASF-Release.cfg]... Scanning files starting at

[GitHub] mrutkows opened a new pull request #40: Fix example.c to have correct ASF license header.

mrutkows opened a new pull request #40: Fix example.c to have correct ASF license header. URL: https://github.com/apache/incubator-openwhisk-runtime-docker/pull/40 and also fix the Dockerfile headers to remove the bad '#' character

[GitHub] dgrove-oss closed pull request #204: ignore mycluster.yaml

dgrove-oss closed pull request #204: ignore mycluster.yaml URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/204 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[GitHub] csantanapr commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file."

csantanapr commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file." URL: https://github.com/apache/incubator-openwhisk/pull/3705#issuecomment-392118771 @cbickel @jeremiaswerner since you reverted and took the ASF license header, What do you think of

[GitHub] csantanapr commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file."

csantanapr commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file." URL: https://github.com/apache/incubator-openwhisk/pull/3705#issuecomment-392118771 @cbickel @jeremiaswerner since you reverted and took the ASF license header out, What do you think

[GitHub] csantanapr commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file."

csantanapr commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file." URL: https://github.com/apache/incubator-openwhisk/pull/3705#issuecomment-392119132 Now that I think about it how come Travis didn't catch this problem in the PR that introduce the

[GitHub] houshengbo closed pull request #3708: Single build.gradle file missing ASF license header

houshengbo closed pull request #3708: Single build.gradle file missing ASF license header URL: https://github.com/apache/incubator-openwhisk/pull/3708 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] tysonnorris commented on issue #3707: enable concurrent runs on ActionContainer test utility

tysonnorris commented on issue #3707: enable concurrent runs on ActionContainer test utility URL: https://github.com/apache/incubator-openwhisk/pull/3707#issuecomment-392131196 @rabbah I removed the unused `maxConcurrent` parameter

[GitHub] csantanapr commented on issue #204: ignore mycluster.yaml

csantanapr commented on issue #204: ignore mycluster.yaml URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/204#issuecomment-392140419 merge away sir @dgrove-oss This is an automated message from the

[GitHub] houshengbo closed pull request #162: clean up repos. and file exclusion listings.

houshengbo closed pull request #162: clean up repos. and file exclusion listings. URL: https://github.com/apache/incubator-openwhisk-release/pull/162 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] houshengbo closed pull request #42: Update repo. files to latest ASF license scanning standards to match OW project policies

houshengbo closed pull request #42: Update repo. files to latest ASF license scanning standards to match OW project policies URL: https://github.com/apache/incubator-openwhisk-utilities/pull/42 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it

[GitHub] houshengbo closed pull request #40: Fix example.c to have correct ASF license header.

houshengbo closed pull request #40: Fix example.c to have correct ASF license header. URL: https://github.com/apache/incubator-openwhisk-runtime-docker/pull/40 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 #3703: Add the license and notice for ConcurrentMapBackedCache.scala

codecov-io commented on issue #3703: Add the license and notice for ConcurrentMapBackedCache.scala URL: https://github.com/apache/incubator-openwhisk/pull/3703#issuecomment-391831090 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3703?src=pr=h1) Report > Merging

[GitHub] csantanapr opened a new pull request #204: ignore mycluster.yaml

csantanapr opened a new pull request #204: ignore mycluster.yaml URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/204 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] csantanapr commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file."

csantanapr commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file." URL: https://github.com/apache/incubator-openwhisk/pull/3705#issuecomment-392125914 I see it looks like it only cause problems with generating the logs. But the gradle task

[GitHub] csantanapr commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file."

csantanapr commented on issue #3705: Revert "Add ASF license to optional XML files and last gradle file." URL: https://github.com/apache/incubator-openwhisk/pull/3705#issuecomment-392125914 I see it looks like it only cause problems with generating the logs. But the gradle task

[GitHub] csantanapr commented on issue #3689: Enable extending environment variables of Controller and Invoker

csantanapr commented on issue #3689: Enable extending environment variables of Controller and Invoker URL: https://github.com/apache/incubator-openwhisk/pull/3689#issuecomment-392206350 Thanks @chetanmeh for the changes. PG1 2954 ⏳

[GitHub] csantanapr commented on issue #304: API Gateway Path Parameters For Query Params?

csantanapr commented on issue #304: API Gateway Path Parameters For Query Params? URL: https://github.com/apache/incubator-openwhisk-apigateway/issues/304#issuecomment-392210799 query parameters are usually used as optional information like filters and overrides. Absolutely they

[GitHub] csantanapr commented on issue #3689: Enable extending environment variables of Controller and Invoker

csantanapr commented on issue #3689: Enable extending environment variables of Controller and Invoker URL: https://github.com/apache/incubator-openwhisk/pull/3689#issuecomment-392206350 Thanks @chetanmeh for the changes. PG1 2954 ⏳

[GitHub] csantanapr commented on issue #3689: Enable extending environment variables of Controller and Invoker

csantanapr commented on issue #3689: Enable extending environment variables of Controller and Invoker URL: https://github.com/apache/incubator-openwhisk/pull/3689#issuecomment-392219418 Hey @chetanmeh please review the commit I just pushed

[GitHub] chetanmeh commented on issue #3689: Enable extending environment variables of Controller and Invoker

chetanmeh commented on issue #3689: Enable extending environment variables of Controller and Invoker URL: https://github.com/apache/incubator-openwhisk/pull/3689#issuecomment-392226848 @csantanapr I needed this support for travis specially. There I want to have a way where just with a

[GitHub] codecov-io commented on issue #3689: Enable extending environment variables of Controller and Invoker

codecov-io commented on issue #3689: Enable extending environment variables of Controller and Invoker URL: https://github.com/apache/incubator-openwhisk/pull/3689#issuecomment-391266601 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3689?src=pr=h1) Report > Merging

[GitHub] codecov-io commented on issue #3703: Add the license and notice for ConcurrentMapBackedCache.scala

codecov-io commented on issue #3703: Add the license and notice for ConcurrentMapBackedCache.scala URL: https://github.com/apache/incubator-openwhisk/pull/3703#issuecomment-391831090 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3703?src=pr=h1) Report > Merging

[GitHub] mrutkows opened a new pull request #164: Fix or document last remaining exclusions across all release repos.

mrutkows opened a new pull request #164: Fix or document last remaining exclusions across all release repos. URL: https://github.com/apache/incubator-openwhisk-release/pull/164 This is an automated message from the Apache

[GitHub] matt-boyd commented on issue #3691: README: Available typo

matt-boyd commented on issue #3691: README: Available typo URL: https://github.com/apache/incubator-openwhisk/pull/3691#issuecomment-391957655 Yes! Signed the CLA! This is an automated message from the Apache Git Service.

[GitHub] chetanmeh commented on a change in pull request #2282: Distributed tracing support #2192

chetanmeh commented on a change in pull request #2282: Distributed tracing support #2192 URL: https://github.com/apache/incubator-openwhisk/pull/2282#discussion_r190799810 ## File path: core/controller/src/main/scala/whisk/core/controller/Controller.scala ## @@ -203,6

[GitHub] chetanmeh commented on a change in pull request #2282: Distributed tracing support #2192

chetanmeh commented on a change in pull request #2282: Distributed tracing support #2192 URL: https://github.com/apache/incubator-openwhisk/pull/2282#discussion_r190802179 ## File path: common/scala/src/main/scala/whisk/common/tracing/OpenTracingProvider.scala ## @@

[GitHub] chetanmeh commented on a change in pull request #2282: Distributed tracing support #2192

chetanmeh commented on a change in pull request #2282: Distributed tracing support #2192 URL: https://github.com/apache/incubator-openwhisk/pull/2282#discussion_r190803840 ## File path: common/scala/src/main/scala/whisk/common/tracing/OpenTracingProvider.scala ## @@

[GitHub] chetanmeh commented on a change in pull request #2282: Distributed tracing support #2192

chetanmeh commented on a change in pull request #2282: Distributed tracing support #2192 URL: https://github.com/apache/incubator-openwhisk/pull/2282#discussion_r190799562 ## File path: common/scala/src/main/scala/whisk/common/tracing/OpenTracingProvider.scala ## @@

[GitHub] chetanmeh commented on a change in pull request #2282: Distributed tracing support #2192

chetanmeh commented on a change in pull request #2282: Distributed tracing support #2192 URL: https://github.com/apache/incubator-openwhisk/pull/2282#discussion_r190804603 ## File path: common/scala/src/main/scala/whisk/common/tracing/OpenTracingProvider.scala ## @@

[GitHub] chetanmeh commented on a change in pull request #2282: Distributed tracing support #2192

chetanmeh commented on a change in pull request #2282: Distributed tracing support #2192 URL: https://github.com/apache/incubator-openwhisk/pull/2282#discussion_r190802858 ## File path: common/scala/src/main/scala/whisk/common/tracing/OpenTracingProvider.scala ## @@

[GitHub] chetanmeh commented on a change in pull request #2282: Distributed tracing support #2192

chetanmeh commented on a change in pull request #2282: Distributed tracing support #2192 URL: https://github.com/apache/incubator-openwhisk/pull/2282#discussion_r190800720 ## File path: common/scala/src/main/scala/whisk/common/tracing/OpenTracingProvider.scala ## @@

[GitHub] chetanmeh commented on a change in pull request #2282: Distributed tracing support #2192

chetanmeh commented on a change in pull request #2282: Distributed tracing support #2192 URL: https://github.com/apache/incubator-openwhisk/pull/2282#discussion_r190802724 ## File path: common/scala/src/main/scala/whisk/common/tracing/OpenTracingProvider.scala ## @@

[GitHub] codecov-io commented on issue #3699: Add gatling-throughput test for cold invocations.

codecov-io commented on issue #3699: Add gatling-throughput test for cold invocations. URL: https://github.com/apache/incubator-openwhisk/pull/3699#issuecomment-391705629 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3699?src=pr=h1) Report > Merging

[GitHub] cbickel commented on issue #3696: Move the files under performance to tests/performance

cbickel commented on issue #3696: Move the files under performance to tests/performance URL: https://github.com/apache/incubator-openwhisk/pull/3696#issuecomment-391970454 @csantanapr @houshengbo Why is this change necessary? This breaks everyone, that is already using the performance

[GitHub] cbickel commented on issue #3696: Move the files under performance to tests/performance

cbickel commented on issue #3696: Move the files under performance to tests/performance URL: https://github.com/apache/incubator-openwhisk/pull/3696#issuecomment-391970454 @csantanapr @houshengbo Why is this change necessary? This breaks everyone, that is already using the performance

[GitHub] codecov-io commented on issue #3622: make containers' timezone sync with the docker host

codecov-io commented on issue #3622: make containers' timezone sync with the docker host URL: https://github.com/apache/incubator-openwhisk/pull/3622#issuecomment-387274607 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3622?src=pr=h1) Report > Merging

[GitHub] codecov-io commented on issue #3699: Add gatling-throughput test for cold invocations.

codecov-io commented on issue #3699: Add gatling-throughput test for cold invocations. URL: https://github.com/apache/incubator-openwhisk/pull/3699#issuecomment-391705629 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3699?src=pr=h1) Report > Merging