[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] 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] 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] 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] 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] 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] 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] 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] 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] 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] 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] 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] 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] 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] 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, `AdditionalHeaders` 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, AdditionalHeader is meant for consumers of client-go that do not have access to `requestUrl()`.

[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] 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 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] 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] 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] 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] 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] 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 #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] 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] 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] 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] 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] 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 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] 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] 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] 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] 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] 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 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_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] 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 #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] 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-371204992 rebased + added doc update with an example @dubeejw pg2/2899

[GitHub] dubeejw closed pull request #3405: Install CLI with edge.yml

dubeejw closed pull request #3405: Install CLI with edge.yml URL: https://github.com/apache/incubator-openwhisk/pull/3405 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] csantanapr commented on issue #3407: WIP: No longer need to build 'whisk/*' runtimes for builtins.

csantanapr commented on issue #3407: WIP: No longer need to build 'whisk/*' runtimes for builtins. URL: https://github.com/apache/incubator-openwhisk/pull/3407#issuecomment-371208056 I wan to see the flexibility of defining in the manifest json the runtimes eaxh can come different

[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-371207019 @dubeejw is the intent to provide per-request specific headers? if so, it might be more flexible to include the additional headers via the

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

rabbah commented on issue #3388: Update require-whisk-auth behavior to secure web action URL: https://github.com/apache/incubator-openwhisk/pull/3388#issuecomment-371206530 I dont think you need a pg. This is an automated

[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-371204992 rebased + added doc update with an example pg2/2899

[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-371204992 rebased + added doc update with an example This

[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-371200382 @rabbah @mkouba all I was trying to avoid doing via `main(String[] args)` is to

[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-371200360 @mdeuser This is an automated message from the Apache Git Service. To

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

csantanapr 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-371198689 @rabbah this is similar in spirit what I?m doin in swift4 with Codable IN and Codable

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

jpkessle 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_r172898928 ## File path: docs/apigateway.md ## @@ -192,14 +205,16 @@ Now let's restore the

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

jpkessle 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_r172898998 ## File path: docs/apigateway.md ## @@ -192,14 +205,16 @@ Now let's restore the

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

jpkessle 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_r172898965 ## File path: docs/apigateway.md ## @@ -192,14 +205,16 @@ Now let's restore the

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

jpkessle 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_r172898856 ## File path: docs/apigateway.md ## @@ -192,14 +205,16 @@ Now let's restore the

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

jpkessle 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_r172898796 ## File path: docs/apigateway.md ## @@ -192,14 +205,16 @@ Now let's restore the

[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-371188993 > I assume you also mean there is a a serializer/deserializer from Foo to JSON object?

[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-371187087 > I'd try to avoid main(String args[]) if possible. Instead, we could allow user to

[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-371187087 > I'd try to avoid main(String args[]) if possible. Instead, we could allow user to

[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] 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-371187087 > I'd try to avoid main(String args[]) if possible. Instead, we could allow user to

[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] 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-371184620 @rabbah Ok, thanks. This

[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] 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] 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] 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] dgrove-oss commented on issue #3409: Remove namespace configuration; use invoker's instead

dgrove-oss commented on issue #3409: Remove namespace configuration; use invoker's instead URL: https://github.com/apache/incubator-openwhisk/pull/3409#issuecomment-371179109 PG1 / 2688 ? This is an automated message from

[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-371179202 > I can try to send a PR if you are interested ;-) you are most welcome

[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] 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] 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] 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-371173849 As a general note: It would be useful in the long run to consolidate this and scripts like it into wskadmin (and

[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] 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-371173021 Note: Whisk actions must return json objects. If you change the signature you?ll need to

[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] 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_r172873079 ## File path: actionRuntimes/actionProxy/README.md ## @@ -1,37 +0,0 @@

[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] 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] 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_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] 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_r172873079 ## File path: actionRuntimes/actionProxy/README.md ## @@ -1,37 +0,0 @@

[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] 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] dgrove-oss opened a new issue #239: More helpful error message when unable to verify certificates

dgrove-oss opened a new issue #239: More helpful error message when unable to verify certificates URL: https://github.com/apache/incubator-openwhisk-cli/issues/239 See https://github.com/apache/incubator-openwhisk-deploy-kube/issues/161. It would be helpful for new users if the

[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

[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] 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] 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] 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] 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] 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

  1   2   >