[GitHub] pritidesai opened a new pull request #910: Fix for 909

2018-05-17 Thread GitBox
pritidesai opened a new pull request #910: Fix for 909 URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/910 Closes #909 Fixing License in all .yaml and action source files under `tests/` This is an

[GitHub] mrutkows opened a new pull request #24: Fix travis yaml file asf header, change to mini.

2018-05-17 Thread GitBox
mrutkows opened a new pull request #24: Fix travis yaml file asf header, change to mini. URL: https://github.com/apache/incubator-openwhisk-runtime-python/pull/24 This is an automated message from the Apache Git Service. To

[GitHub] mrutkows opened a new pull request #61: Fix travis yaml file asf header, change to mini.

2018-05-17 Thread GitBox
mrutkows opened a new pull request #61: Fix travis yaml file asf header, change to mini. URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/61 This is an automated message from the Apache Git Service. To

[GitHub] mrutkows opened a new pull request #22: Fix travis yaml file asf header, change to mini.

2018-05-17 Thread GitBox
mrutkows opened a new pull request #22: Fix travis yaml file asf header, change to mini. URL: https://github.com/apache/incubator-openwhisk-runtime-php/pull/22 This is an automated message from the Apache Git Service. To

[GitHub] mrutkows opened a new pull request #53: Fix travis yaml file asf header, change to mini.

2018-05-17 Thread GitBox
mrutkows opened a new pull request #53: Fix travis yaml file asf header, change to mini. URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/53 This is an automated message from the Apache Git Service. To

[GitHub] mrutkows opened a new pull request #38: Fix travis yaml file asf header, change to mini.

2018-05-17 Thread GitBox
mrutkows opened a new pull request #38: Fix travis yaml file asf header, change to mini. URL: https://github.com/apache/incubator-openwhisk-runtime-docker/pull/38 This is an automated message from the Apache Git Service. To

[GitHub] mrutkows closed issue #76: Review NOTICE file for incubator-openwhisk-cli

2018-05-17 Thread GitBox
mrutkows closed issue #76: Review NOTICE file for incubator-openwhisk-cli URL: https://github.com/apache/incubator-openwhisk-release/issues/76 This is an automated message from the Apache Git Service. To respond to the

[GitHub] mrutkows commented on issue #76: Review NOTICE file for incubator-openwhisk-cli

2018-05-17 Thread GitBox
mrutkows commented on issue #76: Review NOTICE file for incubator-openwhisk-cli URL: https://github.com/apache/incubator-openwhisk-release/issues/76#issuecomment-390027401 Closing in favor of tracking with LICENSE issues here:

[GitHub] mrutkows commented on issue #84: Review NOTICE file for incubator-openwhisk-runtime-python

2018-05-17 Thread GitBox
mrutkows commented on issue #84: Review NOTICE file for incubator-openwhisk-runtime-python URL: https://github.com/apache/incubator-openwhisk-release/issues/84#issuecomment-390024931 Closing in favor of tracking along with LICENSE in issue here:

[GitHub] mrutkows closed issue #84: Review NOTICE file for incubator-openwhisk-runtime-python

2018-05-17 Thread GitBox
mrutkows closed issue #84: Review NOTICE file for incubator-openwhisk-runtime-python URL: https://github.com/apache/incubator-openwhisk-release/issues/84 This is an automated message from the Apache Git Service. To respond

[GitHub] mrutkows closed issue #25: Some source files miss Apache license headers

2018-05-17 Thread GitBox
mrutkows closed issue #25: Some source files miss Apache license headers URL: https://github.com/apache/incubator-openwhisk-runtime-java/issues/25 This is an automated message from the Apache Git Service. To respond to the

[GitHub] mrutkows closed issue #81: Review NOTICE file for incubator-openwhisk-runtime-java

2018-05-17 Thread GitBox
mrutkows closed issue #81: Review NOTICE file for incubator-openwhisk-runtime-java URL: https://github.com/apache/incubator-openwhisk-release/issues/81 This is an automated message from the Apache Git Service. To respond to

[GitHub] mrutkows commented on issue #81: Review NOTICE file for incubator-openwhisk-runtime-java

2018-05-17 Thread GitBox
mrutkows commented on issue #81: Review NOTICE file for incubator-openwhisk-runtime-java URL: https://github.com/apache/incubator-openwhisk-release/issues/81#issuecomment-390023591 Closing in favor of tracking issues with the LICENSE file

[GitHub] mrutkows closed issue #80: Review NOTICE file for incubator-openwhisk-runtime-docker

2018-05-17 Thread GitBox
mrutkows closed issue #80: Review NOTICE file for incubator-openwhisk-runtime-docker URL: https://github.com/apache/incubator-openwhisk-release/issues/80 This is an automated message from the Apache Git Service. To respond

[GitHub] mrutkows commented on issue #80: Review NOTICE file for incubator-openwhisk-runtime-docker

2018-05-17 Thread GitBox
mrutkows commented on issue #80: Review NOTICE file for incubator-openwhisk-runtime-docker URL: https://github.com/apache/incubator-openwhisk-release/issues/80#issuecomment-390022844 Closing in favor of tracking with LICENSE issues here:

[GitHub] mrutkows closed issue #85: Review NOTICE file for incubator-openwhisk-runtime-swift

2018-05-17 Thread GitBox
mrutkows closed issue #85: Review NOTICE file for incubator-openwhisk-runtime-swift URL: https://github.com/apache/incubator-openwhisk-release/issues/85 This is an automated message from the Apache Git Service. To respond

[GitHub] mrutkows commented on issue #85: Review NOTICE file for incubator-openwhisk-runtime-swift

2018-05-17 Thread GitBox
mrutkows commented on issue #85: Review NOTICE file for incubator-openwhisk-runtime-swift URL: https://github.com/apache/incubator-openwhisk-release/issues/85#issuecomment-390022290 Closing in favor of tracking with LICENSE issues here:

[GitHub] mrutkows commented on issue #83: Review NOTICE file for incubator-openwhisk-runtime-php

2018-05-17 Thread GitBox
mrutkows commented on issue #83: Review NOTICE file for incubator-openwhisk-runtime-php URL: https://github.com/apache/incubator-openwhisk-release/issues/83#issuecomment-390021149 Close in favor of tracking with LICENSE file here:

[GitHub] mrutkows closed issue #83: Review NOTICE file for incubator-openwhisk-runtime-php

2018-05-17 Thread GitBox
mrutkows closed issue #83: Review NOTICE file for incubator-openwhisk-runtime-php URL: https://github.com/apache/incubator-openwhisk-release/issues/83 This is an automated message from the Apache Git Service. To respond to

[GitHub] mrutkows commented on issue #82: Review NOTICE file for incubator-openwhisk-runtime-nodejs

2018-05-17 Thread GitBox
mrutkows commented on issue #82: Review NOTICE file for incubator-openwhisk-runtime-nodejs URL: https://github.com/apache/incubator-openwhisk-release/issues/82#issuecomment-390018933 Close in favor of tracking with LICENSE issues here:

[GitHub] mrutkows closed issue #82: Review NOTICE file for incubator-openwhisk-runtime-nodejs

2018-05-17 Thread GitBox
mrutkows closed issue #82: Review NOTICE file for incubator-openwhisk-runtime-nodejs URL: https://github.com/apache/incubator-openwhisk-release/issues/82 This is an automated message from the Apache Git Service. To respond

[GitHub] mrutkows commented on issue #48: Update doc to remove experimental references

2018-05-17 Thread GitBox
mrutkows commented on issue #48: Update doc to remove experimental references URL: https://github.com/apache/incubator-openwhisk-client-go/issues/48#issuecomment-390017789 PR: https://github.com/apache/incubator-openwhisk-client-go/pull/86 fixed issue Issue:

[GitHub] mrutkows commented on issue #48: Update doc to remove experimental references

2018-05-17 Thread GitBox
mrutkows commented on issue #48: Update doc to remove experimental references URL: https://github.com/apache/incubator-openwhisk-client-go/issues/48#issuecomment-390017789 Fixed: https://github.com/apache/incubator-openwhisk-client-go/issues/82 PR:

[GitHub] mrutkows closed issue #48: Update doc to remove experimental references

2018-05-17 Thread GitBox
mrutkows closed issue #48: Update doc to remove experimental references URL: https://github.com/apache/incubator-openwhisk-client-go/issues/48 This is an automated message from the Apache Git Service. To respond to the

[GitHub] mrutkows commented on issue #77: Review NOTICE file for incubator-openwhisk-client-go

2018-05-17 Thread GitBox
mrutkows commented on issue #77: Review NOTICE file for incubator-openwhisk-client-go URL: https://github.com/apache/incubator-openwhisk-release/issues/77#issuecomment-390016576 Closing issue to instead track with LICENSE here:

[GitHub] mrutkows closed issue #77: Review NOTICE file for incubator-openwhisk-client-go

2018-05-17 Thread GitBox
mrutkows closed issue #77: Review NOTICE file for incubator-openwhisk-client-go URL: https://github.com/apache/incubator-openwhisk-release/issues/77 This is an automated message from the Apache Git Service. To respond to the

[GitHub] mrutkows commented on issue #78: Review NOTICE file for incubator-openwhisk-apigateway

2018-05-17 Thread GitBox
mrutkows commented on issue #78: Review NOTICE file for incubator-openwhisk-apigateway URL: https://github.com/apache/incubator-openwhisk-release/issues/78#issuecomment-390015350 Closing as this granularity of tracking is not needed, see

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

2018-05-17 Thread GitBox
mrutkows commented on issue #273: A list of source files missing Apache license headers URL: https://github.com/apache/incubator-openwhisk-apigateway/issues/273#issuecomment-390012725 Closing this issue and will open more granular issues for any remaining files that need to have ASF

[GitHub] mrutkows closed issue #273: A list of source files missing Apache license headers

2018-05-17 Thread GitBox
mrutkows closed issue #273: A list of source files missing Apache license headers URL: https://github.com/apache/incubator-openwhisk-apigateway/issues/273 This is an automated message from the Apache Git Service. To respond

[GitHub] mrutkows closed issue #87: Review all (12) NOTICE files for contents

2018-05-17 Thread GitBox
mrutkows closed issue #87: Review all (12) NOTICE files for contents URL: https://github.com/apache/incubator-openwhisk-release/issues/87 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] mrutkows opened a new issue #87: Review all (12) NOTICE files for contents

2018-05-17 Thread GitBox
mrutkows opened a new issue #87: Review all (12) NOTICE files for contents URL: https://github.com/apache/incubator-openwhisk-release/issues/87 Track the review of notice files in these individual issues: openwhisk: https://github.com/apache/incubator-openwhisk-release/issues/75

[GitHub] mrutkows closed issue #75: Review NOTICE file for main openwhisk repo.

2018-05-17 Thread GitBox
mrutkows closed issue #75: Review NOTICE file for main openwhisk repo. URL: https://github.com/apache/incubator-openwhisk-release/issues/75 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] mrutkows closed issue #157: Track LICENSE and NOTICE issues for `incubator-openwhisk-catalog`

2018-05-17 Thread GitBox
mrutkows closed issue #157: Track LICENSE and NOTICE issues for `incubator-openwhisk-catalog` URL: https://github.com/apache/incubator-openwhisk-release/issues/157 This is an automated message from the Apache Git Service.

[GitHub] mrutkows opened a new issue #96: Document LICENSE file Exclusions for incubator-openwhisk-catalog

2018-05-17 Thread GitBox
mrutkows opened a new issue #96: Document LICENSE file Exclusions for incubator-openwhisk-catalog URL: https://github.com/apache/incubator-openwhisk-release/issues/96 Please use this issue to document directories/files (by extension) or individual files we have decided will not require a

[GitHub] mrutkows closed pull request #86: Remove the disclaimer and the reference as experimental

2018-05-17 Thread GitBox
mrutkows closed pull request #86: Remove the disclaimer and the reference as experimental URL: https://github.com/apache/incubator-openwhisk-client-go/pull/86 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] mrutkows closed issue #82: Change README to remove Disclaimer and "experimental" descriptions

2018-05-17 Thread GitBox
mrutkows closed issue #82: Change README to remove Disclaimer and "experimental" descriptions URL: https://github.com/apache/incubator-openwhisk-client-go/issues/82 This is an automated message from the Apache Git Service.

[GitHub] mrutkows closed pull request #155: Add the tests directory back to the source code release

2018-05-17 Thread GitBox
mrutkows closed pull request #155: Add the tests directory back to the source code release URL: https://github.com/apache/incubator-openwhisk-release/pull/155 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] mhamann commented on issue #302: Add option to encrypt redis password

2018-05-17 Thread GitBox
mhamann commented on issue #302: Add option to encrypt redis password URL: https://github.com/apache/incubator-openwhisk-apigateway/pull/302#issuecomment-389996458 @alexsong93 can you update the readme to document how the redis password should be encrypted and what needs to be passed into

[GitHub] dgrove-oss opened a new pull request #201: apigateway working with helm+minikube (NodePort)

2018-05-17 Thread GitBox
dgrove-oss opened a new pull request #201: apigateway working with helm+minikube (NodePort) URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/201 1. rework nginx.conf for NodePort ingress to direct apigw traffic to the appropriate service. 2. Add also invoking

[GitHub] alexsong93 opened a new pull request #302: Add option to encrypt redis password

2018-05-17 Thread GitBox
alexsong93 opened a new pull request #302: Add option to encrypt redis password URL: https://github.com/apache/incubator-openwhisk-apigateway/pull/302 @mhamann This is an automated message from the Apache Git Service. To

[GitHub] mrutkows opened a new issue #157: Track LICENSE and NOTICE issues for incubator-openwhisk-catalog

2018-05-17 Thread GitBox
mrutkows opened a new issue #157: Track LICENSE and NOTICE issues for incubator-openwhisk-catalog URL: https://github.com/apache/incubator-openwhisk-release/issues/157 Please use this issue to document directories/files (by extension) or individual files we have decided will not require a

[GitHub] sciabarracom opened a new pull request #31: Ignore this PR

2018-05-17 Thread GitBox
sciabarracom opened a new pull request #31: Ignore this PR URL: https://github.com/apache/incubator-openwhisk-runtime-go/pull/31 Only to trigger a build and cause errors (if any). This is an automated message from the Apache

[GitHub] mrutkows closed issue #3660: Python (*.py) and YAML (.yml) files missing ASF license headers

2018-05-17 Thread GitBox
mrutkows closed issue #3660: Python (*.py) and YAML (.yml) files missing ASF license headers URL: https://github.com/apache/incubator-openwhisk/issues/3660 This is an automated message from the Apache Git Service. To

[GitHub] mrutkows commented on issue #3666: Use Minified ASF License header in Ansible YAML files.

2018-05-17 Thread GitBox
mrutkows commented on issue #3666: Use Minified ASF License header in Ansible YAML files. URL: https://github.com/apache/incubator-openwhisk/pull/3666#issuecomment-389956352 Closes #3660 This is an automated message from

[GitHub] mrutkows commented on issue #3660: Python (*.py) and YAML (.yml) files missing ASF license headers

2018-05-17 Thread GitBox
mrutkows commented on issue #3660: Python (*.py) and YAML (.yml) files missing ASF license headers URL: https://github.com/apache/incubator-openwhisk/issues/3660#issuecomment-38999 Closed by PR https://github.com/apache/incubator-openwhisk/pull/3666

[GitHub] houshengbo closed pull request #277: Add ASF license header to travis yaml file.

2018-05-17 Thread GitBox
houshengbo closed pull request #277: Add ASF license header to travis yaml file. URL: https://github.com/apache/incubator-openwhisk-catalog/pull/277 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance:

[GitHub] houshengbo closed pull request #301: Add ASF license header to travis yaml file.

2018-05-17 Thread GitBox
houshengbo closed pull request #301: Add ASF license header to travis yaml file. URL: https://github.com/apache/incubator-openwhisk-apigateway/pull/301 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] codecov-io commented on issue #3619: Provide an activation store SPI

2018-05-17 Thread GitBox
codecov-io commented on issue #3619: Provide an activation store SPI URL: https://github.com/apache/incubator-openwhisk/pull/3619#issuecomment-387224274 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3619?src=pr=h1) Report > Merging

[GitHub] pritidesai commented on issue #909: Fix LICENSE headers in all YAML/.js under "tests" folder

2018-05-17 Thread GitBox
pritidesai commented on issue #909: Fix LICENSE headers in all YAML/.js under "tests" folder URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/909#issuecomment-389947247 Assigning to me as I am working on it.

[GitHub] mrutkows opened a new pull request #277: Add ASF license header to travis yaml file.

2018-05-17 Thread GitBox
mrutkows opened a new pull request #277: Add ASF license header to travis yaml file. URL: https://github.com/apache/incubator-openwhisk-catalog/pull/277 This is an automated message from the Apache Git Service. To respond

[GitHub] houshengbo closed pull request #3670: Add ASF Headers to .groovy source files.

2018-05-17 Thread GitBox
houshengbo closed pull request #3670: Add ASF Headers to .groovy source files. URL: https://github.com/apache/incubator-openwhisk/pull/3670 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] codecov-io commented on issue #3670: Add ASF Headers to .groovy source files.

2018-05-17 Thread GitBox
codecov-io commented on issue #3670: Add ASF Headers to .groovy source files. URL: https://github.com/apache/incubator-openwhisk/pull/3670#issuecomment-389938394 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3670?src=pr=h1) Report > Merging

[GitHub] dubee closed pull request #3652: Disable Slf4jLogger startup log in test runs

2018-05-17 Thread GitBox
dubee closed pull request #3652: Disable Slf4jLogger startup log in test runs URL: https://github.com/apache/incubator-openwhisk/pull/3652 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] codecov-io commented on issue #3669: Make stemcells configurable by deployment

2018-05-17 Thread GitBox
codecov-io commented on issue #3669: Make stemcells configurable by deployment URL: https://github.com/apache/incubator-openwhisk/pull/3669#issuecomment-389936897 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3669?src=pr=h1) Report > Merging

[GitHub] codecov-io commented on issue #3669: Make stemcells configurable by deployment

2018-05-17 Thread GitBox
codecov-io commented on issue #3669: Make stemcells configurable by deployment URL: https://github.com/apache/incubator-openwhisk/pull/3669#issuecomment-389936897 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3669?src=pr=h1) Report > Merging

[GitHub] mrutkows opened a new pull request #301: Add ASF license header to travis yaml file.

2018-05-17 Thread GitBox
mrutkows opened a new pull request #301: Add ASF license header to travis yaml file. URL: https://github.com/apache/incubator-openwhisk-apigateway/pull/301 This is an automated message from the Apache Git Service. To

[GitHub] rabbah closed pull request #3232: Making prewarm kind (and count) configurable

2018-05-17 Thread GitBox
rabbah closed pull request #3232: Making prewarm kind (and count) configurable URL: https://github.com/apache/incubator-openwhisk/pull/3232 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] rabbah commented on issue #3232: Making prewarm kind (and count) configurable

2018-05-17 Thread GitBox
rabbah commented on issue #3232: Making prewarm kind (and count) configurable URL: https://github.com/apache/incubator-openwhisk/pull/3232#issuecomment-389926238 @Himavanth thanks for the thumbs up - will close in favor of new PR.

[GitHub] mrutkows opened a new pull request #3670: Add ASF Headers to .groovy source files.

2018-05-17 Thread GitBox
mrutkows opened a new pull request #3670: Add ASF Headers to .groovy source files. URL: https://github.com/apache/incubator-openwhisk/pull/3670 ## Description ## Related issue and scope - [ ] I opened an issue to propose and discuss this change (#)

[GitHub] mdeuser commented on issue #301: Display appropriate message when an activation ID is not returned from a trigger fire.

2018-05-17 Thread GitBox
mdeuser commented on issue #301: Display appropriate message when an activation ID is not returned from a trigger fire. URL: https://github.com/apache/incubator-openwhisk-cli/issues/301#issuecomment-389920959 and should probably pass along a message that indicates the trigger was not

[GitHub] csantanapr closed issue #156: Swift runtime has "SPM" builds (binaries) using ext. code with no license attribution

2018-05-17 Thread GitBox
csantanapr closed issue #156: Swift runtime has "SPM" builds (binaries) using ext. code with no license attribution URL: https://github.com/apache/incubator-openwhisk-release/issues/156 This is an automated message from the

[GitHub] csantanapr commented on issue #156: Swift runtime has "SPM" builds (binaries) using ext. code with no license attribution

2018-05-17 Thread GitBox
csantanapr commented on issue #156: Swift runtime has "SPM" builds (binaries) using ext. code with no license attribution URL: https://github.com/apache/incubator-openwhisk-release/issues/156#issuecomment-389918646 spm-build not related at all to spmjs spm stands for swift package

[GitHub] dubee opened a new issue #301: Display appropriate message when an activation ID is not returned from a trigger fire.

2018-05-17 Thread GitBox
dubee opened a new issue #301: Display appropriate message when an activation ID is not returned from a trigger fire. URL: https://github.com/apache/incubator-openwhisk-cli/issues/301 Executing a trigger returns an activation ID only when there is rule linked to the trigger. As a result,

[GitHub] dubee closed pull request #151: increase wait time for maxTriggers test

2018-05-17 Thread GitBox
dubee closed pull request #151: increase wait time for maxTriggers test URL: https://github.com/apache/incubator-openwhisk-package-alarms/pull/151 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance:

[GitHub] mrutkows opened a new issue #156: Swift runtime has "SPM" builds (binaries) using ext. code with no license attribution

2018-05-17 Thread GitBox
mrutkows opened a new issue #156: Swift runtime has "SPM" builds (binaries) using ext. code with no license attribution URL: https://github.com/apache/incubator-openwhisk-release/issues/156 Both Swift 3.1 and Swift 4 runtimes produce "spm-build" directories indicative of use of 3rd party

[GitHub] tysonnorris commented on issue #2795: enable concurrent activation processing

2018-05-17 Thread GitBox
tysonnorris commented on issue #2795: enable concurrent activation processing URL: https://github.com/apache/incubator-openwhisk/pull/2795#issuecomment-389904732 @markusthoemmes @rabbah @csantanapr this is ready for a PG and another review glance

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

2018-05-17 Thread GitBox
Himavanth commented on issue #3232: Making prewarm kind (and count) configurable URL: https://github.com/apache/incubator-openwhisk/pull/3232#issuecomment-389904344 Sounds good. Thanks @rabbah  This is an automated message

[GitHub] jasonpet opened a new pull request #151: increase wait time for maxTriggers test

2018-05-17 Thread GitBox
jasonpet opened a new pull request #151: increase wait time for maxTriggers test URL: https://github.com/apache/incubator-openwhisk-package-alarms/pull/151 This is an automated message from the Apache Git Service. To respond

[GitHub] houshengbo opened a new pull request #155: Add the tests directory back to the source code release

2018-05-17 Thread GitBox
houshengbo opened a new pull request #155: Add the tests directory back to the source code release URL: https://github.com/apache/incubator-openwhisk-release/pull/155 Since we have already added the valid license header to all the test cases, we restore to add them back to the source

[GitHub] rabbah commented on issue #3669: Make stemcells configurable by deployment

2018-05-17 Thread GitBox
rabbah commented on issue #3669: Make stemcells configurable by deployment URL: https://github.com/apache/incubator-openwhisk/pull/3669#issuecomment-389895111 Should get a fresh PG @csantanapr @dubee  This is an automated

[GitHub] rabbah commented on a change in pull request #3232: Making prewarm kind (and count) configurable

2018-05-17 Thread GitBox
rabbah commented on a change in pull request #3232: Making prewarm kind (and count) configurable URL: https://github.com/apache/incubator-openwhisk/pull/3232#discussion_r188986673 ## File path: core/invoker/src/main/scala/whisk/core/containerpool/ContainerPool.scala ##

[GitHub] rabbah commented on issue #3669: Make stemcells configurable by deployment

2018-05-17 Thread GitBox
rabbah commented on issue #3669: Make stemcells configurable by deployment URL: https://github.com/apache/incubator-openwhisk/pull/3669#issuecomment-389893508 Note to committer: please preserve co-author credits, I preserved much of @Himavanth patches from #3232 and improved on them

[GitHub] rabbah commented on issue #3669: Make stemcells configurable by deployment

2018-05-17 Thread GitBox
rabbah commented on issue #3669: Make stemcells configurable by deployment URL: https://github.com/apache/incubator-openwhisk/pull/3669#issuecomment-389893066 @dgrove-oss @tysonnorris I don't think this will break downstream in terms of a deployment but it will affect performance. A

[GitHub] rabbah commented on issue #3669: Make stemcells configurable by deployment

2018-05-17 Thread GitBox
rabbah commented on issue #3669: Make stemcells configurable by deployment URL: https://github.com/apache/incubator-openwhisk/pull/3669#issuecomment-389893066 @dgrove-oss @tysonnorris I don't think this will break downstream in terms of a deployment but it will affect performance. A

[GitHub] rabbah commented on issue #3669: Make stemcells configurable by deployment

2018-05-17 Thread GitBox
rabbah commented on issue #3669: Make stemcells configurable by deployment URL: https://github.com/apache/incubator-openwhisk/pull/3669#issuecomment-389893066 @dgrove-oss I don't think this will break downstream in terms of a deployment but it will affect performance. A heads up - when

[GitHub] rabbah commented on a change in pull request #3669: Make stemcells configurable by deployment

2018-05-17 Thread GitBox
rabbah commented on a change in pull request #3669: Make stemcells configurable by deployment URL: https://github.com/apache/incubator-openwhisk/pull/3669#discussion_r188985305 ## File path: core/invoker/src/main/scala/whisk/core/containerpool/ContainerPool.scala ## @@

[GitHub] rabbah opened a new pull request #3669: Stemcells

2018-05-17 Thread GitBox
rabbah opened a new pull request #3669: Stemcells URL: https://github.com/apache/incubator-openwhisk/pull/3669 ## Description This PR replaces #3232 and makes stem cell containers configurable from the runtime manifest. See the changes to runtimes.json for an example (or the new

[GitHub] dubee commented on a change in pull request #299: Wait for 2 minutes for logs.

2018-05-17 Thread GitBox
dubee commented on a change in pull request #299: Wait for 2 minutes for logs. URL: https://github.com/apache/incubator-openwhisk-cli/pull/299#discussion_r188983257 ## File path: tests/src/test/scala/whisk/core/cli/test/WskCliBasicUsageTests.scala ## @@ -400,7 +400,7 @@

[GitHub] dubee commented on a change in pull request #3619: Provide an activation store SPI

2018-05-17 Thread GitBox
dubee commented on a change in pull request #3619: Provide an activation store SPI URL: https://github.com/apache/incubator-openwhisk/pull/3619#discussion_r188982201 ## File path: common/scala/src/main/scala/whisk/core/entity/ActivationStore.scala ## @@ -0,0 +1,69 @@ +/*

[GitHub] dubee closed pull request #299: Wait for 2 minutes for logs.

2018-05-17 Thread GitBox
dubee closed pull request #299: Wait for 2 minutes for logs. URL: https://github.com/apache/incubator-openwhisk-cli/pull/299 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] rabbah commented on a change in pull request #299: Wait for 2 minutes for logs.

2018-05-17 Thread GitBox
rabbah commented on a change in pull request #299: Wait for 2 minutes for logs. URL: https://github.com/apache/incubator-openwhisk-cli/pull/299#discussion_r188979941 ## File path: tests/src/test/scala/whisk/core/cli/test/WskCliBasicUsageTests.scala ## @@ -400,7 +400,7 @@

[GitHub] dgrove-oss opened a new pull request #300: Bug fix: Use configured prototcol for API requests

2018-05-17 Thread GitBox
dgrove-oss opened a new pull request #300: Bug fix: Use configured prototcol for API requests URL: https://github.com/apache/incubator-openwhisk-cli/pull/300 Add check to see if Client.Config.Host already specifies the protocol to use before prepending https:// in URL construction.

[GitHub] cbickel opened a new pull request #299: Wait for 2 minutes for logs.

2018-05-17 Thread GitBox
cbickel opened a new pull request #299: Wait for 2 minutes for logs. URL: https://github.com/apache/incubator-openwhisk-cli/pull/299 Increase time to wait for logs, to handle potential delays, that could occure, depending on the store, that is used to store logs.

[GitHub] style95 commented on issue #3663: Support SPI for ContainerPool and ContainerProxy

2018-05-17 Thread GitBox
style95 commented on issue #3663: Support SPI for ContainerPool and ContainerProxy URL: https://github.com/apache/incubator-openwhisk/pull/3663#issuecomment-389860736 @csantanapr thank you for the good tip. This is an

[GitHub] style95 commented on issue #3663: Support SPI for ContainerPool and ContainerProxy

2018-05-17 Thread GitBox
style95 commented on issue #3663: Support SPI for ContainerPool and ContainerProxy URL: https://github.com/apache/incubator-openwhisk/pull/3663#issuecomment-389860736 @csantanpr thank you for the good tip. This is an

[GitHub] chetanmeh commented on issue #3502: Immutable attachments via putAndAttach

2018-05-17 Thread GitBox
chetanmeh commented on issue #3502: Immutable attachments via putAndAttach URL: https://github.com/apache/incubator-openwhisk/pull/3502#issuecomment-389851869 Rebased to current master. @rabbah and @dubee would be helpful if you can review the proposed changes now

[GitHub] codecov-io commented on issue #3502: Immutable attachments via putAndAttach

2018-05-17 Thread GitBox
codecov-io commented on issue #3502: Immutable attachments via putAndAttach URL: https://github.com/apache/incubator-openwhisk/pull/3502#issuecomment-389761107 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3502?src=pr=h1) Report > Merging

[GitHub] csantanapr closed pull request #3654: Issues/3653 - Ansible playbook openwhisk.yml fails to find whisk/cont…

2018-05-17 Thread GitBox
csantanapr closed pull request #3654: Issues/3653 - Ansible playbook openwhisk.yml fails to find whisk/cont… URL: https://github.com/apache/incubator-openwhisk/pull/3654 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for

[GitHub] csantanapr commented on issue #10: Can't run tests in OSX

2018-05-17 Thread GitBox
csantanapr commented on issue #10: Can't run tests in OSX URL: https://github.com/apache/incubator-openwhisk-runtime-go/issues/10#issuecomment-389839011 @sciabarra I went ahead and enabled Travis to run on master branch daily to see how often it fails and which tests fail This way we

[GitHub] csantanapr commented on issue #3502: Immutable attachments via putAndAttach

2018-05-17 Thread GitBox
csantanapr commented on issue #3502: Immutable attachments via putAndAttach URL: https://github.com/apache/incubator-openwhisk/pull/3502#issuecomment-389835826 What’s the latest status on this PR? @dubee Does this affect or related to your ArtifactStore work?

[GitHub] csantanapr commented on issue #3663: Support SPI for ContainerPool and ContainerProxy

2018-05-17 Thread GitBox
csantanapr commented on issue #3663: Support SPI for ContainerPool and ContainerProxy URL: https://github.com/apache/incubator-openwhisk/pull/3663#issuecomment-389834596 @style95 Closing the PR and then re-opening the PR will trigger the build again in Travis

[GitHub] csantanapr commented on issue #10: Can't run tests in OSX

2018-05-17 Thread GitBox
csantanapr commented on issue #10: Can't run tests in OSX URL: https://github.com/apache/incubator-openwhisk-runtime-go/issues/10#issuecomment-389832783 Tests sometimes fail on Travis also And also fail in OSX also Maybe there is a race condition on the tests with concurrent

[GitHub] junoyoon commented on issue #3570: ArtifactStore implementation for MongoDB

2018-05-17 Thread GitBox
junoyoon commented on issue #3570: ArtifactStore implementation for MongoDB URL: https://github.com/apache/incubator-openwhisk/pull/3570#issuecomment-389820765 rebase please. This is an automated message from the Apache Git

[GitHub] codecov-io commented on issue #3502: Immutable attachments via putAndAttach

2018-05-17 Thread GitBox
codecov-io commented on issue #3502: Immutable attachments via putAndAttach URL: https://github.com/apache/incubator-openwhisk/pull/3502#issuecomment-389761107 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3502?src=pr=h1) Report > Merging

[GitHub] chetanmeh commented on a change in pull request #3619: Provide an activation store SPI

2018-05-17 Thread GitBox
chetanmeh commented on a change in pull request #3619: Provide an activation store SPI URL: https://github.com/apache/incubator-openwhisk/pull/3619#discussion_r188849747 ## File path: common/scala/src/main/scala/whisk/core/entity/ActivationStore.scala ## @@ -0,0 +1,69 @@

[GitHub] codecov-io commented on issue #3663: Support SPI for ContainerPool and ContainerProxy

2018-05-17 Thread GitBox
codecov-io commented on issue #3663: Support SPI for ContainerPool and ContainerProxy URL: https://github.com/apache/incubator-openwhisk/pull/3663#issuecomment-389757306 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3663?src=pr=h1) Report > Merging

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

2018-05-17 Thread GitBox
rabbah commented on issue #3232: Making prewarm kind (and count) configurable URL: https://github.com/apache/incubator-openwhisk/pull/3232#issuecomment-389755413 @Himavanth take a look at this branch:

[GitHub] sciabarra commented on issue #10: Can't run tests in OSX

2018-05-17 Thread GitBox
sciabarra commented on issue #10: Can't run tests in OSX URL: https://github.com/apache/incubator-openwhisk-runtime-go/issues/10#issuecomment-389755164 Tests now runs for me in all the case after I added the "fixture" (TestMain) to compile only once.