[GitHub] csantanapr opened a new pull request #3404: bump dockerskeleton to pick up curl and wget 1.2.0

2018-03-06 Thread GitBox
csantanapr opened a new pull request #3404: bump dockerskeleton to pick up curl and wget 1.2.0 URL: https://github.com/apache/incubator-openwhisk/pull/3404 ## Description Updating dockskeleton image to pick up curl and wget. ## My changes affect the following components - [x]

[GitHub] rabbah commented on a change in pull request #3366: Adds method deleteAttachments to ArtifactStore

2018-03-06 Thread GitBox
rabbah commented on a change in pull request #3366: Adds method deleteAttachments to ArtifactStore URL: https://github.com/apache/incubator-openwhisk/pull/3366#discussion_r172757550 ## File path: common/scala/src/main/scala/whisk/core/entity/WhiskAction.scala ## @@

[GitHub] csantanapr closed pull request #21: add curl and wget

2018-03-06 Thread GitBox
csantanapr closed pull request #21: add curl and wget URL: https://github.com/apache/incubator-openwhisk-runtime-docker/pull/21 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] chetanmeh commented on issue #3369: Use pureconfig for CouchDbRestStore

2018-03-06 Thread GitBox
chetanmeh commented on issue #3369: Use pureconfig for CouchDbRestStore URL: https://github.com/apache/incubator-openwhisk/pull/3369#issuecomment-371045007 Thanks @markusthoemmes . Let me know if it needs any further change

[GitHub] chetanmeh commented on issue #3356: Add method to delete attachment to ArtifactStore

2018-03-06 Thread GitBox
chetanmeh commented on issue #3356: Add method to delete attachment to ArtifactStore URL: https://github.com/apache/incubator-openwhisk/issues/3356#issuecomment-371044769 Closed via #3366 This is an automated message from

[GitHub] chetanmeh closed issue #3356: Add method to delete attachment to ArtifactStore

2018-03-06 Thread GitBox
chetanmeh closed issue #3356: Add method to delete attachment to ArtifactStore URL: https://github.com/apache/incubator-openwhisk/issues/3356 This is an automated message from the Apache Git Service. To respond to the

[GitHub] chetanmeh commented on a change in pull request #3366: Adds method deleteAttachments to ArtifactStore

2018-03-06 Thread GitBox
chetanmeh commented on a change in pull request #3366: Adds method deleteAttachments to ArtifactStore URL: https://github.com/apache/incubator-openwhisk/pull/3366#discussion_r172755939 ## File path: common/scala/src/main/scala/whisk/core/entity/WhiskAction.scala ## @@

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

2018-03-06 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-371022697 @daisy-ycguo You need to run first package_source-code.sh to have OPENWHISK_CLEANED_SOURCE_DIR

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

2018-03-06 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-371022697 @daisy-ycguo You need to run first package_source-code.sh to have OPENWHISK_CLEANED_SOURCE_DIR

[GitHub] paulcastro closed pull request #23: Add Codable Support for Swift 4.x

2018-03-06 Thread GitBox
paulcastro closed pull request #23: Add Codable Support for Swift 4.x URL: https://github.com/apache/incubator-openwhisk-runtime-swift/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

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

2018-03-06 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-371011153 From @dubeejw >Can we open an issue at least to update the doc with examples of using the web action auth?

[GitHub] csantanapr commented on a change in pull request #23: Add Codable Support for Swift 4.x

2018-03-06 Thread GitBox
csantanapr commented on a change in pull request #23: Add Codable Support for Swift 4.x URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/23#discussion_r172730781 ## File path: core/swift40Action/epilogue.swift ## @@ -21,24 +21,100 @@ import

[GitHub] mrutkows closed pull request #52: Add license verification document

2018-03-06 Thread GitBox
mrutkows closed pull request #52: Add license verification document URL: https://github.com/apache/incubator-openwhisk-release/pull/52 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] paulcastro commented on a change in pull request #23: Add Codable Support for Swift 4.x

2018-03-06 Thread GitBox
paulcastro commented on a change in pull request #23: Add Codable Support for Swift 4.x URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/23#discussion_r172728876 ## File path: core/swift40Action/epilogue.swift ## @@ -21,24 +21,100 @@ import

[GitHub] paulcastro commented on a change in pull request #23: Add Codable Support for Swift 4.x

2018-03-06 Thread GitBox
paulcastro commented on a change in pull request #23: Add Codable Support for Swift 4.x URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/23#discussion_r172728354 ## File path: README.md ## @@ -39,6 +39,76 @@ func main(args: [String:Any]) ->

[GitHub] paulcastro commented on a change in pull request #23: Add Codable Support for Swift 4.x

2018-03-06 Thread GitBox
paulcastro commented on a change in pull request #23: Add Codable Support for Swift 4.x URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/23#discussion_r172728354 ## File path: README.md ## @@ -39,6 +39,76 @@ func main(args: [String:Any]) ->

[GitHub] csantanapr commented on issue #3345: ansible 2.4 compatibility

2018-03-06 Thread GitBox
csantanapr commented on issue #3345: ansible 2.4 compatibility URL: https://github.com/apache/incubator-openwhisk/issues/3345#issuecomment-371000840 Yes moving to latest ansible sounds like a good idea, like always if you have some internal pipeline inside your company coordinate with

[GitHub] daisy-ycguo opened a new issue #25: Some source files miss Apache license headers

2018-03-06 Thread GitBox
daisy-ycguo opened a new issue #25: Some source files miss Apache license headers URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/issues/25 Following [Apache license header guideline](http://www.apache.org/legal/src-headers.html), **all human-readable Apache-developed

[GitHub] daisy-ycguo opened a new issue #31: Some source files miss Apache license headers

2018-03-06 Thread GitBox
daisy-ycguo opened a new issue #31: Some source files miss Apache license headers URL: https://github.com/apache/incubator-openwhisk-runtime-swift/issues/31 Following [Apache license header guideline](http://www.apache.org/legal/src-headers.html), **all human-readable Apache-developed

[GitHub] daisy-ycguo opened a new issue #25: Some source files miss Apache license headers

2018-03-06 Thread GitBox
daisy-ycguo opened a new issue #25: Some source files miss Apache license headers URL: https://github.com/apache/incubator-openwhisk-runtime-java/issues/25 Following [Apache license header guideline](http://www.apache.org/legal/src-headers.html), **all human-readable Apache-developed

[GitHub] daisy-ycguo opened a new issue #20: Some source files miss Apache license headers

2018-03-06 Thread GitBox
daisy-ycguo opened a new issue #20: Some source files miss Apache license headers URL: https://github.com/apache/incubator-openwhisk-runtime-docker/issues/20 Following [Apache license header guideline](http://www.apache.org/legal/src-headers.html), **all human-readable Apache-developed

[GitHub] daisy-ycguo opened a new issue #10: Some source files miss Apache license headers

2018-03-06 Thread GitBox
daisy-ycguo opened a new issue #10: Some source files miss Apache license headers URL: https://github.com/apache/incubator-openwhisk-runtime-python/issues/10 Following [Apache license header guideline](http://www.apache.org/legal/src-headers.html), **all human-readable Apache-developed

[GitHub] daisy-ycguo opened a new issue #8: Some source files miss Apache license headers

2018-03-06 Thread GitBox
daisy-ycguo opened a new issue #8: Some source files miss Apache license headers URL: https://github.com/apache/incubator-openwhisk-runtime-php/issues/8 Following [Apache license header guideline](http://www.apache.org/legal/src-headers.html), **all human-readable Apache-developed files

[GitHub] daisy-ycguo opened a new issue #68: Some source files miss Apache license headers

2018-03-06 Thread GitBox
daisy-ycguo opened a new issue #68: Some source files miss Apache license headers URL: https://github.com/apache/incubator-openwhisk-client-go/issues/68 Following [Apache license header guideline](http://www.apache.org/legal/src-headers.html), **all human-readable Apache-developed files

[GitHub] pritidesai opened a new issue #773: How to address go get failures due to breaking changes in Go Client?

2018-03-06 Thread GitBox
pritidesai opened a new issue #773: How to address go get failures due to breaking changes in Go Client? URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/773 There was a change in the function signature of Get action in apache/incubator-openwhisk-client-go@72bf712

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

2018-03-06 Thread GitBox
daisy-ycguo commented on issue #273: A list of source files missing Apache license headers URL: https://github.com/apache/incubator-openwhisk-apigateway/issues/273#issuecomment-370991462 @mhamann Apache Rat could do that, yet still need a human verification.

[GitHub] daisy-ycguo commented on issue #716: Some source files are missing Apache license headers

2018-03-06 Thread GitBox
daisy-ycguo commented on issue #716: Some source files are missing Apache license headers URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/716#issuecomment-370990878 More files: ``` openwhisk-wskdeploy/tests/src/integration/apigateway/src/list-members.js

[GitHub] pritidesai opened a new pull request #772: upgrading go client

2018-03-06 Thread GitBox
pritidesai opened a new pull request #772: upgrading go client URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/772 This is an automated message from the Apache Git Service. To respond to the message, please

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

2018-03-06 Thread GitBox
rabbah commented on issue #3402: Create '_users' database for singleton ephemeral couchdb v 2+ URL: https://github.com/apache/incubator-openwhisk/pull/3402#issuecomment-370969346 @bbrowning didn't you do this at some point?

[GitHub] jonpspri commented on issue #3345: ansible 2.4 compatibility

2018-03-06 Thread GitBox
jonpspri commented on issue #3345: ansible 2.4 compatibility URL: https://github.com/apache/incubator-openwhisk/issues/3345#issuecomment-370939183 So we need a timeframe for when we are going to support ansible 2.4+ only. It seems like now is as good a time as any, which would then let

[GitHub] jonpspri opened a new pull request #3402: Create '_users' database for singleton ephemeral couchdb v 2+

2018-03-06 Thread GitBox
jonpspri opened a new pull request #3402: Create '_users' database for singleton ephemeral couchdb v 2+ URL: https://github.com/apache/incubator-openwhisk/pull/3402 ## Description Add a step to the 'ansible/roles/couchdb/deploy.yml' playbook that will create a

[GitHub] csantanapr commented on issue #3138: Add bash kind action

2018-03-06 Thread GitBox
csantanapr commented on issue #3138: Add bash kind action URL: https://github.com/apache/incubator-openwhisk/pull/3138#issuecomment-366863338 Recent changes to dockerSkeleton runtime now allows for better support for bash actions. Using `--native` with a simple script is the recommend

[GitHub] jonpspri commented on issue #3363: Ansible image name overrides and bump NGinx to 1.13.9

2018-03-06 Thread GitBox
jonpspri commented on issue #3363: Ansible image name overrides and bump NGinx to 1.13.9 URL: https://github.com/apache/incubator-openwhisk/pull/3363#issuecomment-370918926 Just rebased in a hope that #3393 fixed the Kafka heisenbug.

[GitHub] pritidesai closed pull request #765: Update deploymentreader.go tests and fix bind logic.

2018-03-06 Thread GitBox
pritidesai closed pull request #765: Update deploymentreader.go tests and fix bind logic. URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/765 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 a change in pull request #765: Update deploymentreader.go tests and fix bind logic.

2018-03-06 Thread GitBox
pritidesai commented on a change in pull request #765: Update deploymentreader.go tests and fix bind logic. URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/765#discussion_r172615337 ## File path: deployers/deploymentreader.go ## @@ -68,48 +66,62 @@ func

[GitHub] pritidesai commented on a change in pull request #765: Update deploymentreader.go tests and fix bind logic.

2018-03-06 Thread GitBox
pritidesai commented on a change in pull request #765: Update deploymentreader.go tests and fix bind logic. URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/765#discussion_r172614569 ## File path: cmd/report.go ## @@ -62,6 +62,8 @@ func init() {

[GitHub] pritidesai commented on a change in pull request #765: Update deploymentreader.go tests and fix bind logic.

2018-03-06 Thread GitBox
pritidesai commented on a change in pull request #765: Update deploymentreader.go tests and fix bind logic. URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/765#discussion_r172614096 ## File path: .travis.yml ## @@ -30,7 +30,7 @@ install: - go get -u

[GitHub] mrutkows opened a new issue #771: TestInterpolateStringWithEnvVar has printlns(); need to be converted to Trace output

2018-03-06 Thread GitBox
mrutkows opened a new issue #771: TestInterpolateStringWithEnvVar has printlns(); need to be converted to Trace output URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/771 TestInterpolateStringWithEnvVar always produces this output: `` NoDollar oh, dollars!

[GitHub] mdeuser opened a new issue #3401: Consolidate action annotation constants

2018-03-06 Thread GitBox
mdeuser opened a new issue #3401: Consolidate action annotation constants URL: https://github.com/apache/incubator-openwhisk/issues/3401 Currently, the defined action annotation names are not centrally defined. Consolidate them as constants into a single location and change the existing

[GitHub] mdeuser commented on a change in pull request #3388: Update require-whisk-auth behavior to secure web action

2018-03-06 Thread GitBox
mdeuser commented on a change in pull request #3388: Update require-whisk-auth behavior to secure web action URL: https://github.com/apache/incubator-openwhisk/pull/3388#discussion_r172628041 ## File path: core/controller/src/main/scala/whisk/core/controller/WebActions.scala

[GitHub] markusthoemmes commented on a change in pull request #3391: Add namespace-blacklist to invoker.

2018-03-06 Thread GitBox
markusthoemmes commented on a change in pull request #3391: Add namespace-blacklist to invoker. URL: https://github.com/apache/incubator-openwhisk/pull/3391#discussion_r172624728 ## File path: ansible/files/namespace_throttlings_design_document_for_subjects_db.json ##

[GitHub] rabbah commented on a change in pull request #3391: Add namespace-blacklist to invoker.

2018-03-06 Thread GitBox
rabbah commented on a change in pull request #3391: Add namespace-blacklist to invoker. URL: https://github.com/apache/incubator-openwhisk/pull/3391#discussion_r172623147 ## File path: ansible/files/namespace_throttlings_design_document_for_subjects_db.json ## @@ -0,0

[GitHub] rabbah commented on issue #3391: Add namespace-blacklist to invoker.

2018-03-06 Thread GitBox
rabbah commented on issue #3391: Add namespace-blacklist to invoker. URL: https://github.com/apache/incubator-openwhisk/pull/3391#issuecomment-370888539 Thanks @chetanmeh, used the gradle task you added, here is the view function fwiw: ```javascript function (doc) { if

[GitHub] rabbah closed issue #3315: view functions extracted to source files should be excluded from scancode

2018-03-06 Thread GitBox
rabbah closed issue #3315: view functions extracted to source files should be excluded from scancode URL: https://github.com/apache/incubator-openwhisk/issues/3315 This is an automated message from the Apache Git Service.

[GitHub] rabbah closed pull request #3366: Adds method deleteAttachments to ArtifactStore

2018-03-06 Thread GitBox
rabbah closed pull request #3366: Adds method deleteAttachments to ArtifactStore URL: https://github.com/apache/incubator-openwhisk/pull/3366 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] rabbah commented on a change in pull request #3366: Adds method deleteAttachments to ArtifactStore

2018-03-06 Thread GitBox
rabbah commented on a change in pull request #3366: Adds method deleteAttachments to ArtifactStore URL: https://github.com/apache/incubator-openwhisk/pull/3366#discussion_r172620716 ## File path: common/scala/src/main/scala/whisk/core/entity/WhiskAction.scala ## @@

[GitHub] rabbah commented on issue #3366: Adds method deleteAttachments to ArtifactStore

2018-03-06 Thread GitBox
rabbah commented on issue #3366: Adds method deleteAttachments to ArtifactStore URL: https://github.com/apache/incubator-openwhisk/pull/3366#issuecomment-370886240 PG1/2682 is ok. This is an automated message from the Apache

[GitHub] pritidesai opened a new issue #770: Doc - Env. Variable Interpolation in JSON Inputs

2018-03-06 Thread GitBox
pritidesai opened a new issue #770: Doc - Env. Variable Interpolation in JSON Inputs URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/770 With a recent PR #766, we are adding support to interpolate input values using env. variables such that: ```

[GitHub] pritidesai opened a new issue #769: Update Spec - Env. Variable Interpolation in JSON Inputs

2018-03-06 Thread GitBox
pritidesai opened a new issue #769: Update Spec - Env. Variable Interpolation in JSON Inputs URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/769 With a recent PR #766, we are adding support to interpolate input values with env. variables such that: ```

[GitHub] markusthoemmes opened a new pull request #3400: Use ShardingContainerPoolBalancer as the default.

2018-03-06 Thread GitBox
markusthoemmes opened a new pull request #3400: Use ShardingContainerPoolBalancer as the default. URL: https://github.com/apache/incubator-openwhisk/pull/3400 ## Description As proposed on the dev-list, this switches the default loadbalancer to be the sharding implementation.

[GitHub] duynguyen commented on issue #2: Add DC/OS universe for OpenWhisk and orchestration scripts

2018-03-06 Thread GitBox
duynguyen commented on issue #2: Add DC/OS universe for OpenWhisk and orchestration scripts URL: https://github.com/apache/incubator-openwhisk-deploy-mesos/pull/2#issuecomment-370863417 @ddragosd wdyt? This is an automated

[GitHub] mdeuser commented on a change in pull request #3388: Update require-whisk-auth behavior to secure web action

2018-03-06 Thread GitBox
mdeuser commented on a change in pull request #3388: Update require-whisk-auth behavior to secure web action URL: https://github.com/apache/incubator-openwhisk/pull/3388#discussion_r172507692 ## File path: common/scala/src/main/scala/whisk/core/entity/WhiskAction.scala ##

[GitHub] duynguyen opened a new pull request #2: Add DC/OS universe for OpenWhisk and orchestration scripts

2018-03-06 Thread GitBox
duynguyen opened a new pull request #2: Add DC/OS universe for OpenWhisk and orchestration scripts URL: https://github.com/apache/incubator-openwhisk-deploy-mesos/pull/2 The universe contains packages for: * API Gateway * CouchDB * OpenWhisk controller * OpenWhisk invoker

[GitHub] duynguyen commented on issue #1: Set up project skeleton

2018-03-06 Thread GitBox
duynguyen commented on issue #1: Set up project skeleton URL: https://github.com/apache/incubator-openwhisk-deploy-mesos/pull/1#issuecomment-370861866 @ddragosd is it enough for the project structure? This is an automated

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

2018-03-06 Thread GitBox
dubeejw commented on issue #3394: fix deploy.yml URL: https://github.com/apache/incubator-openwhisk/pull/3394#issuecomment-370854911 Trying https://github.com/apache/incubator-openwhisk/pull/3399. This is an automated

[GitHub] dubeejw closed pull request #3399: WIP: CLI deployment failure due to permissions

2018-03-06 Thread GitBox
dubeejw closed pull request #3399: WIP: CLI deployment failure due to permissions URL: https://github.com/apache/incubator-openwhisk/pull/3399 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] dubeejw commented on issue #3394: fix deploy.yml

2018-03-06 Thread GitBox
dubeejw commented on issue #3394: fix deploy.yml URL: https://github.com/apache/incubator-openwhisk/pull/3394#issuecomment-370853787 ``` TASK [Gathering Facts] * [WARNING]: Consider using unarchive module rather than running

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

2018-03-06 Thread GitBox
dubeejw commented on issue #3394: fix deploy.yml URL: https://github.com/apache/incubator-openwhisk/pull/3394#issuecomment-370853396 Travis keeps failing here. This is an automated message from the Apache Git Service. To

[GitHub] mdeuser opened a new pull request #3399: WIP: CLI deployment failure due to permissions

2018-03-06 Thread GitBox
mdeuser opened a new pull request #3399: WIP: CLI deployment failure due to permissions URL: https://github.com/apache/incubator-openwhisk/pull/3399 ## Description Update cli role .yml Local testing via vagrant VM Travis testing ## My changes affect the following

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

2018-03-06 Thread GitBox
kameshsampath commented on issue #24: AdoptOpenJDK Eclipse OpenJ9 as alternative Java runtime URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/24#issuecomment-370836429 @Param-S @csantanapr check this https://github.com/kameshsampath/adoptopenjdk , I have made the

[GitHub] kameshsampath commented on issue #18: Rebase OW JavaRuntime to use OpenJDK 8

2018-03-06 Thread GitBox
kameshsampath commented on issue #18: Rebase OW JavaRuntime to use OpenJDK 8 URL: https://github.com/apache/incubator-openwhisk-runtime-java/issues/18#issuecomment-370833566 @csantanapr https://github.com/kameshsampath/adoptopenjdk check that project that has the AdoptOpenJDK based

[GitHub] cbickel commented on issue #3391: Add namespace-blacklist to invoker.

2018-03-06 Thread GitBox
cbickel commented on issue #3391: Add namespace-blacklist to invoker. URL: https://github.com/apache/incubator-openwhisk/pull/3391#issuecomment-370832230 PG5#140 is OK. This is an automated message from the Apache Git

[GitHub] markusthoemmes commented on issue #3369: Use pureconfig for CouchDbRestStore

2018-03-06 Thread GitBox
markusthoemmes commented on issue #3369: Use pureconfig for CouchDbRestStore URL: https://github.com/apache/incubator-openwhisk/pull/3369#issuecomment-370818611 @chetanmeh I'll fix the conflicts. This is an automated message

[GitHub] rabbah commented on a change in pull request #3388: Update require-whisk-auth behavior to secure web action

2018-03-06 Thread GitBox
rabbah commented on a change in pull request #3388: Update require-whisk-auth behavior to secure web action URL: https://github.com/apache/incubator-openwhisk/pull/3388#discussion_r172546821 ## File path: common/scala/src/main/scala/whisk/core/entity/WhiskAction.scala ##

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

2018-03-06 Thread GitBox
RSulzmann commented on issue #3394: fix deploy.yml URL: https://github.com/apache/incubator-openwhisk/pull/3394#issuecomment-370804629 Observations: Temporary directories ?quivalent to the local {{openwhisk_build_dir}} were created on the remote server but not deleted after the

[GitHub] davglass commented on issue #2220: Ansible error while deploying to distributed environment

2018-03-06 Thread GitBox
davglass commented on issue #2220: Ansible error while deploying to distributed environment URL: https://github.com/apache/incubator-openwhisk/issues/2220#issuecomment-370803021 I'm having a similar error, I wrestled with it all day yesterday and still can't figure out the issue. I have

[GitHub] markusthoemmes commented on a change in pull request #3394: fix deploy.yml

2018-03-06 Thread GitBox
markusthoemmes commented on a change in pull request #3394: fix deploy.yml URL: https://github.com/apache/incubator-openwhisk/pull/3394#discussion_r172536083 ## File path: ansible/roles/cli/tasks/deploy.yml ## @@ -48,10 +50,7 @@ -C {{ openwhisk_build_dir }}/{{

[GitHub] markusthoemmes commented on a change in pull request #3394: fix deploy.yml

2018-03-06 Thread GitBox
markusthoemmes commented on a change in pull request #3394: fix deploy.yml URL: https://github.com/apache/incubator-openwhisk/pull/3394#discussion_r172536083 ## File path: ansible/roles/cli/tasks/deploy.yml ## @@ -48,10 +50,7 @@ -C {{ openwhisk_build_dir }}/{{

[GitHub] mdeuser commented on a change in pull request #3388: Update require-whisk-auth behavior to secure web action

2018-03-06 Thread GitBox
mdeuser commented on a change in pull request #3388: Update require-whisk-auth behavior to secure web action URL: https://github.com/apache/incubator-openwhisk/pull/3388#discussion_r172513170 ## File path: core/controller/src/main/scala/whisk/core/controller/WebActions.scala

[GitHub] mdeuser commented on a change in pull request #3388: Update require-whisk-auth behavior to secure web action

2018-03-06 Thread GitBox
mdeuser commented on a change in pull request #3388: Update require-whisk-auth behavior to secure web action URL: https://github.com/apache/incubator-openwhisk/pull/3388#discussion_r172507692 ## File path: common/scala/src/main/scala/whisk/core/entity/WhiskAction.scala ##

[GitHub] rabbah opened a new issue #3398: action container tests observed to fail often since change to use port mapping

2018-03-06 Thread GitBox
rabbah opened a new issue #3398: action container tests observed to fail often since change to use port mapping URL: https://github.com/apache/incubator-openwhisk/issues/3398 The change I made in #3335 replaced docker inspect with port forwarding. I also added a brute sleep to deal with

[GitHub] rabbah opened a new pull request #3397: Partially revert change to route all action proxy tests through port mapping

2018-03-06 Thread GitBox
rabbah opened a new pull request #3397: Partially revert change to route all action proxy tests through port mapping URL: https://github.com/apache/incubator-openwhisk/pull/3397 The change I made in #3335 perhaps not surprisingly has been observed to fail a few times with Connection Reset

[GitHub] rabbah commented on a change in pull request #3388: Update require-whisk-auth behavior to secure web action

2018-03-06 Thread GitBox
rabbah commented on a change in pull request #3388: Update require-whisk-auth behavior to secure web action URL: https://github.com/apache/incubator-openwhisk/pull/3388#discussion_r172499506 ## File path: docs/annotations.md ## @@ -48,7 +48,7 @@ and must be present and

[GitHub] rabbah commented on a change in pull request #3388: Update require-whisk-auth behavior to secure web action

2018-03-06 Thread GitBox
rabbah commented on a change in pull request #3388: Update require-whisk-auth behavior to secure web action URL: https://github.com/apache/incubator-openwhisk/pull/3388#discussion_r172500051 ## File path: tests/src/test/scala/whisk/core/controller/test/WebActionsApiTests.scala

[GitHub] rabbah commented on a change in pull request #3388: Update require-whisk-auth behavior to secure web action

2018-03-06 Thread GitBox
rabbah commented on a change in pull request #3388: Update require-whisk-auth behavior to secure web action URL: https://github.com/apache/incubator-openwhisk/pull/3388#discussion_r172499869 ## File path: tests/src/test/scala/whisk/core/controller/test/WebActionsApiTests.scala

[GitHub] rabbah commented on a change in pull request #3388: Update require-whisk-auth behavior to secure web action

2018-03-06 Thread GitBox
rabbah commented on a change in pull request #3388: Update require-whisk-auth behavior to secure web action URL: https://github.com/apache/incubator-openwhisk/pull/3388#discussion_r172498719 ## File path: core/controller/src/main/scala/whisk/core/controller/WebActions.scala

[GitHub] rabbah commented on a change in pull request #3388: Update require-whisk-auth behavior to secure web action

2018-03-06 Thread GitBox
rabbah commented on a change in pull request #3388: Update require-whisk-auth behavior to secure web action URL: https://github.com/apache/incubator-openwhisk/pull/3388#discussion_r172499335 ## File path: core/controller/src/main/scala/whisk/core/controller/WebActions.scala

[GitHub] rabbah commented on a change in pull request #3388: Update require-whisk-auth behavior to secure web action

2018-03-06 Thread GitBox
rabbah commented on a change in pull request #3388: Update require-whisk-auth behavior to secure web action URL: https://github.com/apache/incubator-openwhisk/pull/3388#discussion_r172498942 ## File path: core/controller/src/main/scala/whisk/core/controller/WebActions.scala

[GitHub] rabbah commented on a change in pull request #3388: Update require-whisk-auth behavior to secure web action

2018-03-06 Thread GitBox
rabbah commented on a change in pull request #3388: Update require-whisk-auth behavior to secure web action URL: https://github.com/apache/incubator-openwhisk/pull/3388#discussion_r172498687 ## File path: core/controller/src/main/scala/whisk/core/controller/WebActions.scala

[GitHub] rabbah commented on a change in pull request #3388: Update require-whisk-auth behavior to secure web action

2018-03-06 Thread GitBox
rabbah commented on a change in pull request #3388: Update require-whisk-auth behavior to secure web action URL: https://github.com/apache/incubator-openwhisk/pull/3388#discussion_r172498687 ## File path: core/controller/src/main/scala/whisk/core/controller/WebActions.scala

[GitHub] dgrove-oss commented on issue #3378: LogStoreProvider using invokerAgent for KubernetesContainerPool

2018-03-06 Thread GitBox
dgrove-oss commented on issue #3378: LogStoreProvider using invokerAgent for KubernetesContainerPool URL: https://github.com/apache/incubator-openwhisk/pull/3378#issuecomment-370764054 PG4 / 1440 ? This is an automated

[GitHub] markusthoemmes opened a new pull request #3396: Self-generate random strings without going through a UUID.

2018-03-06 Thread GitBox
markusthoemmes opened a new pull request #3396: Self-generate random strings without going through a UUID. URL: https://github.com/apache/incubator-openwhisk/pull/3396 ## Description ActivationIDs are generated via UUID.randomUUID() today. Looking through the code of that you'll

[GitHub] markusthoemmes opened a new issue #3395: Generate ActivationId string directly without going through UUID.

2018-03-06 Thread GitBox
markusthoemmes opened a new issue #3395: Generate ActivationId string directly without going through UUID. URL: https://github.com/apache/incubator-openwhisk/issues/3395 ActivationIDs are generated via UUID.randomUUID() today. Looking through the code of that you'll notice that it only

[GitHub] rabbah commented on a change in pull request #3211: implement ContainerFactory.cpuShare to fix #3110

2018-03-06 Thread GitBox
rabbah commented on a change in pull request #3211: implement ContainerFactory.cpuShare to fix #3110 URL: https://github.com/apache/incubator-openwhisk/pull/3211#discussion_r172493830 ## File path: core/invoker/src/main/resources/application.conf ## @@ -39,7 +49,6 @@

[GitHub] rabbah commented on a change in pull request #3211: implement ContainerFactory.cpuShare to fix #3110

2018-03-06 Thread GitBox
rabbah commented on a change in pull request #3211: implement ContainerFactory.cpuShare to fix #3110 URL: https://github.com/apache/incubator-openwhisk/pull/3211#discussion_r172494186 ## File path:

[GitHub] rabbah commented on a change in pull request #3211: implement ContainerFactory.cpuShare to fix #3110

2018-03-06 Thread GitBox
rabbah commented on a change in pull request #3211: implement ContainerFactory.cpuShare to fix #3110 URL: https://github.com/apache/incubator-openwhisk/pull/3211#discussion_r172495016 ## File path: tests/src/test/scala/whisk/core/containerpool/test/ContainerArgsConfigTest.scala

[GitHub] rabbah commented on a change in pull request #3211: implement ContainerFactory.cpuShare to fix #3110

2018-03-06 Thread GitBox
rabbah commented on a change in pull request #3211: implement ContainerFactory.cpuShare to fix #3110 URL: https://github.com/apache/incubator-openwhisk/pull/3211#discussion_r172494890 ## File path:

[GitHub] rabbah closed pull request #3371: Make AuthKey parsing leaner.

2018-03-06 Thread GitBox
rabbah closed pull request #3371: Make AuthKey parsing leaner. URL: https://github.com/apache/incubator-openwhisk/pull/3371 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 closed pull request #3362: Make ActivationId implementation leaner.

2018-03-06 Thread GitBox
rabbah closed pull request #3362: Make ActivationId implementation leaner. URL: https://github.com/apache/incubator-openwhisk/pull/3362 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] RSulzmann commented on issue #3394: fix deploy.yml

2018-03-06 Thread GitBox
RSulzmann commented on issue #3394: fix deploy.yml URL: https://github.com/apache/incubator-openwhisk/pull/3394#issuecomment-370751879 edit (@rabbah): I moved the comment to the description. This is an automated message from

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

2018-03-06 Thread GitBox
RSulzmann commented on issue #3394: fix deploy.yml URL: https://github.com/apache/incubator-openwhisk/pull/3394#issuecomment-370751879 Problem: the directory created on the remote system has no write access when created. the downloaded and extracted files needs to be copied from

[GitHub] cbickel closed pull request #3393: Turn off Kafka auto-commits.

2018-03-06 Thread GitBox
cbickel closed pull request #3393: Turn off Kafka auto-commits. URL: https://github.com/apache/incubator-openwhisk/pull/3393 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] csantanapr closed issue #258: Move Feed Test from BasicHealthTest to MessageHubFeedTests

2018-03-06 Thread GitBox
csantanapr closed issue #258: Move Feed Test from BasicHealthTest to MessageHubFeedTests URL: https://github.com/apache/incubator-openwhisk-package-kafka/issues/258 This is an automated message from the Apache Git Service.

[GitHub] csantanapr closed pull request #259: Move Feed Update Test

2018-03-06 Thread GitBox
csantanapr closed pull request #259: Move Feed Update Test URL: https://github.com/apache/incubator-openwhisk-package-kafka/pull/259 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] daisy-ycguo commented on issue #52: Add license verification document

2018-03-06 Thread GitBox
daisy-ycguo commented on issue #52: Add license verification document URL: https://github.com/apache/incubator-openwhisk-release/pull/52#issuecomment-370726502 The document is ready for review. @mrutkows @houshengbo This is

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

2018-03-06 Thread GitBox
daisy-ycguo commented on issue #53: Change to use RAT maven plugin to support ASF short format header URL: https://github.com/apache/incubator-openwhisk-release/pull/53#issuecomment-370725972 @houshengbo Hi, in verify_source_code.sh, I use OPENWHISK_SOURCE_DIR instead of

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

2018-03-06 Thread GitBox
daisy-ycguo commented on issue #53: Change to use RAT maven plugin to support ASF short format header URL: https://github.com/apache/incubator-openwhisk-release/pull/53#issuecomment-370725972 @houshengbo Hi, in `verify_source_code.sh`, I use OPENWHISK_SOURCE_DIR instead of

[GitHub] markusthoemmes commented on a change in pull request #3391: Add namespace-blacklist to invoker.

2018-03-06 Thread GitBox
markusthoemmes commented on a change in pull request #3391: Add namespace-blacklist to invoker. URL: https://github.com/apache/incubator-openwhisk/pull/3391#discussion_r172450254 ## File path: core/invoker/src/main/scala/whisk/core/invoker/NamespaceBlacklist.scala ## @@

[GitHub] markusthoemmes closed issue #3376: Create PR checklist

2018-03-06 Thread GitBox
markusthoemmes closed issue #3376: Create PR checklist URL: https://github.com/apache/incubator-openwhisk/issues/3376 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub

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

2018-03-06 Thread GitBox
markusthoemmes commented on issue #3394: fix deploy.yml URL: https://github.com/apache/incubator-openwhisk/pull/3394#issuecomment-370709364 Please add a description describing what fails and how you fixed the issue. This is

  1   2   >