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

2018-03-07 Thread GitBox
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 p

[GitHub] mrutkows closed issue #705: Unable to use ENV VARS in input JSON

2018-03-07 Thread GitBox
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, ple

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

2018-03-07 Thread GitBox
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] jonpspri commented on issue #3407: WIP: No longer need to build 'whisk/*' runtimes for builtins.

2018-03-07 Thread GitBox
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 to

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

2018-03-07 Thread GitBox
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] lionelvillard opened a new issue #774: support for deploying/undeploying headless managed project

2018-03-07 Thread GitBox
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 `--man

[GitHub] houshengbo opened a new issue #54: Add the missing license header to the source files in each repo to be released

2018-03-07 Thread GitBox
houshengbo opened a new 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 There are 12 repos we plan to release for openwhisk: "openwhisk", "openwhisk-wskdeploy", "openw

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

2018-03-07 Thread GitBox
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 provenan

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

2018-03-07 Thread GitBox
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. Th

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

2018-03-07 Thread GitBox
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-371274102 @rabbah yes I have signed the CLA. I will talk to @cbickel regarding your suggestion for consolidation. ---

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

2018-03-07 Thread GitBox
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] pritidesai closed pull request #23: Add the license headers

2018-03-07 Thread GitBox
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 fore

[GitHub] lionelvillard opened a new pull request #775: add --name project name flag

2018-03-07 Thread GitBox
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 mess

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

2018-03-07 Thread GitBox
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 messa

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

2018-03-07 Thread GitBox
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] houshengbo closed pull request #3383: Add the license headers

2018-03-07 Thread GitBox
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

2018-03-07 Thread GitBox
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 Gi

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

2018-03-07 Thread GitBox
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 Servic

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

2018-03-07 Thread GitBox
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] mdeuser commented on a change in pull request #3352: API GW doc update to include path parameter example

2018-03-07 Thread GitBox
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] houshengbo commented on a change in pull request #53: Change to use RAT maven plugin to support ASF short format header

2018-03-07 Thread GitBox
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 opened a new pull request #3410: Fix typos in api gw doc

2018-03-07 Thread GitBox
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 c

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

2018-03-07 Thread GitBox
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

2018-03-07 Thread GitBox
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 #3388: Update require-whisk-auth behavior to secure web action

2018-03-07 Thread GitBox
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] csantanapr closed pull request #3410: Fix typos in api gw doc

2018-03-07 Thread GitBox
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] csantanapr closed pull request #3388: Update require-whisk-auth behavior to secure web action

2018-03-07 Thread GitBox
csantanapr closed pull request #3388: Update require-whisk-auth behavior to secure web action URL: https://github.com/apache/incubator-openwhisk/pull/3388 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 proven

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

2018-03-07 Thread GitBox
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 t

[GitHub] tysonnorris commented on issue #2833: MesosContainerFactory

2018-03-07 Thread GitBox
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 s

[GitHub] rabbah commented on issue #2833: MesosContainerFactory

2018-03-07 Thread GitBox
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] rabbah commented on issue #2833: MesosContainerFactory

2018-03-07 Thread GitBox
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] dgrove-oss commented on issue #3378: LogStoreProvider using invokerAgent for KubernetesContainerPool

2018-03-07 Thread GitBox
dgrove-oss commented on issue #3378: LogStoreProvider using invokerAgent for KubernetesContainerPool URL: https://github.com/apache/incubator-openwhisk/pull/3378#issuecomment-371343603 Superseded by #3411. This is an automa

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

2018-03-07 Thread GitBox
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 o

[GitHub] dgrove-oss opened a new pull request #3411: refactor KubernetesClient to separate invokerAgent; add forwarding LogStoreProvider

2018-03-07 Thread GitBox
dgrove-oss opened a new pull request #3411: refactor KubernetesClient to separate invokerAgent; add forwarding LogStoreProvider URL: https://github.com/apache/incubator-openwhisk/pull/3411 ## Description 1. refactor KubernetesClient to move invokerAgent into a subclass to get a better s

[GitHub] dgrove-oss commented on issue #3411: refactor KubernetesClient to separate invokerAgent; add forwarding LogStoreProvider

2018-03-07 Thread GitBox
dgrove-oss commented on issue #3411: refactor KubernetesClient to separate invokerAgent; add forwarding LogStoreProvider URL: https://github.com/apache/incubator-openwhisk/pull/3411#issuecomment-371346769 @jcrossley3 please take a look and let me know what you think. @markusthoemmes sugg

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

2018-03-07 Thread GitBox
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: core/invoker/src/main/scala/whisk/core/conta

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

2018-03-07 Thread GitBox
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 commented on a change in pull request #3411: refactor KubernetesClient to separate invokerAgent; add forwarding LogStoreProvider

2018-03-07 Thread GitBox
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: core/invoker/src/main/scala/whisk/core/conta

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

2018-03-07 Thread GitBox
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: core/invoker/src/main/scala/whisk/core/conta

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

2018-03-07 Thread GitBox
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 automa

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

2018-03-07 Thread GitBox
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) o

[GitHub] dgrove-oss commented on issue #3411: refactor KubernetesClient to separate invokerAgent; add forwarding LogStoreProvider

2018-03-07 Thread GitBox
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] dubeejw closed pull request #67: Additional HTTP Headers

2018-03-07 Thread GitBox
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 pu

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

2018-03-07 Thread GitBox
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 automate

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

2018-03-07 Thread GitBox
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 wi

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

2018-03-07 Thread GitBox
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 @@ +DCOS_

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

2018-03-07 Thread GitBox
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 f

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

2018-03-07 Thread GitBox
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 a

<    1   2