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 per
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 provena
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 init
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?
---
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&el=h1)
Report
> Merging
[#3
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 duri
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:
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 bra
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 mak
mgencur commented on issue #3951: Docs fixes for action invocations
URL:
https://github.com/apache/incubator-openwhisk/pull/3951#issuecomment-412020929
hey @rabbah, wdyt now?
This is an automated message from the Apache Git S
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:
core/invoker/src/main/scala/whisk/core/containerpool/kubernetes/KubernetesContai
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.
Th
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
action-devel
markusthoemmes commented on issue #3951: Docs fixes for action invocations
URL:
https://github.com/apache/incubator-openwhisk/pull/3951#issuecomment-412023366
@rabbah Clarified in the other PR, I don't see a reason not to make them
both behave the same. One could argue that an action timeo
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:
core/invoker/src/main/scala/whisk/core/containerpool/kubernetes/KubernetesContainer.sc
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 e
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 to
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.
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&el=h1)
Report
> Merging
[#3951](https://codec
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 little
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&el=h1)
Report
> Merging
[#3961](https://co
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&el=h1)
Report
> Merging
[#3960](htt
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&el=h1)
Report
> Merging
[#39
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&el=h1)
Report
> Merging
[#36
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 t
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&el=h1)
Report
> Merging
[#3814]
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.
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 provena
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 per
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 provenan
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&el=h1)
Report
> Merging
[#39
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&el=h1)
Report
> Merging
[#3
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&el=h1)
Report
> Merging
[#3
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.
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 ad
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:
StrayCat1 opened a new issue #963: Unable to deploy tests/usecases/triggerrule
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/963
I installed the `wskdeploy` version 0.9.7 from the zip, configured the
client to point to my local openwhisk instance, cloned and navigated
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 res
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
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.
---
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:
https://github.com/apache/incubator-openwhisk/blob/8f1dc2e9c848deb9f72da5b636895cbc5c
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
https://github.com/apache/incubator-openwhisk/blob/f6046721acd801ea1777cb5cb040dfaa6922a18e/tests/src/test/s
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 prove
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
tysonnorris removed a comment on issue #3941: to address #3918, reuse a
container on applicationError
URL:
https://github.com/apache/incubator-openwhisk/pull/3941#issuecomment-412193457
In renaming `containerError` to `developerError`, it suddenly becomes
strange to me that the other erro
tysonnorris commented on issue #3941: to address #3918, reuse a container on
applicationError
URL:
https://github.com/apache/incubator-openwhisk/pull/3941#issuecomment-412193457
In renaming `containerError` to `developerError`, it suddenly becomes
strange to me that the other error is `ap
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 instead
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 vol
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 mess
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!
-
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 bel
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 below
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 respond
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)
![screen
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 re
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 blocke
56 matches
Mail list logo