[GitHub] houshengbo closed pull request #3383: Add the license headers

houshengbo closed pull request #3383: Add the license headers URL: https://github.com/apache/incubator-openwhisk/pull/3383 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] houshengbo closed issue #54: Add the missing license header to the source files in each repo to be released

houshengbo closed issue #54: Add the missing license header to the source files in each repo to be released URL: https://github.com/apache/incubator-openwhisk-release/issues/54 This is an automated message from the Apache

[GitHub] mrutkows commented on a change in pull request #766: adding support for using env. variables in inputs JSON

mrutkows commented on a change in pull request #766: adding support for using env. variables in inputs JSON URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/766#discussion_r172929780 ## File path: parsers/parameters.go ## @@ -322,7 +325,7 @@ func

[GitHub] mrutkows commented on a change in pull request #766: adding support for using env. variables in inputs JSON

mrutkows commented on a change in pull request #766: adding support for using env. variables in inputs JSON URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/766#discussion_r172930016 ## File path: tests/dat/manifest_validate_multiline_params.yaml ## @@

[GitHub] mrutkows commented on a change in pull request #766: adding support for using env. variables in inputs JSON

mrutkows commented on a change in pull request #766: adding support for using env. variables in inputs JSON URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/766#discussion_r172930215 ## File path: wskenv/environment.go ## @@ -63,15 +63,21 @@ func

[GitHub] mrutkows commented on a change in pull request #766: adding support for using env. variables in inputs JSON

mrutkows commented on a change in pull request #766: adding support for using env. variables in inputs JSON URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/766#discussion_r172929604 ## File path: parsers/manifest_parser_test.go ## @@ -1032,7 +1054,7 @@

[GitHub] mrutkows closed pull request #766: adding support for using env. variables in inputs JSON

mrutkows closed pull request #766: adding support for using env. variables in inputs JSON URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/766 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 #705: Unable to use ENV VARS in input JSON

mrutkows closed issue #705: Unable to use ENV VARS in input JSON URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/705 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] dgrove-oss closed pull request #3409: Remove namespace configuration; use invoker's instead

dgrove-oss closed pull request #3409: Remove namespace configuration; use invoker's instead URL: https://github.com/apache/incubator-openwhisk/pull/3409 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] lionelvillard opened a new pull request #775: add --name project name flag

lionelvillard opened a new pull request #775: add --name project name flag URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/775 This is an automated message from the Apache Git Service. To respond to the

[GitHub] jonpspri commented on issue #3407: WIP: No longer need to build 'whisk/*' runtimes for builtins.

jonpspri commented on issue #3407: WIP: No longer need to build 'whisk/*' runtimes for builtins. URL: https://github.com/apache/incubator-openwhisk/pull/3407#issuecomment-371236194 @csantanapr The capability of putting prefix and tag are already in the manifest.json file. We just need

[GitHub] lue-kle commented on a change in pull request #3382: Cleanup script for unused entities in the whisks database.

lue-kle commented on a change in pull request #3382: Cleanup script for unused entities in the whisks database. URL: https://github.com/apache/incubator-openwhisk/pull/3382#discussion_r172975758 ## File path: tools/db/cleanUpWhisks.py ## @@ -0,0 +1,150 @@ +#!/usr/bin/env

[GitHub] mdeuser commented on a change in pull request #3352: API GW doc update to include path parameter example

mdeuser commented on a change in pull request #3352: API GW doc update to include path parameter example URL: https://github.com/apache/incubator-openwhisk/pull/3352#discussion_r172987431 ## File path: docs/apigateway.md ## @@ -192,14 +205,16 @@ Now let's restore the API

[GitHub] pritidesai closed pull request #23: Add the license headers

pritidesai closed pull request #23: Add the license headers URL: https://github.com/apache/incubator-openwhisk-runtime-java/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: As this is a

[GitHub] hsane2001 closed issue #2570: Cannot install APIgateway if http_proxy variable set in native environment

hsane2001 closed issue #2570: Cannot install APIgateway if http_proxy variable set in native environment URL: https://github.com/apache/incubator-openwhisk/issues/2570 This is an automated message from the Apache Git

[GitHub] mdeuser commented on issue #3388: Update require-whisk-auth behavior to secure web action

mdeuser commented on issue #3388: Update require-whisk-auth behavior to secure web action URL: https://github.com/apache/incubator-openwhisk/pull/3388#issuecomment-371285212 or pg4/1451 This is an automated message from the

[GitHub] lionelvillard opened a new issue #774: support for deploying/undeploying headless managed project

lionelvillard opened a new issue #774: support for deploying/undeploying headless managed project URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/774 usecase: deploying assets into an existing project without updating the manifest. wskdeploy should accept

[GitHub] dubeejw commented on issue #3388: Update require-whisk-auth behavior to secure web action

dubeejw commented on issue #3388: Update require-whisk-auth behavior to secure web action URL: https://github.com/apache/incubator-openwhisk/pull/3388#issuecomment-371279807 PG3 1969 ? This is an automated message from the

[GitHub] jasonpet opened a new pull request #136: do not retry a throttled alarms trigger

jasonpet opened a new pull request #136: do not retry a throttled alarms trigger URL: https://github.com/apache/incubator-openwhisk-package-alarms/pull/136 This is an automated message from the Apache Git Service. To respond

[GitHub] houshengbo commented on issue #53: Change to use RAT maven plugin to support ASF short format header

houshengbo commented on issue #53: Change to use RAT maven plugin to support ASF short format header URL: https://github.com/apache/incubator-openwhisk-release/pull/53#issuecomment-371267701 @daisy-ycguo Please rebase it.

[GitHub] pritidesai commented on a change in pull request #766: adding support for using env. variables in inputs JSON

pritidesai commented on a change in pull request #766: adding support for using env. variables in inputs JSON URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/766#discussion_r172931132 ## File path: tests/dat/manifest_validate_multiline_params.yaml ## @@

[GitHub] dubeejw commented on issue #3388: Update require-whisk-auth behavior to secure web action

dubeejw commented on issue #3388: Update require-whisk-auth behavior to secure web action URL: https://github.com/apache/incubator-openwhisk/pull/3388#issuecomment-371279253 Can't get a PG to work. This is an automated

[GitHub] dubeejw commented on issue #3388: Update require-whisk-auth behavior to secure web action

dubeejw commented on issue #3388: Update require-whisk-auth behavior to secure web action URL: https://github.com/apache/incubator-openwhisk/pull/3388#issuecomment-371279807 PG3 1969 ? This is an automated message from the

[GitHub] dubeejw commented on issue #67: Additional HTTP Headers

dubeejw commented on issue #67: Additional HTTP Headers URL: https://github.com/apache/incubator-openwhisk-client-go/pull/67#issuecomment-371300767 @mdeuser, AdditionalHeader is meant for consumers of client-go that do not have access to `requestUrl()`.

[GitHub] dubeejw commented on issue #67: Additional HTTP Headers

dubeejw commented on issue #67: Additional HTTP Headers URL: https://github.com/apache/incubator-openwhisk-client-go/pull/67#issuecomment-371300767 @mdeuser, `AdditionalHeaders` is meant for consumers of client-go that do not have access to `requestUrl()`.

[GitHub] tysonnorris commented on issue #2833: MesosContainerFactory

tysonnorris commented on issue #2833: MesosContainerFactory URL: https://github.com/apache/incubator-openwhisk/pull/2833#issuecomment-371325712 @rabbah any comments on this? I have some upcoming changes for a separate/new PR to deal with HA deploy scenarios in mesos, that will make more

[GitHub] csantanapr commented on issue #3388: Update require-whisk-auth behavior to secure web action

csantanapr commented on issue #3388: Update require-whisk-auth behavior to secure web action URL: https://github.com/apache/incubator-openwhisk/pull/3388#issuecomment-371324124 pg4/1451 ? This is an automated message from

[GitHub] rabbah commented on issue #2833: MesosContainerFactory

rabbah commented on issue #2833: MesosContainerFactory URL: https://github.com/apache/incubator-openwhisk/pull/2833#issuecomment-371326169 My apologies. I should?ve switched the assignee so it?s on my queue. Will spend the time in this tomorrow, pg etc.

[GitHub] mdeuser opened a new pull request #3410: Fix typos in api gw doc

mdeuser opened a new pull request #3410: Fix typos in api gw doc URL: https://github.com/apache/incubator-openwhisk/pull/3410 ## Description Documentation fix ## Related issue and scope - [ ] I opened an issue to propose and discuss this change (#) ## My

[GitHub] dgrove-oss closed pull request #3378: LogStoreProvider using invokerAgent for KubernetesContainerPool

dgrove-oss closed pull request #3378: LogStoreProvider using invokerAgent for KubernetesContainerPool URL: https://github.com/apache/incubator-openwhisk/pull/3378 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake

[GitHub] csantanapr closed pull request #3410: Fix typos in api gw doc

csantanapr closed pull request #3410: Fix typos in api gw doc URL: https://github.com/apache/incubator-openwhisk/pull/3410 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] rabbah commented on issue #2833: MesosContainerFactory

rabbah commented on issue #2833: MesosContainerFactory URL: https://github.com/apache/incubator-openwhisk/pull/2833#issuecomment-371326169 My apologies. I should?ve switched the assignee so it?s on my queue. Will spend the time on this tomorrow, pg etc.

[GitHub] houshengbo commented on a change in pull request #53: Change to use RAT maven plugin to support ASF short format header

houshengbo commented on a change in pull request #53: Change to use RAT maven plugin to support ASF short format header URL: https://github.com/apache/incubator-openwhisk-release/pull/53#discussion_r172995310 ## File path: tools/verify_source_code.sh ## @@ -24,13 +24,10

[GitHub] mdeuser commented on issue #67: Additional HTTP Headers

mdeuser commented on issue #67: Additional HTTP Headers URL: https://github.com/apache/incubator-openwhisk-client-go/pull/67#issuecomment-371355681 i think i understand. this is for setting a header that will appear in any/all subsequent request to the backend; not for setting header(s)

[GitHub] dgrove-oss commented on a change in pull request #3411: refactor KubernetesClient to separate invokerAgent; add forwarding LogStoreProvider

dgrove-oss commented on a change in pull request #3411: refactor KubernetesClient to separate invokerAgent; add forwarding LogStoreProvider URL: https://github.com/apache/incubator-openwhisk/pull/3411#discussion_r173040613 ## File path:

[GitHub] dgrove-oss commented on a change in pull request #3411: refactor KubernetesClient to separate invokerAgent; add forwarding LogStoreProvider

dgrove-oss commented on a change in pull request #3411: refactor KubernetesClient to separate invokerAgent; add forwarding LogStoreProvider URL: https://github.com/apache/incubator-openwhisk/pull/3411#discussion_r173040613 ## File path:

[GitHub] dubeejw closed pull request #67: Additional HTTP Headers

dubeejw closed pull request #67: Additional HTTP Headers URL: https://github.com/apache/incubator-openwhisk-client-go/pull/67 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] dgrove-oss commented on issue #3411: refactor KubernetesClient to separate invokerAgent; add forwarding LogStoreProvider

dgrove-oss commented on issue #3411: refactor KubernetesClient to separate invokerAgent; add forwarding LogStoreProvider URL: https://github.com/apache/incubator-openwhisk/pull/3411#issuecomment-371357346 PG3 / 1970 ? This

[GitHub] chetanmeh commented on issue #3249: #3243 - Use gradle scoverage to collect test coverage stats

chetanmeh commented on issue #3249: #3243 - Use gradle scoverage to collect test coverage stats URL: https://github.com/apache/incubator-openwhisk/pull/3249#issuecomment-371380551 Rebased to latest master This is an

[GitHub] kameshsampath commented on issue #26: Multiple Distros for Java Runtimes

kameshsampath commented on issue #26: Multiple Distros for Java Runtimes URL: https://github.com/apache/incubator-openwhisk-runtime-java/issues/26#issuecomment-371398396 > But do we need to support images based on different distributions. Can't we have single distribution across multiple

[GitHub] pritidesai opened a new pull request #776: Adding support for Inline Code and reading action source from URL

pritidesai opened a new pull request #776: Adding support for Inline Code and reading action source from URL URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/776 Closes #745 Closes #622 This is an

[GitHub] ddragosd commented on a change in pull request #2: Add DC/OS universe for OpenWhisk and orchestration scripts

ddragosd commented on a change in pull request #2: Add DC/OS universe for OpenWhisk and orchestration scripts URL: https://github.com/apache/incubator-openwhisk-deploy-mesos/pull/2#discussion_r173073325 ## File path: README.md ## @@ -0,0 +1,17 @@ +# OpenWhisk Deployment

[GitHub] ddragosd commented on a change in pull request #2: Add DC/OS universe for OpenWhisk and orchestration scripts

ddragosd commented on a change in pull request #2: Add DC/OS universe for OpenWhisk and orchestration scripts URL: https://github.com/apache/incubator-openwhisk-deploy-mesos/pull/2#discussion_r173073819 ## File path: dcos-deploy-script/Makefile ## @@ -0,0 +1,127 @@

[GitHub] jonpspri opened a new pull request #3412: Changes to runtimes variables in support of retiring 'whisk/*' runtimes

jonpspri opened a new pull request #3412: Changes to runtimes variables in support of retiring 'whisk/*' runtimes URL: https://github.com/apache/incubator-openwhisk/pull/3412 Changes to ansible processes for determining and maybe pulling runtime images, as a preliminary to eliminating the

[GitHub] Param-S commented on issue #26: Multiple Distros for Java Runtimes

Param-S commented on issue #26: Multiple Distros for Java Runtimes URL: https://github.com/apache/incubator-openwhisk-runtime-java/issues/26#issuecomment-371381160 I see, 2 requirements, 1. Supporting different architectures 2. Supporting different distributions Yes, I agree

[GitHub] csantanapr commented on issue #26: Multiple Distros for Java Runtimes

csantanapr commented on issue #26: Multiple Distros for Java Runtimes URL: https://github.com/apache/incubator-openwhisk-runtime-java/issues/26#issuecomment-371213696 Also need to consider architect to the matrix of images s390x ppc64 arm (ie raspberry pi/edge device)

[GitHub] codecov-io commented on issue #3249: #3243 - Use gradle scoverage to collect test coverage stats

codecov-io commented on issue #3249: #3243 - Use gradle scoverage to collect test coverage stats URL: https://github.com/apache/incubator-openwhisk/pull/3249#issuecomment-365147821 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3249?src=pr=h1) Report >

[GitHub] csantanapr commented on issue #3404: bump dockerskeleton to pick up curl and wget 1.2.0

csantanapr commented on issue #3404: bump dockerskeleton to pick up curl and wget 1.2.0 URL: https://github.com/apache/incubator-openwhisk/pull/3404#issuecomment-371061484 PG3 1965 ? This is an automated message from the

[GitHub] csantanapr commented on issue #3404: bump dockerskeleton to pick up curl and wget 1.2.0

csantanapr commented on issue #3404: bump dockerskeleton to pick up curl and wget 1.2.0 URL: https://github.com/apache/incubator-openwhisk/pull/3404#issuecomment-371061484 PG3 1964 ? This is an automated message from the

[GitHub] csantanapr commented on issue #3388: Update require-whisk-auth behavior to secure web action

csantanapr commented on issue #3388: Update require-whisk-auth behavior to secure web action URL: https://github.com/apache/incubator-openwhisk/pull/3388#issuecomment-371057673 PG4 1446 ? This is an automated message from

[GitHub] daisy-ycguo commented on issue #53: Change to use RAT maven plugin to support ASF short format header

daisy-ycguo commented on issue #53: Change to use RAT maven plugin to support ASF short format header URL: https://github.com/apache/incubator-openwhisk-release/pull/53#issuecomment-371064998 Thanks @houshengbo . This PR is ready for review and merge. Please help.

[GitHub] dubeejw commented on issue #3405: Update CLI Deploy Script

dubeejw commented on issue #3405: Update CLI Deploy Script URL: https://github.com/apache/incubator-openwhisk/pull/3405#issuecomment-371070125 @RSulzmann, can you give an example? I tried doing what you had for ` shell: cp -r {{ openwhisk_build_dir }}/{{ openwhisk_cli.archive_name }}/ {{

[GitHub] dubeejw commented on issue #3405: Update CLI Deploy Script

dubeejw commented on issue #3405: Update CLI Deploy Script URL: https://github.com/apache/incubator-openwhisk/pull/3405#issuecomment-371075255 I see something about a `release` directory that may not work with local mode. Seems like we default to remote mode by default. Can fix that

[GitHub] dubeejw commented on issue #3405: Update CLI Deploy Script

dubeejw commented on issue #3405: Update CLI Deploy Script URL: https://github.com/apache/incubator-openwhisk/pull/3405#issuecomment-371059444 @RSulzmann, here is another attempt. This is an automated message from the Apache

[GitHub] dubeejw commented on issue #3405: Update CLI Deploy Script

dubeejw commented on issue #3405: Update CLI Deploy Script URL: https://github.com/apache/incubator-openwhisk/pull/3405#issuecomment-371059358 https://github.com/apache/incubator-openwhisk/pull/3394 This is an automated

[GitHub] dubeejw commented on issue #3405: Install CLI with edge.yml

dubeejw commented on issue #3405: Install CLI with edge.yml URL: https://github.com/apache/incubator-openwhisk/pull/3405#issuecomment-371085608 Going with @cbickel changes. Repurposing this one to install the CLI with edge.

[GitHub] csantanapr commented on issue #3388: Update require-whisk-auth behavior to secure web action

csantanapr commented on issue #3388: Update require-whisk-auth behavior to secure web action URL: https://github.com/apache/incubator-openwhisk/pull/3388#issuecomment-371057673 PG4 1446 ? This is an automated message from

[GitHub] dubeejw commented on issue #3406: Fix deployment of the CLI if edge is on a remote machine.

dubeejw commented on issue #3406: Fix deployment of the CLI if edge is on a remote machine. URL: https://github.com/apache/incubator-openwhisk/pull/3406#issuecomment-371082080 PG4 1447 ? This is an automated message from

[GitHub] dubeejw commented on issue #3405: Update CLI Deploy Script

dubeejw commented on issue #3405: Update CLI Deploy Script URL: https://github.com/apache/incubator-openwhisk/pull/3405#issuecomment-371058615 PG1 2686 ? This is an automated message from the Apache Git Service. To respond

[GitHub] cbickel commented on issue #3406: Fix deployment of the CLI if edge is on a remote machine.

cbickel commented on issue #3406: Fix deployment of the CLI if edge is on a remote machine. URL: https://github.com/apache/incubator-openwhisk/pull/3406#issuecomment-371082982 PG5#145 running. This is an automated message

[GitHub] dubeejw commented on issue #3405: Install CLI with edge.yml

dubeejw commented on issue #3405: Install CLI with edge.yml URL: https://github.com/apache/incubator-openwhisk/pull/3405#issuecomment-371085608 Going with @cbickel's changes. Repurposing this one to install the CLI with edge.

[GitHub] rabbah commented on a change in pull request #3407: WIP: No longer need to build 'whisk/*' runtimes for builtins.

rabbah commented on a change in pull request #3407: WIP: No longer need to build 'whisk/*' runtimes for builtins. URL: https://github.com/apache/incubator-openwhisk/pull/3407#discussion_r172848192 ## File path: actionRuntimes/actionProxy/invoke.py ## @@ -1,136 +0,0 @@

[GitHub] lue-kle commented on issue #3382: Cleanup script for unused entities in the whisks database.

lue-kle commented on issue #3382: Cleanup script for unused entities in the whisks database. URL: https://github.com/apache/incubator-openwhisk/pull/3382#issuecomment-371144282 PG3/1966 run successfully on this fork/branch.

[GitHub] kameshsampath opened a new issue #26: Multiple Distros for Java Runtimes

kameshsampath opened a new issue #26: Multiple Distros for Java Runtimes URL: https://github.com/apache/incubator-openwhisk-runtime-java/issues/26 It will be nice to have OpenWhisk JDK runtimes available on multiple distributions * centos * debian * alpine The initial work

[GitHub] csantanapr commented on a change in pull request #24: AdoptOpenJDK Eclipse OpenJ9 as alternative Java runtime

csantanapr commented on a change in pull request #24: AdoptOpenJDK Eclipse OpenJ9 as alternative Java runtime URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/24#discussion_r172862568 ## File path: gradle/docker.gradle ## @@ -29,8 +29,8 @@ ext {

[GitHub] jonpspri commented on issue #3406: Fix deployment of the CLI if edge is on a remote machine.

jonpspri commented on issue #3406: Fix deployment of the CLI if edge is on a remote machine. URL: https://github.com/apache/incubator-openwhisk/pull/3406#issuecomment-371134460 _Nice_. I'm a little surprised it works given all the untar problems I had, but kudos that it does. Will keep

[GitHub] markusthoemmes closed pull request #3397: Partially revert change to route all action proxy tests through port mapping

markusthoemmes closed pull request #3397: Partially revert change to route all action proxy tests through port mapping URL: https://github.com/apache/incubator-openwhisk/pull/3397 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed

[GitHub] rabbah commented on a change in pull request #3407: WIP: No longer need to build 'whisk/*' runtimes for builtins.

rabbah commented on a change in pull request #3407: WIP: No longer need to build 'whisk/*' runtimes for builtins. URL: https://github.com/apache/incubator-openwhisk/pull/3407#discussion_r172848571 ## File path: actionRuntimes/actionProxy/README.md ## @@ -1,37 +0,0 @@

[GitHub] rabbah commented on issue #3382: Cleanup script for unused entities in the whisks database.

rabbah commented on issue #3382: Cleanup script for unused entities in the whisks database. URL: https://github.com/apache/incubator-openwhisk/pull/3382#issuecomment-371152049 @lue-kle do you have an Apache CLA? This is an

[GitHub] kameshsampath commented on issue #15: More flexibility to parsing and passing JSON input/out in Java runtime

kameshsampath commented on issue #15: More flexibility to parsing and passing JSON input/out in Java runtime URL: https://github.com/apache/incubator-openwhisk-runtime-java/issues/15#issuecomment-371158635 @mkouba thanks for suggestion, we are working on to adhere to standard Java

[GitHub] Param-S commented on issue #24: AdoptOpenJDK Eclipse OpenJ9 as alternative Java runtime

Param-S commented on issue #24: AdoptOpenJDK Eclipse OpenJ9 as alternative Java runtime URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/24#issuecomment-371144210 @kameshsampath It will be useful. But, I think, we can have separate discussion as we may need to focus

[GitHub] lue-kle commented on issue #3382: Cleanup script for unused entities in the whisks database.

lue-kle commented on issue #3382: Cleanup script for unused entities in the whisks database. URL: https://github.com/apache/incubator-openwhisk/pull/3382#issuecomment-371144282 Playground3 (Build #1966) run successfully on this fork/branch.

[GitHub] rabbah commented on a change in pull request #3407: WIP: No longer need to build 'whisk/*' runtimes for builtins.

rabbah commented on a change in pull request #3407: WIP: No longer need to build 'whisk/*' runtimes for builtins. URL: https://github.com/apache/incubator-openwhisk/pull/3407#discussion_r172848987 ## File path: ansible/group_vars/all ## @@ -207,6 +207,9 @@ docker:

[GitHub] rabbah commented on a change in pull request #3407: WIP: No longer need to build 'whisk/*' runtimes for builtins.

rabbah commented on a change in pull request #3407: WIP: No longer need to build 'whisk/*' runtimes for builtins. URL: https://github.com/apache/incubator-openwhisk/pull/3407#discussion_r172848987 ## File path: ansible/group_vars/all ## @@ -207,6 +207,9 @@ docker:

[GitHub] markusthoemmes commented on issue #3407: WIP: No longer need to build 'whisk/*' runtimes for builtins.

markusthoemmes commented on issue #3407: WIP: No longer need to build 'whisk/*' runtimes for builtins. URL: https://github.com/apache/incubator-openwhisk/pull/3407#issuecomment-371146038 I would split this in at least two PRs. Removing the actionRuntimes from the repo is imho not part of

[GitHub] mkouba commented on issue #15: More flexibility to parsing and passing JSON input/out in Java runtime

mkouba commented on issue #15: More flexibility to parsing and passing JSON input/out in Java runtime URL: https://github.com/apache/incubator-openwhisk-runtime-java/issues/15#issuecomment-371154325 Also the current return type (`com.google.gson.JsonObject`) of a Java action signature

[GitHub] dgrove-oss commented on issue #3402: Create '_users' database for singleton ephemeral couchdb v 2+

dgrove-oss commented on issue #3402: Create '_users' database for singleton ephemeral couchdb v 2+ URL: https://github.com/apache/incubator-openwhisk/pull/3402#issuecomment-371158446 @rabbah - Maybe you are thinking of the kube-couchdb docker image? @bbrowning added code to create the

[GitHub] RSulzmann closed pull request #3394: fix deploy.yml

RSulzmann closed pull request #3394: fix deploy.yml URL: https://github.com/apache/incubator-openwhisk/pull/3394 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 request

[GitHub] RSulzmann commented on issue #3394: fix deploy.yml

RSulzmann commented on issue #3394: fix deploy.yml URL: https://github.com/apache/incubator-openwhisk/pull/3394#issuecomment-371133153 fixed in https://github.com/apache/incubator-openwhisk/pull/3406 closing PR This is an

[GitHub] jonpspri opened a new pull request #3407: WIP: No longer need to build 'whisk/*' runtimes for builtins.

jonpspri opened a new pull request #3407: WIP: No longer need to build 'whisk/*' runtimes for builtins. URL: https://github.com/apache/incubator-openwhisk/pull/3407 Adjust the build and invoker to pull and deploy runtimes based on their fully-qualified Docker image names. Runtimes

[GitHub] cbickel commented on issue #3406: Fix deployment of the CLI if edge is on a remote machine.

cbickel commented on issue #3406: Fix deployment of the CLI if edge is on a remote machine. URL: https://github.com/apache/incubator-openwhisk/pull/3406#issuecomment-371082982 PG5#145 ? . This is an automated message from

[GitHub] jonpspri opened a new pull request #3408: Fine-tuning Dockerfile for alpine 'scala' image

jonpspri opened a new pull request #3408: Fine-tuning Dockerfile for alpine 'scala' image URL: https://github.com/apache/incubator-openwhisk/pull/3408 Linting-type improvements to the alpine 'scala' Dockerfile. No functional changes. ## Description Two changes to the Dockerfile

[GitHub] jonpspri commented on a change in pull request #3405: Install CLI with edge.yml

jonpspri commented on a change in pull request #3405: Install CLI with edge.yml URL: https://github.com/apache/incubator-openwhisk/pull/3405#discussion_r172836207 ## File path: ansible/edge.yml ## @@ -8,3 +8,4 @@ roles: - nginx - cli + - cli-install Review

[GitHub] jeremiaswerner closed pull request #3406: Fix deployment of the CLI if edge is on a remote machine.

jeremiaswerner closed pull request #3406: Fix deployment of the CLI if edge is on a remote machine. URL: https://github.com/apache/incubator-openwhisk/pull/3406 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] jonpspri commented on a change in pull request #3405: Install CLI with edge.yml

jonpspri commented on a change in pull request #3405: Install CLI with edge.yml URL: https://github.com/apache/incubator-openwhisk/pull/3405#discussion_r172836598 ## File path: ansible/edge.yml ## @@ -8,3 +8,4 @@ roles: - nginx - cli + - cli-install Review

[GitHub] rabbah closed pull request #3391: Add namespace-blacklist to invoker.

rabbah closed pull request #3391: Add namespace-blacklist to invoker. URL: https://github.com/apache/incubator-openwhisk/pull/3391 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] markusthoemmes commented on a change in pull request #3382: Cleanup script for unused entities in the whisks database.

markusthoemmes commented on a change in pull request #3382: Cleanup script for unused entities in the whisks database. URL: https://github.com/apache/incubator-openwhisk/pull/3382#discussion_r172871483 ## File path: tools/db/cleanUpWhisks.py ## @@ -0,0 +1,150 @@

[GitHub] jonpspri commented on a change in pull request #3407: WIP: No longer need to build 'whisk/*' runtimes for builtins.

jonpspri commented on a change in pull request #3407: WIP: No longer need to build 'whisk/*' runtimes for builtins. URL: https://github.com/apache/incubator-openwhisk/pull/3407#discussion_r172873290 ## File path: ansible/group_vars/all ## @@ -207,6 +207,9 @@ docker:

[GitHub] jcrossley3 opened a new pull request #3409: Remove namespace configuration; use invoker's instead

jcrossley3 opened a new pull request #3409: Remove namespace configuration; use invoker's instead URL: https://github.com/apache/incubator-openwhisk/pull/3409 Making this configurable -- and defaulting to something that may not be the ns in which the invoker is running -- is more

[GitHub] mkouba commented on issue #15: More flexibility to parsing and passing JSON input/out in Java runtime

mkouba commented on issue #15: More flexibility to parsing and passing JSON input/out in Java runtime URL: https://github.com/apache/incubator-openwhisk-runtime-java/issues/15#issuecomment-371173793 @rabbah Why is that? Is the contract for OW actions defined somewhere?

[GitHub] mkouba commented on issue #15: More flexibility to parsing and passing JSON input/out in Java runtime

mkouba commented on issue #15: More flexibility to parsing and passing JSON input/out in Java runtime URL: https://github.com/apache/incubator-openwhisk-runtime-java/issues/15#issuecomment-371179496 @kameshsampath Yep, I've seen this code already. We can wrap the resulting

[GitHub] jonpspri commented on issue #3407: WIP: No longer need to build 'whisk/*' runtimes for builtins.

jonpspri commented on issue #3407: WIP: No longer need to build 'whisk/*' runtimes for builtins. URL: https://github.com/apache/incubator-openwhisk/pull/3407#issuecomment-371171374 Agreed this is pretty massive. I'll see what I can do to break it up as it leaves WIP; some of the changes

[GitHub] jonpspri commented on issue #3407: WIP: No longer need to build 'whisk/*' runtimes for builtins.

jonpspri commented on issue #3407: WIP: No longer need to build 'whisk/*' runtimes for builtins. URL: https://github.com/apache/incubator-openwhisk/pull/3407#issuecomment-371171374 Agreed this is pretty massive. I'll see what I can do to break it up as it leaves WIP; some of the changes

[GitHub] jonpspri commented on a change in pull request #3407: WIP: No longer need to build 'whisk/*' runtimes for builtins.

jonpspri commented on a change in pull request #3407: WIP: No longer need to build 'whisk/*' runtimes for builtins. URL: https://github.com/apache/incubator-openwhisk/pull/3407#discussion_r172872933 ## File path: actionRuntimes/actionProxy/invoke.py ## @@ -1,136 +0,0 @@

[GitHub] dgrove-oss commented on a change in pull request #3407: WIP: No longer need to build 'whisk/*' runtimes for builtins.

dgrove-oss commented on a change in pull request #3407: WIP: No longer need to build 'whisk/*' runtimes for builtins. URL: https://github.com/apache/incubator-openwhisk/pull/3407#discussion_r172875542 ## File path: ansible/group_vars/all ## @@ -207,6 +207,9 @@ docker:

[GitHub] mkouba commented on issue #15: More flexibility to parsing and passing JSON input/out in Java runtime

mkouba commented on issue #15: More flexibility to parsing and passing JSON input/out in Java runtime URL: https://github.com/apache/incubator-openwhisk-runtime-java/issues/15#issuecomment-371173308 I'd try to avoid `main(String args[])` if possible. Instead, we could allow user to

[GitHub] kameshsampath commented on issue #15: More flexibility to parsing and passing JSON input/out in Java runtime

kameshsampath commented on issue #15: More flexibility to parsing and passing JSON input/out in Java runtime URL: https://github.com/apache/incubator-openwhisk-runtime-java/issues/15#issuecomment-371177266 @mkouba - this is where it needs to be fixed further

[GitHub] rabbah commented on issue #15: More flexibility to parsing and passing JSON input/out in Java runtime

rabbah commented on issue #15: More flexibility to parsing and passing JSON input/out in Java runtime URL: https://github.com/apache/incubator-openwhisk-runtime-java/issues/15#issuecomment-371183079 @mkouba

[GitHub] jonpspri commented on a change in pull request #3407: WIP: No longer need to build 'whisk/*' runtimes for builtins.

jonpspri commented on a change in pull request #3407: WIP: No longer need to build 'whisk/*' runtimes for builtins. URL: https://github.com/apache/incubator-openwhisk/pull/3407#discussion_r172887567 ## File path: ansible/group_vars/all ## @@ -207,6 +207,9 @@ docker:

[GitHub] dgrove-oss commented on issue #161: x509: certificate signed by unknown authority accessing Nginx in Windows 10

dgrove-oss commented on issue #161: x509: certificate signed by unknown authority accessing Nginx in Windows 10 URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/161#issuecomment-371162716 hi. When using the self-signed certificate generated by certs.sh, you need to

  1   2   >