[GitHub] markusthoemmes closed pull request #4110: Exclude unused transitive dependencies

markusthoemmes closed pull request #4110: Exclude unused transitive dependencies URL: https://github.com/apache/incubator-openwhisk/pull/4110 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] chetanmeh commented on issue #4111: Update to Gradle 4.10.2

chetanmeh commented on issue #4111: Update to Gradle 4.10.2 URL: https://github.com/apache/incubator-openwhisk/pull/4111#issuecomment-438926058 @vvraskin Can you trigger a PG run for this PR This is an automated message from

[GitHub] chetanmeh closed pull request #4104: Cache empty auth results to reduce db load

chetanmeh closed pull request #4104: Cache empty auth results to reduce db load URL: https://github.com/apache/incubator-openwhisk/pull/4104 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 edited a comment on issue #4114: Update KindRestrictor to merge namespace and default whitelists

codecov-io edited a comment on issue #4114: Update KindRestrictor to merge namespace and default whitelists URL: https://github.com/apache/incubator-openwhisk/pull/4114#issuecomment-438879407 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/4114?src=pr=h1) Report >

[GitHub] codecov-io commented on issue #4114: Update KindRestrictor to merge namespace and default whitelists

codecov-io commented on issue #4114: Update KindRestrictor to merge namespace and default whitelists URL: https://github.com/apache/incubator-openwhisk/pull/4114#issuecomment-438879407 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/4114?src=pr=h1) Report > Merging

[GitHub] tysonnorris commented on a change in pull request #4114: Update KindRestrictor to merge namespace and default whitelists

tysonnorris commented on a change in pull request #4114: Update KindRestrictor to merge namespace and default whitelists URL: https://github.com/apache/incubator-openwhisk/pull/4114#discussion_r233677573 ## File path:

[GitHub] asteed opened a new pull request #4114: Update KindRestrictor to merge subject and default whitelists

asteed opened a new pull request #4114: Update KindRestrictor to merge subject and default whitelists URL: https://github.com/apache/incubator-openwhisk/pull/4114 When performing `KindRestrictor.check()`, merge the contents of both the namespace's `allowedKinds` limit with the

[GitHub] tysonnorris opened a new pull request #383: enable use of concurrency action limits via cli

tysonnorris opened a new pull request #383: enable use of concurrency action limits via cli URL: https://github.com/apache/incubator-openwhisk-cli/pull/383 enable --concurrency flag to set intra-container concurrency limit via wsk cli requires client-go PR first:

[GitHub] tysonnorris commented on issue #94: Add Concurrency to Limits

tysonnorris commented on issue #94: Add Concurrency to Limits URL: https://github.com/apache/incubator-openwhisk-client-go/pull/94#issuecomment-438853625 This PR is required to move the openwhisk-cli to support concurrency changes introduced via

[GitHub] dubee closed issue #102: fatal error: concurrent map writes

dubee closed issue #102: fatal error: concurrent map writes URL: https://github.com/apache/incubator-openwhisk-client-go/issues/102 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] dubee closed pull request #103: Allow NewClient to run concurrently

dubee closed pull request #103: Allow NewClient to run concurrently URL: https://github.com/apache/incubator-openwhisk-client-go/pull/103 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

[GitHub] dgrove-oss opened a new issue #356: dynamic volume provisioning for kubeadm-dind-cluster

dgrove-oss opened a new issue #356: dynamic volume provisioning for kubeadm-dind-cluster URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/356 Document the steps for defining a default storage class and associated provisioner to kubeadm-dind-cluster setup. Then

[GitHub] dgrove-oss opened a new issue #355: better readiness probe for kafka

dgrove-oss opened a new issue #355: better readiness probe for kafka URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/355 We need a good external check for Kafka being ready that can be used in the `openwhisk.readiness.waitForKafka` macro in _readiness.tpl. The

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

rabbah commented on issue #2795: enable concurrent activation processing URL: https://github.com/apache/incubator-openwhisk/pull/2795#issuecomment-438819035 @tysonnorris thanks for being patient with this and working through all the nuances. Looking forward to feedback from production on

[GitHub] mdeuser closed pull request #104: Update Go Version

mdeuser closed pull request #104: Update Go Version URL: https://github.com/apache/incubator-openwhisk-client-go/pull/104 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull

[GitHub] dgrove-oss opened a new issue #354: kafka: enable replicated kafka pods

dgrove-oss opened a new issue #354: kafka: enable replicated kafka pods URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/354 For production deployments, the kakfa StatefulSet should have a replicaCount > 1. To enable this, we need to do similar changes as we did

[GitHub] codecov-io edited a comment on issue #3974: Support for start-of-activation log sentinel

codecov-io edited a comment on issue #3974: Support for start-of-activation log sentinel URL: https://github.com/apache/incubator-openwhisk/pull/3974#issuecomment-413638224 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3974?src=pr=h1) Report > Merging

[GitHub] dubee commented on issue #104: Update Go Version

dubee commented on issue #104: Update Go Version URL: https://github.com/apache/incubator-openwhisk-client-go/pull/104#issuecomment-438798408 @mdeuser, anything else? This is an automated message from the Apache Git Service.

[GitHub] chetanmeh commented on issue #4104: Cache empty auth results to reduce db load

chetanmeh commented on issue #4104: Cache empty auth results to reduce db load URL: https://github.com/apache/incubator-openwhisk/pull/4104#issuecomment-438795814 @rabbah @markusthoemmes can you review cache changes once

[GitHub] chetanmeh commented on issue #4110: Exclude unused transitive dependencies

chetanmeh commented on issue #4110: Exclude unused transitive dependencies URL: https://github.com/apache/incubator-openwhisk/pull/4110#issuecomment-438795340 Scala compiler was primary target as seen few reports of vulnerability in that in past. So lesser unused code ... Lesser the

[GitHub] dgrove-oss commented on issue #3974: Support for start-of-activation log sentinel

dgrove-oss commented on issue #3974: Support for start-of-activation log sentinel URL: https://github.com/apache/incubator-openwhisk/pull/3974#issuecomment-438794664 since it was easy, I rebased to resolve conflicts & squashed back to a single commit.

[GitHub] mrutkows commented on issue #342: Updated home.html to include .NET runtime reference.

mrutkows commented on issue #342: Updated home.html to include .NET runtime reference. URL: https://github.com/apache/incubator-openwhisk-website/pull/342#issuecomment-438789355 This will have to be reworked a tiny nit to support the new HTML layout...

[GitHub] dgrove-oss commented on issue #3974: Support for start-of-activation log sentinel

dgrove-oss commented on issue #3974: Support for start-of-activation log sentinel URL: https://github.com/apache/incubator-openwhisk/pull/3974#issuecomment-438786143 Not much has happened. I do still think having a start sentinel is potentially useful, but there were some additional

[GitHub] dubee commented on issue #4112: Create trigger activation only when active rule(s) are present

dubee commented on issue #4112: Create trigger activation only when active rule(s) are present URL: https://github.com/apache/incubator-openwhisk/pull/4112#issuecomment-438785733 TY @markusthoemmes! This is an automated

[GitHub] dubee commented on issue #4088: Make activation polling for blocking invocations configurable

dubee commented on issue #4088: Make activation polling for blocking invocations configurable URL: https://github.com/apache/incubator-openwhisk/pull/4088#issuecomment-438785460 Change this back to a WIP state as the two active ack changes do not currently allow DB polling to be

[GitHub] dubee edited a comment on issue #4088: Make activation polling for blocking invocations configurable

dubee edited a comment on issue #4088: Make activation polling for blocking invocations configurable URL: https://github.com/apache/incubator-openwhisk/pull/4088#issuecomment-438785460 Changed this back to a WIP state as the two active ack changes do not currently allow DB polling to be

[GitHub] dubee commented on issue #103: Allow NewClient to run concurrently

dubee commented on issue #103: Allow NewClient to run concurrently URL: https://github.com/apache/incubator-openwhisk-client-go/pull/103#issuecomment-438784406 Travis is currently broken. Waiting for https://github.com/apache/incubator-openwhisk-client-go/pull/104 before this one is

[GitHub] dgrove-oss opened a new issue #353: Need ability to pin versions of all upstream artifacts in Helm Chart

dgrove-oss opened a new issue #353: Need ability to pin versions of all upstream artifacts in Helm Chart URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/353 To make a useful release of the Helm chart we need the ability to pin docker images, downloads, git clones,

[GitHub] markusthoemmes commented on issue #3991: Provide Artifact with File Storage Activation Store

markusthoemmes commented on issue #3991: Provide Artifact with File Storage Activation Store URL: https://github.com/apache/incubator-openwhisk/pull/3991#issuecomment-438777665 This has gotten very stale. Still a valid PR?

[GitHub] markusthoemmes commented on issue #3974: Support for start-of-activation log sentinel

markusthoemmes commented on issue #3974: Support for start-of-activation log sentinel URL: https://github.com/apache/incubator-openwhisk/pull/3974#issuecomment-438775322 This has been sitting in WIP, got stale and is inactive since 2 months. Any update?

[GitHub] markusthoemmes commented on issue #3977: Elasticsearch Activation Store

markusthoemmes commented on issue #3977: Elasticsearch Activation Store URL: https://github.com/apache/incubator-openwhisk/pull/3977#issuecomment-438775196 This has been sitting in WIP, got stale and is inactive since 2 months. Any update?

[GitHub] markusthoemmes commented on issue #3996: Move sequence component activation IDs to annotations

markusthoemmes commented on issue #3996: Move sequence component activation IDs to annotations URL: https://github.com/apache/incubator-openwhisk/pull/3996#issuecomment-438775006 This has been sitting in WIP, got stale and is inactive since 2 months. Any update?

[GitHub] markusthoemmes commented on issue #3997: Move conductor component activation IDs to annotations

markusthoemmes commented on issue #3997: Move conductor component activation IDs to annotations URL: https://github.com/apache/incubator-openwhisk/pull/3997#issuecomment-438775072 This has been sitting in WIP, got stale and is inactive since 2 months. Any update?

[GitHub] markusthoemmes commented on issue #3994: Optionally return a small portion of user logs for blocking invocations

markusthoemmes commented on issue #3994: Optionally return a small portion of user logs for blocking invocations URL: https://github.com/apache/incubator-openwhisk/pull/3994#issuecomment-438774837 This has been sitting in WIP, got stale and is inactive since 2 months. Any update?

[GitHub] markusthoemmes commented on issue #3995: Move trigger component activation IDs to annotations

markusthoemmes commented on issue #3995: Move trigger component activation IDs to annotations URL: https://github.com/apache/incubator-openwhisk/pull/3995#issuecomment-438774914 This has been sitting in WIP, got stale and is inactive since 2 months. Any update?

[GitHub] markusthoemmes closed pull request #4112: Create trigger activation only when active rule(s) are present

markusthoemmes closed pull request #4112: Create trigger activation only when active rule(s) are present URL: https://github.com/apache/incubator-openwhisk/pull/4112 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] dubee commented on issue #4112: Create trigger activation only when active rule(s) are present

dubee commented on issue #4112: Create trigger activation only when active rule(s) are present URL: https://github.com/apache/incubator-openwhisk/pull/4112#issuecomment-438771257 PG1 3618  This is an automated message

[GitHub] dubee commented on a change in pull request #104: Update Go Version

dubee commented on a change in pull request #104: Update Go Version URL: https://github.com/apache/incubator-openwhisk-client-go/pull/104#discussion_r233564769 ## File path: .travis.yml ## @@ -22,7 +22,7 @@ matrix: - os: linux sudo: required

[GitHub] dgrove-oss commented on issue #352: Streamline Helm install process + linting fixes

dgrove-oss commented on issue #352: Streamline Helm install process + linting fixes URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/352#issuecomment-438765815 This is now ready for review and merge.

[GitHub] pritidesai closed pull request #1009: adding pkg and action version number with interpolation capability

pritidesai closed pull request #1009: adding pkg and action version number with interpolation capability URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/1009 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below

[GitHub] codecov-io edited a comment on issue #2795: enable concurrent activation processing

codecov-io edited a comment 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] codecov-io edited a comment on issue #2795: enable concurrent activation processing

codecov-io edited a comment 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] mrutkows opened a new pull request #348: Update Action documentation sections and prioritize Developer role

mrutkows opened a new pull request #348: Update Action documentation sections and prioritize Developer role URL: https://github.com/apache/incubator-openwhisk-website/pull/348 Additionally, be very clear on Developer pre-reqs and encourage the use of Kube for local development platform.

[GitHub] mdeuser commented on a change in pull request #104: Update Go Version

mdeuser commented on a change in pull request #104: Update Go Version URL: https://github.com/apache/incubator-openwhisk-client-go/pull/104#discussion_r233526708 ## File path: .travis.yml ## @@ -22,7 +22,7 @@ matrix: - os: linux sudo: required

[GitHub] pritidesai opened a new pull request #1009: adding pkg and action version number with interpolation capability

pritidesai opened a new pull request #1009: adding pkg and action version number with interpolation capability URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/1009 This is an automated message from the

[GitHub] dubee commented on issue #380: Refactor cli to support go 1.11

dubee commented on issue #380: Refactor cli to support go 1.11 URL: https://github.com/apache/incubator-openwhisk-cli/issues/380#issuecomment-438719142 @vvraskin, probably a good idea that we fix these. Apparently only the latest two released version of Go are actually supported.

[GitHub] dubee opened a new pull request #104: Update Go Version

dubee opened a new pull request #104: Update Go Version URL: https://github.com/apache/incubator-openwhisk-client-go/pull/104 Travis is failing due to golint as shown below. It appears Go 1.8 is no longer supported (https://github.com/golang/go/issues/28291). ``` $ go get -u

[GitHub] dubee closed pull request #381: allow api gw access token override

dubee closed pull request #381: allow api gw access token override URL: https://github.com/apache/incubator-openwhisk-cli/pull/381 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] vvraskin commented on issue #4113: Documentation corrections

vvraskin commented on issue #4113: Documentation corrections URL: https://github.com/apache/incubator-openwhisk/issues/4113#issuecomment-438707480 Hi @atsag , great that you found it! Do you mind opening a PR to fix it?

[GitHub] atsag opened a new issue #4113: Documentation corrections

atsag opened a new issue #4113: Documentation corrections URL: https://github.com/apache/incubator-openwhisk/issues/4113 Some minor corrections to the README.md file: - In the "Using CLI from outside the VM section"

[GitHub] dubee commented on a change in pull request #4112: Create trigger activation only when active rule(s) are present

dubee commented on a change in pull request #4112: Create trigger activation only when active rule(s) are present URL: https://github.com/apache/incubator-openwhisk/pull/4112#discussion_r233492077 ## File path:

[GitHub] dubee commented on issue #4112: Create trigger activation only when active rule(s) are present

dubee commented on issue #4112: Create trigger activation only when active rule(s) are present URL: https://github.com/apache/incubator-openwhisk/pull/4112#issuecomment-438681487 @mdeuser, the activation is not written to the database, it is just needlessly created in the controller

[GitHub] mhenke1 commented on issue #4104: Cache empty auth results to reduce db load

mhenke1 commented on issue #4104: Cache empty auth results to reduce db load URL: https://github.com/apache/incubator-openwhisk/pull/4104#issuecomment-438655141 @chetanmeh I have changed the code as suggested This is an

[GitHub] mhenke1 edited a comment on issue #4104: Cache empty auth results to reduce db load

mhenke1 edited a comment on issue #4104: Cache empty auth results to reduce db load URL: https://github.com/apache/incubator-openwhisk/pull/4104#issuecomment-438654979 [PG 2:3878 ok] This is an automated message from the

[GitHub] mhenke1 commented on issue #4104: Cache empty auth results to reduce db load

mhenke1 commented on issue #4104: Cache empty auth results to reduce db load URL: https://github.com/apache/incubator-openwhisk/pull/4104#issuecomment-438654979 [PG 2:3878] This is an automated message from the Apache Git

[GitHub] codecov-io edited a comment on issue #4104: Cache empty auth results to reduce db load

codecov-io edited a comment on issue #4104: Cache empty auth results to reduce db load URL: https://github.com/apache/incubator-openwhisk/pull/4104#issuecomment-43700 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/4104?src=pr=h1) Report > Merging

[GitHub] codecov-io edited a comment on issue #4078: Add ability to not store activations for specified namespace.

codecov-io edited a comment on issue #4078: Add ability to not store activations for specified namespace. URL: https://github.com/apache/incubator-openwhisk/pull/4078#issuecomment-432238083 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/4078?src=pr=h1) Report >