[GitHub] tysonnorris closed pull request #3941: to address #3918, reuse a container on applicationError

2018-08-10 Thread GitBox
tysonnorris closed pull request #3941: to address #3918, reuse a container on applicationError URL: https://github.com/apache/incubator-openwhisk/pull/3941 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] ygalashov commented on issue #272: Controller crash loop

2018-08-10 Thread GitBox
ygalashov commented on issue #272: Controller crash loop URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/272#issuecomment-412124179 Yes it's failing liveness probe, here is describe output: ``` Name: controller-0 Namespace: openwhisk Node:

[GitHub] rabbah commented on issue #3941: to address #3918, reuse a container on applicationError

2018-08-10 Thread GitBox
rabbah commented on issue #3941: to address #3918, reuse a container on applicationError URL: https://github.com/apache/incubator-openwhisk/pull/3941#issuecomment-412188985 See

[GitHub] tysonnorris commented on issue #3941: to address #3918, reuse a container on applicationError

2018-08-10 Thread GitBox
tysonnorris commented on issue #3941: to address #3918, reuse a container on applicationError URL: https://github.com/apache/incubator-openwhisk/pull/3941#issuecomment-412188927 I guess this is the test:

[GitHub] dgrove-oss opened a new pull request #273: Use whisk.auth from secrets instead of the default auths in the git repo

2018-08-10 Thread GitBox
dgrove-oss opened a new pull request #273: Use whisk.auth from secrets instead of the default auths in the git repo URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/273 A modification of the fix suggested in PR#271 that mounts the whisk.auth secrets in a volume

[GitHub] mrutkows opened a new pull request #298: Create new graphic for scaling-alarms-capacity

2018-08-10 Thread GitBox
mrutkows opened a new pull request #298: Create new graphic for scaling-alarms-capacity URL: https://github.com/apache/incubator-openwhisk-website/pull/298 This is an automated message from the Apache Git Service. To

[GitHub] mrutkows opened a new pull request #297: Update home page illustrations based upon feedback

2018-08-10 Thread GitBox
mrutkows opened a new pull request #297: Update home page illustrations based upon feedback URL: https://github.com/apache/incubator-openwhisk-website/pull/297 This is an automated message from the Apache Git Service. To

[GitHub] StrayCat1 closed issue #963: Unable to deploy tests/usecases/triggerrule

2018-08-10 Thread GitBox
StrayCat1 closed issue #963: Unable to deploy tests/usecases/triggerrule URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/963 This is an automated message from the Apache Git Service. To respond to the

[GitHub] dgrove-oss commented on issue #271: Correctly writing custom OpenWhisk authentication keys (for the guest…

2018-08-10 Thread GitBox
dgrove-oss commented on issue #271: Correctly writing custom OpenWhisk authentication keys (for the guest… URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/271#issuecomment-412201884 Closing in favor of #273 -- thank you for the problem report & suggested fix!

[GitHub] dgrove-oss closed pull request #271: Correctly writing custom OpenWhisk authentication keys (for the guest…

2018-08-10 Thread GitBox
dgrove-oss closed pull request #271: Correctly writing custom OpenWhisk authentication keys (for the guest… URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/271 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed

[GitHub] tysonnorris opened a new pull request #3941: to address #3918, reuse a container on applicationError

2018-08-10 Thread GitBox
tysonnorris opened a new pull request #3941: to address #3918, reuse a container on applicationError URL: https://github.com/apache/incubator-openwhisk/pull/3941 Reuse a container on applicationError (graceful error from action), only during /run (any error during /init still destroys the

[GitHub] dgrove-oss commented on issue #271: Correctly writing custom OpenWhisk authentication keys (for the guest…

2018-08-10 Thread GitBox
dgrove-oss commented on issue #271: Correctly writing custom OpenWhisk authentication keys (for the guest… URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/271#issuecomment-412196210 I did a slightly different version of this fix in #273 that mounts the secrets in a

[GitHub] tysonnorris commented on issue #3941: to address #3918, reuse a container on applicationError

2018-08-10 Thread GitBox
tysonnorris commented on issue #3941: to address #3918, reuse a container on applicationError URL: https://github.com/apache/incubator-openwhisk/pull/3941#issuecomment-412187822 Is there a test existing for timeout on run? I don't see one but will try to add it (please let me know if you

[GitHub] rabbah commented on issue #3941: to address #3918, reuse a container on applicationError

2018-08-10 Thread GitBox
rabbah commented on issue #3941: to address #3918, reuse a container on applicationError URL: https://github.com/apache/incubator-openwhisk/pull/3941#issuecomment-412188134 I think there is such a test I’ll try to find it. For the second part I don’t think it matters.

[GitHub] rabbah closed pull request #273: Use whisk.auth from secrets instead of the default auths in the git repo

2018-08-10 Thread GitBox
rabbah closed pull request #273: Use whisk.auth from secrets instead of the default auths in the git repo URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/273 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed

[GitHub] dgrove-oss commented on issue #272: Controller crash loop

2018-08-10 Thread GitBox
dgrove-oss commented on issue #272: Controller crash loop URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/272#issuecomment-412103068 Its not obvious from the log why the controller is crashing. Could you do a `kubectl describe` on the pod, maybe there will be some

[GitHub] pritidesai opened a new pull request #299: Adding GitHub repo structure

2018-08-10 Thread GitBox
pritidesai opened a new pull request #299: Adding GitHub repo structure URL: https://github.com/apache/incubator-openwhisk-website/pull/299 ![screen shot 2018-08-10 at 5 02 24 pm](https://user-images.githubusercontent.com/206285/43985820-47dabd7a-9cbf-11e8-9553-479be5b05b7b.png)

[GitHub] ddragosd closed issue #139: Meet minio file access denied when waiting for the controller

2018-08-10 Thread GitBox
ddragosd closed issue #139: Meet minio file access denied when waiting for the controller URL: https://github.com/apache/incubator-openwhisk-devtools/issues/139 This is an automated message from the Apache Git Service. To

[GitHub] JasonShin opened a new pull request #148: adding a note about -i

2018-08-10 Thread GitBox
JasonShin opened a new pull request #148: adding a note about -i URL: https://github.com/apache/incubator-openwhisk-devtools/pull/148 Hey guys, thanks for the great project! I was going through the documentation to set up a playground environment locally but I was immediately

[GitHub] markusthoemmes opened a new pull request #3960: Make gradle and dependency downloads more stable.

2018-08-10 Thread GitBox
markusthoemmes opened a new pull request #3960: Make gradle and dependency downloads more stable. URL: https://github.com/apache/incubator-openwhisk/pull/3960 Gradle wrapper and dependency downloads are subject to intermittent failures, which can get annoying when you want to test your

[GitHub] keunseob opened a new pull request #3961: Add length of podName checking code for K8s

2018-08-10 Thread GitBox
keunseob opened a new pull request #3961: Add length of podName checking code for K8s URL: https://github.com/apache/incubator-openwhisk/pull/3961 Add maximum length of pod name checking and fixing code. ## Description In case of kubenetes deployment environment, invokers

[GitHub] codecov-io edited a comment on issue #3951: Docs fixes for action invocations

2018-08-10 Thread GitBox
codecov-io edited a comment on issue #3951: Docs fixes for action invocations URL: https://github.com/apache/incubator-openwhisk/pull/3951#issuecomment-411695432 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3951?src=pr=h1) Report > Merging

[GitHub] jthomas commented on issue #268: Add Quotes (somewhere) and remove old pages

2018-08-10 Thread GitBox
jthomas commented on issue #268: Add Quotes (somewhere) and remove old pages URL: https://github.com/apache/incubator-openwhisk-website/issues/268#issuecomment-412007902 @mrutkows Could we contact those non-anonymous quotees and ask them for a more recent version?

[GitHub] codecov-io edited a comment on issue #3656: Adjust the behavior when update controller cluster

2018-08-10 Thread GitBox
codecov-io edited a comment on issue #3656: Adjust the behavior when update controller cluster URL: https://github.com/apache/incubator-openwhisk/pull/3656#issuecomment-389115907 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3656?src=pr=h1) Report > Merging

[GitHub] ygalashov opened a new issue #272: Controller crash loop

2018-08-10 Thread GitBox
ygalashov opened a new issue #272: Controller crash loop URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/272 I cant run openwhisk on single node k8s clustre. I'm installing helm chart according to guidelines. Here is my clusterconfig.yaml: ``` whisk:

[GitHub] markusthoemmes commented on a change in pull request #3961: Add length of podName checking code for K8s

2018-08-10 Thread GitBox
markusthoemmes commented on a change in pull request #3961: Add length of podName checking code for K8s URL: https://github.com/apache/incubator-openwhisk/pull/3961#discussion_r209192008 ## File path:

[GitHub] keunseob commented on a change in pull request #3961: Add length of podName checking code for K8s

2018-08-10 Thread GitBox
keunseob commented on a change in pull request #3961: Add length of podName checking code for K8s URL: https://github.com/apache/incubator-openwhisk/pull/3961#discussion_r209194261 ## File path:

[GitHub] rabbah commented on issue #3941: to address #3918, reuse a container on applicationError

2018-08-10 Thread GitBox
rabbah commented on issue #3941: to address #3918, reuse a container on applicationError URL: https://github.com/apache/incubator-openwhisk/pull/3941#issuecomment-412024187 It’s better - would you want to reuse the container if it timed out? You’d end up with a concurrent activation for

[GitHub] mgencur edited a comment on issue #3941: to address #3918, reuse a container on applicationError

2018-08-10 Thread GitBox
mgencur edited a comment on issue #3941: to address #3918, reuse a container on applicationError URL: https://github.com/apache/incubator-openwhisk/pull/3941#issuecomment-412000192 Hi, I'm wondering what's the rationale behind throwing different types of errors when there's a timeout

[GitHub] rabbah commented on issue #3951: Docs fixes for action invocations

2018-08-10 Thread GitBox
rabbah commented on issue #3951: Docs fixes for action invocations URL: https://github.com/apache/incubator-openwhisk/pull/3951#issuecomment-412022526 LGTM now - I’ll leave it to Markus to clarify the init vs run timeout.

[GitHub] markusthoemmes commented on issue #3941: to address #3918, reuse a container on applicationError

2018-08-10 Thread GitBox
markusthoemmes commented on issue #3941: to address #3918, reuse a container on applicationError URL: https://github.com/apache/incubator-openwhisk/pull/3941#issuecomment-412023047 Good questions @mgencur. I agree, a timeout on run should be made a container-error (aka

[GitHub] rabbah commented on issue #147: Use 0.9.0 release as default sources

2018-08-10 Thread GitBox
rabbah commented on issue #147: Use 0.9.0 release as default sources URL: https://github.com/apache/incubator-openwhisk-devtools/pull/147#issuecomment-412025143 :100: This is an automated message from the Apache Git Service.

[GitHub] fmaschler commented on issue #147: Use 0.9.0 release as default sources

2018-08-10 Thread GitBox
fmaschler commented on issue #147: Use 0.9.0 release as default sources URL: https://github.com/apache/incubator-openwhisk-devtools/pull/147#issuecomment-412024933 Sure, I didn't know this is used by Travis to test the master. Should the master remain default? What about a new make rule

[GitHub] mgencur commented on issue #3951: Docs fixes for action invocations

2018-08-10 Thread GitBox
mgencur commented on issue #3951: Docs fixes for action invocations URL: https://github.com/apache/incubator-openwhisk/pull/3951#issuecomment-412031946 @rabbah @markusthoemmes alright, thanks. So let's wait with this PR until the other one is completely sorted out. Then I can make a

[GitHub] jiangpengcheng opened a new pull request #3656: Adjust the behavior when update controller cluster

2018-08-10 Thread GitBox
jiangpengcheng opened a new pull request #3656: Adjust the behavior when update controller cluster URL: https://github.com/apache/incubator-openwhisk/pull/3656 Set the max permits for each invokerSlot instead of creating new ForcibleSemaphore. New ForcibleSemaphore will ignore acquired

[GitHub] jiangpengcheng closed pull request #3656: Adjust the behavior when update controller cluster

2018-08-10 Thread GitBox
jiangpengcheng closed pull request #3656: Adjust the behavior when update controller cluster URL: https://github.com/apache/incubator-openwhisk/pull/3656 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] mgencur commented on issue #3941: to address #3918, reuse a container on applicationError

2018-08-10 Thread GitBox
mgencur commented on issue #3941: to address #3918, reuse a container on applicationError URL: https://github.com/apache/incubator-openwhisk/pull/3941#issuecomment-412000192 Hi, I'm wondering what's the rationale behind throwing different types of errors when there's a timeout during

[GitHub] codecov-io edited a comment on issue #3656: Adjust the behavior when update controller cluster

2018-08-10 Thread GitBox
codecov-io edited a comment on issue #3656: Adjust the behavior when update controller cluster URL: https://github.com/apache/incubator-openwhisk/pull/3656#issuecomment-389115907 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3656?src=pr=h1) Report > Merging

[GitHub] codecov-io edited a comment on issue #3656: Adjust the behavior when update controller cluster

2018-08-10 Thread GitBox
codecov-io edited a comment on issue #3656: Adjust the behavior when update controller cluster URL: https://github.com/apache/incubator-openwhisk/pull/3656#issuecomment-389115907 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3656?src=pr=h1) Report > Merging

[GitHub] fmaschler commented on issue #139: Meet minio file access denied when waiting for the controller

2018-08-10 Thread GitBox
fmaschler commented on issue #139: Meet minio file access denied when waiting for the controller URL: https://github.com/apache/incubator-openwhisk-devtools/issues/139#issuecomment-412078764 I see... Well then this issue seems to be solved. The related #140 is merged anyway.

[GitHub] markusthoemmes closed pull request #3698: Make amount of parallel docker runs configurable.

2018-08-10 Thread GitBox
markusthoemmes closed pull request #3698: Make amount of parallel docker runs configurable. URL: https://github.com/apache/incubator-openwhisk/pull/3698 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] codecov-io edited a comment on issue #3960: Make gradle and dependency downloads more stable.

2018-08-10 Thread GitBox
codecov-io edited a comment on issue #3960: Make gradle and dependency downloads more stable. URL: https://github.com/apache/incubator-openwhisk/pull/3960#issuecomment-412035494 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3960?src=pr=h1) Report > Merging

[GitHub] codecov-io commented on issue #3961: Add length of podName checking code for K8s

2018-08-10 Thread GitBox
codecov-io commented on issue #3961: Add length of podName checking code for K8s URL: https://github.com/apache/incubator-openwhisk/pull/3961#issuecomment-412035276 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3961?src=pr=h1) Report > Merging

[GitHub] codecov-io edited a comment on issue #3698: Make amount of parallel docker runs configurable.

2018-08-10 Thread GitBox
codecov-io edited a comment on issue #3698: Make amount of parallel docker runs configurable. URL: https://github.com/apache/incubator-openwhisk/pull/3698#issuecomment-392048718 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3698?src=pr=h1) Report > Merging

[GitHub] cbickel commented on issue #3698: Make amount of parallel docker runs configurable.

2018-08-10 Thread GitBox
cbickel commented on issue #3698: Make amount of parallel docker runs configurable. URL: https://github.com/apache/incubator-openwhisk/pull/3698#issuecomment-412051346 PG2#3492 looks good. This is an automated message from

[GitHub] codecov-io edited a comment on issue #3814: wskadmin scala db command to export and import

2018-08-10 Thread GitBox
codecov-io edited a comment on issue #3814: wskadmin scala db command to export and import URL: https://github.com/apache/incubator-openwhisk/pull/3814#issuecomment-400666862 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3814?src=pr=h1) Report > Merging

[GitHub] chetanmeh commented on issue #3945: Treat action code as attachments

2018-08-10 Thread GitBox
chetanmeh commented on issue #3945: Treat action code as attachments URL: https://github.com/apache/incubator-openwhisk/pull/3945#issuecomment-412055273 @dubee new actions would be stored with `code` as object. That format was already being used by Java actions so should work fine.

[GitHub] codecov-io commented on issue #3960: Make gradle and dependency downloads more stable.

2018-08-10 Thread GitBox
codecov-io commented on issue #3960: Make gradle and dependency downloads more stable. URL: https://github.com/apache/incubator-openwhisk/pull/3960#issuecomment-412035494 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3960?src=pr=h1) Report > Merging

[GitHub] codecov-io edited a comment on issue #3960: Make gradle and dependency downloads more stable.

2018-08-10 Thread GitBox
codecov-io edited a comment on issue #3960: Make gradle and dependency downloads more stable. URL: https://github.com/apache/incubator-openwhisk/pull/3960#issuecomment-412035494 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3960?src=pr=h1) Report > Merging

[GitHub] jiangpengcheng opened a new pull request #3656: Adjust the behavior when update controller cluster

2018-08-10 Thread GitBox
jiangpengcheng opened a new pull request #3656: Adjust the behavior when update controller cluster URL: https://github.com/apache/incubator-openwhisk/pull/3656 Set the max permits for each invokerSlot instead of creating new ForcibleSemaphore. New ForcibleSemaphore will ignore acquired