[GitHub] dgrove-oss closed pull request #380: Document ICP and Dynamic DNS Provisioning

2018-11-30 Thread GitBox
dgrove-oss closed pull request #380: Document ICP and Dynamic DNS Provisioning URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/380 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 a change in pull request #388: Save Code Associated with Blackbox Actions

2018-11-30 Thread GitBox
dubee commented on a change in pull request #388: Save Code Associated with Blackbox Actions URL: https://github.com/apache/incubator-openwhisk-cli/pull/388#discussion_r237898444 ## File path: commands/action.go ## @@ -692,7 +692,7 @@ func saveCode(action whisk.Action,

[GitHub] dgrove-oss commented on issue #378: cleanup specification of storageclasses and persistence

2018-11-30 Thread GitBox
dgrove-oss commented on issue #378: cleanup specification of storageclasses and persistence URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/378#issuecomment-443255218 i'm going to merge #380 first, then reconcile the changed variable names and enhance the docs as

[GitHub] dubee commented on a change in pull request #388: Save Code Associated with Blackbox Actions

2018-11-30 Thread GitBox
dubee commented on a change in pull request #388: Save Code Associated with Blackbox Actions URL: https://github.com/apache/incubator-openwhisk-cli/pull/388#discussion_r237894295 ## File path: commands/action.go ## @@ -692,7 +692,7 @@ func saveCode(action whisk.Action,

[GitHub] moritzraho opened a new pull request #330: Security: Disable use of X-Accel-Redirect header

2018-11-30 Thread GitBox
moritzraho opened a new pull request #330: Security: Disable use of X-Accel-Redirect header URL: https://github.com/apache/incubator-openwhisk-apigateway/pull/330 This PR addresses a security issue in which web actions could access internal locations defined in the nginx config files by

[GitHub] dubee commented on a change in pull request #388: Save Code Associated with Blackbox Actions

2018-11-30 Thread GitBox
dubee commented on a change in pull request #388: Save Code Associated with Blackbox Actions URL: https://github.com/apache/incubator-openwhisk-cli/pull/388#discussion_r237899789 ## File path: commands/action.go ## @@ -692,7 +692,7 @@ func saveCode(action whisk.Action,

[GitHub] dubee commented on a change in pull request #388: Save Code Associated with Blackbox Actions

2018-11-30 Thread GitBox
dubee commented on a change in pull request #388: Save Code Associated with Blackbox Actions URL: https://github.com/apache/incubator-openwhisk-cli/pull/388#discussion_r237898444 ## File path: commands/action.go ## @@ -692,7 +692,7 @@ func saveCode(action whisk.Action,

[GitHub] dubee commented on a change in pull request #388: Save Code Associated with Blackbox Actions

2018-11-30 Thread GitBox
dubee commented on a change in pull request #388: Save Code Associated with Blackbox Actions URL: https://github.com/apache/incubator-openwhisk-cli/pull/388#discussion_r237898444 ## File path: commands/action.go ## @@ -692,7 +692,7 @@ func saveCode(action whisk.Action,

[GitHub] dubee commented on a change in pull request #388: Save Code Associated with Blackbox Actions

2018-11-30 Thread GitBox
dubee commented on a change in pull request #388: Save Code Associated with Blackbox Actions URL: https://github.com/apache/incubator-openwhisk-cli/pull/388#discussion_r237899789 ## File path: commands/action.go ## @@ -692,7 +692,7 @@ func saveCode(action whisk.Action,

[GitHub] pritidesai opened a new issue #1018: Find out how to package runtimes.json into wskdeploy binary

2018-11-30 Thread GitBox
pritidesai opened a new issue #1018: Find out how to package runtimes.json into wskdeploy binary URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/1018 As a quick fix in PR #1017, I have moved runtime `json` into `runtimes.go` but we want to bring back `runtimes.json`

[GitHub] pritidesai commented on issue #1017: Deleting static runtimes data file, instead, adding data to go file

2018-11-30 Thread GitBox
pritidesai commented on issue #1017: Deleting static runtimes data file, instead, adding data to go file URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/1017#issuecomment-443309146 We are merging this PR but please address this issue

[GitHub] dubee commented on a change in pull request #388: Save Code Associated with Blackbox Actions

2018-11-30 Thread GitBox
dubee commented on a change in pull request #388: Save Code Associated with Blackbox Actions URL: https://github.com/apache/incubator-openwhisk-cli/pull/388#discussion_r237894295 ## File path: commands/action.go ## @@ -692,7 +692,7 @@ func saveCode(action whisk.Action,

[GitHub] starpit opened a new issue #13: conductor should obey INSECURE_SSL in wskprops

2018-11-30 Thread GitBox
starpit opened a new issue #13: conductor should obey INSECURE_SSL in wskprops URL: https://github.com/apache/incubator-openwhisk-composer/issues/13 conductor parses out wskprops, but ignores the INSECURE_SSL setting that might be asserted in that file:

[GitHub] dgrove-oss opened a new issue #382: Improve configuration of invoker.containerFactory.nameservers

2018-11-30 Thread GitBox
dgrove-oss opened a new issue #382: Improve configuration of invoker.containerFactory.nameservers URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/382 When the providers are enabled with the DockerContainerFactory, we should probably default the nameserver

[GitHub] rabbah commented on a change in pull request #4129: Adding YARNContainerFactory. This allows OpenWhisk to run actions on Apache Hadoop clusters.

2018-11-30 Thread GitBox
rabbah commented on a change in pull request #4129: Adding YARNContainerFactory. This allows OpenWhisk to run actions on Apache Hadoop clusters. URL: https://github.com/apache/incubator-openwhisk/pull/4129#discussion_r237980462 ## File path:

[GitHub] rabbah commented on a change in pull request #4129: Adding YARNContainerFactory. This allows OpenWhisk to run actions on Apache Hadoop clusters.

2018-11-30 Thread GitBox
rabbah commented on a change in pull request #4129: Adding YARNContainerFactory. This allows OpenWhisk to run actions on Apache Hadoop clusters. URL: https://github.com/apache/incubator-openwhisk/pull/4129#discussion_r237980039 ## File path:

[GitHub] rabbah commented on a change in pull request #4129: Adding YARNContainerFactory. This allows OpenWhisk to run actions on Apache Hadoop clusters.

2018-11-30 Thread GitBox
rabbah commented on a change in pull request #4129: Adding YARNContainerFactory. This allows OpenWhisk to run actions on Apache Hadoop clusters. URL: https://github.com/apache/incubator-openwhisk/pull/4129#discussion_r237981086 ## File path:

[GitHub] rabbah commented on a change in pull request #4129: Adding YARNContainerFactory. This allows OpenWhisk to run actions on Apache Hadoop clusters.

2018-11-30 Thread GitBox
rabbah commented on a change in pull request #4129: Adding YARNContainerFactory. This allows OpenWhisk to run actions on Apache Hadoop clusters. URL: https://github.com/apache/incubator-openwhisk/pull/4129#discussion_r237981794 ## File path:

[GitHub] dgrove-oss commented on issue #378: cleanup specification of storageclasses and persistence

2018-11-30 Thread GitBox
dgrove-oss commented on issue #378: cleanup specification of storageclasses and persistence URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/378#issuecomment-443342337 ready to be merged. This is an

[GitHub] dgrove-oss commented on issue #379: Where is docs/ingress.md?

2018-11-30 Thread GitBox
dgrove-oss commented on issue #379: Where is docs/ingress.md? URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/379#issuecomment-443348326 @xieweicarl2018 - thanks for reporting the dangling reference in the docs. I have submitted a PR to fix values.yaml (as

[GitHub] jonpspri opened a new pull request #109: Add CA certificate processing

2018-11-30 Thread GitBox
jonpspri opened a new pull request #109: Add CA certificate processing URL: https://github.com/apache/incubator-openwhisk-client-go/pull/109 Create a '--cacert' flag that allows `wsk` to honor self-signing CAs if the certificate is provided. While not eliminating the need for `wsk -i`

[GitHub] jonpspri opened a new pull request #389: Add CA certificate processing.

2018-11-30 Thread GitBox
jonpspri opened a new pull request #389: Add CA certificate processing. URL: https://github.com/apache/incubator-openwhisk-cli/pull/389 Create a '--cacert' flag that allows wsk to honor self-signing CAs if the certificate is provided. While not eliminating the need for wsk -i this does

[GitHub] dgrove-oss opened a new pull request #351: deploy kube now documents both IBM Cloud and IBM Cloud Private

2018-11-30 Thread GitBox
dgrove-oss opened a new pull request #351: deploy kube now documents both IBM Cloud and IBM Cloud Private URL: https://github.com/apache/incubator-openwhisk-website/pull/351 This is an automated message from the Apache Git

[GitHub] akrabat opened a new pull request #4149: Do not link the `/run/runc` volume unless invoker_use_runc is set

2018-11-30 Thread GitBox
akrabat opened a new pull request #4149: Do not link the `/run/runc` volume unless invoker_use_runc is set URL: https://github.com/apache/incubator-openwhisk/pull/4149 On Mac, `/run/runc` doesn't exist and trying to set it as a volume causes an error. ## My changes affect

[GitHub] dgrove-oss closed pull request #4144: add wskdeploy to scriptRunner Dockerfile

2018-11-30 Thread GitBox
dgrove-oss closed pull request #4144: add wskdeploy to scriptRunner Dockerfile URL: https://github.com/apache/incubator-openwhisk/pull/4144 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] csantanapr closed pull request #331: package rename for whisk.core is now org.apache.openwhisk.core

2018-11-30 Thread GitBox
csantanapr closed pull request #331: package rename for whisk.core is now org.apache.openwhisk.core URL: https://github.com/apache/incubator-openwhisk-apigateway/pull/331 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for

[GitHub] csantanapr closed pull request #351: deploy kube now documents both IBM Cloud and IBM Cloud Private

2018-11-30 Thread GitBox
csantanapr closed pull request #351: deploy kube now documents both IBM Cloud and IBM Cloud Private URL: https://github.com/apache/incubator-openwhisk-website/pull/351 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 a change in pull request #4142: tools/dev - updated intellij script to accommodate docker-compose

2018-11-30 Thread GitBox
ddragosd commented on a change in pull request #4142: tools/dev - updated intellij script to accommodate docker-compose URL: https://github.com/apache/incubator-openwhisk/pull/4142#discussion_r238041104 ## File path:

[GitHub] ddragosd commented on a change in pull request #4142: tools/dev - updated intellij script to accommodate docker-compose

2018-11-30 Thread GitBox
ddragosd commented on a change in pull request #4142: tools/dev - updated intellij script to accommodate docker-compose URL: https://github.com/apache/incubator-openwhisk/pull/4142#discussion_r238041114 ## File path:

[GitHub] ddragosd commented on a change in pull request #4142: tools/dev - updated intellij script to accommodate docker-compose

2018-11-30 Thread GitBox
ddragosd commented on a change in pull request #4142: tools/dev - updated intellij script to accommodate docker-compose URL: https://github.com/apache/incubator-openwhisk/pull/4142#discussion_r238041201 ## File path:

[GitHub] chetanmeh commented on issue #3246: ActionLimitsTests memory test failing

2018-11-30 Thread GitBox
chetanmeh commented on issue #3246: ActionLimitsTests memory test failing URL: https://github.com/apache/incubator-openwhisk/issues/3246#issuecomment-443404992 On my local Ubuntu box ``` Kernel Version: 4.15.0-39-generic Operating System: Ubuntu 16.04.5 LTS Total Memory:

[GitHub] csantanapr closed issue #300: .dockerignore should not leave out LICENSE.txt

2018-11-30 Thread GitBox
csantanapr closed issue #300: .dockerignore should not leave out LICENSE.txt URL: https://github.com/apache/incubator-openwhisk-apigateway/issues/300 This is an automated message from the Apache Git Service. To respond to

[GitHub] csantanapr closed pull request #328: Include license in container image

2018-11-30 Thread GitBox
csantanapr closed pull request #328: Include license in container image URL: https://github.com/apache/incubator-openwhisk-apigateway/pull/328 This is an automated message from the Apache Git Service. To respond to the

[GitHub] csantanapr closed pull request #319: Add package tzdata to the image

2018-11-30 Thread GitBox
csantanapr closed pull request #319: Add package tzdata to the image URL: https://github.com/apache/incubator-openwhisk-apigateway/pull/319 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] dgrove-oss opened a new pull request #381: fix dangling links to docs/ingress.md (was split into multiple files).

2018-11-30 Thread GitBox
dgrove-oss opened a new pull request #381: fix dangling links to docs/ingress.md (was split into multiple files). URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/381 Fixes #379. This is an automated

[GitHub] pritidesai closed pull request #350: Adjust Home page and add Swift Action examples

2018-11-30 Thread GitBox
pritidesai closed pull request #350: Adjust Home page and add Swift Action examples URL: https://github.com/apache/incubator-openwhisk-website/pull/350 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] jthomas commented on issue #325: Support for invocations based on websocket messages

2018-11-30 Thread GitBox
jthomas commented on issue #325: Support for invocations based on websocket messages URL: https://github.com/apache/incubator-openwhisk-apigateway/issues/325#issuecomment-443140036 I can't find any documentation about this feature in AWS at the moment. We'll have to wait and see what

[GitHub] mdeuser commented on a change in pull request #388: Save Code Associated with Blackbox Actions

2018-11-30 Thread GitBox
mdeuser commented on a change in pull request #388: Save Code Associated with Blackbox Actions URL: https://github.com/apache/incubator-openwhisk-cli/pull/388#discussion_r237851733 ## File path: commands/action.go ## @@ -692,7 +692,7 @@ func saveCode(action whisk.Action,

[GitHub] daisy-ycguo commented on a change in pull request #378: cleanup specification of storageclasses and persistence

2018-11-30 Thread GitBox
daisy-ycguo commented on a change in pull request #378: cleanup specification of storageclasses and persistence URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/378#discussion_r237733194 ## File path: docs/k8s-ibm-public.md ## @@ -97,12 +97,15 @@

[GitHub] jonpspri opened a new pull request #380: Document ICP and Dynamic DNS Provisioning

2018-11-30 Thread GitBox
jonpspri opened a new pull request #380: Document ICP and Dynamic DNS Provisioning URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/380 Edits to documentation to address two concerns: 1. Directions for deploying OpenWhisk onto IBM Cloud Private, including

[GitHub] daisy-ycguo commented on issue #378: cleanup specification of storageclasses and persistence

2018-11-30 Thread GitBox
daisy-ycguo commented on issue #378: cleanup specification of storageclasses and persistence URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/378#issuecomment-443207834 Sorry for my misleading messages. Finally I verified this PR worked well for IKS 10.8, just **with

[GitHub] lionelvillard opened a new issue #53: support for .py license header

2018-11-30 Thread GitBox
lionelvillard opened a new issue #53: support for .py license header URL: https://github.com/apache/incubator-openwhisk-utilities/issues/53 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] dgrove-oss commented on a change in pull request #381: fix dangling links to docs/ingress.md (was split into multiple files).

2018-11-30 Thread GitBox
dgrove-oss commented on a change in pull request #381: fix dangling links to docs/ingress.md (was split into multiple files). URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/381#discussion_r238007888 ## File path: helm/openwhisk/templates/ow-whisk-cm.yaml