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

2018-05-23 Thread GitBox
csantanapr commented on issue #3696: Move the files under performance to tests/performance URL: https://github.com/apache/incubator-openwhisk/pull/3696#issuecomment-391484203 PG  PG1 2945 This is an automated message from

[GitHub] mrutkows closed issue #803: Report - Parameters in JSON form

2018-05-23 Thread GitBox
mrutkows closed issue #803: Report - Parameters in JSON form URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/803 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] rabbah commented on issue #924: binary releases for a platform include binaries for all other platforms

2018-05-23 Thread GitBox
rabbah commented on issue #924: binary releases for a platform include binaries for all other platforms URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/924#issuecomment-391502867

[GitHub] mdeuser closed pull request #304: Run unit and native tests separately

2018-05-23 Thread GitBox
mdeuser closed pull request #304: Run unit and native tests separately URL: https://github.com/apache/incubator-openwhisk-cli/pull/304 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] rabbah opened a new pull request #3697: Add equals method to ensure that ByteSize instances satisfy Comparable contract

2018-05-23 Thread GitBox
rabbah opened a new pull request #3697: Add equals method to ensure that ByteSize instances satisfy Comparable contract URL: https://github.com/apache/incubator-openwhisk/pull/3697 The ByteSize method did not satisfy this requirement for Comparables: ``` * It is strongly

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

2018-05-23 Thread GitBox
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] codecov-io commented on issue #3697: Add equals method to ensure that ByteSize instances satisfy Comparable contract

2018-05-23 Thread GitBox
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] rabbah commented on issue #809: wskdeploy does not respect WSK_CONFIG_FILE

2018-05-23 Thread GitBox
rabbah commented on issue #809: wskdeploy does not respect WSK_CONFIG_FILE URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/809#issuecomment-391506667 suggested workaround is using `--config` flag. This is

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

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

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

2018-05-23 Thread GitBox
pritidesai commented on issue #898: wskdeploy export doesn't support multiple packages in a single project URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/898#issuecomment-391496807 again, @kpavel has fixed it, let me know if you run into any issues.

[GitHub] rabbah commented on issue #924: binary releases for a platform include binaries for all other platforms

2018-05-23 Thread GitBox
rabbah commented on issue #924: binary releases for a platform include binaries for all other platforms URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/924#issuecomment-391503205 worse, the binary is the wrong format: ``` file wskdeploy wskdeploy: ELF

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

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

[GitHub] mdeuser commented on issue #91: Fix invalid warning message

2018-05-23 Thread GitBox
mdeuser commented on issue #91: Fix invalid warning message URL: https://github.com/apache/incubator-openwhisk-client-go/pull/91#issuecomment-391493877 @dubee - is there a pg for this? This is an automated message from the

[GitHub] rabbah opened a new issue #925: better error messages

2018-05-23 Thread GitBox
rabbah opened a new issue #925: better error messages URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/925 run `wskdeploy` with no manifest file and you get this ``` Error: root.go [140]: [ERROR_MANIFEST_FILE_NOT_FOUND]: File:

[GitHub] mrutkows opened a new issue #161: Verify we do not include any tooling dot `.` configuration files in source release

2018-05-23 Thread GitBox
mrutkows opened a new issue #161: Verify we do not include any tooling dot `.` configuration files in source release URL: https://github.com/apache/incubator-openwhisk-release/issues/161 Please make sure we do not include the following files which are for config. tooling: ``` .bin

[GitHub] houshengbo opened a new pull request #3696: Move the files under performance to tests/performance

2018-05-23 Thread GitBox
houshengbo opened a new pull request #3696: Move the files under performance to tests/performance URL: https://github.com/apache/incubator-openwhisk/pull/3696 ## Description ## Related issue and scope - [ ] I opened an issue to propose and discuss this change

[GitHub] codecov-io commented on issue #3695: Use method links and correct method name.

2018-05-23 Thread GitBox
codecov-io commented on issue #3695: Use method links and correct method name. URL: https://github.com/apache/incubator-openwhisk/pull/3695#issuecomment-391476286 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3695?src=pr=h1) Report > Merging

[GitHub] houshengbo commented on issue #152: Lightbend copyright appears, no LICENSE attribution

2018-05-23 Thread GitBox
houshengbo commented on issue #152: Lightbend copyright appears, no LICENSE attribution URL: https://github.com/apache/incubator-openwhisk-release/issues/152#issuecomment-391495974 @markusthoemmes Could you change the implementation of this file? Change it or remove it. If not, we

[GitHub] mdeuser commented on a change in pull request #91: Fix invalid warning message

2018-05-23 Thread GitBox
mdeuser commented on a change in pull request #91: Fix invalid warning message URL: https://github.com/apache/incubator-openwhisk-client-go/pull/91#discussion_r190398547 ## File path: whisk/client.go ## @@ -652,10 +652,16 @@ func IsHttpRespSuccess(r *http.Response) bool {

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

2018-05-23 Thread GitBox
pritidesai commented on issue #899: wskdeploy export fails to export the `main` attribute for Java actions URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/899#issuecomment-391496171 Hi @brunogirin I have verified that @kpavel's changes has fixed this issue, the main

[GitHub] rabbah opened a new issue #924: binary releases for a platform include binaries for all other platforms

2018-05-23 Thread GitBox
rabbah opened a new issue #924: binary releases for a platform include binaries for all other platforms URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/924 ``` > unzip openwhisk_wskdeploy-latest-mac-amd64.zip > ls openwhisk_wskdeploy-latest-mac-amd64

[GitHub] rabbah commented on issue #809: wskdeploy does not respect WSK_CONFIG_FILE

2018-05-23 Thread GitBox
rabbah commented on issue #809: wskdeploy does not respect WSK_CONFIG_FILE URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/809#issuecomment-391505734 bump - this is really a show stopper trying to deploy to multiple configurations. Is there a suggested

[GitHub] mrutkows closed pull request #923: Enable API Gateway integration test in Travis

2018-05-23 Thread GitBox
mrutkows closed pull request #923: Enable API Gateway integration test in Travis URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/923 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] mrutkows closed issue #725: API Gateway - Enable integration test

2018-05-23 Thread GitBox
mrutkows closed issue #725: API Gateway - Enable integration test URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/725 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] rabbah commented on issue #809: wskdeploy does not respect WSK_CONFIG_FILE

2018-05-23 Thread GitBox
rabbah commented on issue #809: wskdeploy does not respect WSK_CONFIG_FILE URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/809#issuecomment-391505734 bump - this is really a show stopper trying to deploy to multiple configurations. Is there a suggested

[GitHub] rabbah commented on issue #809: wskdeploy does not respect WSK_CONFIG_FILE

2018-05-23 Thread GitBox
rabbah commented on issue #809: wskdeploy does not respect WSK_CONFIG_FILE URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/809#issuecomment-391506667 suggested workaround is using `-c` flag. This is an

[GitHub] chetanmeh opened a new pull request #3689: Extend env 3477

2018-05-23 Thread GitBox
chetanmeh opened a new pull request #3689: Extend env 3477 URL: https://github.com/apache/incubator-openwhisk/pull/3689 Provides a way to extend the environment variable set for Controller and Invoker ## Description As mentioned in #3477 for some set of deployments like those

[GitHub] style95 closed pull request #3671: Activation id in header

2018-05-23 Thread GitBox
style95 closed pull request #3671: Activation id in header URL: https://github.com/apache/incubator-openwhisk/pull/3671 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull

[GitHub] style95 opened a new pull request #3671: Activation id in header

2018-05-23 Thread GitBox
style95 opened a new pull request #3671: Activation id in header URL: https://github.com/apache/incubator-openwhisk/pull/3671 This closes #3582 ## Description This change will add activation id in the response headers. Since same logic needs to be used in both

[GitHub] chetanmeh commented on a change in pull request #3689: Enable extending environment variables of Controller and Invoker

2018-05-23 Thread GitBox
chetanmeh commented on a change in pull request #3689: Enable extending environment variables of Controller and Invoker URL: https://github.com/apache/incubator-openwhisk/pull/3689#discussion_r190155278 ## File path: ansible/roles/controller/tasks/deploy.yml ## @@ -236,6

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

2018-05-23 Thread GitBox
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] chetanmeh commented on a change in pull request #3562: ArtifactStore implementation for CosmosDB

2018-05-23 Thread GitBox
chetanmeh commented on a change in pull request #3562: ArtifactStore implementation for CosmosDB URL: https://github.com/apache/incubator-openwhisk/pull/3562#discussion_r190470634 ## File path:

[GitHub] augustin-taiwan opened a new pull request #128: Remove duplicated port setting

2018-05-23 Thread GitBox
augustin-taiwan opened a new pull request #128: Remove duplicated port setting URL: https://github.com/apache/incubator-openwhisk-devtools/pull/128 Port 2888 is duplicated. This is an automated message from the Apache Git

[GitHub] chetanmeh commented on a change in pull request #3689: Enable extending environment variables of Controller and Invoker

2018-05-23 Thread GitBox
chetanmeh commented on a change in pull request #3689: Enable extending environment variables of Controller and Invoker URL: https://github.com/apache/incubator-openwhisk/pull/3689#discussion_r190466946 ## File path: ansible/environments/local/group_vars/all ## @@ -2,6

[GitHub] chetanmeh commented on issue #3523: KafkaConnectorTests heisenbug

2018-05-23 Thread GitBox
chetanmeh commented on issue #3523: KafkaConnectorTests heisenbug URL: https://github.com/apache/incubator-openwhisk/issues/3523#issuecomment-391594542 Another failure seen at https://scans.gradle.com/s/6sfxa7kccvlji/tests/xaqugzlgs2zw6-6k6x5jmp5qe2c due to same reason as one above

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

2018-05-23 Thread GitBox
chetanmeh commented on issue #3562: ArtifactStore implementation for CosmosDB URL: https://github.com/apache/incubator-openwhisk/pull/3562#issuecomment-391595062 @tysonnorris I agree current ref count based shutdown handling is not desirable and we should relook into how `ArtifactStore`

[GitHub] sechunOH commented on issue #3579: Using non root user in controller

2018-05-23 Thread GitBox
sechunOH commented on issue #3579: Using non root user in controller URL: https://github.com/apache/incubator-openwhisk/pull/3579#issuecomment-391595004 @fmaschler not processes as root on the host, but as root in containers. It is for preventing privilege-escalation attacks from

[GitHub] pritidesai closed pull request #160: Clean up Openwhisk repo. ASF license header inclusion/exclusion tables

2018-05-23 Thread GitBox
pritidesai closed pull request #160: Clean up Openwhisk repo. ASF license header inclusion/exclusion tables URL: https://github.com/apache/incubator-openwhisk-release/pull/160 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below

[GitHub] rabbah opened a new issue #929: if manifest read fails, wskdeploy should fail

2018-05-23 Thread GitBox
rabbah opened a new issue #929: if manifest read fails, wskdeploy should fail URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/929 wskdeploy fetches the runtime manifest from hardcoded values here should it fail reading from the API host.

[GitHub] chetanmeh commented on issue #3502: Immutable attachments via putAndAttach

2018-05-23 Thread GitBox
chetanmeh commented on issue #3502: Immutable attachments via putAndAttach URL: https://github.com/apache/incubator-openwhisk/pull/3502#issuecomment-391295950 Can this PR be reviewed and approved? As this is required for some other PRs #3453, Attachment In lining and

[GitHub] codecov-io commented on issue #2282: Distributed tracing support #2192

2018-05-23 Thread GitBox
codecov-io commented on issue #2282: Distributed tracing support #2192 URL: https://github.com/apache/incubator-openwhisk/pull/2282#issuecomment-391308983 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/2282?src=pr=h1) Report > Merging

[GitHub] allustin commented on issue #126: Docker-Compose: Migrate to dockerize instead of curl

2018-05-23 Thread GitBox
allustin commented on issue #126: Docker-Compose: Migrate to dockerize instead of curl URL: https://github.com/apache/incubator-openwhisk-devtools/issues/126#issuecomment-391334866 this is a loop wtihout timeout, for example

[GitHub] duglin opened a new issue #3690: Change current working dir to location of exec

2018-05-23 Thread GitBox
duglin opened a new issue #3690: Change current working dir to location of exec URL: https://github.com/apache/incubator-openwhisk/issues/3690 I have a function that I deploy via a Docker container (ie. I do: `wsk action update mydemo exec.zip --native`). Inside of that zip I have some

[GitHub] fmaschler commented on issue #3579: Using non root user in controller

2018-05-23 Thread GitBox
fmaschler commented on issue #3579: Using non root user in controller URL: https://github.com/apache/incubator-openwhisk/pull/3579#issuecomment-391314107 I'm not an expert in namespaces but I see what you mean. Though those container processes run as root on the host. Even if they may not

[GitHub] allustin opened a new issue #126: Docker-Compose: Migrate to dockerize instead of curl

2018-05-23 Thread GitBox
allustin opened a new issue #126: Docker-Compose: Migrate to dockerize instead of curl URL: https://github.com/apache/incubator-openwhisk-devtools/issues/126 in current makefile there is curl --silent with un-ended loop. please migrate to https://github.com/jwilder/dockerize - it

[GitHub] pritidesai opened a new pull request #926: Overwriting Deployment file inputs

2018-05-23 Thread GitBox
pritidesai opened a new pull request #926: Overwriting Deployment file inputs URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/926 This is an automated message from the Apache Git Service. To respond to the

[GitHub] rabbah opened a new issue #930: wskdeploy should fail if host info does not parse or cannot be retrieved,

2018-05-23 Thread GitBox
rabbah opened a new issue #930: wskdeploy should fail if host info does not parse or cannot be retrieved, URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/930 This method should exist `wskdeploy` if there is an error - it should not proceed and surely not silently

[GitHub] rabbah opened a new issue #927: action deploy even when runtime is deprecated

2018-05-23 Thread GitBox
rabbah opened a new issue #927: action deploy even when runtime is deprecated URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/927 I can reproduce by deploying openwhisk with the following manifest: ``` > cat ansible/files/runtimes.json { "runtimes": {

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

2018-05-23 Thread GitBox
csantanapr commented on issue #3696: Move the files under performance to tests/performance URL: https://github.com/apache/incubator-openwhisk/pull/3696#issuecomment-391551311 @houshengbo Travis is failing to deploy.sh You need the relative directory here

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

2018-05-23 Thread GitBox
csantanapr commented on issue #3696: Move the files under performance to tests/performance URL: https://github.com/apache/incubator-openwhisk/pull/3696#issuecomment-391551311 @houshengbo You need the relative directory here

[GitHub] houshengbo closed pull request #926: Overwriting Deployment file inputs

2018-05-23 Thread GitBox
houshengbo closed pull request #926: Overwriting Deployment file inputs URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/926 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

[GitHub] augustin-taiwan opened a new issue #127: there are two 2888. it is duplicated.

2018-05-23 Thread GitBox
augustin-taiwan opened a new issue #127: there are two 2888. it is duplicated. URL: https://github.com/apache/incubator-openwhisk-devtools/issues/127 https://github.com/apache/incubator-openwhisk-devtools/blob/3227c032a0cee9a52c13ae5f8257f7d9a4b1485c/docker-compose/Makefile#L134

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

2018-05-23 Thread GitBox
codecov-io commented on issue #3696: Move the files under performance to tests/performance URL: https://github.com/apache/incubator-openwhisk/pull/3696#issuecomment-391564547 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3696?src=pr=h1) Report > Merging

[GitHub] pritidesai closed pull request #921: Add ASF license header to .gradle files.

2018-05-23 Thread GitBox
pritidesai closed pull request #921: Add ASF license header to .gradle files. URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/921 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] rabbah opened a new issue #928: hardcoded protocol does not allow for http to be used

2018-05-23 Thread GitBox
rabbah opened a new issue #928: hardcoded protocol does not allow for http to be used URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/928 this line hardcodes `https` as the protocol.

[GitHub] rabbah commented on issue #925: better error messages

2018-05-23 Thread GitBox
rabbah commented on issue #925: better error messages URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/925#issuecomment-391536541 many error messages could be reported better fwiw. This is an automated

[GitHub] rabbah opened a new issue #931: schema encoding of the runtime manifest schema uses uint16

2018-05-23 Thread GitBox
rabbah opened a new issue #931: schema encoding of the runtime manifest schema uses uint16 URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/931

[GitHub] augustin-taiwan commented on a change in pull request #120: removed port check for ports 80 and 443

2018-05-23 Thread GitBox
augustin-taiwan commented on a change in pull request #120: removed port check for ports 80 and 443 URL: https://github.com/apache/incubator-openwhisk-devtools/pull/120#discussion_r190450107 ## File path: docker-compose/Makefile ## @@ -131,7 +131,7 @@ print-host:

[GitHub] rabbah commented on issue #3635: Recommendations for configuration of pre-warmed pool

2018-05-23 Thread GitBox
rabbah commented on issue #3635: Recommendations for configuration of pre-warmed pool URL: https://github.com/apache/incubator-openwhisk/issues/3635#issuecomment-391295292 This pr https://github.com/apache/incubator-openwhisk/pull/3669 will allow you to set stem cells for any runtime.

[GitHub] rabbah commented on issue #3691: README: Available typo

2018-05-23 Thread GitBox
rabbah commented on issue #3691: README: Available typo URL: https://github.com/apache/incubator-openwhisk/pull/3691#issuecomment-391355597 Thanks! This is an automated message from the Apache Git Service. To respond to the

[GitHub] houshengbo closed pull request #23: Last files to need ASF License headers.

2018-05-23 Thread GitBox
houshengbo closed pull request #23: Last files to need ASF License headers. URL: https://github.com/apache/incubator-openwhisk-runtime-php/pull/23 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] rabbah commented on issue #3691: README: Available typo

2018-05-23 Thread GitBox
rabbah commented on issue #3691: README: Available typo URL: https://github.com/apache/incubator-openwhisk/pull/3691#issuecomment-391355597 Thanks. This is an automated message from the Apache Git Service. To respond to the

[GitHub] houshengbo closed pull request #39: Last files to need ASF License headers.

2018-05-23 Thread GitBox
houshengbo closed pull request #39: Last files to need ASF License headers. URL: https://github.com/apache/incubator-openwhisk-runtime-docker/pull/39 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] houshengbo closed pull request #54: Last files to need ASF License headers.

2018-05-23 Thread GitBox
houshengbo closed pull request #54: Last files to need ASF License headers. URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/54 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] houshengbo closed pull request #54: Last files to need ASF License headers.

2018-05-23 Thread GitBox
houshengbo closed pull request #54: Last files to need ASF License headers. URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/54 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] houshengbo closed pull request #63: Last files to need ASF License headers.

2018-05-23 Thread GitBox
houshengbo closed pull request #63: Last files to need ASF License headers. URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/63 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] houshengbo closed pull request #278: Add ASF license header to .gradle files.

2018-05-23 Thread GitBox
houshengbo closed pull request #278: Add ASF license header to .gradle files. URL: https://github.com/apache/incubator-openwhisk-catalog/pull/278 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

[GitHub] mrutkows commented on issue #10: Some source files miss Apache license headers

2018-05-23 Thread GitBox
mrutkows commented on issue #10: Some source files miss Apache license headers URL: https://github.com/apache/incubator-openwhisk-runtime-python/issues/10#issuecomment-391377174 @daisy-ycguo can we close this issue? This is

[GitHub] starpit opened a new issue #119: add support for retries

2018-05-23 Thread GitBox
starpit opened a new issue #119: add support for retries URL: https://github.com/apache/incubator-openwhisk-client-js/issues/119 in some scenarios, the connection to openwhisk may be unreliable. for example, certain versions of nodejs on linux yield routine ETIMEDOUT or ESOCKETTIMEOUT or

[GitHub] dgrove-oss opened a new issue #203: travis smoke test should include apigw

2018-05-23 Thread GitBox
dgrove-oss opened a new issue #203: travis smoke test should include apigw URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/203 The smoke tests run by tools/travis/build-helm.sh should include `wsk api create` and invoking an API via curl. There's a commented out

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

2018-05-23 Thread GitBox
starpit 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] starpit opened a new pull request #118: add User-Agent to request header

2018-05-23 Thread GitBox
starpit opened a new pull request #118: add User-Agent to request header URL: https://github.com/apache/incubator-openwhisk-client-js/pull/118 This imposes a default User-Agent, if the client did not specify one. We could change this to leave it blank if not specified; but my opinion right

[GitHub] dgrove-oss commented on issue #202: Kuberentes v1.9.7 Deployment

2018-05-23 Thread GitBox
dgrove-oss commented on issue #202: Kuberentes v1.9.7 Deployment URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/202#issuecomment-391420724 ICLA now on file; merging PR. Thanks for the contribution!

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

2018-05-23 Thread GitBox
starpit 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] starpit opened a new pull request #118: add User-Agent to request header

2018-05-23 Thread GitBox
starpit opened a new pull request #118: add User-Agent to request header URL: https://github.com/apache/incubator-openwhisk-client-js/pull/118 This imposes a default User-Agent, if the client did not specify one. We could change this to leave it blank if not specified; but my opinion right

[GitHub] dgrove-oss closed pull request #202: Kuberentes v1.9.7 Deployment

2018-05-23 Thread GitBox
dgrove-oss closed pull request #202: Kuberentes v1.9.7 Deployment URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/202 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] matt-boyd opened a new pull request #3691: README: Available typo

2018-05-23 Thread GitBox
matt-boyd opened a new pull request #3691: README: Available typo URL: https://github.com/apache/incubator-openwhisk/pull/3691 Updating avaiable to available. ## Description ## Related issue and scope - [ ] I opened an issue to propose and discuss

[GitHub] jthomas commented on issue #126: Docker-Compose: Migrate to dockerize instead of curl

2018-05-23 Thread GitBox
jthomas commented on issue #126: Docker-Compose: Migrate to dockerize instead of curl URL: https://github.com/apache/incubator-openwhisk-devtools/issues/126#issuecomment-391354524 Hello @allustin, thanks for raising this issue. Is this something you are familiar with? Would you

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

2018-05-23 Thread GitBox
matt-boyd commented on issue #3691: README: Available typo URL: https://github.com/apache/incubator-openwhisk/pull/3691#issuecomment-391356727 No problem! :+1: This is an automated message from the Apache Git Service.

[GitHub] codecov-io commented on issue #3691: README: Available typo

2018-05-23 Thread GitBox
codecov-io commented on issue #3691: README: Available typo URL: https://github.com/apache/incubator-openwhisk/pull/3691#issuecomment-391367357 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3691?src=pr=h1) Report > Merging

[GitHub] houshengbo closed pull request #25: Last files to need ASF License headers.

2018-05-23 Thread GitBox
houshengbo closed pull request #25: Last files to need ASF License headers. URL: https://github.com/apache/incubator-openwhisk-runtime-python/pull/25 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] houshengbo closed pull request #305: Add ASF license header to .gradle files.

2018-05-23 Thread GitBox
houshengbo closed pull request #305: Add ASF license header to .gradle files. URL: https://github.com/apache/incubator-openwhisk-cli/pull/305 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

[GitHub] starpit opened a new pull request #118: add User-Agent to request header

2018-05-23 Thread GitBox
starpit opened a new pull request #118: add User-Agent to request header URL: https://github.com/apache/incubator-openwhisk-client-js/pull/118 Fixes #117 This is an automated message from the Apache Git Service. To respond to

[GitHub] markusthoemmes commented on a change in pull request #3669: Make stemcells configurable by deployment

2018-05-23 Thread GitBox
markusthoemmes commented on a change in pull request #3669: Make stemcells configurable by deployment URL: https://github.com/apache/incubator-openwhisk/pull/3669#discussion_r190341384 ## File path: common/scala/src/main/scala/whisk/core/entity/ExecManifest.scala ## @@

[GitHub] markusthoemmes commented on a change in pull request #3669: Make stemcells configurable by deployment

2018-05-23 Thread GitBox
markusthoemmes commented on a change in pull request #3669: Make stemcells configurable by deployment URL: https://github.com/apache/incubator-openwhisk/pull/3669#discussion_r190336091 ## File path: common/scala/src/main/scala/whisk/core/entity/ExecManifest.scala ## @@

[GitHub] markusthoemmes commented on a change in pull request #3669: Make stemcells configurable by deployment

2018-05-23 Thread GitBox
markusthoemmes commented on a change in pull request #3669: Make stemcells configurable by deployment URL: https://github.com/apache/incubator-openwhisk/pull/3669#discussion_r190343332 ## File path: common/scala/src/main/scala/whisk/core/entity/ExecManifest.scala ## @@

[GitHub] markusthoemmes commented on a change in pull request #3669: Make stemcells configurable by deployment

2018-05-23 Thread GitBox
markusthoemmes commented on a change in pull request #3669: Make stemcells configurable by deployment URL: https://github.com/apache/incubator-openwhisk/pull/3669#discussion_r190335820 ## File path: common/scala/src/main/scala/whisk/core/entity/ExecManifest.scala ## @@

[GitHub] rabbah opened a new issue #3693: new invokers cause load balanacer to reallocate black box invokers

2018-05-23 Thread GitBox
rabbah opened a new issue #3693: new invokers cause load balanacer to reallocate black box invokers URL: https://github.com/apache/incubator-openwhisk/issues/3693 The logic to update the set of managed and black box invokers will cause new invokers (when a sufficient number is added) to

[GitHub] tysonnorris commented on a change in pull request #3562: ArtifactStore implementation for CosmosDB

2018-05-23 Thread GitBox
tysonnorris commented on a change in pull request #3562: ArtifactStore implementation for CosmosDB URL: https://github.com/apache/incubator-openwhisk/pull/3562#discussion_r190358559 ## File path:

[GitHub] dgrove-oss opened a new pull request #3692: Slightly expand description of ICLA process.

2018-05-23 Thread GitBox
dgrove-oss opened a new pull request #3692: Slightly expand description of ICLA process. URL: https://github.com/apache/incubator-openwhisk/pull/3692 ## Description ## Related issue and scope - [ ] I opened an issue to propose and discuss this change (#)

[GitHub] dgrove-oss commented on issue #201: apigateway working with helm+minikube (NodePort)

2018-05-23 Thread GitBox
dgrove-oss commented on issue #201: apigateway working with helm+minikube (NodePort) URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/201#issuecomment-391426736 @csantanapr -- this is ready to merge when you have time. I gave up on adding the apigateway test

[GitHub] dgrove-oss closed issue #63: Helm chart to orchestrate deployment

2018-05-23 Thread GitBox
dgrove-oss closed issue #63: Helm chart to orchestrate deployment URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/63 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] csantanapr commented on a change in pull request #3689: Enable extending environment variables of Controller and Invoker

2018-05-23 Thread GitBox
csantanapr commented on a change in pull request #3689: Enable extending environment variables of Controller and Invoker URL: https://github.com/apache/incubator-openwhisk/pull/3689#discussion_r190335870 ## File path: ansible/environments/local/group_vars/all ## @@ -2,6

[GitHub] codecov-io commented on issue #3692: Slightly expand description of ICLA process.

2018-05-23 Thread GitBox
codecov-io commented on issue #3692: Slightly expand description of ICLA process. URL: https://github.com/apache/incubator-openwhisk/pull/3692#issuecomment-391436468 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3692?src=pr=h1) Report > Merging

[GitHub] pritidesai commented on issue #919: GitHub "revert" task/code fails; "code: already_exists"

2018-05-23 Thread GitBox
pritidesai commented on issue #919: GitHub "revert" task/code fails; "code: already_exists" URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/919#issuecomment-391455560 It creates this weird draft release on release page:

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

2018-05-23 Thread GitBox
tysonnorris commented on issue #3562: ArtifactStore implementation for CosmosDB URL: https://github.com/apache/incubator-openwhisk/pull/3562#issuecomment-391457832 I think adding `ArtifactStoreProvider.shutdown()` seems like a fine addition, if calling `client.close()` multiple times is

[GitHub] rabbah commented on a change in pull request #3669: Make stemcells configurable by deployment

2018-05-23 Thread GitBox
rabbah commented on a change in pull request #3669: Make stemcells configurable by deployment URL: https://github.com/apache/incubator-openwhisk/pull/3669#discussion_r190365020 ## File path: common/scala/src/main/scala/whisk/core/entity/ExecManifest.scala ## @@ -285,6

[GitHub] rabbah commented on a change in pull request #3669: Make stemcells configurable by deployment

2018-05-23 Thread GitBox
rabbah commented on a change in pull request #3669: Make stemcells configurable by deployment URL: https://github.com/apache/incubator-openwhisk/pull/3669#discussion_r190364979 ## File path: common/scala/src/main/scala/whisk/core/entity/ExecManifest.scala ## @@ -285,6

[GitHub] rabbah commented on a change in pull request #3669: Make stemcells configurable by deployment

2018-05-23 Thread GitBox
rabbah commented on a change in pull request #3669: Make stemcells configurable by deployment URL: https://github.com/apache/incubator-openwhisk/pull/3669#discussion_r190365264 ## File path: common/scala/src/main/scala/whisk/core/entity/ExecManifest.scala ## @@ -262,13

[GitHub] pritidesai opened a new pull request #923: an attempt to enable api gateway test in travis

2018-05-23 Thread GitBox
pritidesai opened a new pull request #923: an attempt to enable api gateway test in travis URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/923 This is an automated message from the Apache Git Service. To

  1   2   >