[GitHub] codecov-io commented on issue #3590: Collect test reports as part of log collection

2018-05-03 Thread GitBox
codecov-io commented on issue #3590: Collect test reports as part of log collection URL: https://github.com/apache/incubator-openwhisk/pull/3590#issuecomment-384934946 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3590?src=pr=h1) Report > Merging

[GitHub] codecov-io commented on issue #3588: Make count with skip to work with CouchDB

2018-05-03 Thread GitBox
codecov-io commented on issue #3588: Make count with skip to work with CouchDB URL: https://github.com/apache/incubator-openwhisk/pull/3588#issuecomment-384704462 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3588?src=pr=h1) Report > Merging

[GitHub] chetanmeh commented on issue #3590: Collect test reports as part of log collection

2018-05-03 Thread GitBox
chetanmeh commented on issue #3590: Collect test reports as part of log collection URL: https://github.com/apache/incubator-openwhisk/pull/3590#issuecomment-386505581 @csantanapr I followed the pattern used for other log task and made report collection rely on condition `when: "'tests'

[GitHub] chetanmeh commented on a change in pull request #3588: Make count with skip to work with CouchDB

2018-05-03 Thread GitBox
chetanmeh commented on a change in pull request #3588: Make count with skip to work with CouchDB URL: https://github.com/apache/incubator-openwhisk/pull/3588#discussion_r185989973 ## File path: common/scala/src/main/scala/whisk/core/database/CouchDbRestStore.scala ## @@

[GitHub] codecov-io commented on issue #3525: Re-enable invokers docker_container module

2018-05-03 Thread GitBox
codecov-io commented on issue #3525: Re-enable invokers docker_container module URL: https://github.com/apache/incubator-openwhisk/pull/3525#issuecomment-386497161 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3525?src=pr=h1) Report > Merging

[GitHub] houshengbo commented on issue #3607: Change the JDK to use openjdk8-openj9

2018-05-03 Thread GitBox
houshengbo commented on issue #3607: Change the JDK to use openjdk8-openj9 URL: https://github.com/apache/incubator-openwhisk/issues/3607#issuecomment-386478210 @mrutkows Apache policy seems to raise caution of BCL: ``` BCL Oracle's (originally Sun's) Binary Code License

[GitHub] houshengbo commented on issue #3607: Change the JDK to use openjdk8-openj9

2018-05-03 Thread GitBox
houshengbo commented on issue #3607: Change the JDK to use openjdk8-openj9 URL: https://github.com/apache/incubator-openwhisk/issues/3607#issuecomment-386478210 @mrutkows Apache policy seems to raise caution of BCL: ``` BCL Oracle's (originally Sun's) Binary Code License

[GitHub] houshengbo commented on issue #3607: Change the JDK to use openjdk8-openj9

2018-05-03 Thread GitBox
houshengbo commented on issue #3607: Change the JDK to use openjdk8-openj9 URL: https://github.com/apache/incubator-openwhisk/issues/3607#issuecomment-386478210 @mrutkows Apache policy seems to raise caution of BCL: ``` BCL Oracle's (originally Sun's) Binary Code License

[GitHub] pritidesai commented on issue #890: Resolves issue #880

2018-05-03 Thread GitBox
pritidesai commented on issue #890: Resolves issue #880 URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/890#issuecomment-386472766 Hi @kpavel I am testing these changes and able to generate `manifest.yaml` after running: ``` ./wskdeploy sync -m

[GitHub] pritidesai opened a new pull request #894: adding list of YAML key/values which supports env. variables

2018-05-03 Thread GitBox
pritidesai opened a new pull request #894: adding list of YAML key/values which supports env. variables URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/894 Closes #892 This is an automated message from the

[GitHub] mrutkows commented on issue #3607: Change the JDK to use openjdk8-openj9

2018-05-03 Thread GitBox
mrutkows commented on issue #3607: Change the JDK to use openjdk8-openj9 URL: https://github.com/apache/incubator-openwhisk/issues/3607#issuecomment-386452951 @markusthoemmes @houshengbo Some things that seem clear to me: - We do not plan to distribute source code, only binary

[GitHub] mrutkows commented on issue #3607: Change the JDK to use openjdk8-openj9

2018-05-03 Thread GitBox
mrutkows commented on issue #3607: Change the JDK to use openjdk8-openj9 URL: https://github.com/apache/incubator-openwhisk/issues/3607#issuecomment-386452951 @markusthoemmes @houshengbo Some things that seem clear to me: - We do not plan to distribute source code, only binary

[GitHub] mrutkows commented on issue #3607: Change the JDK to use openjdk8-openj9

2018-05-03 Thread GitBox
mrutkows commented on issue #3607: Change the JDK to use openjdk8-openj9 URL: https://github.com/apache/incubator-openwhisk/issues/3607#issuecomment-386452951 @markusthoemmes @houshengbo Some things that seem clear to me: - We do not plan to distribute source code, only binary

[GitHub] mrutkows commented on issue #3607: Change the JDK to use openjdk8-openj9

2018-05-03 Thread GitBox
mrutkows commented on issue #3607: Change the JDK to use openjdk8-openj9 URL: https://github.com/apache/incubator-openwhisk/issues/3607#issuecomment-386452951 @markusthoemmes @houshengbo Some things that seem clear to me: - We do not plan to distribute source code, only binary

[GitHub] mrutkows commented on issue #3607: Change the JDK to use openjdk8-openj9

2018-05-03 Thread GitBox
mrutkows commented on issue #3607: Change the JDK to use openjdk8-openj9 URL: https://github.com/apache/incubator-openwhisk/issues/3607#issuecomment-386452951 @markusthoemmes @houshengbo Some things that seem clear to me: - We do not plan to distribute source code, only binary

[GitHub] mrutkows commented on issue #3607: Change the JDK to use openjdk8-openj9

2018-05-03 Thread GitBox
mrutkows commented on issue #3607: Change the JDK to use openjdk8-openj9 URL: https://github.com/apache/incubator-openwhisk/issues/3607#issuecomment-386452951 @markusthoemmes @houshengbo Some things that seem clear to me: - We do not plan to distribute source code, only binary

[GitHub] csantanapr commented on issue #2147: Parse `+json` content types suffix for web action bodies

2018-05-03 Thread GitBox
csantanapr commented on issue #2147: Parse `+json` content types suffix for web action bodies URL: https://github.com/apache/incubator-openwhisk/issues/2147#issuecomment-386458099 @rabbah what are the next steps to fix this in your opinion?

[GitHub] mrutkows commented on issue #3607: Change the JDK to use openjdk8-openj9

2018-05-03 Thread GitBox
mrutkows commented on issue #3607: Change the JDK to use openjdk8-openj9 URL: https://github.com/apache/incubator-openwhisk/issues/3607#issuecomment-386452951 @markusthoemmes @houshengbo Some things that seem clear to me: - We do not plan to distribute source code, only binary

[GitHub] mrutkows commented on issue #3607: Change the JDK to use openjdk8-openj9

2018-05-03 Thread GitBox
mrutkows commented on issue #3607: Change the JDK to use openjdk8-openj9 URL: https://github.com/apache/incubator-openwhisk/issues/3607#issuecomment-386452951 @markusthoemmes @houshengbo Some things that seem clear to me: - We do not plan to distribute source code, only binary

[GitHub] mrutkows commented on issue #3607: Change the JDK to use openjdk8-openj9

2018-05-03 Thread GitBox
mrutkows commented on issue #3607: Change the JDK to use openjdk8-openj9 URL: https://github.com/apache/incubator-openwhisk/issues/3607#issuecomment-386452951 @markusthoemmes @houshengbo Some things that seem clear to me: - We do not plan to distribute source code, only binary

[GitHub] mrutkows commented on issue #3607: Change the JDK to use openjdk8-openj9

2018-05-03 Thread GitBox
mrutkows commented on issue #3607: Change the JDK to use openjdk8-openj9 URL: https://github.com/apache/incubator-openwhisk/issues/3607#issuecomment-386452951 @markusthoemmes @houshengbo Some things that seem clear to me: - We do not plan to distribute source code, only binary

[GitHub] mrutkows commented on issue #3607: Change the JDK to use openjdk8-openj9

2018-05-03 Thread GitBox
mrutkows commented on issue #3607: Change the JDK to use openjdk8-openj9 URL: https://github.com/apache/incubator-openwhisk/issues/3607#issuecomment-386452951 @markusthoemmes @houshengbo Some things that seem clear to me: - The intent of the Oracle Binary Code License Agreement

[GitHub] mrutkows commented on issue #3607: Change the JDK to use openjdk8-openj9

2018-05-03 Thread GitBox
mrutkows commented on issue #3607: Change the JDK to use openjdk8-openj9 URL: https://github.com/apache/incubator-openwhisk/issues/3607#issuecomment-386452951 @markusthoemmes @houshengbo Some things that seem clear to me: - The intent of the Oracle Binary Code License Agreement

[GitHub] mrutkows commented on issue #3607: Change the JDK to use openjdk8-openj9

2018-05-03 Thread GitBox
mrutkows commented on issue #3607: Change the JDK to use openjdk8-openj9 URL: https://github.com/apache/incubator-openwhisk/issues/3607#issuecomment-386452951 @markusthoemmes @houshengbo Some things that seem clear to me: - The intent of the Oracle Binary Code License Agreement

[GitHub] duglin commented on issue #2147: Parse `+json` content types suffix for web action bodies

2018-05-03 Thread GitBox
duglin commented on issue #2147: Parse `+json` content types suffix for web action bodies URL: https://github.com/apache/incubator-openwhisk/issues/2147#issuecomment-386450729 I just ran into this today (I think its either the same, or related, issue). My function is in a docker

[GitHub] houshengbo commented on issue #3607: Change the JDK to use openjdk8-openj9

2018-05-03 Thread GitBox
houshengbo commented on issue #3607: Change the JDK to use openjdk8-openj9 URL: https://github.com/apache/incubator-openwhisk/issues/3607#issuecomment-386446086 @markusthoemmes That is good question I am currently seeking the answer as well. So far, my answer tends to be fine with

[GitHub] houshengbo opened a new issue #53: What is the license of ibmcom/swift-ubuntu:3.1.1

2018-05-03 Thread GitBox
houshengbo opened a new issue #53: What is the license of ibmcom/swift-ubuntu:3.1.1 URL: https://github.com/apache/incubator-openwhisk-runtime-swift/issues/53 https://hub.docker.com/r/ibmcom/swift-ubuntu-runtime/ We use this image, but we are not sure of the license.

[GitHub] markusthoemmes commented on issue #3607: Change the JDK to use openjdk8-openj9

2018-05-03 Thread GitBox
markusthoemmes commented on issue #3607: Change the JDK to use openjdk8-openj9 URL: https://github.com/apache/incubator-openwhisk/issues/3607#issuecomment-386440722 Is this also an issue if the mentioned alternative JDKs are not part of our release? We're moving to use a prebuilt docker

[GitHub] mcdan opened a new issue #3608: Unnecessary "becomes" pollutes global python install

2018-05-03 Thread GitBox
mcdan opened a new issue #3608: Unnecessary "becomes" pollutes global python install URL: https://github.com/apache/incubator-openwhisk/issues/3608 ## Environment details: * local deployment, vagrant, native ubuntu, Mac OS, Bluemix, ... * version of docker, vagrant,

[GitHub] houshengbo commented on issue #137: JDK license issue for the runtime Java

2018-05-03 Thread GitBox
houshengbo commented on issue #137: JDK license issue for the runtime Java URL: https://github.com/apache/incubator-openwhisk-release/issues/137#issuecomment-386434735 Tracking this issue to be resolved: OpenWhisk: https://github.com/apache/incubator-openwhisk/issues/3607

[GitHub] houshengbo opened a new issue #3607: Change the JDK to use openjdk8-openj9

2018-05-03 Thread GitBox
houshengbo opened a new issue #3607: Change the JDK to use openjdk8-openj9 URL: https://github.com/apache/incubator-openwhisk/issues/3607 Apache project will have issue using JDK licensed under oracle BCL or GNU GPL. For docker image with JDK, we have to switch to use

[GitHub] houshengbo opened a new pull request #272: Remove the unnecessary dependencies

2018-05-03 Thread GitBox
houshengbo opened a new pull request #272: Remove the unnecessary dependencies URL: https://github.com/apache/incubator-openwhisk-catalog/pull/272 This is an automated message from the Apache Git Service. To respond to the

[GitHub] houshengbo commented on issue #137: JDK license issue for the runtime Java

2018-05-03 Thread GitBox
houshengbo commented on issue #137: JDK license issue for the runtime Java URL: https://github.com/apache/incubator-openwhisk-release/issues/137#issuecomment-386424740 Based on my observation, the momentum of openwhisk runtime java, changing from oracle JDK to Docker Images for Eclipse

[GitHub] eweiter opened a new issue #293: CLI handling of Trigger parameters vs Feed parameters

2018-05-03 Thread GitBox
eweiter opened a new issue #293: CLI handling of Trigger parameters vs Feed parameters URL: https://github.com/apache/incubator-openwhisk-cli/issues/293 This issue is the culmination of discussions and two other issues that were opened, #289 & #290 . A proposal was talked about for

[GitHub] abaruni commented on issue #289: Unable to view Trigger parameters from the CLI

2018-05-03 Thread GitBox
abaruni commented on issue #289: Unable to view Trigger parameters from the CLI URL: https://github.com/apache/incubator-openwhisk-cli/issues/289#issuecomment-386421778 feed and triggers are fundamentally two distinct concepts with a feed being composed of a trigger and some event source

[GitHub] abaruni commented on issue #289: Unable to view Trigger parameters from the CLI

2018-05-03 Thread GitBox
abaruni commented on issue #289: Unable to view Trigger parameters from the CLI URL: https://github.com/apache/incubator-openwhisk-cli/issues/289#issuecomment-386419785 my proposal here would be to no longer mix the concepts of a feed and a trigger and have a separate whisk entity for

[GitHub] abaruni commented on issue #290: Unable to update Trigger parameters from CLI

2018-05-03 Thread GitBox
abaruni commented on issue #290: Unable to update Trigger parameters from CLI URL: https://github.com/apache/incubator-openwhisk-cli/issues/290#issuecomment-386419169 my proposal here would be to no longer mix the concepts of a feed and a trigger and have a separate whisk entity for

[GitHub] dubee opened a new pull request #292: Account for variable length timestamps for log stripping

2018-05-03 Thread GitBox
dubee opened a new pull request #292: Account for variable length timestamps for log stripping URL: https://github.com/apache/incubator-openwhisk-cli/pull/292 See issue for details. Closes https://github.com/apache/incubator-openwhisk-cli/issues/291

[GitHub] abaruni commented on issue #289: Unable to view Trigger parameters from the CLI

2018-05-03 Thread GitBox
abaruni commented on issue #289: Unable to view Trigger parameters from the CLI URL: https://github.com/apache/incubator-openwhisk-cli/issues/289#issuecomment-386419785 my proposal here would be to no longer mix the concepts of a feed and a trigger and have a separate whisk entity for

[GitHub] abaruni commented on issue #290: Unable to update Trigger parameters from CLI

2018-05-03 Thread GitBox
abaruni commented on issue #290: Unable to update Trigger parameters from CLI URL: https://github.com/apache/incubator-openwhisk-cli/issues/290#issuecomment-386419169 my proposal here would be to no longer mix the concepts of a feed and a trigger and have a separate whisk entity for

[GitHub] abaruni commented on issue #290: Unable to update Trigger parameters from CLI

2018-05-03 Thread GitBox
abaruni commented on issue #290: Unable to update Trigger parameters from CLI URL: https://github.com/apache/incubator-openwhisk-cli/issues/290#issuecomment-386419169 my proposal here would be to not mix the concepts of a feed and a trigger and have a separate whisk entity for feeds. then

[GitHub] dubee opened a new issue #291: Activation log --strip command malforms output

2018-05-03 Thread GitBox
dubee opened a new issue #291: Activation log --strip command malforms output URL: https://github.com/apache/incubator-openwhisk-cli/issues/291 When displaying activation logs with the `--strip` flag, the first 39 characters of a log line are removed to exclude the log timestamp and pipe.

[GitHub] mrutkows closed pull request #893: adding env. variables in action function

2018-05-03 Thread GitBox
mrutkows closed pull request #893: adding env. variables in action function URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/893 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] mrutkows closed issue #891: Cannot use environment variables in action function parameters.

2018-05-03 Thread GitBox
mrutkows closed issue #891: Cannot use environment variables in action function parameters. URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/891 This is an automated message from the Apache Git Service.

[GitHub] rabbah closed pull request #189: add pointer to Helm deployment to top-level README.md

2018-05-03 Thread GitBox
rabbah closed pull request #189: add pointer to Helm deployment to top-level README.md URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/189 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 opened a new pull request #893: adding env. variables in action function

2018-05-03 Thread GitBox
pritidesai opened a new pull request #893: adding env. variables in action function URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/893 Closes #891 This is an automated message from the Apache Git Service.

[GitHub] pritidesai commented on issue #884: passing parameters between manifest and deploy.yml

2018-05-03 Thread GitBox
pritidesai commented on issue #884: passing parameters between manifest and deploy.yml URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/884#issuecomment-386391735 @jasonpet nope action doesn't show such parameter, also, @dubee referenced couple of issues from CLI repo

[GitHub] pritidesai closed pull request #887: WIP: please do not merge: an attempt to cleanup list of deps

2018-05-03 Thread GitBox
pritidesai closed pull request #887: WIP: please do not merge: an attempt to cleanup list of deps URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/887 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] pritidesai commented on issue #887: WIP: please do not merge: an attempt to cleanup list of deps

2018-05-03 Thread GitBox
pritidesai commented on issue #887: WIP: please do not merge: an attempt to cleanup list of deps URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/887#issuecomment-386388149 need to revisit this with may be better tooling to catch cross references ...

[GitHub] pritidesai closed pull request #888: Change the LICENSE file into the format defined in release

2018-05-03 Thread GitBox
pritidesai closed pull request #888: Change the LICENSE file into the format defined in release URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/888 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] eweiter commented on issue #289: Unable to view Trigger parameters from the CLI

2018-05-03 Thread GitBox
eweiter commented on issue #289: Unable to view Trigger parameters from the CLI URL: https://github.com/apache/incubator-openwhisk-cli/issues/289#issuecomment-386380543 If we expose the "trigger_payload" at the top level of a Trigger that would also address this issue, but fundamentally

[GitHub] eweiter commented on issue #289: Unable to view Trigger parameters from the CLI

2018-05-03 Thread GitBox
eweiter commented on issue #289: Unable to view Trigger parameters from the CLI URL: https://github.com/apache/incubator-openwhisk-cli/issues/289#issuecomment-386380007 More context on this issue: This issue is related to #290 It is dealing with Triggers that have associated Feeds.

[GitHub] dubee commented on a change in pull request #3588: Make count with skip to work with CouchDB

2018-05-03 Thread GitBox
dubee commented on a change in pull request #3588: Make count with skip to work with CouchDB URL: https://github.com/apache/incubator-openwhisk/pull/3588#discussion_r185884555 ## File path: common/scala/src/main/scala/whisk/core/database/CouchDbRestStore.scala ## @@

[GitHub] eweiter commented on issue #290: Unable to update Trigger parameters from CLI

2018-05-03 Thread GitBox
eweiter commented on issue #290: Unable to update Trigger parameters from CLI URL: https://github.com/apache/incubator-openwhisk-cli/issues/290#issuecomment-386376369 So one proposal for how to handle this would be the introduction of a new parameter, something like --feed-param/-fp ,

[GitHub] eweiter commented on issue #290: Unable to update Trigger parameters from CLI

2018-05-03 Thread GitBox
eweiter commented on issue #290: Unable to update Trigger parameters from CLI URL: https://github.com/apache/incubator-openwhisk-cli/issues/290#issuecomment-386375596 The commands to reproduce are: `wsk trigger create alarm --feed whisk.system/alarms/alarm -p cron "* * * * *"` `wsk

[GitHub] eweiter commented on issue #290: Unable to update Trigger parameters from CLI

2018-05-03 Thread GitBox
eweiter commented on issue #290: Unable to update Trigger parameters from CLI URL: https://github.com/apache/incubator-openwhisk-cli/issues/290#issuecomment-386375246 @csantanapr @abaruni Sorry for lacking context the first time. So this issue originated with an issue outside of the

[GitHub] mrutkows commented on issue #892: Create markdown page describing where Interpolation is supported

2018-05-03 Thread GitBox
mrutkows commented on issue #892: Create markdown page describing where Interpolation is supported URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/892#issuecomment-386366614 Once this issue is addressed, we can open a subsequent issue against the spec. to pickup the

[GitHub] mrutkows opened a new issue #892: Create markdown page describing where Interpolation is supported

2018-05-03 Thread GitBox
mrutkows opened a new issue #892: Create markdown page describing where Interpolation is supported URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/892 This issue #[891](https://github.com/apache/incubator-openwhisk-wskdeploy/issues/891) reminded me we discussed

[GitHub] mrutkows commented on issue #891: Cannot use environment variables in action function parameters.

2018-05-03 Thread GitBox
mrutkows commented on issue #891: Cannot use environment variables in action function parameters. URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/891#issuecomment-386365465 @pritidesai we have been adding interpolation for keys/values in many places (and thankfully

[GitHub] dubee commented on issue #222: A list of source files missing Apache license headers

2018-05-03 Thread GitBox
dubee commented on issue #222: A list of source files missing Apache license headers URL: https://github.com/apache/incubator-openwhisk-cli/issues/222#issuecomment-386364957 @daisy-ycguo, @mrutkows, can this issue be closed?

[GitHub] dubee commented on issue #290: Unable to update Trigger parameters from CLI

2018-05-03 Thread GitBox
dubee commented on issue #290: Unable to update Trigger parameters from CLI URL: https://github.com/apache/incubator-openwhisk-cli/issues/290#issuecomment-386364426 Related to https://github.com/apache/incubator-openwhisk-wskdeploy/issues/884#issuecomment-386106835.

[GitHub] dubee commented on issue #289: Unable to view Trigger parameters from the CLI

2018-05-03 Thread GitBox
dubee commented on issue #289: Unable to view Trigger parameters from the CLI URL: https://github.com/apache/incubator-openwhisk-cli/issues/289#issuecomment-386364464 Related to https://github.com/apache/incubator-openwhisk-wskdeploy/issues/884#issuecomment-386106835.

[GitHub] tysonnorris commented on a change in pull request #2795: enable concurrent activation processing

2018-05-03 Thread GitBox
tysonnorris commented on a change in pull request #2795: enable concurrent activation processing URL: https://github.com/apache/incubator-openwhisk/pull/2795#discussion_r185868550 ## File path: performance/wrk_tests/throughput-async.sh ## @@ -0,0 +1,31 @@ +#!/bin/sh +set

[GitHub] csantanapr commented on issue #3588: Make count with skip to work with CouchDB

2018-05-03 Thread GitBox
csantanapr commented on issue #3588: Make count with skip to work with CouchDB URL: https://github.com/apache/incubator-openwhisk/pull/3588#issuecomment-386361482 PG2 3081  This is an automated message from the Apache Git

[GitHub] mdeuser commented on issue #289: Unable to view Trigger parameters from the CLI

2018-05-03 Thread GitBox
mdeuser commented on issue #289: Unable to view Trigger parameters from the CLI URL: https://github.com/apache/incubator-openwhisk-cli/issues/289#issuecomment-386356047 @csantanapr - isn't the `trigger_payload` unique to the alarms trigger feed actions? i agree that these seem to be a

[GitHub] abaruni commented on issue #290: Unable to update Trigger parameters from CLI

2018-05-03 Thread GitBox
abaruni commented on issue #290: Unable to update Trigger parameters from CLI URL: https://github.com/apache/incubator-openwhisk-cli/issues/290#issuecomment-386345022 @eweiter could you provide a little bit of context? i.e. the feed you're working with, the command you're running, the

[GitHub] Logicon211 opened a new issue #891: Cannot use environment variables in action function parameters.

2018-05-03 Thread GitBox
Logicon211 opened a new issue #891: Cannot use environment variables in action function parameters. URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/891 It took a bit of time to figure out that I couldn't use environment variables in the function path of an action.

[GitHub] kpavel commented on issue #3525: Re-enable invokers docker_container module

2018-05-03 Thread GitBox
kpavel commented on issue #3525: Re-enable invokers docker_container module URL: https://github.com/apache/incubator-openwhisk/pull/3525#issuecomment-386345051 @chetanmeh I'm waiting for input from @dubee as i'm not sure what is blocking it from to be merged There new conflicts which i

[GitHub] csantanapr commented on issue #289: Unable to view Trigger parameters from the CLI

2018-05-03 Thread GitBox
csantanapr commented on issue #289: Unable to view Trigger parameters from the CLI URL: https://github.com/apache/incubator-openwhisk-cli/issues/289#issuecomment-386345009 Once change that has being discussed is to store the `trigger_payload` as individual parameters in the trigger

[GitHub] csantanapr commented on issue #289: Unable to view Trigger parameters from the CLI

2018-05-03 Thread GitBox
csantanapr commented on issue #289: Unable to view Trigger parameters from the CLI URL: https://github.com/apache/incubator-openwhisk-cli/issues/289#issuecomment-386344308 Is this a custom/standard trigger or a trigger feed? For trigger feed the parameters for the functions are

[GitHub] csantanapr commented on issue #289: Unable to view Trigger parameters from the CLI

2018-05-03 Thread GitBox
csantanapr commented on issue #289: Unable to view Trigger parameters from the CLI URL: https://github.com/apache/incubator-openwhisk-cli/issues/289#issuecomment-386344308 Is this a custom/standard trigger or a trigger feed? For trigger feed the parameters for the functions are

[GitHub] dubee commented on issue #290: Unable to update Trigger parameters from CLI

2018-05-03 Thread GitBox
dubee commented on issue #290: Unable to update Trigger parameters from CLI URL: https://github.com/apache/incubator-openwhisk-cli/issues/290#issuecomment-386341825 @mdeuser, @csantanapr, @drcariel, @abaruni This is an

[GitHub] dubee commented on issue #289: Unable to view Trigger parameters from the CLI

2018-05-03 Thread GitBox
dubee commented on issue #289: Unable to view Trigger parameters from the CLI URL: https://github.com/apache/incubator-openwhisk-cli/issues/289#issuecomment-386341780 @mdeuser, @csantanapr, @drcariel, @abaruni This is an

[GitHub] chetanmeh commented on issue #3525: Re-enable invokers docker_container module

2018-05-03 Thread GitBox
chetanmeh commented on issue #3525: Re-enable invokers docker_container module URL: https://github.com/apache/incubator-openwhisk/pull/3525#issuecomment-386339135 @kpavel Just checking if you are still working on this PR?

[GitHub] jasonpet commented on issue #884: passing parameters between manifest and deploy.yml

2018-05-03 Thread GitBox
jasonpet commented on issue #884: passing parameters between manifest and deploy.yml URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/884#issuecomment-386338048 the action should also show the trigger_payload. It will be under a param called `payload` though just as

[GitHub] kpavel opened a new pull request #890: Resolves issue #880

2018-05-03 Thread GitBox
kpavel opened a new pull request #890: Resolves issue #880 URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/890 - fix for rules actions when action from different package, e.g. dependency - fix for dependencies when dependency from different namespace note: if the

[GitHub] codecov-io commented on issue #3562: ArtifactStore implementation for CosmosDB

2018-05-03 Thread GitBox
codecov-io commented on issue #3562: ArtifactStore implementation for CosmosDB URL: https://github.com/apache/incubator-openwhisk/pull/3562#issuecomment-384942423 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3562?src=pr=h1) Report > Merging

[GitHub] eweiter opened a new issue #290: Unable to update Trigger parameters from CLI

2018-05-03 Thread GitBox
eweiter opened a new issue #290: Unable to update Trigger parameters from CLI URL: https://github.com/apache/incubator-openwhisk-cli/issues/290 When inspecting the `wsk trigger update ` command it was shown that you can specify a `-p` to update parameters. When I tried to do this

[GitHub] mrutkows commented on issue #102: Document LICENSE file Exclusions for incubator-openwhisk-runtime-java

2018-05-03 Thread GitBox
mrutkows commented on issue #102: Document LICENSE file Exclusions for incubator-openwhisk-runtime-java URL: https://github.com/apache/incubator-openwhisk-release/issues/102#issuecomment-386333018 @houshengbo Apparently, we have switched (or are in the process of switching) from openjdk

[GitHub] eweiter opened a new issue #289: Unable to view Trigger parameters from the CLI

2018-05-03 Thread GitBox
eweiter opened a new issue #289: Unable to view Trigger parameters from the CLI URL: https://github.com/apache/incubator-openwhisk-cli/issues/289 A trigger is capable of having parameters set on it. When a user does a `wsk trigger get ` they should receive data back that includes the

[GitHub] houshengbo opened a new issue #137: JDK license issue for the runtime Java

2018-05-03 Thread GitBox
houshengbo opened a new issue #137: JDK license issue for the runtime Java URL: https://github.com/apache/incubator-openwhisk-release/issues/137 Openwhisk is under apache license 2.0. We should be cautious about using JDK in runtime java repo or any other repo. Oracle java is licensed

[GitHub] vielmetti commented on issue #285: Multi-architecture build of API Gateway.

2018-05-03 Thread GitBox
vielmetti commented on issue #285: Multi-architecture build of API Gateway. URL: https://github.com/apache/incubator-openwhisk-apigateway/pull/285#issuecomment-386325471 > We intend to support arm64 eventually, but there is a deep bug in the OpenRESTy use of LuaJIT that needs to be

[GitHub] markusthoemmes closed pull request #3606: Increase nginx security by adapting ssl_ciphers.

2018-05-03 Thread GitBox
markusthoemmes closed pull request #3606: Increase nginx security by adapting ssl_ciphers. URL: https://github.com/apache/incubator-openwhisk/pull/3606 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] houshengbo closed pull request #49: WIP: Add the uninstall script for slack package

2018-05-03 Thread GitBox
houshengbo closed pull request #49: WIP: Add the uninstall script for slack package URL: https://github.com/apache/incubator-openwhisk-catalog/pull/49 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] houshengbo closed pull request #64: Add the support to package source code

2018-05-03 Thread GitBox
houshengbo closed pull request #64: Add the support to package source code URL: https://github.com/apache/incubator-openwhisk-client-go/pull/64 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] houshengbo closed pull request #46: Add retry support to every request

2018-05-03 Thread GitBox
houshengbo closed pull request #46: Add retry support to every request URL: https://github.com/apache/incubator-openwhisk-client-go/pull/46 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] houshengbo closed pull request #38: Add test cases regarding the action

2018-05-03 Thread GitBox
houshengbo closed pull request #38: Add test cases regarding the action URL: https://github.com/apache/incubator-openwhisk-cli/pull/38 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] houshengbo closed pull request #194: Update README regarding the environment variable OPENWHISK_HOME

2018-05-03 Thread GitBox
houshengbo closed pull request #194: Update README regarding the environment variable OPENWHISK_HOME URL: https://github.com/apache/incubator-openwhisk-cli/pull/194 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] houshengbo closed pull request #279: WIP: Check dependencies

2018-05-03 Thread GitBox
houshengbo closed pull request #279: WIP: Check dependencies URL: https://github.com/apache/incubator-openwhisk-cli/pull/279 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] houshengbo closed pull request #219: Add the support to package source code with Gradle

2018-05-03 Thread GitBox
houshengbo closed pull request #219: Add the support to package source code with Gradle URL: https://github.com/apache/incubator-openwhisk-cli/pull/219 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 commented on issue #880: wskdeploy export produces empty file

2018-05-03 Thread GitBox
pritidesai commented on issue #880: wskdeploy export produces empty file URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/880#issuecomment-386301297 @brunogirin I think export functionality is not supported for a project with multiple packages. @kpavel please confirm.

[GitHub] dgrove-oss opened a new pull request #189: add pointer to Helm deployment to top-level README.md

2018-05-03 Thread GitBox
dgrove-oss opened a new pull request #189: add pointer to Helm deployment to top-level README.md URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/189 This is an automated message from the Apache Git

[GitHub] codecov-io commented on issue #3606: Increase nginx security by adapting ssl_ciphers.

2018-05-03 Thread GitBox
codecov-io commented on issue #3606: Increase nginx security by adapting ssl_ciphers. URL: https://github.com/apache/incubator-openwhisk/pull/3606#issuecomment-386226233 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3606?src=pr=h1) Report > :exclamation: No

[GitHub] cbickel closed pull request #3592: Make limit overcommit relative to the actual cluster size.

2018-05-03 Thread GitBox
cbickel closed pull request #3592: Make limit overcommit relative to the actual cluster size. URL: https://github.com/apache/incubator-openwhisk/pull/3592 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] markusthoemmes commented on issue #3592: Make limit overcommit relative to the actual cluster size.

2018-05-03 Thread GitBox
markusthoemmes commented on issue #3592: Make limit overcommit relative to the actual cluster size. URL: https://github.com/apache/incubator-openwhisk/pull/3592#issuecomment-386260365 PG2 3086  This is an automated

[GitHub] markusthoemmes closed pull request #3199: One tid for the whole system.

2018-05-03 Thread GitBox
markusthoemmes closed pull request #3199: One tid for the whole system. URL: https://github.com/apache/incubator-openwhisk/pull/3199 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] codecov-io commented on issue #3199: One tid for the whole system.

2018-05-03 Thread GitBox
codecov-io commented on issue #3199: One tid for the whole system. URL: https://github.com/apache/incubator-openwhisk/pull/3199#issuecomment-386216631 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3199?src=pr=h1) Report > :exclamation: No coverage uploaded for pull

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

2018-05-03 Thread GitBox
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 > :exclamation: No coverage

[GitHub] codecov-io commented on issue #3606: Increase nginx security by adapting ssl_ciphers.

2018-05-03 Thread GitBox
codecov-io commented on issue #3606: Increase nginx security by adapting ssl_ciphers. URL: https://github.com/apache/incubator-openwhisk/pull/3606#issuecomment-386226233 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3606?src=pr=h1) Report > :exclamation: No

[GitHub] codecov-io commented on issue #3606: Increase nginx security by adapting ssl_ciphers.

2018-05-03 Thread GitBox
codecov-io commented on issue #3606: Increase nginx security by adapting ssl_ciphers. URL: https://github.com/apache/incubator-openwhisk/pull/3606#issuecomment-386226233 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3606?src=pr=h1) Report > :exclamation: No

  1   2   >