[GitHub] houshengbo closed pull request #148: Better describe LICENSE policies, rules and document exclusions by repo.

houshengbo closed pull request #148: Better describe LICENSE policies, rules and document exclusions by repo. URL: https://github.com/apache/incubator-openwhisk-release/pull/148 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed

[GitHub] mrutkows closed pull request #275: Restore the LICENSE and NOTICE to fit the source code release

mrutkows closed pull request #275: Restore the LICENSE and NOTICE to fit the source code release URL: https://github.com/apache/incubator-openwhisk-catalog/pull/275 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] mrutkows closed pull request #58: Restore the LICENSE and NOTICE to fit the source code release

mrutkows closed pull request #58: Restore the LICENSE and NOTICE to fit the source code release URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/58 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] mrutkows closed pull request #84: Restore the LICENSE and NOTICE to fit the source code release

mrutkows closed pull request #84: Restore the LICENSE and NOTICE to fit the source code release URL: https://github.com/apache/incubator-openwhisk-client-go/pull/84 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] mrutkows closed pull request #295: Restore the LICENSE and NOTICE to fit the source code release

mrutkows closed pull request #295: Restore the LICENSE and NOTICE to fit the source code release URL: https://github.com/apache/incubator-openwhisk-cli/pull/295 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] mrutkows commented on a change in pull request #21: Restore the LICENSE and NOTICE to fit the source code release

mrutkows commented on a change in pull request #21: Restore the LICENSE and NOTICE to fit the source code release URL: https://github.com/apache/incubator-openwhisk-runtime-php/pull/21#discussion_r187642038 ## File path: LICENSE.txt ## @@ -187,7 +187,7 @@ same

[GitHub] mrutkows closed pull request #23: Restore the LICENSE and NOTICE to fit the source code release

mrutkows closed pull request #23: Restore the LICENSE and NOTICE to fit the source code release URL: https://github.com/apache/incubator-openwhisk-runtime-python/pull/23 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for

[GitHub] mrutkows closed pull request #21: Restore the LICENSE and NOTICE to fit the source code release

mrutkows closed pull request #21: Restore the LICENSE and NOTICE to fit the source code release URL: https://github.com/apache/incubator-openwhisk-runtime-php/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

[GitHub] mrutkows closed pull request #51: Restore the LICENSE and NOTICE to fit the source code release

mrutkows closed pull request #51: Restore the LICENSE and NOTICE to fit the source code release URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/51 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

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

csantanapr commented on issue #3232: Making prewarm kind (and count) configurable URL: https://github.com/apache/incubator-openwhisk/pull/3232#issuecomment-388490617 From reading the source code, it will use the kind set by default. So it looks what needs to be specified in the ansible

[GitHub] codecov-io commented on issue #2795: enable concurrent activation processing

codecov-io commented on issue #2795: enable concurrent activation processing URL: https://github.com/apache/incubator-openwhisk/pull/2795#issuecomment-386889684 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/2795?src=pr=h1) Report > Merging

[GitHub] dgrove-oss opened a new pull request #197: Reorganize values.yaml and simplify helper usage

dgrove-oss opened a new pull request #197: Reorganize values.yaml and simplify helper usage URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/197 Make a pass through values.yaml to unify naming conventions and group into per-service hierarchies. Inline trivial helper

[GitHub] eweiter opened a new issue #3646: Swift 4.1 issue discovered around Structs with two Strings

eweiter opened a new issue #3646: Swift 4.1 issue discovered around Structs with two Strings URL: https://github.com/apache/incubator-openwhisk/issues/3646 It was reported that the Swift 4.1 runtime was exhibiting a strange behavior. The behavior in question occurs when a Struct has

[GitHub] mrutkows commented on issue #194: Adopt latest scancode ASF release config. (enforce *.md, .conf, .sh).

mrutkows commented on issue #194: Adopt latest scancode ASF release config. (enforce *.md, .conf, .sh). URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/194#issuecomment-388482338 @dgrove-oss thanks Dave!

[GitHub] mrutkows closed pull request #902: Restore the LICENSE and NOTICE to fit the source code release

mrutkows closed pull request #902: Restore the LICENSE and NOTICE to fit the source code release URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/902 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] jasonpet closed pull request #267: new gradle task

jasonpet closed pull request #267: new gradle task URL: https://github.com/apache/incubator-openwhisk-package-kafka/pull/267 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] abaruni opened a new pull request #267: new gradle task

abaruni opened a new pull request #267: new gradle task URL: https://github.com/apache/incubator-openwhisk-package-kafka/pull/267 add additional gradle task to allow running test suite without any tests relying upon health endpoints

[GitHub] codecov-io commented on issue #3286: Treat action code as attachments for created/updated actions

codecov-io commented on issue #3286: Treat action code as attachments for created/updated actions URL: https://github.com/apache/incubator-openwhisk/pull/3286#issuecomment-385776180 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3286?src=pr=h1) Report > Merging

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

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

[GitHub] shazron closed issue #3207: Remove Consul from "The Internal flow of processing" section diagram in about.md

shazron closed issue #3207: Remove Consul from "The Internal flow of processing" section diagram in about.md URL: https://github.com/apache/incubator-openwhisk/issues/3207 This is an automated message from the Apache Git

[GitHub] codecov-io commented on issue #3619: Provide an activation store SPI

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] csantanapr commented on issue #3638: Remove Consul from "The Internal flow of processing" section diagram in about.md

csantanapr commented on issue #3638: Remove Consul from "The Internal flow of processing" section diagram in about.md URL: https://github.com/apache/incubator-openwhisk/pull/3638#issuecomment-388419055 Sweet  @shazron

[GitHub] csantanapr closed pull request #3638: Remove Consul from "The Internal flow of processing" section diagram in about.md

csantanapr closed pull request #3638: Remove Consul from "The Internal flow of processing" section diagram in about.md URL: https://github.com/apache/incubator-openwhisk/pull/3638 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed

[GitHub] shazron commented on issue #3638: Remove Consul from "The Internal flow of processing" section diagram in about.md

shazron commented on issue #3638: Remove Consul from "The Internal flow of processing" section diagram in about.md URL: https://github.com/apache/incubator-openwhisk/pull/3638#issuecomment-388418932 Rebased. All checks pass.

[GitHub] codecov-io commented on issue #3638: Remove Consul from "The Internal flow of processing" section diagram in about.md

codecov-io commented on issue #3638: Remove Consul from "The Internal flow of processing" section diagram in about.md URL: https://github.com/apache/incubator-openwhisk/pull/3638#issuecomment-387984721 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3638?src=pr=h1)

[GitHub] codecov-io commented on issue #3638: Remove Consul from "The Internal flow of processing" section diagram in about.md

codecov-io commented on issue #3638: Remove Consul from "The Internal flow of processing" section diagram in about.md URL: https://github.com/apache/incubator-openwhisk/pull/3638#issuecomment-387984721 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3638?src=pr=h1)

[GitHub] mrutkows opened a new pull request #148: Better describe LICENSE policies, rules and document exclusions by repo.

mrutkows opened a new pull request #148: Better describe LICENSE policies, rules and document exclusions by repo. URL: https://github.com/apache/incubator-openwhisk-release/pull/148 This is an automated message from the

[GitHub] dgrove-oss closed pull request #194: Adopt latest scancode ASF release config. (enforce *.md, .conf, .sh).

dgrove-oss closed pull request #194: Adopt latest scancode ASF release config. (enforce *.md, .conf, .sh). URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/194 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed

[GitHub] vvraskin opened a new issue #3645: Create a single list of all deployment parameters with description

vvraskin opened a new issue #3645: Create a single list of all deployment parameters with description URL: https://github.com/apache/incubator-openwhisk/issues/3645 We have several places where OW deployment parameters could be configured (pureConf, ansible vars), some of the params

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

rabbah commented on issue #3232: Making prewarm kind (and count) configurable URL: https://github.com/apache/incubator-openwhisk/pull/3232#issuecomment-388403914 @csantanapr this will need a PG. I'll go through it today or this weekend but it's good enough for jenkins.

[GitHub] markusthoemmes commented on issue #3644: Allow more special characters in EntityName.

markusthoemmes commented on issue #3644: Allow more special characters in EntityName. URL: https://github.com/apache/incubator-openwhisk/pull/3644#issuecomment-388403600 @rabbah pushed a change to allow all characters except "/" (to avoid ambiguous URLs). Fixing the underscore

[GitHub] dgrove-oss commented on issue #194: Adopt latest scancode ASF release config. (enforce *.md, .conf, .sh).

dgrove-oss commented on issue #194: Adopt latest scancode ASF release config. (enforce *.md, .conf, .sh). URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/194#issuecomment-388398048 @mrutkows try rebasing again when you get a chance. I believe setup.sh shouldn't touch

[GitHub] rabbah commented on a change in pull request #3644: Allow more special characters in EntityName.

rabbah commented on a change in pull request #3644: Allow more special characters in EntityName. URL: https://github.com/apache/incubator-openwhisk/pull/3644#discussion_r187649246 ## File path: common/scala/src/main/scala/whisk/core/entity/EntityPath.scala ## @@ -189,13

[GitHub] markusthoemmes commented on a change in pull request #3644: Allow more special characters in EntityName.

markusthoemmes commented on a change in pull request #3644: Allow more special characters in EntityName. URL: https://github.com/apache/incubator-openwhisk/pull/3644#discussion_r187648786 ## File path: common/scala/src/main/scala/whisk/core/entity/EntityPath.scala ## @@

[GitHub] rabbah commented on a change in pull request #3644: Allow more special characters in EntityName.

rabbah commented on a change in pull request #3644: Allow more special characters in EntityName. URL: https://github.com/apache/incubator-openwhisk/pull/3644#discussion_r187648510 ## File path: common/scala/src/main/scala/whisk/core/entity/EntityPath.scala ## @@ -189,13

[GitHub] rabbah commented on a change in pull request #3644: Allow more special characters in EntityName.

rabbah commented on a change in pull request #3644: Allow more special characters in EntityName. URL: https://github.com/apache/incubator-openwhisk/pull/3644#discussion_r187648510 ## File path: common/scala/src/main/scala/whisk/core/entity/EntityPath.scala ## @@ -189,13

[GitHub] csantanapr closed pull request #196: perform all downloads in /tmp to avoid adding spurious files into clo…

csantanapr closed pull request #196: perform all downloads in /tmp to avoid adding spurious files into clo… URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/196 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed

[GitHub] csantanapr closed pull request #193: Flatten and simplify Helm charts

csantanapr closed pull request #193: Flatten and simplify Helm charts URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/193 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 a change in pull request #3644: Allow more special characters in EntityName.

csantanapr commented on a change in pull request #3644: Allow more special characters in EntityName. URL: https://github.com/apache/incubator-openwhisk/pull/3644#discussion_r187645739 ## File path: common/scala/src/main/scala/whisk/core/entity/EntityPath.scala ## @@

[GitHub] csantanapr commented on issue #3644: Allow more special characters in EntityName.

csantanapr commented on issue #3644: Allow more special characters in EntityName. URL: https://github.com/apache/incubator-openwhisk/pull/3644#issuecomment-388394275 @markusthoemmes Is this a fix for #3637 ? This is an

[GitHub] rabbah commented on a change in pull request #3644: Allow more special characters in EntityName.

rabbah commented on a change in pull request #3644: Allow more special characters in EntityName. URL: https://github.com/apache/incubator-openwhisk/pull/3644#discussion_r187645162 ## File path: common/scala/src/main/scala/whisk/core/entity/EntityPath.scala ## @@ -189,13

[GitHub] markusthoemmes opened a new pull request #3644: Allow more special characters in EntityName.

markusthoemmes opened a new pull request #3644: Allow more special characters in EntityName. URL: https://github.com/apache/incubator-openwhisk/pull/3644 There's no reason not to allow these characters in EntityName, so let's not be narrower on our users than we need to be. ## My

[GitHub] dgrove-oss commented on issue #196: perform all downloads in /tmp to avoid adding spurious files into clo…

dgrove-oss commented on issue #196: perform all downloads in /tmp to avoid adding spurious files into clo… URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/196#issuecomment-388393338 we could move scancode.sh earlier, but it is also probably better to the install of

[GitHub] codecov-io commented on issue #3643: Restore the LICENSE and NOTICE to fit the source code release

codecov-io commented on issue #3643: Restore the LICENSE and NOTICE to fit the source code release URL: https://github.com/apache/incubator-openwhisk/pull/3643#issuecomment-388390995 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3643?src=pr=h1) Report > Merging

[GitHub] mrutkows commented on issue #194: Adopt latest scancode ASF release config. (enforce *.md, .conf, .sh).

mrutkows commented on issue #194: Adopt latest scancode ASF release config. (enforce *.md, .conf, .sh). URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/194#issuecomment-388388278 @csantanapr likely we should be doing less in setup.sh (i.e., during the "before_install"

[GitHub] mrutkows commented on issue #194: Adopt latest scancode ASF release config. (enforce *.md, .conf, .sh).

mrutkows commented on issue #194: Adopt latest scancode ASF release config. (enforce *.md, .conf, .sh). URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/194#issuecomment-388388278 @csantanapr likely we should be doing less in setup.sh (i.e., during the "before_install"

[GitHub] mrutkows commented on issue #194: Adopt latest scancode ASF release config. (enforce *.md, .conf, .sh).

mrutkows commented on issue #194: Adopt latest scancode ASF release config. (enforce *.md, .conf, .sh). URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/194#issuecomment-388388278 @csantanapr likely we should be doing less in setup.sh (i.e., during the "before_install"

[GitHub] houshengbo opened a new pull request #902: Restore the LICENSE and NOTICE to fit the source code release

houshengbo opened a new pull request #902: Restore the LICENSE and NOTICE to fit the source code release URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/902 All the work we have done for binary dependencies have been saved in openwhisk release. We will resume the work

[GitHub] houshengbo opened a new pull request #84: Restore the LICENSE and NOTICE to fit the source code release

houshengbo opened a new pull request #84: Restore the LICENSE and NOTICE to fit the source code release URL: https://github.com/apache/incubator-openwhisk-client-go/pull/84 All the work we have done for binary dependencies have been saved in openwhisk release. We will resume the work

[GitHub] houshengbo opened a new pull request #295: Restore the LICENSE and NOTICE to fit the source code release

houshengbo opened a new pull request #295: Restore the LICENSE and NOTICE to fit the source code release URL: https://github.com/apache/incubator-openwhisk-cli/pull/295 All the work we have done for binary dependencies have been saved in openwhisk release. We will resume the work later

[GitHub] houshengbo opened a new pull request #50: Restore the LICENSE and NOTICE to fit the source code release

houshengbo opened a new pull request #50: Restore the LICENSE and NOTICE to fit the source code release URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/50 All the work we have done for binary dependencies have been saved in openwhisk release. We will resume the

[GitHub] houshengbo opened a new pull request #37: Restore the LICENSE and NOTICE to fit the source code release

houshengbo opened a new pull request #37: Restore the LICENSE and NOTICE to fit the source code release URL: https://github.com/apache/incubator-openwhisk-runtime-docker/pull/37 All the work we have done for binary dependencies have been saved in openwhisk release. We will resume the

[GitHub] houshengbo opened a new pull request #58: Restore the LICENSE and NOTICE to fit the source code release

houshengbo opened a new pull request #58: Restore the LICENSE and NOTICE to fit the source code release URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/58 All the work we have done for binary dependencies have been saved in openwhisk release. We will resume the

[GitHub] houshengbo opened a new pull request #21: Restore the LICENSE and NOTICE to fit the source code release

houshengbo opened a new pull request #21: Restore the LICENSE and NOTICE to fit the source code release URL: https://github.com/apache/incubator-openwhisk-runtime-php/pull/21 All the work we have done for binary dependencies have been saved in openwhisk release. We will resume the work

[GitHub] houshengbo opened a new pull request #23: Restore the LICENSE and NOTICE to fit the source code release

houshengbo opened a new pull request #23: Restore the LICENSE and NOTICE to fit the source code release URL: https://github.com/apache/incubator-openwhisk-runtime-python/pull/23 All the work we have done for binary dependencies have been saved in openwhisk release. We will resume the

[GitHub] houshengbo opened a new pull request #51: Restore the LICENSE and NOTICE to fit the source code release

houshengbo opened a new pull request #51: Restore the LICENSE and NOTICE to fit the source code release URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/51 All the work we have done for binary dependencies have been saved in openwhisk release. We will resume the work

[GitHub] houshengbo opened a new pull request #275: Restore the LICENSE and NOTICE to fit the source code release

houshengbo opened a new pull request #275: Restore the LICENSE and NOTICE to fit the source code release URL: https://github.com/apache/incubator-openwhisk-catalog/pull/275 All the work we have done for binary dependencies have been saved in openwhisk release. We will resume the work

[GitHub] dgrove-oss opened a new pull request #196: perform all downloads in /tmp to avoid adding spurious files into clo…

dgrove-oss opened a new pull request #196: perform all downloads in /tmp to avoid adding spurious files into clo… URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/196 …ned repo This is an automated message

[GitHub] houshengbo opened a new pull request #3643: Restore the LICENSE and NOTICE to fit the source code release

houshengbo opened a new pull request #3643: Restore the LICENSE and NOTICE to fit the source code release URL: https://github.com/apache/incubator-openwhisk/pull/3643 All the work we have done for binary dependencies have been saved in openwhisk release. We will resume the work later in

[GitHub] brunogirin opened a new issue #901: Allow project credential entry to be an environment variable

brunogirin opened a new issue #901: Allow project credential entry to be an environment variable URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/901 When creating a manifest with project information to use with `wskdeploy sync`, it requires credentials to be specified

[GitHub] rabbah closed pull request #3642: Fix README.md's scanCode issues.

rabbah closed pull request #3642: Fix README.md's scanCode issues. URL: https://github.com/apache/incubator-openwhisk/pull/3642 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 issue #3642: Fix README.md's scanCode issues.

rabbah commented on issue #3642: Fix README.md's scanCode issues. URL: https://github.com/apache/incubator-openwhisk/pull/3642#issuecomment-388343824 @markusthoemmes it was a pr that had passed travis before the scan code changes.

[GitHub] codecov-io commented on issue #3642: Fix README.md's scanCode issues.

codecov-io commented on issue #3642: Fix README.md's scanCode issues. URL: https://github.com/apache/incubator-openwhisk/pull/3642#issuecomment-388336050 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3642?src=pr=h1) Report > Merging

[GitHub] csantanapr commented on issue #294: Fix the copyryright IBM issue in the headers

csantanapr commented on issue #294: Fix the copyryright IBM issue in the headers URL: https://github.com/apache/incubator-openwhisk-apigateway/issues/294#issuecomment-388331960 All source code currently in all the ASF repos that came from IBM is donated to ASF.

[GitHub] csantanapr commented on a change in pull request #193: Flatten and simplify Helm charts

csantanapr commented on a change in pull request #193: Flatten and simplify Helm charts URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/193#discussion_r187582480 ## File path: helm/Chart.yaml ## @@ -5,5 +5,7 @@ version: 0.1.0 keywords: - Apache

[GitHub] csantanapr commented on a change in pull request #193: Flatten and simplify Helm charts

csantanapr commented on a change in pull request #193: Flatten and simplify Helm charts URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/193#discussion_r187582480 ## File path: helm/Chart.yaml ## @@ -5,5 +5,7 @@ version: 0.1.0 keywords: - Apache

[GitHub] csantanapr commented on issue #193: Flatten and simplify Helm charts

csantanapr commented on issue #193: Flatten and simplify Helm charts URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/193#issuecomment-388330366 Problems with Travis This is an automated message from the

[GitHub] csantanapr commented on issue #3638: Remove Consul from "The Internal flow of processing" section diagram in about.md

csantanapr commented on issue #3638: Remove Consul from "The Internal flow of processing" section diagram in about.md URL: https://github.com/apache/incubator-openwhisk/pull/3638#issuecomment-388327278 Ah ok @shazron different readme problem, scancode changed and is out of this repo

[GitHub] markusthoemmes commented on issue #3638: Remove Consul from "The Internal flow of processing" section diagram in about.md

markusthoemmes commented on issue #3638: Remove Consul from "The Internal flow of processing" section diagram in about.md URL: https://github.com/apache/incubator-openwhisk/pull/3638#issuecomment-388326575 @shazron should be fine after #3642 is merged.

[GitHub] markusthoemmes opened a new pull request #3642: Fix README.md's scanCode issues.

markusthoemmes opened a new pull request #3642: Fix README.md's scanCode issues. URL: https://github.com/apache/incubator-openwhisk/pull/3642 `master` is broken due to trailing whitespace. @mrutkows did scanCode's config change without updating the main repo?

[GitHub] csantanapr commented on issue #194: Adopt latest scancode ASF release config. (enforce *.md, .conf, .sh).

csantanapr commented on issue #194: Adopt latest scancode ASF release config. (enforce *.md, .conf, .sh). URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/194#issuecomment-388326466 The scancode step should be done much earlier on the pure git repo source

[GitHub] shazron commented on issue #3638: Remove Consul from "The Internal flow of processing" section diagram in about.md

shazron commented on issue #3638: Remove Consul from "The Internal flow of processing" section diagram in about.md URL: https://github.com/apache/incubator-openwhisk/pull/3638#issuecomment-388324822 @csantanapr not sure that will help, since the last two commits from master itself failed

[GitHub] csantanapr commented on issue #3638: Remove Consul from "The Internal flow of processing" section diagram in about.md

csantanapr commented on issue #3638: Remove Consul from "The Internal flow of processing" section diagram in about.md URL: https://github.com/apache/incubator-openwhisk/pull/3638#issuecomment-388323804 I think you need to update your branch with latest from master. The error is

[GitHub] shazron commented on issue #3638: Remove Consul from "The Internal flow of processing" section diagram in about.md

shazron commented on issue #3638: Remove Consul from "The Internal flow of processing" section diagram in about.md URL: https://github.com/apache/incubator-openwhisk/pull/3638#issuecomment-388322618 Not sure why one of the builds in the build matrix failed, this is just a doc change...

[GitHub] sven-lange-last commented on issue #3606: Increase nginx security by adapting ssl_ciphers.

sven-lange-last commented on issue #3606: Increase nginx security by adapting ssl_ciphers. URL: https://github.com/apache/incubator-openwhisk/pull/3606#issuecomment-388310902 @ningyougang you are using [OpenJDK 1.8.0 update 102](http://openjdk.java.net/projects/jdk8u/releases/8u102.html)

[GitHub] sven-lange-last commented on issue #3606: Increase nginx security by adapting ssl_ciphers.

sven-lange-last commented on issue #3606: Increase nginx security by adapting ssl_ciphers. URL: https://github.com/apache/incubator-openwhisk/pull/3606#issuecomment-388310902 @ningyougang you are using [OpenJDK 1.8.0 update 102](http://openjdk.java.net/projects/jdk8u/releases/8u102.html)

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

markusthoemmes commented on issue #3607: Change the JDK to use openjdk8-openj9 URL: https://github.com/apache/incubator-openwhisk/issues/3607#issuecomment-388269801 Good one @dgrove-oss! Didn't think about all the other stuff in a docker image. I think its illusionary to have a docker