[GitHub] ningyougang commented on issue #3606: Increase nginx security by adapting ssl_ciphers.

2018-05-10 Thread GitBox
ningyougang commented on issue #3606: Increase nginx security by adapting ssl_ciphers. URL: https://github.com/apache/incubator-openwhisk/pull/3606#issuecomment-388244316 oh. @cbickel @markusthoemmes , after rebased, some test cases can't be passed in my local build env. for example

[GitHub] ningyougang commented on issue #3606: Increase nginx security by adapting ssl_ciphers.

2018-05-10 Thread GitBox
ningyougang commented on issue #3606: Increase nginx security by adapting ssl_ciphers. URL: https://github.com/apache/incubator-openwhisk/pull/3606#issuecomment-388244316 oh. @cbickel @markusthoemmes , after rebased, some test cases can't be passed in my local build env. for example

[GitHub] ningyougang commented on issue #3606: Increase nginx security by adapting ssl_ciphers.

2018-05-10 Thread GitBox
ningyougang commented on issue #3606: Increase nginx security by adapting ssl_ciphers. URL: https://github.com/apache/incubator-openwhisk/pull/3606#issuecomment-388244316 oh. @cbickel @markusthoemmes , after rebased, some test cases can't be passed in my local build env. for example

[GitHub] shazron commented on issue #3638: Remove Consul from "The Internal flow of processing" section diagram in about.md

2018-05-10 Thread GitBox
shazron commented on issue #3638: Remove Consul from "The Internal flow of processing" section diagram in about.md URL: https://github.com/apache/incubator-openwhisk/pull/3638#issuecomment-388240629 @csantanapr Updated to remove process numbers.

[GitHub] dgrove-oss commented on issue #194: Adopt latest scancode ASF release config. (enforce *.md, .conf, .sh).

2018-05-10 Thread GitBox
dgrove-oss commented on issue #194: Adopt latest scancode ASF release config. (enforce *.md, .conf, .sh). URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/194#issuecomment-388233632 ok. I'll do something similar to the gethelm fix you just merged for me and have

[GitHub] mrutkows commented on issue #194: Adopt latest scancode ASF release config. (enforce *.md, .conf, .sh).

2018-05-10 Thread GitBox
mrutkows commented on issue #194: Adopt latest scancode ASF release config. (enforce *.md, .conf, .sh). URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/194#issuecomment-388205602 @dgrove-oss apparently the openwhisk CLI repo's README.md is overwriting this project's

[GitHub] mrutkows commented on issue #194: Adopt latest scancode ASF release config. (enforce *.md, .conf, .sh).

2018-05-10 Thread GitBox
mrutkows commented on issue #194: Adopt latest scancode ASF release config. (enforce *.md, .conf, .sh). URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/194#issuecomment-388205602 @dgrove-oss apparently the openwhisk CLI repo's README.md is overwriting this project's

[GitHub] mrutkows commented on issue #194: Adopt latest scancode ASF release config. (enforce *.md, .conf, .sh).

2018-05-10 Thread GitBox
mrutkows commented on issue #194: Adopt latest scancode ASF release config. (enforce *.md, .conf, .sh). URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/194#issuecomment-388205602 @dgrove-oss apparently the openwhisk CLI repo's README.md is overwriting this project's

[GitHub] mrutkows commented on issue #194: Adopt latest scancode ASF release config. (enforce *.md, .conf, .sh).

2018-05-10 Thread GitBox
mrutkows commented on issue #194: Adopt latest scancode ASF release config. (enforce *.md, .conf, .sh). URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/194#issuecomment-388205602 @dgrove-oss apparently the openwhisk CLI repo's README.md is overwriting this project's

[GitHub] mrutkows commented on issue #194: Adopt latest scancode ASF release config. (enforce *.md, .conf, .sh).

2018-05-10 Thread GitBox
mrutkows commented on issue #194: Adopt latest scancode ASF release config. (enforce *.md, .conf, .sh). URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/194#issuecomment-388205602 @dgrove-oss apparently the openwhisk CLI repo's README.md is overwriting this project's

[GitHub] csantanapr commented on issue #3616: Use ‘docker.host’ java property as a way to set Docker access for Runtime unit tests

2018-05-10 Thread GitBox
csantanapr commented on issue #3616: Use ‘docker.host’ java property as a way to set Docker access for Runtime unit tests URL: https://github.com/apache/incubator-openwhisk/pull/3616#issuecomment-388192413 New PG1 2889 ⌛️

[GitHub] dgrove-oss commented on issue #3607: Change the JDK to use openjdk8-openj9

2018-05-10 Thread GitBox
dgrove-oss commented on issue #3607: Change the JDK to use openjdk8-openj9 URL: https://github.com/apache/incubator-openwhisk/issues/3607#issuecomment-388189327 Quoting from the adoptopenjdk license file (https://hub.docker.com/r/adoptopenjdk/openjdk8-openj9/). > > The

[GitHub] mrutkows closed pull request #195: download/install helm from /tmp

2018-05-10 Thread GitBox
mrutkows closed pull request #195: download/install helm from /tmp URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/195 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this

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

2018-05-10 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-388186291 @kpavel we have `action. Exec.Main` set in such cases, please populate `main` in

[GitHub] brunogirin opened a new issue #900: jar files are exported for each Java action even when they are shared

2018-05-10 Thread GitBox
brunogirin opened a new issue #900: jar files are exported for each Java action even when they are shared URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/900 When running `wskdeploy export`, the tool downloads all artefacts related to the project's package. If the

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

2018-05-10 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-388186291 @kpavel we have `action. Exec.Main` set in such cases, please populate `main` in

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

2018-05-10 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-388185146 @kpavel can you please add support for multiple packages while exporting ow entities?

[GitHub] brunogirin opened a new issue #899: wskdeploy export fails to export the `main` attribute for Java actions

2018-05-10 Thread GitBox
brunogirin opened a new issue #899: wskdeploy export fails to export the `main` attribute for Java actions URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/899 When running `wskdeploy` on a project that has Java actions, it fails to export the `main` attribute for those

[GitHub] brunogirin opened a new issue #898: wskdeploy export doesn't support multiple packages in a single project

2018-05-10 Thread GitBox
brunogirin opened a new issue #898: wskdeploy export doesn't support multiple packages in a single project URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/898 When running `wskdeploy` on a project that contains multiple packages, I get the following message: ```

[GitHub] brunogirin commented on issue #890: Resolves issue #880

2018-05-10 Thread GitBox
brunogirin commented on issue #890: Resolves issue #880 URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/890#issuecomment-388179719 As just discussed on Slack, I was doing something wrong and have now got it working, thanks @pritidesai for the help! That said, I've got a

[GitHub] ddragosd closed pull request #3541: Point Getting Started to docker-compose

2018-05-10 Thread GitBox
ddragosd closed pull request #3541: Point Getting Started to docker-compose URL: https://github.com/apache/incubator-openwhisk/pull/3541 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] dgrove-oss opened a new pull request #195: download/install helm from /tmp

2018-05-10 Thread GitBox
dgrove-oss opened a new pull request #195: download/install helm from /tmp URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/195 This is an automated message from the Apache Git Service. To respond to the

[GitHub] mrutkows commented on issue #194: Adopt latest scancode ASF release config. (enforce *.md, .conf, .sh).

2018-05-10 Thread GitBox
mrutkows commented on issue #194: Adopt latest scancode ASF release config. (enforce *.md, .conf, .sh). URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/194#issuecomment-388171140 @dgrove-oss just saw same in travis logs... those paths do not make sense... the raw

[GitHub] mrutkows commented on issue #194: Adopt latest scancode ASF release config. (enforce *.md, .conf, .sh).

2018-05-10 Thread GitBox
mrutkows commented on issue #194: Adopt latest scancode ASF release config. (enforce *.md, .conf, .sh). URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/194#issuecomment-388171140 @dgrove-oss just saw same in travis logs, thought I updated my fork before the push

[GitHub] dgrove-oss commented on issue #194: Adopt latest scancode ASF release config. (enforce *.md, .conf, .sh).

2018-05-10 Thread GitBox
dgrove-oss commented on issue #194: Adopt latest scancode ASF release config. (enforce *.md, .conf, .sh). URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/194#issuecomment-388170942 ``` Scan detected 2 error(s) in 2 file(s):

[GitHub] mrutkows opened a new pull request #194: Adopt latest scancode ASF release config. (enforce *.md, .conf, .sh).

2018-05-10 Thread GitBox
mrutkows opened a new pull request #194: Adopt latest scancode ASF release config. (enforce *.md, .conf, .sh). URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/194 This is an automated message from the

[GitHub] pritidesai commented on issue #890: Resolves issue #880

2018-05-10 Thread GitBox
pritidesai commented on issue #890: Resolves issue #880 URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/890#issuecomment-388161147 I am still running into document update conflict on IBM Cloud Functions: ```"response":{"result":{"error":{"error":"error creating

[GitHub] pritidesai commented on issue #890: Resolves issue #880

2018-05-10 Thread GitBox
pritidesai commented on issue #890: Resolves issue #880 URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/890#issuecomment-388158828 @brunogirin the latest build has @kpavel's changes. You dont have to build it from source. I am trying to test them with cloudant use case

[GitHub] dgrove-oss opened a new pull request #193: Flatten and simplify Helm charts

2018-05-10 Thread GitBox
dgrove-oss opened a new pull request #193: Flatten and simplify Helm charts URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/193 Consolidate into a single level of charts to simplify structure. This is an

[GitHub] houshengbo closed pull request #34: Align both versions of release configuration and remove Hash license with typo.

2018-05-10 Thread GitBox
houshengbo closed pull request #34: Align both versions of release configuration and remove Hash license with typo. URL: https://github.com/apache/incubator-openwhisk-utilities/pull/34 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is

[GitHub] houshengbo opened a new pull request #147: Archive the license and notice work for openwhisk projects

2018-05-10 Thread GitBox
houshengbo opened a new pull request #147: Archive the license and notice work for openwhisk projects URL: https://github.com/apache/incubator-openwhisk-release/pull/147 This is an automated message from the Apache Git

[GitHub] mrutkows opened a new pull request #34: Align both versions of release configuration and remove Hash license with typo.

2018-05-10 Thread GitBox
mrutkows opened a new pull request #34: Align both versions of release configuration and remove Hash license with typo. URL: https://github.com/apache/incubator-openwhisk-utilities/pull/34 This is an automated message from

[GitHub] ddragosd commented on issue #3541: Point Getting Started to docker-compose

2018-05-10 Thread GitBox
ddragosd commented on issue #3541: Point Getting Started to docker-compose URL: https://github.com/apache/incubator-openwhisk/pull/3541#issuecomment-388149270 @rabbah thanks for testing on a fresh system. The `WARNING` messages from Ansible should be fine b.c ansible usage is limited to

[GitHub] rabbah commented on issue #3541: Point Getting Started to docker-compose

2018-05-10 Thread GitBox
rabbah commented on issue #3541: Point Getting Started to docker-compose URL: https://github.com/apache/incubator-openwhisk/pull/3541#issuecomment-388147804 On a fresh system, got one error and some warnings: ``` cat: /Users/rabbah/tmp/openwhisk/local.env: No such file or

[GitHub] daisy-ycguo closed pull request #145: Fix markdown files with a typo. in the Hash license header.

2018-05-10 Thread GitBox
daisy-ycguo closed pull request #145: Fix markdown files with a typo. in the Hash license header. URL: https://github.com/apache/incubator-openwhisk-release/pull/145 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] ddragosd commented on issue #3541: Point Getting Started to docker-compose

2018-05-10 Thread GitBox
ddragosd commented on issue #3541: Point Getting Started to docker-compose URL: https://github.com/apache/incubator-openwhisk/pull/3541#issuecomment-388140456 With the latest additions to `docker-compose` which now supports the API Management, I think we can safely merge this, as anyone

[GitHub] mrutkows commented on issue #146: Get rid of Maven, since it is useless

2018-05-10 Thread GitBox
mrutkows commented on issue #146: Get rid of Maven, since it is useless URL: https://github.com/apache/incubator-openwhisk-release/pull/146#issuecomment-388133853 This solved "apt-get" fetch error (somehow travis/ubuntu image was using an old dependency list): ``` Failed to fetch

[GitHub] mrutkows closed pull request #146: Get rid of Maven, since it is useless

2018-05-10 Thread GitBox
mrutkows closed pull request #146: Get rid of Maven, since it is useless URL: https://github.com/apache/incubator-openwhisk-release/pull/146 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this

[GitHub] mrutkows commented on issue #146: Get rid of Maven, since it is useless

2018-05-10 Thread GitBox
mrutkows commented on issue #146: Get rid of Maven, since it is useless URL: https://github.com/apache/incubator-openwhisk-release/pull/146#issuecomment-388133853 This solved "apt-get" fetch error (somehow travis/ubuntu image was using an old dependency list): ``` Failed to fetch

[GitHub] brunogirin commented on issue #890: Resolves issue #880

2018-05-10 Thread GitBox
brunogirin commented on issue #890: Resolves issue #880 URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/890#issuecomment-388120976 I still have the same problem when running the export with `latest` so I must be doing something wrong. Has the fix made it into latest yet

[GitHub] rabbah commented on issue #3629: [Feature requests] action protection

2018-05-10 Thread GitBox
rabbah commented on issue #3629: [Feature requests] action protection URL: https://github.com/apache/incubator-openwhisk/issues/3629#issuecomment-388104701 There is a general issue in that referenced entities are not tracked at all today - applies to rules, packages, sequences, and

[GitHub] mrutkows closed pull request #3633: Fix markdown files with a typo. in the Hash license header.

2018-05-10 Thread GitBox
mrutkows closed pull request #3633: Fix markdown files with a typo. in the Hash license header. URL: https://github.com/apache/incubator-openwhisk/pull/3633 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 opened a new pull request #146: WIP: fix the travis issues

2018-05-10 Thread GitBox
houshengbo opened a new pull request #146: WIP: fix the travis issues URL: https://github.com/apache/incubator-openwhisk-release/pull/146 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] codecov-io commented on issue #3633: Fix markdown files with a typo. in the Hash license header.

2018-05-10 Thread GitBox
codecov-io commented on issue #3633: Fix markdown files with a typo. in the Hash license header. URL: https://github.com/apache/incubator-openwhisk/pull/3633#issuecomment-387887974 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3633?src=pr=h1) Report > Merging

[GitHub] houshengbo closed pull request #144: Update header scan policy document

2018-05-10 Thread GitBox
houshengbo closed pull request #144: Update header scan policy document URL: https://github.com/apache/incubator-openwhisk-release/pull/144 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this

[GitHub] brunogirin commented on issue #890: Resolves issue #880

2018-05-10 Thread GitBox
brunogirin commented on issue #890: Resolves issue #880 URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/890#issuecomment-388096977 @pritidesai will do as soon as I can This is an automated message from the

[GitHub] ddragosd commented on issue #3629: [Feature requests] action protection

2018-05-10 Thread GitBox
ddragosd commented on issue #3629: [Feature requests] action protection URL: https://github.com/apache/incubator-openwhisk/issues/3629#issuecomment-388096059 I also feel like a developer should have full control over the action. Maybe a warning like - “you’re deleting an action

[GitHub] rabbah opened a new issue #3641: deprecation warning in couch playbook

2018-05-10 Thread GitBox
rabbah opened a new issue #3641: deprecation warning in couch playbook URL: https://github.com/apache/incubator-openwhisk/issues/3641 ``` TASK [couchdb : create '_users' database for singleton mode]

[GitHub] rabbah opened a new issue #3640: variable warning in controller playbook

2018-05-10 Thread GitBox
rabbah opened a new issue #3640: variable warning in controller playbook URL: https://github.com/apache/incubator-openwhisk/issues/3640 ``` TASK [controller : add seed nodes to controller environment]

[GitHub] rabbah opened a new issue #3639: jinja warning from controller playbook

2018-05-10 Thread GitBox
rabbah opened a new issue #3639: jinja warning from controller playbook URL: https://github.com/apache/incubator-openwhisk/issues/3639 ``` TASK [nginx : copy controller cert for authentication]

[GitHub] mrutkows opened a new pull request #145: Fix markdown files with a typo. in the Hash license header.

2018-05-10 Thread GitBox
mrutkows opened a new pull request #145: Fix markdown files with a typo. in the Hash license header. URL: https://github.com/apache/incubator-openwhisk-release/pull/145 This is an automated message from the Apache Git

[GitHub] houshengbo closed pull request #274: Fix markdown files with a typo. in the Hash license header.

2018-05-10 Thread GitBox
houshengbo closed pull request #274: Fix markdown files with a typo. in the Hash license header. URL: https://github.com/apache/incubator-openwhisk-catalog/pull/274 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] codecov-io commented on issue #3619: Provide an activation store SPI

2018-05-10 Thread GitBox
codecov-io commented on issue #3619: Provide an activation store SPI URL: https://github.com/apache/incubator-openwhisk/pull/3619#issuecomment-387224274 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3619?src=pr=h1) Report > Merging

[GitHub] houshengbo closed pull request #83: Fix markdown files with a typo. in the Hash license header.

2018-05-10 Thread GitBox
houshengbo closed pull request #83: Fix markdown files with a typo. in the Hash license header. URL: https://github.com/apache/incubator-openwhisk-client-go/pull/83 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] houshengbo closed pull request #36: Fix markdown files with a typo. in the Hash license header.

2018-05-10 Thread GitBox
houshengbo closed pull request #36: Fix markdown files with a typo. in the Hash license header. URL: https://github.com/apache/incubator-openwhisk-runtime-docker/pull/36 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for

[GitHub] houshengbo closed pull request #50: Fix markdown files with a typo. in the Hash license header.

2018-05-10 Thread GitBox
houshengbo closed pull request #50: Fix markdown files with a typo. in the Hash license header. URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/50 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] houshengbo closed pull request #49: Fix markdown files with a typo. in the Hash license header.

2018-05-10 Thread GitBox
houshengbo closed pull request #49: Fix markdown files with a typo. in the Hash license header. URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/49 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for

[GitHub] houshengbo closed pull request #20: Fix markdown files with a typo. in the Hash license header.

2018-05-10 Thread GitBox
houshengbo closed pull request #20: Fix markdown files with a typo. in the Hash license header. URL: https://github.com/apache/incubator-openwhisk-runtime-php/pull/20 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] houshengbo closed pull request #22: Fix markdown files with a typo. in the Hash license header.

2018-05-10 Thread GitBox
houshengbo closed pull request #22: Fix markdown files with a typo. in the Hash license header. URL: https://github.com/apache/incubator-openwhisk-runtime-python/pull/22 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for

[GitHub] houshengbo closed pull request #57: Fix markdown files with a typo. in the Hash license header.

2018-05-10 Thread GitBox
houshengbo closed pull request #57: Fix markdown files with a typo. in the Hash license header. URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/57 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] mrutkows opened a new pull request #274: Fix markdown files with a typo. in the Hash license header.

2018-05-10 Thread GitBox
mrutkows opened a new pull request #274: Fix markdown files with a typo. in the Hash license header. URL: https://github.com/apache/incubator-openwhisk-catalog/pull/274 This is an automated message from the Apache Git

[GitHub] mrutkows opened a new pull request #83: Fix markdown files with a typo. in the Hash license header.

2018-05-10 Thread GitBox
mrutkows opened a new pull request #83: Fix markdown files with a typo. in the Hash license header. URL: https://github.com/apache/incubator-openwhisk-client-go/pull/83 This is an automated message from the Apache Git

[GitHub] dubee closed pull request #3636: update docker runtime from 1.2.0 to 1.3.0

2018-05-10 Thread GitBox
dubee closed pull request #3636: update docker runtime from 1.2.0 to 1.3.0 URL: https://github.com/apache/incubator-openwhisk/pull/3636 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] mrutkows opened a new pull request #36: Fix markdown files with a typo. in the Hash license header.

2018-05-10 Thread GitBox
mrutkows opened a new pull request #36: Fix markdown files with a typo. in the Hash license header. URL: https://github.com/apache/incubator-openwhisk-runtime-docker/pull/36 This is an automated message from the Apache Git

[GitHub] dubee commented on issue #2847: Treat action code as attachments for created/updated actions

2018-05-10 Thread GitBox
dubee commented on issue #2847: Treat action code as attachments for created/updated actions URL: https://github.com/apache/incubator-openwhisk/pull/2847#issuecomment-388063998 Closed in favor of https://github.com/apache/incubator-openwhisk/pull/3286

[GitHub] dubee closed pull request #2847: Treat action code as attachments for created/updated actions

2018-05-10 Thread GitBox
dubee closed pull request #2847: Treat action code as attachments for created/updated actions URL: https://github.com/apache/incubator-openwhisk/pull/2847 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] dubee commented on issue #3286: Treat action code as attachments for created/updated actions

2018-05-10 Thread GitBox
dubee commented on issue #3286: Treat action code as attachments for created/updated actions URL: https://github.com/apache/incubator-openwhisk/pull/3286#issuecomment-388063938 Closing https://github.com/apache/incubator-openwhisk/pull/2847 in favor of this PR as it was easier to rebase

[GitHub] mrutkows opened a new pull request #50: Fix markdown files with a typo. in the Hash license header.

2018-05-10 Thread GitBox
mrutkows opened a new pull request #50: Fix markdown files with a typo. in the Hash license header. URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/50 This is an automated message from the Apache Git

[GitHub] mrutkows opened a new pull request #49: Fix markdown files with a typo. in the Hash license header.

2018-05-10 Thread GitBox
mrutkows opened a new pull request #49: Fix markdown files with a typo. in the Hash license header. URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/49 This is an automated message from the Apache Git

[GitHub] mrutkows opened a new pull request #20: Fix markdown files with a typo. in the Hash license header.

2018-05-10 Thread GitBox
mrutkows opened a new pull request #20: Fix markdown files with a typo. in the Hash license header. URL: https://github.com/apache/incubator-openwhisk-runtime-php/pull/20 This is an automated message from the Apache Git

[GitHub] mrutkows opened a new pull request #22: Fix markdown files with a typo. in the Hash license header.

2018-05-10 Thread GitBox
mrutkows opened a new pull request #22: Fix markdown files with a typo. in the Hash license header. URL: https://github.com/apache/incubator-openwhisk-runtime-python/pull/22 This is an automated message from the Apache Git

[GitHub] mrutkows opened a new pull request #57: Fix markdown files with a typo. in the Hash license header.

2018-05-10 Thread GitBox
mrutkows opened a new pull request #57: Fix markdown files with a typo. in the Hash license header. URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/57 This is an automated message from the Apache Git

[GitHub] jasonpet commented on issue #146: Add worker to logging

2018-05-10 Thread GitBox
jasonpet commented on issue #146: Add worker to logging URL: https://github.com/apache/incubator-openwhisk-package-alarms/pull/146#issuecomment-388052693 yeah, I definitely agree that they are important for anyone using the HA support for alarms

[GitHub] csantanapr commented on issue #3207: Remove Consul from "The Internal flow of processing" section diagram in about.md

2018-05-10 Thread GitBox
csantanapr commented on issue #3207: Remove Consul from "The Internal flow of processing" section diagram in about.md URL: https://github.com/apache/incubator-openwhisk/issues/3207#issuecomment-388040747 Nice  This is an

[GitHub] mkouba commented on issue #42: Add support for Kotlin actions

2018-05-10 Thread GitBox
mkouba commented on issue #42: Add support for Kotlin actions URL: https://github.com/apache/incubator-openwhisk-runtime-java/issues/42#issuecomment-388030669 > The experimental runtime also adds a data class based API so that you can avoid having to work with JSON directly. This

[GitHub] seabaylea commented on issue #42: Add support for Kotlin actions

2018-05-10 Thread GitBox
seabaylea commented on issue #42: Add support for Kotlin actions URL: https://github.com/apache/incubator-openwhisk-runtime-java/issues/42#issuecomment-388029720 There's really two reasons for having a specific Kotlin runtime: 1. Platform and dependencies As @mkouba says, it

[GitHub] codecov-io commented on issue #3638: Remove Consul from "The Internal flow of processing" section diagram in about.md

2018-05-10 Thread GitBox
codecov-io commented on issue #3638: Remove Consul from "The Internal flow of processing" section diagram in about.md URL: https://github.com/apache/incubator-openwhisk/pull/3638#issuecomment-387984721 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3638?src=pr=h1)

[GitHub] shazron commented on issue #3207: Remove Consul from "The Internal flow of processing" section diagram in about.md

2018-05-10 Thread GitBox
shazron commented on issue #3207: Remove Consul from "The Internal flow of processing" section diagram in about.md URL: https://github.com/apache/incubator-openwhisk/issues/3207#issuecomment-387974802 Actually, filed #3638 with the source just in case I don't get back to this issue in a

[GitHub] shazron opened a new pull request #3638: Remove Consul from "The Internal flow of processing" section diagram in about.md

2018-05-10 Thread GitBox
shazron opened a new pull request #3638: Remove Consul from "The Internal flow of processing" section diagram in about.md URL: https://github.com/apache/incubator-openwhisk/pull/3638 … Added draw.io source file as well. All logos are vectors. ## Description This changes the

[GitHub] KeonHee opened a new issue #3637: Recognize code as binary

2018-05-10 Thread GitBox
KeonHee opened a new issue #3637: Recognize code as binary URL: https://github.com/apache/incubator-openwhisk/issues/3637 ## Environment details: * CentOS distributed environment * Mac OS cli ## Steps to reproduce the issue: 1. Create a `test.js` file

[GitHub] shazron commented on issue #3207: Remove Consul from "The Internal flow of processing" section diagram in about.md

2018-05-10 Thread GitBox
shazron commented on issue #3207: Remove Consul from "The Internal flow of processing" section diagram in about.md URL: https://github.com/apache/incubator-openwhisk/issues/3207#issuecomment-387970928 How's this for the diagram I created in draw.io? If the numbering is right, I'll send a

[GitHub] shazron commented on issue #3207: Remove Consul from "The Internal flow of processing" section diagram in about.md

2018-05-10 Thread GitBox
shazron commented on issue #3207: Remove Consul from "The Internal flow of processing" section diagram in about.md URL: https://github.com/apache/incubator-openwhisk/issues/3207#issuecomment-387970928 How's this for the diagram I created in draw.io? If the numbering is right, I'll send a

[GitHub] KeonHee commented on issue #146: Add worker to logging

2018-05-10 Thread GitBox
KeonHee commented on issue #146: Add worker to logging URL: https://github.com/apache/incubator-openwhisk-package-alarms/pull/146#issuecomment-387965890 @jasonpet Thank you for your review. I think it is good to include workers and hosts in the logs explicitly because they are