[GitHub] codecov-io commented on issue #3636: update docker runtime from 1.2.0 to 1.3.0

codecov-io commented on issue #3636: update docker runtime from 1.2.0 to 1.3.0 URL: https://github.com/apache/incubator-openwhisk/pull/3636#issuecomment-387950453 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3636?src=pr=h1) Report > :exclamation: No coverage

[GitHub] csantanapr commented on issue #3636: update docker runtime from 1.2.0 to 1.3.0

csantanapr commented on issue #3636: update docker runtime from 1.2.0 to 1.3.0 URL: https://github.com/apache/incubator-openwhisk/pull/3636#issuecomment-387944982 PG2 3114 ⌛️ This is an automated message from the Apache Git

[GitHub] csantanapr opened a new pull request #3636: update docker runtime from 1.2.0 to 1.3.0

csantanapr opened a new pull request #3636: update docker runtime from 1.2.0 to 1.3.0 URL: https://github.com/apache/incubator-openwhisk/pull/3636 ## Description Bump dockerskeleton default image diff

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

csantanapr commented on issue #3616: WIP: 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-387939251 PG2 3111 

[GitHub] csantanapr closed issue #2672: Action attachments are not cached and end up repeatedly fetched even if not needed

csantanapr closed issue #2672: Action attachments are not cached and end up repeatedly fetched even if not needed URL: https://github.com/apache/incubator-openwhisk/issues/2672 This is an automated message from the Apache

[GitHub] csantanapr closed pull request #2855: Cache database attachments

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

[GitHub] dubee closed pull request #35: update changelog for openssh-client

dubee closed pull request #35: update changelog for openssh-client URL: https://github.com/apache/incubator-openwhisk-runtime-docker/pull/35 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] tddg opened a new issue #3635: Recommendations for configuration of pre-warmed pool

tddg opened a new issue #3635: Recommendations for configuration of pre-warmed pool URL: https://github.com/apache/incubator-openwhisk/issues/3635 I am wondering if there are any empirical rules when coming to configure the pre-warmed pool for each invoker? Right now pre-warmed pool is

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

csantanapr commented on issue #3616: WIP: 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-387914897 PG2 3111 ⌛️

[GitHub] csantanapr commented on issue #3610: Fixes #3608 - Use --user flag to install locally for the current user

csantanapr commented on issue #3610: Fixes #3608 - Use --user flag to install locally for the current user URL: https://github.com/apache/incubator-openwhisk/pull/3610#issuecomment-387914471 PG3 2240 ⌛️ This is an

[GitHub] csantanapr commented on issue #3627: remove unused imports or dead variables as reported in travis build.

csantanapr commented on issue #3627: remove unused imports or dead variables as reported in travis build. URL: https://github.com/apache/incubator-openwhisk/pull/3627#issuecomment-387914198 PG3 2239 OK This is an automated

[GitHub] csantanapr closed pull request #3627: remove unused imports or dead variables as reported in travis build.

csantanapr closed pull request #3627: remove unused imports or dead variables as reported in travis build. URL: https://github.com/apache/incubator-openwhisk/pull/3627 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] csantanapr opened a new pull request #35: update changelog for openssh-client

csantanapr opened a new pull request #35: update changelog for openssh-client URL: https://github.com/apache/incubator-openwhisk-runtime-docker/pull/35 This is an automated message from the Apache Git Service. To respond to

[GitHub] csantanapr closed issue #3611: Activation list API should provide descriptive error for negative skip values

csantanapr closed issue #3611: Activation list API should provide descriptive error for negative skip values URL: https://github.com/apache/incubator-openwhisk/issues/3611 This is an automated message from the Apache Git

[GitHub] csantanapr closed pull request #3615: Terminate route for invalid list skip values

csantanapr closed pull request #3615: Terminate route for invalid list skip values URL: https://github.com/apache/incubator-openwhisk/pull/3615 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] dubee commented on issue #3615: Terminate route for invalid list skip values

dubee commented on issue #3615: Terminate route for invalid list skip values URL: https://github.com/apache/incubator-openwhisk/pull/3615#issuecomment-387895473 PG5 305  This is an automated message from the Apache Git

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

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] dubee closed issue #26: git clone not working when using ssh git url (add ssh client)

dubee closed issue #26: git clone not working when using ssh git url (add ssh client) URL: https://github.com/apache/incubator-openwhisk-runtime-docker/issues/26 This is an automated message from the Apache Git Service. To

[GitHub] dubee closed pull request #34: add openssh client for dockerskeleton

dubee closed pull request #34: add openssh client for dockerskeleton URL: https://github.com/apache/incubator-openwhisk-runtime-docker/pull/34 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] codecov-io commented on issue #3615: Terminate route for invalid list skip values

codecov-io commented on issue #3615: Terminate route for invalid list skip values URL: https://github.com/apache/incubator-openwhisk/pull/3615#issuecomment-386757956 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3615?src=pr=h1) Report > Merging

[GitHub] codecov-io commented on issue #3631: bump new version of ios staterapp

codecov-io commented on issue #3631: bump new version of ios staterapp URL: https://github.com/apache/incubator-openwhisk/pull/3631#issuecomment-387874978 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3631?src=pr=h1) Report > Merging

[GitHub] dubee closed issue #3630: bump new version of swift/ios starter app

dubee closed issue #3630: bump new version of swift/ios starter app URL: https://github.com/apache/incubator-openwhisk/issues/3630 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] dubee commented on issue #3615: Terminate route for invalid list skip values

dubee commented on issue #3615: Terminate route for invalid list skip values URL: https://github.com/apache/incubator-openwhisk/pull/3615#issuecomment-387865022 PG5 304 ⏳ This is an automated message from the Apache Git

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

csantanapr commented on issue #3616: WIP: 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-387861398 @jonpspri your done with WIP and ready for review?

[GitHub] codecov-io commented on issue #3632: Add license header to build-failures.md

codecov-io commented on issue #3632: Add license header to build-failures.md URL: https://github.com/apache/incubator-openwhisk/pull/3632#issuecomment-387859895 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3632?src=pr=h1) Report > Merging

[GitHub] dubee closed pull request #3632: Add license header to build-failures.md

dubee closed pull request #3632: Add license header to build-failures.md URL: https://github.com/apache/incubator-openwhisk/pull/3632 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] greenfireman opened a new issue #3634: Creating an action from zip: 413 Request Entity Too Large

greenfireman opened a new issue #3634: Creating an action from zip: 413 Request Entity Too Large URL: https://github.com/apache/incubator-openwhisk/issues/3634 ## Environment details: * Docker based deployment - https://github.com/apache/incubator-openwhisk-devtools *

[GitHub] pritidesai closed issue #880: wskdeploy export produces empty file

pritidesai closed issue #880: wskdeploy export produces empty file URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/880 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] pritidesai closed pull request #890: Resolves issue #880

pritidesai closed pull request #890: Resolves issue #880 URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/890 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign

[GitHub] mrutkows opened a new pull request #897: Fix all files with Hash license header to fix typo.

mrutkows opened a new pull request #897: Fix all files with Hash license header to fix typo. URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/897 This is an automated message from the Apache Git Service. To

[GitHub] pritidesai closed pull request #298: Change hash header to latest version in .conf files.

pritidesai closed pull request #298: Change hash header to latest version in .conf files. URL: https://github.com/apache/incubator-openwhisk-apigateway/pull/298 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 closed pull request #150: do not assume env var HOST_INDEX begins with 'host'

dubee closed pull request #150: do not assume env var HOST_INDEX begins with 'host' URL: https://github.com/apache/incubator-openwhisk-package-alarms/pull/150 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 closed pull request #174: do not assume env var HOST_INDEX begins with 'host'

dubee closed pull request #174: do not assume env var HOST_INDEX begins with 'host' URL: https://github.com/apache/incubator-openwhisk-package-cloudant/pull/174 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 opened a new pull request #3632: Add license header to build-failures.md

dubee opened a new pull request #3632: Add license header to build-failures.md URL: https://github.com/apache/incubator-openwhisk/pull/3632 ## Description ## Related issue and scope - [ ] I opened an issue to propose and discuss this change (#) ## My

[GitHub] pritidesai closed pull request #48: Add ASF license headers to all Scala .conf files.

pritidesai closed pull request #48: Add ASF license headers to all Scala .conf files. URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/48 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] pritidesai closed pull request #33: Support .conf files in default ASF license scanning configurations

pritidesai closed pull request #33: Support .conf files in default ASF license scanning configurations URL: https://github.com/apache/incubator-openwhisk-utilities/pull/33 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for

[GitHub] jasonpet opened a new pull request #174: do not assume env var HOST_INDEX begins with 'host'

jasonpet opened a new pull request #174: do not assume env var HOST_INDEX begins with 'host' URL: https://github.com/apache/incubator-openwhisk-package-cloudant/pull/174 This is an automated message from the Apache Git

[GitHub] dubee commented on issue #2855: Cache database attachments

dubee commented on issue #2855: Cache database attachments URL: https://github.com/apache/incubator-openwhisk/pull/2855#issuecomment-387806564 @csantanapr, just need a . This is an automated message from the Apache Git

[GitHub] gnodet commented on issue #294: Apihost property not used

gnodet commented on issue #294: Apihost property not used URL: https://github.com/apache/incubator-openwhisk-cli/issues/294#issuecomment-387804079 Oops, sorry, I had an alias for `wsk` that was forcing the apihost to `172.17.0.1`

[GitHub] gnodet closed issue #294: Apihost property not used

gnodet closed issue #294: Apihost property not used URL: https://github.com/apache/incubator-openwhisk-cli/issues/294 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub

[GitHub] mdeuser commented on issue #294: Apihost property not used

mdeuser commented on issue #294: Apihost property not used URL: https://github.com/apache/incubator-openwhisk-cli/issues/294#issuecomment-387803197 @gnodet - set the `WSK_CLI_DEBUG` environment variable (to any value) to enable additional tracing to see where the cli is pulling the

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

mcdan commented on issue #3579: Using non root user in controller URL: https://github.com/apache/incubator-openwhisk/pull/3579#issuecomment-387798889 @fmaschler okay, sounds good. This is an automated message from the Apache

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

mcdan commented on issue #3579: Using non root user in controller URL: https://github.com/apache/incubator-openwhisk/pull/3579#issuecomment-387775075 @fmaschler no I mean the invoker reading "/var/lib/docker/containers/< containerid >/< containerid >-json.log I thought they were

[GitHub] jasonpet opened a new pull request #150: do not assume env var HOST_INDEX begins with 'host'

jasonpet opened a new pull request #150: do not assume env var HOST_INDEX begins with 'host' URL: https://github.com/apache/incubator-openwhisk-package-alarms/pull/150 This is an automated message from the Apache Git

[GitHub] mrutkows opened a new pull request #298: Change hash header to latest version in .conf files.

mrutkows opened a new pull request #298: Change hash header to latest version in .conf files. URL: https://github.com/apache/incubator-openwhisk-apigateway/pull/298 This is an automated message from the Apache Git Service.

[GitHub] dubee commented on issue #3619: Provide an activation store SPI

dubee commented on issue #3619: Provide an activation store SPI URL: https://github.com/apache/incubator-openwhisk/pull/3619#issuecomment-387794678 @chetanmeh, yes we want to be able to use a different store to fetch activations from other than CouchDB. I plan to send some more

[GitHub] dubee commented on issue #3625: Bump Ansible to 2.5.2

dubee commented on issue #3625: Bump Ansible to 2.5.2 URL: https://github.com/apache/incubator-openwhisk/pull/3625#issuecomment-387788086 @csantanapr, please review. This is an automated message from the Apache Git Service.

[GitHub] gnodet opened a new issue #294: Apihost property not used

gnodet opened a new issue #294: Apihost property not used URL: https://github.com/apache/incubator-openwhisk-cli/issues/294 The following shell excerpt exhibits the problem. The `apihost` property has been correctly set along with the auth property, but it's not used when invoking the

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

mcdan commented on issue #3579: Using non root user in controller URL: https://github.com/apache/incubator-openwhisk/pull/3579#issuecomment-387775075 @fmaschler no I mean the invoker reading "/var/lib/docker/containers/< containerid >/< containerid >-json.log I thought they were

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

mcdan commented on issue #3579: Using non root user in controller URL: https://github.com/apache/incubator-openwhisk/pull/3579#issuecomment-387775075 @fmaschler no I mean the invoker reading "/var/lib/docker/containers//-json.log I thought they were written by the user who runs the

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

fmaschler commented on issue #3579: Using non root user in controller URL: https://github.com/apache/incubator-openwhisk/pull/3579#issuecomment-387765815 I checked the namespaces but on the host the process still runs as root: ``` $ docker exec -i -t controller0 /bin/bash

[GitHub] csantanapr commented on issue #2855: Cache database attachments

csantanapr commented on issue #2855: Cache database attachments URL: https://github.com/apache/incubator-openwhisk/pull/2855#issuecomment-387758352 @dubee @rabbah this ready to merge it would be a step forward This is an

[GitHub] csantanapr closed pull request #3506: Create troubleshooting guide for dependency download failures

csantanapr closed pull request #3506: Create troubleshooting guide for dependency download failures URL: https://github.com/apache/incubator-openwhisk/pull/3506 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] csantanapr commented on issue #3627: remove unused imports or dead variables as reported in travis build.

csantanapr commented on issue #3627: remove unused imports or dead variables as reported in travis build. URL: https://github.com/apache/incubator-openwhisk/pull/3627#issuecomment-387744294 Travis should be enough to cover this

[GitHub] csantanapr commented on issue #3628: Revert "Replace WALA Pair with clean new Pair."

csantanapr commented on issue #3628: Revert "Replace WALA Pair with clean new Pair." URL: https://github.com/apache/incubator-openwhisk/pull/3628#issuecomment-387743499 @mrutkows in this case was not about PG. it was that a common Class “common.Pair” was being removed from the common

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

csantanapr commented on issue #3629: [Feature requests] action protection URL: https://github.com/apache/incubator-openwhisk/issues/3629#issuecomment-387740875 Hum there is no point of protecting from delete only and not update the action can accidentally be updated with wrong version of

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

style95 commented on issue #3629: [Feature requests] action protection URL: https://github.com/apache/incubator-openwhisk/issues/3629#issuecomment-387737049 @csantanapr Yes mostly you are correct. Let me clarify issues more. Actually, I am operating in-house version of OpenWhisk

[GitHub] csantanapr closed pull request #149: loosen restrictions on redis values

csantanapr closed pull request #149: loosen restrictions on redis values URL: https://github.com/apache/incubator-openwhisk-package-alarms/pull/149 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] csantanapr closed pull request #173: loosen restrictions on redis values

csantanapr closed pull request #173: loosen restrictions on redis values URL: https://github.com/apache/incubator-openwhisk-package-cloudant/pull/173 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] csantanapr closed pull request #187: include Helm-based deployment in TravisCI matrix

csantanapr closed pull request #187: include Helm-based deployment in TravisCI matrix URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/187 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] csantanapr commented on issue #3629: [Feature requests] action protection

csantanapr commented on issue #3629: [Feature requests] action protection URL: https://github.com/apache/incubator-openwhisk/issues/3629#issuecomment-387724536 @style95 can you elaborate your problem? Today we only generate one master api key. This key is use to create, update and

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

style95 commented on issue #3629: [Feature requests] action protection URL: https://github.com/apache/incubator-openwhisk/issues/3629#issuecomment-387721456 @csantanapr Many VM services provide option to enable VM protection. If user enables that option, no one can delete that VM until

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

csantanapr commented on issue #3629: [Feature requests] action protection URL: https://github.com/apache/incubator-openwhisk/issues/3629#issuecomment-387719397 @style95 could you provide a proposal or more details on the use case? Can you elaborate on “VM services” how do they provide

[GitHub] csantanapr commented on issue #3232: Making prewarm kind (and count) configurable

csantanapr commented on issue #3232: Making prewarm kind (and count) configurable URL: https://github.com/apache/incubator-openwhisk/pull/3232#issuecomment-387718038 Thanks for the updates @Himavanth  This is an automated

[GitHub] rabbah commented on a change in pull request #40: WIP: Remove com.sun.* dependency

rabbah commented on a change in pull request #40: WIP: Remove com.sun.* dependency URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/40#discussion_r187018029 ## File path: java8/proxy/src/main/java/openwhisk/java/action/Proxy.java ## @@ -14,156 +14,198

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

mkouba commented on issue #42: Add support for Kotlin actions URL: https://github.com/apache/incubator-openwhisk-runtime-java/issues/42#issuecomment-387664228 I think the reason is that java actions don't need kotlin standard library and so to keep the java runtime and java action jar

[GitHub] mkouba commented on a change in pull request #40: WIP: Remove com.sun.* dependency

mkouba commented on a change in pull request #40: WIP: Remove com.sun.* dependency URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/40#discussion_r186966102 ## File path: java8/proxy/src/main/java/openwhisk/java/action/Proxy.java ## @@ -14,156 +14,198

[GitHub] codecov-io commented on issue #3570: ArtifactStore implementation for MongoDB

codecov-io commented on issue #3570: ArtifactStore implementation for MongoDB URL: https://github.com/apache/incubator-openwhisk/pull/3570#issuecomment-386234242 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3570?src=pr=h1) Report > Merging

[GitHub] Himavanth commented on issue #3232: Making prewarm kind (and count) configurable

Himavanth commented on issue #3232: Making prewarm kind (and count) configurable URL: https://github.com/apache/incubator-openwhisk/pull/3232#issuecomment-387635067 @rabbah I have added the tests. Do you think this would suffice? @csantanapr Glad to know. I was working on tests and

[GitHub] csantanapr commented on issue #23: FAIL: Example_badinit_nocompiler

csantanapr commented on issue #23: FAIL: Example_badinit_nocompiler URL: https://github.com/apache/incubator-openwhisk-runtime-go/issues/23#issuecomment-387633436 It looks like the test is passing in travis now. Closing...

[GitHub] csantanapr closed issue #23: FAIL: Example_badinit_nocompiler

csantanapr closed issue #23: FAIL: Example_badinit_nocompiler URL: https://github.com/apache/incubator-openwhisk-runtime-go/issues/23 This is an automated message from the Apache Git Service. To respond to the message,