[GitHub] saurabh1284 commented on issue #311: controller and invoker pods are stuck in PodInitializing

saurabh1284 commented on issue #311: controller and invoker pods are stuck in PodInitializing URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/311#issuecomment-436905427 Tried by adding "coredns.kube-system", but still hitting same issue. -> kubectl logs

[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 >

[GitHub] mhenke1 opened a new pull request #4104: Cache empty auth results to reduce db load

mhenke1 opened a new pull request #4104: Cache empty auth results to reduce db load URL: https://github.com/apache/incubator-openwhisk/pull/4104 Cache negative auth responses to avoid going to the DB with every identical call ## Description Since negative auth responses

[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-436992721 [PG3:2985 ok] 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] jasonpet closed pull request #182: Rename the imported package into org.apache.openwhisk.

jasonpet closed pull request #182: Rename the imported package into org.apache.openwhisk. URL: https://github.com/apache/incubator-openwhisk-package-cloudant/pull/182 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] lionelvillard opened a new issue #102: fatal error: concurrent map writes

lionelvillard opened a new issue #102: fatal error: concurrent map writes URL: https://github.com/apache/incubator-openwhisk-client-go/issues/102 The whisk client cannot be use across multiple goroutines, causing `concurrent map writes` panics. The problem seems to be here:

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

rabbah commented on issue #4104: Cache empty auth results to reduce db load URL: https://github.com/apache/incubator-openwhisk/pull/4104#issuecomment-437013817 Agree. Is 5 minutes too high? This is an automated message from

[GitHub] csantanapr closed pull request #167: Rename the imported package to org.apache.openwhisk.

csantanapr closed pull request #167: Rename the imported package to org.apache.openwhisk. URL: https://github.com/apache/incubator-openwhisk-package-alarms/pull/167 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 closed pull request #303: Rename the package into org.apache.openwhisk.

csantanapr closed pull request #303: Rename the package into org.apache.openwhisk. URL: https://github.com/apache/incubator-openwhisk-package-kafka/pull/303 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] dgrove-oss opened a new pull request #346: package rename: whisk ==> org.apache.openwhisk

dgrove-oss opened a new pull request #346: package rename: whisk ==> org.apache.openwhisk URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/346 This is an automated message from the Apache Git Service. To

[GitHub] csantanapr closed pull request #290: Rename the package into org.apache.openwhisk

csantanapr closed pull request #290: Rename the package into org.apache.openwhisk URL: https://github.com/apache/incubator-openwhisk-catalog/pull/290 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] jasonpet closed pull request #99: use stretch instead of jessie for nodejs10

jasonpet closed pull request #99: use stretch instead of jessie for nodejs10 URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/99 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] csantanapr closed pull request #346: package rename: whisk ==> org.apache.openwhisk

csantanapr closed pull request #346: package rename: whisk ==> org.apache.openwhisk URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/346 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] 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-437044918 Caching such negative lookups may be abused to put lots of of entries in cache and put pressure on heap (causing eviction of

[GitHub] dgrove-oss commented on issue #345: [k8s-docker-for-mac] Some nodes hang in `Init` status on Mac book reboot

dgrove-oss commented on issue #345: [k8s-docker-for-mac] Some nodes hang in `Init` status on Mac book reboot URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/345#issuecomment-437112732 Its the same problem. The default settings didn't create persistent storage for

[GitHub] codecov-io commented on issue #4105: Change the order to run the Action Limits Tests

codecov-io commented on issue #4105: Change the order to run the Action Limits Tests URL: https://github.com/apache/incubator-openwhisk/pull/4105#issuecomment-437146938 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/4105?src=pr=h1) Report > Merging

[GitHub] dgrove-oss opened a new pull request #347: Enable persistence by default

dgrove-oss opened a new pull request #347: Enable persistence by default URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/347 Consolidate options around persistence and enable it by default. Kubernetes on Docker for Mac has good support for dynamic provisioning of

[GitHub] mrutkows commented on issue #347: Improve instructs. for config. of CLI and Whisk Deploy

mrutkows commented on issue #347: Improve instructs. for config. of CLI and Whisk Deploy URL: https://github.com/apache/incubator-openwhisk-website/pull/347#issuecomment-437181839 @pritidesai ready for review. This is an

[GitHub] pritidesai closed pull request #347: Improve instructs. for config. of CLI and Whisk Deploy

pritidesai closed pull request #347: Improve instructs. for config. of CLI and Whisk Deploy URL: https://github.com/apache/incubator-openwhisk-website/pull/347 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] dgrove-oss removed a comment on issue #347: Enable persistence by default

dgrove-oss removed a comment on issue #347: Enable persistence by default URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/347#issuecomment-437177521 Fixes #345 This is an automated message from the

[GitHub] dgrove-oss commented on issue #347: Enable persistence by default

dgrove-oss commented on issue #347: Enable persistence by default URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/347#issuecomment-437177521 Fixes #345 This is an automated message from the Apache Git

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

markusthoemmes commented on issue #4104: Cache empty auth results to reduce db load URL: https://github.com/apache/incubator-openwhisk/pull/4104#issuecomment-437047798 @chetanmeh the cache is "bounded" as in its entries are garbage collectable.

[GitHub] markusthoemmes commented on issue #4105: Change the order to run the Action Limits Tests

markusthoemmes commented on issue #4105: Change the order to run the Action Limits Tests URL: https://github.com/apache/incubator-openwhisk/pull/4105#issuecomment-437051310 The title says: "Change the order to run the Action Limits Tests". Can you elaborate why this PR is needed?

[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-437053186 @markusthoemmes All caches are currently configured with `softValues` thereby the entries are evicted (apart from expiry) on

[GitHub] houshengbo commented on issue #4105: Change the order to run the Action Limits Tests

houshengbo commented on issue #4105: Change the order to run the Action Limits Tests URL: https://github.com/apache/incubator-openwhisk/pull/4105#issuecomment-437053451 The tests in ActionLimitsTests.scala used to be run relatively earlier before I changed the package from whisk to

[GitHub] houshengbo closed pull request #378: Rename the imported package into org.apache.openwhisk

houshengbo closed pull request #378: Rename the imported package into org.apache.openwhisk URL: https://github.com/apache/incubator-openwhisk-cli/pull/378 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] markusthoemmes commented on issue #4105: Change the order to run the Action Limits Tests

markusthoemmes commented on issue #4105: Change the order to run the Action Limits Tests URL: https://github.com/apache/incubator-openwhisk/pull/4105#issuecomment-437054382 In general, the order of tests should not matter at all. Should we rather try to find/fix the bug here vs. hiding

[GitHub] houshengbo commented on issue #4105: Change the order to run the Action Limits Tests

houshengbo commented on issue #4105: Change the order to run the Action Limits Tests URL: https://github.com/apache/incubator-openwhisk/pull/4105#issuecomment-437054884 @markusthoemmes Yes. I suspect some tests do not release the memory, but not found it yet.

[GitHub] houshengbo opened a new pull request #4105: Change the order to run the Action Limits Tests

houshengbo opened a new pull request #4105: Change the order to run the Action Limits Tests URL: https://github.com/apache/incubator-openwhisk/pull/4105 ## Description ## Related issue and scope - [ ] I opened an issue to propose and discuss this change

[GitHub] markusthoemmes commented on issue #4105: Change the order to run the Action Limits Tests

markusthoemmes commented on issue #4105: Change the order to run the Action Limits Tests URL: https://github.com/apache/incubator-openwhisk/pull/4105#issuecomment-437074239 @houshengbo this has nothing to do with memory leaking at all. It seems like the test just isn't reliable? From the