[GitHub] ServoKvd commented on issue #2911: Proposal to to reorganize core/ and root(openwhisk/) folders to reflect existing architecture better

2017-11-29 Thread GitBox
ServoKvd commented on issue #2911: Proposal to to reorganize core/ and root(openwhisk/) folders to reflect existing architecture better URL: https://github.com/apache/incubator-openwhisk/issues/2911#issuecomment-347839797 I guess Carol's main concern was with "sticking to ansible scripts"

[GitHub] houshengbo commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations

2017-11-29 Thread GitBox
houshengbo commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations URL: https://github.com/apache/incubator-openwhisk/issues/3021#issuecomment-347897872 @rabbah The failure of the wskdeploy build is from the deploy and

[GitHub] houshengbo commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations

2017-11-29 Thread GitBox
houshengbo commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations URL: https://github.com/apache/incubator-openwhisk/issues/3021#issuecomment-347897872 @rabbah The failure of the wskdeploy build is from the deploy and

[GitHub] houshengbo commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations

2017-11-29 Thread GitBox
houshengbo commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations URL: https://github.com/apache/incubator-openwhisk/issues/3021#issuecomment-347897872 @rabbah The failure of the wskdeploy build is from the deploy and

[GitHub] houshengbo commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations

2017-11-29 Thread GitBox
houshengbo commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations URL: https://github.com/apache/incubator-openwhisk/issues/3021#issuecomment-347897872 @rabbah The failure of the wskdeploy build is from the deploy and

[GitHub] mrutkows commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations

2017-11-29 Thread GitBox
mrutkows commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations URL: https://github.com/apache/incubator-openwhisk/issues/3021#issuecomment-347901169 Also, if this is a client-side/user strategy for this failure that is

[GitHub] rabbah commented on a change in pull request #82: add configuration of Ingress and restructure docs accordingly

2017-11-29 Thread GitBox
rabbah commented on a change in pull request #82: add configuration of Ingress and restructure docs accordingly URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/82#discussion_r153826080 ## File path: kubernetes/nginx/README.md ## @@ -96,6 +107,13 @@

[GitHub] rabbah closed pull request #82: add configuration of Ingress and restructure docs accordingly

2017-11-29 Thread GitBox
rabbah closed pull request #82: add configuration of Ingress and restructure docs accordingly URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/82 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the

[GitHub] rabbah commented on issue #3024: Action using dockerfile without going through dockerhub

2017-11-29 Thread GitBox
rabbah commented on issue #3024: Action using dockerfile without going through dockerhub URL: https://github.com/apache/incubator-openwhisk/issues/3024#issuecomment-347908771 Related to https://github.com/apache/incubator-openwhisk/issues/1362#issuecomment-337943114.

[GitHub] houshengbo commented on issue #3021: Concurrency issue brought in by commit

2017-11-29 Thread GitBox
houshengbo commented on issue #3021: Concurrency issue brought in by commit URL: https://github.com/apache/incubator-openwhisk/issues/3021#issuecomment-347879836 wskdeploy travis build works fine before this commit. When I pin the openwhisk version to this commit, the travis failed:

[GitHub] mrutkows commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations

2017-11-29 Thread GitBox
mrutkows commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations URL: https://github.com/apache/incubator-openwhisk/issues/3021#issuecomment-347897189 @rabbah enhancement to return behavior to its previous state?

[GitHub] rabbah commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations

2017-11-29 Thread GitBox
rabbah commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations URL: https://github.com/apache/incubator-openwhisk/issues/3021#issuecomment-347897132 Re-titled and re-labelled accordingly.

[GitHub] houshengbo commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations

2017-11-29 Thread GitBox
houshengbo commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations URL: https://github.com/apache/incubator-openwhisk/issues/3021#issuecomment-347897872 @rabbah The failure of the wskdeploy build is from just the deploy

[GitHub] mrutkows commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations

2017-11-29 Thread GitBox
mrutkows commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations URL: https://github.com/apache/incubator-openwhisk/issues/3021#issuecomment-347898984 @houshengbo in the meantime we likely need to disable all undeploys in

[GitHub] dgrove-oss commented on issue #92: Error when list action

2017-11-29 Thread GitBox
dgrove-oss commented on issue #92: Error when list action URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/92#issuecomment-347901042 Hi Terry, I will take a look today. We had some hiccups in the CI/CD infrastructure so things are a little bit less stable than usual

[GitHub] rabbah commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations

2017-11-29 Thread GitBox
rabbah commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations URL: https://github.com/apache/incubator-openwhisk/issues/3021#issuecomment-347898952 This is a new feature - the client didn't observe this before because

[GitHub] mrutkows commented on issue #3021: Concurrency issue brought in by commit

2017-11-29 Thread GitBox
mrutkows commented on issue #3021: Concurrency issue brought in by commit URL: https://github.com/apache/incubator-openwhisk/issues/3021#issuecomment-347886549 @houshengbo thanks for working to "pin" commits in our Travis to isolate where this issue was introduced.

[GitHub] mrutkows commented on issue #3021: Concurrency issue brought in by commit

2017-11-29 Thread GitBox
mrutkows commented on issue #3021: Concurrency issue brought in by commit URL: https://github.com/apache/incubator-openwhisk/issues/3021#issuecomment-347891412 @rabbah Jeremias indicated he saw this once after merge, but could not recreate it.

[GitHub] mrutkows commented on issue #3021: Concurrency issue brought in by commit

2017-11-29 Thread GitBox
mrutkows commented on issue #3021: Concurrency issue brought in by commit URL: https://github.com/apache/incubator-openwhisk/issues/3021#issuecomment-347891214 @houshengbo @rabbah so I was informed by Dave Grove that we discovered that we were not publishing Docker images for perhaps 2

[GitHub] mrutkows commented on issue #3021: Concurrency issue brought in by commit

2017-11-29 Thread GitBox
mrutkows commented on issue #3021: Concurrency issue brought in by commit URL: https://github.com/apache/incubator-openwhisk/issues/3021#issuecomment-347895825 @rabbah actually MS Azure adapts at the server (server-side API retry 3x) on all APIs...

[GitHub] drcariel opened a new pull request #3025: allow open ansible to install catalog dynamically

2017-11-29 Thread GitBox
drcariel opened a new pull request #3025: allow open ansible to install catalog dynamically URL: https://github.com/apache/incubator-openwhisk/pull/3025 This is an automated message from the Apache Git Service. To respond

[GitHub] houshengbo commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations

2017-11-29 Thread GitBox
houshengbo commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations URL: https://github.com/apache/incubator-openwhisk/issues/3021#issuecomment-347897872 @rabbah The failure of the wskdeploy build is from just the deploy

[GitHub] mrutkows commented on issue #3021: Concurrency issue brought in by commit

2017-11-29 Thread GitBox
mrutkows commented on issue #3021: Concurrency issue brought in by commit URL: https://github.com/apache/incubator-openwhisk/issues/3021#issuecomment-347895825 @rabbah actually MS Azure adapts at the server (server-side API retry 3x) on all APIs... even with eventual consistency within

[GitHub] mrutkows commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations

2017-11-29 Thread GitBox
mrutkows commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations URL: https://github.com/apache/incubator-openwhisk/issues/3021#issuecomment-347898984 @houshengbo in the meantime we likely need to disable all undeploys in

[GitHub] rabbah closed pull request #94: travis changes: use minikube and adjust sleep times

2017-11-29 Thread GitBox
rabbah closed pull request #94: travis changes: use minikube and adjust sleep times URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/94 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] houshengbo commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations

2017-11-29 Thread GitBox
houshengbo commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations URL: https://github.com/apache/incubator-openwhisk/issues/3021#issuecomment-347897872 @rabbah The failure of the wskdeploy build is from just the deploy

[GitHub] rabbah commented on a change in pull request #82: add configuration of Ingress and restructure docs accordingly

2017-11-29 Thread GitBox
rabbah commented on a change in pull request #82: add configuration of Ingress and restructure docs accordingly URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/82#discussion_r153827604 ## File path: kubernetes/nginx/README.md ## @@ -96,6 +107,13 @@

[GitHub] rabbah closed issue #2813: bug in invoker startup leads to too many stem cells?

2017-11-29 Thread GitBox
rabbah closed issue #2813: bug in invoker startup leads to too many stem cells? URL: https://github.com/apache/incubator-openwhisk/issues/2813 This is an automated message from the Apache Git Service. To respond to the

[GitHub] mrutkows commented on issue #3021: Concurrency issue brought in by commit

2017-11-29 Thread GitBox
mrutkows commented on issue #3021: Concurrency issue brought in by commit URL: https://github.com/apache/incubator-openwhisk/issues/3021#issuecomment-347886171 @rabbah we are seeing "concurrency" errors when trying to "undeploy" or delete resources. We have a series of integration tests

[GitHub] rabbah commented on issue #3021: Concurrency issue brought in by commit

2017-11-29 Thread GitBox
rabbah commented on issue #3021: Concurrency issue brought in by commit URL: https://github.com/apache/incubator-openwhisk/issues/3021#issuecomment-347887291 Kafka is not part of the CRUD operations. So I suspect it?s a red herring.

[GitHub] mrutkows commented on issue #651: Concurrency error: Error code: 153: Concurrent modification to resource detected. (code 379)

2017-11-29 Thread GitBox
mrutkows commented on issue #651: Concurrency error: Error code: 153: Concurrent modification to resource detected. (code 379) URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/651#issuecomment-347938756 @houshengbo @pritidesai I am going to start removing failing

[GitHub] mrutkows commented on issue #651: Concurrency error: Error code: 153: Concurrent modification to resource detected. (code 379)

2017-11-29 Thread GitBox
mrutkows commented on issue #651: Concurrency error: Error code: 153: Concurrent modification to resource detected. (code 379) URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/651#issuecomment-347938756 @houshengbo @pritidesai I am going to start removing failing

[GitHub] mrutkows commented on issue #651: Concurrency error: Error code: 153: Concurrent modification to resource detected. (code 379)

2017-11-29 Thread GitBox
mrutkows commented on issue #651: Concurrency error: Error code: 153: Concurrent modification to resource detected. (code 379) URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/651#issuecomment-347938756 @houshengbo @pritidesai I am going to start removing failing

[GitHub] mrutkows commented on issue #651: Concurrency error: Error code: 153: Concurrent modification to resource detected. (code 379)

2017-11-29 Thread GitBox
mrutkows commented on issue #651: Concurrency error: Error code: 153: Concurrent modification to resource detected. (code 379) URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/651#issuecomment-347938756 @houshengbo @pritidesai I am going to start removing failing

[GitHub] drcariel opened a new pull request #5: Test fix

2017-11-29 Thread GitBox
drcariel opened a new pull request #5: Test fix URL: https://github.com/apache/incubator-openwhisk-package-deploy/pull/5 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] mrutkows commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations

2017-11-29 Thread GitBox
mrutkows commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations URL: https://github.com/apache/incubator-openwhisk/issues/3021#issuecomment-347918157 this is causing deja vu for me, saw this in MS Azure ~6 years ago

[GitHub] rabbah opened a new issue #3026: WskRestPackageTests 409 conflict related test failure

2017-11-29 Thread GitBox
rabbah opened a new issue #3026: WskRestPackageTests 409 conflict related test failure URL: https://github.com/apache/incubator-openwhisk/issues/3026 Failed in this [build](https://travis-ci.org/apache/incubator-openwhisk/builds/308993351) following couchd cluster enablement

[GitHub] rabbah commented on issue #3020: Docs: mac group_vars folder not exist, document need update

2017-11-29 Thread GitBox
rabbah commented on issue #3020: Docs: mac group_vars folder not exist, document need update URL: https://github.com/apache/incubator-openwhisk/issues/3020#issuecomment-347914003 Thanks for finding this - do you intend to submit a pr to correct the doc? We welcome contributions.

[GitHub] dgrove-oss opened a new issue #100: Docs: update to suggest using minikube

2017-11-29 Thread GitBox
dgrove-oss opened a new issue #100: Docs: update to suggest using minikube URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/100 we can run OpenWhisk in minikube now. Update

[GitHub] mrutkows commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations

2017-11-29 Thread GitBox
mrutkows commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations URL: https://github.com/apache/incubator-openwhisk/issues/3021#issuecomment-347918157 this is causing deja vu for me, saw this in MS Azure ~6 years ago

[GitHub] mdeuser commented on issue #3019: WIP: Trigger activation reduction

2017-11-29 Thread GitBox
mdeuser commented on issue #3019: WIP: Trigger activation reduction URL: https://github.com/apache/incubator-openwhisk/pull/3019#issuecomment-347939806 had a chat with @jasonpet. apparently many of the provider tests rely on the current behavior (i.e. fire trigger that has no rule, look

[GitHub] mrutkows commented on issue #651: Concurrency error: Error code: 153: Concurrent modification to resource detected. (code 379)

2017-11-29 Thread GitBox
mrutkows commented on issue #651: Concurrency error: Error code: 153: Concurrent modification to resource detected. (code 379) URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/651#issuecomment-347654572 and now on a different test... ``` === RUN

[GitHub] mrutkows commented on issue #651: Concurrency error: Error code: 153: Concurrent modification to resource detected. (code 379)

2017-11-29 Thread GitBox
mrutkows commented on issue #651: Concurrency error: Error code: 153: Concurrent modification to resource detected. (code 379) URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/651#issuecomment-347938756 @houshengbo @pritidesai I am going to start removing failing

[GitHub] mrutkows commented on issue #651: Concurrency error: Error code: 153: Concurrent modification to resource detected. (code 379)

2017-11-29 Thread GitBox
mrutkows commented on issue #651: Concurrency error: Error code: 153: Concurrent modification to resource detected. (code 379) URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/651#issuecomment-347938756 @houshengbo @pritidesai I am going to start removing failing

[GitHub] mrutkows commented on issue #651: Concurrency error: Error code: 153: Concurrent modification to resource detected. (code 379)

2017-11-29 Thread GitBox
mrutkows commented on issue #651: Concurrency error: Error code: 153: Concurrent modification to resource detected. (code 379) URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/651#issuecomment-347938756 @houshengbo @pritidesai I am going to start removing failing

[GitHub] Terry-Shi commented on issue #92: Error when list action

2017-11-28 Thread GitBox
Terry-Shi commented on issue #92: Error when list action URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/92#issuecomment-347733238 I suggest use specific version for the docker image in yaml file instead of "latest". I understand "latest" is a easy choice since

[GitHub] rabbah commented on issue #3021: Concurrency issue brought in by commit https://github.com/apache/incubator-openwhisk/commit/21d37a1af329544e245d376ad613d1a603fc104c

2017-11-28 Thread GitBox
rabbah commented on issue #3021: Concurrency issue brought in by commit https://github.com/apache/incubator-openwhisk/commit/21d37a1af329544e245d376ad613d1a603fc104c URL: https://github.com/apache/incubator-openwhisk/issues/3021#issuecomment-347739416 Can you clarify and add some details

[GitHub] rabbah opened a new issue #3022: services.KafkaConnectorTests.Kafka heisenbug

2017-11-28 Thread GitBox
rabbah opened a new issue #3022: services.KafkaConnectorTests.Kafka heisenbug URL: https://github.com/apache/incubator-openwhisk/issues/3022 Observed one failure in jenkins: ```bash services.KafkaConnectorTests.Kafka connector should send and receive a kafka message even after

[GitHub] rabbah commented on issue #2911: Proposal to to reorganize core/ and root(openwhisk/) folders to reflect existing architecture better

2017-11-28 Thread GitBox
rabbah commented on issue #2911: Proposal to to reorganize core/ and root(openwhisk/) folders to reflect existing architecture better URL: https://github.com/apache/incubator-openwhisk/issues/2911#issuecomment-347739972 Seems wrong to put a backend/server side deployment in a client repo.

[GitHub] rabbah commented on a change in pull request #3019: WIP: Trigger activation reduction

2017-11-28 Thread GitBox
rabbah commented on a change in pull request #3019: WIP: Trigger activation reduction URL: https://github.com/apache/incubator-openwhisk/pull/3019#discussion_r153685127 ## File path: core/controller/src/main/scala/whisk/core/controller/Triggers.scala ## @@ -143,71

[GitHub] mrutkows commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations

2017-11-29 Thread GitBox
mrutkows commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations URL: https://github.com/apache/incubator-openwhisk/issues/3021#issuecomment-347918157 this is causing deja vu for me, saw this in MS Azure 7 years ago

[GitHub] rabbah commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations

2017-11-29 Thread GitBox
rabbah commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations URL: https://github.com/apache/incubator-openwhisk/issues/3021#issuecomment-347920792 A point to reiterate here is that the CRUD operations are round robin

[GitHub] rabbah opened a new issue #3027: ActivationsApiTests fails with no results from list query

2017-11-29 Thread GitBox
rabbah opened a new issue #3027: ActivationsApiTests fails with no results from list query URL: https://github.com/apache/incubator-openwhisk/issues/3027 Observed in couple of Travis PR builds. ``` whisk.core.controller.test.ActivationsApiTests > Activations API should get

[GitHub] drcariel opened a new pull request #3028: More explicit executable call for Mac/Unix users

2017-11-29 Thread GitBox
drcariel opened a new pull request #3028: More explicit executable call for Mac/Unix users URL: https://github.com/apache/incubator-openwhisk/pull/3028 This is an automated message from the Apache Git Service. To respond to

[GitHub] cbickel closed pull request #2810: Add couchdb clustering

2017-11-29 Thread GitBox
cbickel closed pull request #2810: Add couchdb clustering URL: https://github.com/apache/incubator-openwhisk/pull/2810 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] starpit opened a new issue #89: socket hangup issues due to slow connection open

2017-11-29 Thread GitBox
starpit opened a new issue #89: socket hangup issues due to slow connection open URL: https://github.com/apache/incubator-openwhisk-client-js/issues/89 we inherit needle's default connection-open timeout of 10 seconds. this seems to result in socket hangup errors. i'm not sure what the

[GitHub] jessealva commented on issue #45: WIP: Adding support for path parameters for api gateway

2017-11-28 Thread GitBox
jessealva commented on issue #45: WIP: Adding support for path parameters for api gateway URL: https://github.com/apache/incubator-openwhisk-client-go/pull/45#issuecomment-347663320 @mdeuser Any other comments? This is

[GitHub] rabbah closed issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations

2017-11-30 Thread GitBox
rabbah closed issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations URL: https://github.com/apache/incubator-openwhisk/issues/3021 This is an automated message from

[GitHub] rabbah commented on issue #3032: HA controller leads to fail some tests

2017-11-30 Thread GitBox
rabbah commented on issue #3032: HA controller leads to fail some tests URL: https://github.com/apache/incubator-openwhisk/issues/3032#issuecomment-348059813 Thanks @devbv for the cross link - I think having a sticky session to route CRUD operations to the same controller will help

[GitHub] jasonpet commented on a change in pull request #6: import correction

2017-11-30 Thread GitBox
jasonpet commented on a change in pull request #6: import correction URL: https://github.com/apache/incubator-openwhisk-package-deploy/pull/6#discussion_r154141645 ## File path: tools/travis/build.sh ## @@ -1,9 +1,11 @@ #!/bin/bash +set -e + # Build script for

[GitHub] jasonpet commented on a change in pull request #84: make travis more efficient

2017-11-30 Thread GitBox
jasonpet commented on a change in pull request #84: make travis more efficient URL: https://github.com/apache/incubator-openwhisk-package-pushnotifications/pull/84#discussion_r154146118 ## File path: tools/travis/build.sh ## @@ -3,7 +3,7 @@ SCRIPTDIR=$(cd $(dirname

[GitHub] mrutkows commented on issue #651: Concurrency error: Error code: 153: Concurrent modification to resource detected. (code 379)

2017-11-30 Thread GitBox
mrutkows commented on issue #651: Concurrency error: Error code: 153: Concurrent modification to resource detected. (code 379) URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/651#issuecomment-347679613 3rd and 4th attempt: ``` ==> Executing: wskdeploy undeploy -p

[GitHub] drcariel opened a new pull request #84: make travis more efficient

2017-11-30 Thread GitBox
drcariel opened a new pull request #84: make travis more efficient URL: https://github.com/apache/incubator-openwhisk-package-pushnotifications/pull/84 This is an automated message from the Apache Git Service. To respond to

[GitHub] mrutkows opened a new issue #660: Message indicates "Deployment completed successfully" when actually it failed...

2017-11-30 Thread GitBox
mrutkows opened a new issue #660: Message indicates "Deployment completed successfully" when actually it failed... URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/660 Recently we have seen errors on "undeploy" (due to concurrency issues) in Travis. In several cases,

[GitHub] mrutkows commented on issue #651: Concurrency error: Error code: 153: Concurrent modification to resource detected. (code 379)

2017-11-30 Thread GitBox
mrutkows commented on issue #651: Concurrency error: Error code: 153: Concurrent modification to resource detected. (code 379) URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/651#issuecomment-348228770 @houshengbo @pritidesai Apparently, we are printing:

[GitHub] mrutkows commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations

2017-11-30 Thread GitBox
mrutkows commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations URL: https://github.com/apache/incubator-openwhisk/issues/3021#issuecomment-348264538 @rabbah thanks Rodric for your analysis and 3032 reference. Also, will

[GitHub] houshengbo opened a new pull request #661: Remove unnecessary rule changes

2017-11-30 Thread GitBox
houshengbo opened a new pull request #661: Remove unnecessary rule changes URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/661 A rule is enabled when it is created. There is no need to activate it. A rule is able to delete no matter it is active or inactive. This PR

[GitHub] mhenke1 opened a new issue #3034: Loglevel should be configurable via ansible

2017-11-30 Thread GitBox
mhenke1 opened a new issue #3034: Loglevel should be configurable via ansible URL: https://github.com/apache/incubator-openwhisk/issues/3034 The default log level is set to INFO level in logback.xml To make local debugging easier this level should be adaptable for each environment via

[GitHub] jasonpet commented on a change in pull request #6: import correction

2017-11-30 Thread GitBox
jasonpet commented on a change in pull request #6: import correction URL: https://github.com/apache/incubator-openwhisk-package-deploy/pull/6#discussion_r154131314 ## File path: tools/travis/build.sh ## @@ -43,11 +46,11 @@ WSK_CLI=$WHISKDIR/bin/wsk AUTH_KEY=$(cat

[GitHub] abaruni closed pull request #120: Support trigger update for feed triggers

2017-11-30 Thread GitBox
abaruni closed pull request #120: Support trigger update for feed triggers URL: https://github.com/apache/incubator-openwhisk-cli/pull/120 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] devbv opened a new issue #3032: HA controller leads to fail some tests

2017-11-30 Thread GitBox
devbv opened a new issue #3032: HA controller leads to fail some tests URL: https://github.com/apache/incubator-openwhisk/issues/3032 Since Nginx does proxy in Round-Robin way, some tests which were designed to run sequentially in one controller are failed. Let me explain one

[GitHub] abaruni opened a new pull request #120: Support trigger update for feed triggers

2017-11-30 Thread GitBox
abaruni opened a new pull request #120: Support trigger update for feed triggers URL: https://github.com/apache/incubator-openwhisk-cli/pull/120 This is an automated message from the Apache Git Service. To respond to the

[GitHub] akrabat opened a new pull request #5: Fix mistakes in README

2017-11-30 Thread GitBox
akrabat opened a new pull request #5: Fix mistakes in README URL: https://github.com/apache/incubator-openwhisk-runtime-python/pull/5 A couple of missed substitutions? This is an automated message from the Apache Git Service.

[GitHub] rabbah commented on issue #1815: Concurrent updates: race can easily result in "silent squash" failures

2017-11-30 Thread GitBox
rabbah commented on issue #1815: Concurrent updates: race can easily result in "silent squash" failures URL: https://github.com/apache/incubator-openwhisk/issues/1815#issuecomment-277845535 1-3 are all invasive. 4 is potentially lowest hanging fruit as we already have a lockable entry in

[GitHub] abaruni commented on issue #120: Support trigger update for feed triggers

2017-11-30 Thread GitBox
abaruni commented on issue #120: Support trigger update for feed triggers URL: https://github.com/apache/incubator-openwhisk-cli/pull/120#issuecomment-348244538 @jessealva will you take a look at this, please? This is an

[GitHub] abaruni commented on issue #120: Support trigger update for feed triggers

2017-11-30 Thread GitBox
abaruni commented on issue #120: Support trigger update for feed triggers URL: https://github.com/apache/incubator-openwhisk-cli/pull/120#issuecomment-348254998 @csantanapr can you have a look and merge? This is an automated

[GitHub] mrutkows opened a new issue #659: Add flag to suppress Warnings -warning false

2017-11-30 Thread GitBox
mrutkows opened a new issue #659: Add flag to suppress Warnings -warning false URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/659 my wish to "de-clutter" Travis build logs where we repeatedly report warnings pulling API keys from "whisk.properties"... ```

[GitHub] mrutkows commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations

2017-11-30 Thread GitBox
mrutkows commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations URL: https://github.com/apache/incubator-openwhisk/issues/3021#issuecomment-348264538 @rabbah thanks Rodric for your analysis and 3032 reference

[GitHub] jasonpet commented on a change in pull request #84: make travis more efficient

2017-11-30 Thread GitBox
jasonpet commented on a change in pull request #84: make travis more efficient URL: https://github.com/apache/incubator-openwhisk-package-pushnotifications/pull/84#discussion_r154157426 ## File path: .travis.yml ## @@ -18,10 +18,7 @@ notifications: install: true

[GitHub] dgrove-oss commented on issue #99: Enable pod/node affinity/anti-affinity in the deployment

2017-11-30 Thread GitBox
dgrove-oss commented on issue #99: Enable pod/node affinity/anti-affinity in the deployment URL: https://github.com/apache/incubator-openwhisk-deploy-kube/issues/99#issuecomment-348244815 This makes sense. We will pick up some of these deployment policies when we merge the daemon set

[GitHub] jasonpet commented on a change in pull request #84: make travis more efficient

2017-11-30 Thread GitBox
jasonpet commented on a change in pull request #84: make travis more efficient URL: https://github.com/apache/incubator-openwhisk-package-pushnotifications/pull/84#discussion_r154146118 ## File path: tools/travis/build.sh ## @@ -3,7 +3,7 @@ SCRIPTDIR=$(cd $(dirname

[GitHub] mrutkows commented on issue #651: Concurrency error: Error code: 153: Concurrent modification to resource detected. (code 379)

2017-11-30 Thread GitBox
mrutkows commented on issue #651: Concurrency error: Error code: 153: Concurrent modification to resource detected. (code 379) URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/651#issuecomment-348228590 So... with the first 3 int. tests commented out above, we had 2

[GitHub] mrutkows commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations

2017-11-30 Thread GitBox
mrutkows commented on issue #3021: Consider automatic server-side retry on concurrent modification of assets during CRUD operations URL: https://github.com/apache/incubator-openwhisk/issues/3021#issuecomment-348264538 @rabbah thanks Rodric for your analysis and 3032 reference. Also, will

[GitHub] mrutkows commented on issue #651: Concurrency error: Error code: 153: Concurrent modification to resource detected. (code 379)

2017-11-30 Thread GitBox
mrutkows commented on issue #651: Concurrency error: Error code: 153: Concurrent modification to resource detected. (code 379) URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/651#issuecomment-348255470 Retry approach being address here in PR

[GitHub] style95 commented on issue #3029: couchdb clustering not working

2017-11-29 Thread GitBox
style95 commented on issue #3029: couchdb clustering not working URL: https://github.com/apache/incubator-openwhisk/issues/3029#issuecomment-348065868 @csantanapr Are you using dns name in db section of `hosts` file? This is

[GitHub] style95 commented on issue #3029: couchdb clustering not working

2017-11-29 Thread GitBox
style95 commented on issue #3029: couchdb clustering not working URL: https://github.com/apache/incubator-openwhisk/issues/3029#issuecomment-348073217 I have discussed with @csantanapr. Actually there is limitation in #2810. With that change, we can't setup multiple couchdb nodes

[GitHub] style95 commented on issue #3029: couchdb clustering not working

2017-11-29 Thread GitBox
style95 commented on issue #3029: couchdb clustering not working URL: https://github.com/apache/incubator-openwhisk/issues/3029#issuecomment-348073217 I have discussed with @csantanapr. Actually there is limitation in #2810. With that change, we can't setup multiple couchdb nodes

[GitHub] csantanapr closed pull request #3030: attempt to fix couchdb clustering

2017-11-29 Thread GitBox
csantanapr closed pull request #3030: attempt to fix couchdb clustering URL: https://github.com/apache/incubator-openwhisk/pull/3030 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] csantanapr commented on issue #3030: attempt to fix couchdb clustering

2017-11-29 Thread GitBox
csantanapr commented on issue #3030: attempt to fix couchdb clustering URL: https://github.com/apache/incubator-openwhisk/pull/3030#issuecomment-348072256 Discussed with @style95 and couchdb clustering is not possible in single machine

[GitHub] joelmarty commented on issue #2237: web action should not return 200 status on action error

2017-11-30 Thread GitBox
joelmarty commented on issue #2237: web action should not return 200 status on action error URL: https://github.com/apache/incubator-openwhisk/issues/2237#issuecomment-348113404 I can report my experience rather than make a suggestion: I have a function, very simple, that returns a

[GitHub] jeremiaswerner opened a new pull request #3033: add some logs statements to the test to hunt heisenbug #3022

2017-11-30 Thread GitBox
jeremiaswerner opened a new pull request #3033: add some logs statements to the test to hunt heisenbug #3022 URL: https://github.com/apache/incubator-openwhisk/pull/3033 This is an automated message from the Apache Git

[GitHub] mhenke1 commented on a change in pull request #2990: Change log level dynamically

2017-11-30 Thread GitBox
mhenke1 commented on a change in pull request #2990: Change log level dynamically URL: https://github.com/apache/incubator-openwhisk/pull/2990#discussion_r154031014 ## File path: common/scala/src/main/scala/whisk/common/Logging.scala ## @@ -37,7 +37,11 @@ trait Logging {

[GitHub] abaruni commented on a change in pull request #229: verify trigger fire responds with any successful status code

2017-11-30 Thread GitBox
abaruni commented on a change in pull request #229: verify trigger fire responds with any successful status code URL: https://github.com/apache/incubator-openwhisk-package-kafka/pull/229#discussion_r154183037 ## File path: provider/consumer.py ## @@ -353,8 +353,12 @@ def

[GitHub] abaruni opened a new pull request #229: verify trigger fire responds with any successful status code

2017-11-30 Thread GitBox
abaruni opened a new pull request #229: verify trigger fire responds with any successful status code URL: https://github.com/apache/incubator-openwhisk-package-kafka/pull/229 This is an automated message from the Apache Git

[GitHub] rabbah commented on a change in pull request #2832: Do not cache invalidate when adding a DB attachment

2017-11-30 Thread GitBox
rabbah commented on a change in pull request #2832: Do not cache invalidate when adding a DB attachment URL: https://github.com/apache/incubator-openwhisk/pull/2832#discussion_r153965476 ## File path: common/scala/src/main/scala/whisk/core/database/DocumentFactory.scala

[GitHub] rabbah commented on a change in pull request #2832: Do not cache invalidate when adding a DB attachment

2017-11-30 Thread GitBox
rabbah commented on a change in pull request #2832: Do not cache invalidate when adding a DB attachment URL: https://github.com/apache/incubator-openwhisk/pull/2832#discussion_r153964961 ## File path: common/scala/src/main/scala/whisk/core/database/DocumentFactory.scala

[GitHub] rabbah commented on a change in pull request #2832: Do not cache invalidate when adding a DB attachment

2017-11-30 Thread GitBox
rabbah commented on a change in pull request #2832: Do not cache invalidate when adding a DB attachment URL: https://github.com/apache/incubator-openwhisk/pull/2832#discussion_r153965091 ## File path: common/scala/src/main/scala/whisk/core/entity/WhiskAction.scala ## @@

[GitHub] mrutkows commented on a change in pull request #46: Add retry support to every request

2017-11-30 Thread GitBox
mrutkows commented on a change in pull request #46: Add retry support to every request URL: https://github.com/apache/incubator-openwhisk-client-go/pull/46#discussion_r154175005 ## File path: whisk/client.go ## @@ -316,98 +319,118 @@ func BodyTruncator(body

[GitHub] rabbah commented on a change in pull request #46: Add retry support to every request

2017-11-30 Thread GitBox
rabbah commented on a change in pull request #46: Add retry support to every request URL: https://github.com/apache/incubator-openwhisk-client-go/pull/46#discussion_r154176550 ## File path: whisk/client.go ## @@ -316,98 +319,118 @@ func BodyTruncator(body io.ReadCloser)

[GitHub] rabbah commented on a change in pull request #46: Add retry support to every request

2017-11-30 Thread GitBox
rabbah commented on a change in pull request #46: Add retry support to every request URL: https://github.com/apache/incubator-openwhisk-client-go/pull/46#discussion_r154176550 ## File path: whisk/client.go ## @@ -316,98 +319,118 @@ func BodyTruncator(body io.ReadCloser)

<    3   4   5   6   7   8   9   10   11   12   >