rabbah commented on issue #2605: experimental web apigw cleanup
URL:
https://github.com/apache/incubator-openwhisk/pull/2605#issuecomment-323252139
@drcariel applied `wip` label for you - what's left?
This is an automated
lzbj closed issue #295: wskdeploy -p finds manifest.yaml, but not manifest.yml
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/295
This is an automated message from the Apache Git Service.
To respond to
rabbah commented on issue #2630: JSON encoding is inconsistent for web actions
URL:
https://github.com/apache/incubator-openwhisk/issues/2630#issuecomment-323243833
suggested patch for patch+json:
```bash
diff --git
rabbah commented on issue #2630: JSON encoding is inconsistent for web actions
URL:
https://github.com/apache/incubator-openwhisk/issues/2630#issuecomment-323243833
suggested patch
```bash
diff --git
a/core/controller/src/main/scala/whisk/core/controller/WebActions.scala
rabbah commented on issue #2630: JSON encoding is inconsistent for web actions
URL:
https://github.com/apache/incubator-openwhisk/issues/2630#issuecomment-323243833
suggested patch
```bash
diff --git
a/core/controller/src/main/scala/whisk/core/controller/WebActions.scala
rabbah commented on issue #2630: JSON encoding is inconsistent for web actions
URL:
https://github.com/apache/incubator-openwhisk/issues/2630#issuecomment-323243833
suggested patch
```bash
diff --git
a/core/controller/src/main/scala/whisk/core/controller/WebActions.scala
rabbah commented on issue #2630: JSON encoding is inconsistent for web actions
URL:
https://github.com/apache/incubator-openwhisk/issues/2630#issuecomment-323243833
suggested patch
```bash
diff --git
a/core/controller/src/main/scala/whisk/core/controller/WebActions.scala
lzbj commented on a change in pull request #310: Support both manifest.yaml and
manifest.yml
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/pull/310#discussion_r133870776
##
File path: tests/src/integration/common/wskdeploy.go
##
@@ -18,38 +18,46 @@
rabbah commented on issue #2630: JSON encoding is inconsistent for web actions
URL:
https://github.com/apache/incubator-openwhisk/issues/2630#issuecomment-323243833
suggested patch
```bash
diff --git
a/core/controller/src/main/scala/whisk/core/controller/WebActions.scala
rabbah commented on issue #2630: JSON encoding is inconsistent for web actions
URL:
https://github.com/apache/incubator-openwhisk/issues/2630#issuecomment-323242708
`application/problem+json` is not a recognized type in akka-http
rabbah commented on issue #2638: Preserve rule status on update according to
#1840.
URL:
https://github.com/apache/incubator-openwhisk/pull/2638#issuecomment-323241869
LG but needs tests.
This is an automated message from
rabbah commented on issue #2642: Update webhook.
URL:
https://github.com/apache/incubator-openwhisk/pull/2642#issuecomment-323241600
@csantanapr confirmed this is working as desired.
This is an automated message from the
ningyougang commented on issue #2629: Nginx component doesn't support HA
URL:
https://github.com/apache/incubator-openwhisk/issues/2629#issuecomment-323235751
Currently, i know `couchdb` and `kafka` themself support cluster.
But in openwhisk, they don't support.
Has any plan to
ningyougang commented on issue #2631: Change controller cluster work mode
configurable
URL:
https://github.com/apache/incubator-openwhisk/pull/2631#issuecomment-323232132
this PR may has relation with below PR:
https://github.com/apache/incubator-openwhisk/pull/2624
rabbah commented on a change in pull request #2624: Add cache invalidation
between controllers
URL:
https://github.com/apache/incubator-openwhisk/pull/2624#discussion_r133848952
##
File path:
common/scala/src/main/scala/whisk/core/database/MultipleReadersSingleWriterCache.scala
rabbah commented on a change in pull request #2624: Add cache invalidation
between controllers
URL:
https://github.com/apache/incubator-openwhisk/pull/2624#discussion_r133848996
##
File path: common/scala/src/main/scala/whisk/core/entity/AuthKey.scala
##
@@ -39,6 +39,7
rabbah commented on a change in pull request #2624: Add cache invalidation
between controllers
URL:
https://github.com/apache/incubator-openwhisk/pull/2624#discussion_r133849127
##
File path: common/scala/src/main/scala/whisk/core/entity/CacheKey.scala
##
@@ -0,0 +1,37
rabbah commented on a change in pull request #2624: Add cache invalidation
between controllers
URL:
https://github.com/apache/incubator-openwhisk/pull/2624#discussion_r133849169
##
File path: common/scala/src/main/scala/whisk/core/entity/DocInfo.scala
##
@@ -83,6 +83,11
rabbah opened a new pull request #2643: Pretty print webaction JSON response.
URL: https://github.com/apache/incubator-openwhisk/pull/2643
Fixes #2635.
This is an automated message from the Apache Git Service.
To respond to
rabbah opened a new pull request #2642: Update webhook.
URL: https://github.com/apache/incubator-openwhisk/pull/2642
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub
mrutkows closed pull request #310: Support both manifest.yaml and manifest.yml
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/310
This is an automated message from the Apache Git Service.
To respond to
mrutkows commented on issue #310: Support both manifest.yaml and manifest.yml
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/pull/310#issuecomment-323195735
@lzbj @priti we have reason to believe that all the integration tests are
only running up until they get ready to
zachschultz opened a new issue #365: Bluemix Stage 1 errors creating triggers
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/365
Using the two blueprints
([Cloudant](https://github.com/zachschultz/owbp-cloudant-trigger) and
csantanapr closed pull request #2641: Install composer in its own layer
URL: https://github.com/apache/incubator-openwhisk/pull/2641
This is an automated message from the Apache Git Service.
To respond to the message,
houshengbo commented on a change in pull request #2592: Make the result of
response consistent for both Wsk CLI and REST
URL:
https://github.com/apache/incubator-openwhisk/pull/2592#discussion_r133816983
##
File path: tests/src/test/scala/common/rest/WskRest.scala
##
@@
houshengbo commented on a change in pull request #2592: Make the result of
response consistent for both Wsk CLI and REST
URL:
https://github.com/apache/incubator-openwhisk/pull/2592#discussion_r133816983
##
File path: tests/src/test/scala/common/rest/WskRest.scala
##
@@
houshengbo commented on a change in pull request #2592: Make the result of
response consistent for both Wsk CLI and REST
URL:
https://github.com/apache/incubator-openwhisk/pull/2592#discussion_r133816983
##
File path: tests/src/test/scala/common/rest/WskRest.scala
##
@@
houshengbo commented on a change in pull request #2592: Make the result of
response consistent for both Wsk CLI and REST
URL:
https://github.com/apache/incubator-openwhisk/pull/2592#discussion_r133816983
##
File path: tests/src/test/scala/common/rest/WskRest.scala
##
@@
houshengbo commented on a change in pull request #2592: Make the result of
response consistent for both Wsk CLI and REST
URL:
https://github.com/apache/incubator-openwhisk/pull/2592#discussion_r133816480
##
File path: tests/src/test/scala/common/rest/WskRest.scala
##
@@
drcariel commented on a change in pull request #2605: experimental web apigw
cleanup
URL:
https://github.com/apache/incubator-openwhisk/pull/2605#discussion_r133815612
##
File path: tests/src/test/scala/whisk/core/cli/test/WskWebActionsTests.scala
##
@@ -47,94 +47,14 @@
drcariel commented on a change in pull request #2605: experimental web apigw
cleanup
URL:
https://github.com/apache/incubator-openwhisk/pull/2605#discussion_r133815277
##
File path: tests/src/test/scala/whisk/core/cli/test/WskWebActionsTests.scala
##
@@ -47,94 +47,14 @@
rabbah commented on a change in pull request #2605: experimental web apigw
cleanup
URL:
https://github.com/apache/incubator-openwhisk/pull/2605#discussion_r133814687
##
File path: tests/src/test/scala/whisk/core/cli/test/WskWebActionsTests.scala
##
@@ -47,94 +47,14 @@
drcariel commented on issue #2605: experimental web apigw cleanup
URL:
https://github.com/apache/incubator-openwhisk/pull/2605#issuecomment-323175392
PG1~1936
This is an automated message from the Apache Git Service.
To
csantanapr commented on issue #2641: Install composer in its own layer
URL:
https://github.com/apache/incubator-openwhisk/pull/2641#issuecomment-323168449
The issue is hard to explain when docker tries to use a cache layer, for
some reason is not finding composer.
The only think I can
akrabat opened a new pull request #2641: Install composer in its own layer
URL: https://github.com/apache/incubator-openwhisk/pull/2641
Composer installation seems to fail sometimes in the CI build
environment. Move the installation of composer to a separate layer and
separate the
bjustin-ibm commented on issue #2640: activation write can delay HTTP response
for trigger fire
URL:
https://github.com/apache/incubator-openwhisk/issues/2640#issuecomment-323159586
FYI @abaruni
This is an automated
bjustin-ibm opened a new issue #2640: activation write can delay HTTP response
for trigger fire
URL: https://github.com/apache/incubator-openwhisk/issues/2640
I have a client (the [Kafka feed
provider](https://github.com/apache/incubator-openwhisk-package-kafka)
actually) which fires
dubeejw commented on issue #2544: Allow CLI to Save Code from Action
URL:
https://github.com/apache/incubator-openwhisk/pull/2544#issuecomment-323155591
PG4 698 ?
This is an automated message from the Apache Git Service.
csantanapr closed pull request #2637: Loosen formdata check.
URL: https://github.com/apache/incubator-openwhisk/pull/2637
This is an automated message from the Apache Git Service.
To respond to the message, please log on
csantanapr closed issue #2634: Form URL encoded parameters are not properly
processed by web actions
URL: https://github.com/apache/incubator-openwhisk/issues/2634
This is an automated message from the Apache Git Service.
mdeuser opened a new issue #255: Provide two swaggers to download admin vs.
consumer
URL: https://github.com/apache/incubator-openwhisk-apigateway/issues/255
Update the `/apis` endpoint to return the swagger in either the API
developer format or the API consumer format. Currently, the
mdeuser opened a new issue #2639: 'wsk api get' to return developer or consumer
swagger
URL: https://github.com/apache/incubator-openwhisk/issues/2639
Currently, the API swagger definition returned by the `wsk api get` command
is only useful for the API developer, not for the consumer of
mdeuser commented on issue #2591: User-Agent CLI/version header
URL:
https://github.com/apache/incubator-openwhisk/pull/2591#issuecomment-323140866
Fixes #2568
Fixes #2571
This is an automated message from the Apache
houshengbo closed pull request #364: Add ASF header to action testcase
javascript file.
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/364
This is an automated message from the Apache Git Service.
To
csantanapr closed pull request #98: Sync openwhisk
URL: https://github.com/apache/incubator-openwhisk-cli/pull/98
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and
mdeuser commented on a change in pull request #2591: User-Agent CLI/version
header
URL:
https://github.com/apache/incubator-openwhisk/pull/2591#discussion_r133774090
##
File path: docs/rest_api.md
##
@@ -80,11 +80,14 @@ wsk namespace list -v
```
```
REQUEST:
-[GET]
mrutkows opened a new pull request #364: Add ASF header to action testcase
javascript file.
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/364
This is an automated message from the Apache Git Service.
To
drcariel commented on a change in pull request #2591: User-Agent CLI/version
header
URL:
https://github.com/apache/incubator-openwhisk/pull/2591#discussion_r133768141
##
File path: docs/rest_api.md
##
@@ -80,11 +80,14 @@ wsk namespace list -v
```
```
REQUEST:
-[GET]
mrutkows commented on issue #2575: Use Configuration File with Scan Code and
Add Missing License Header
URL:
https://github.com/apache/incubator-openwhisk/pull/2575#issuecomment-323124331
@bjustin-ibm @dubeejw The reason the ASF header was not found by scancode
was that the default .cfg
csantanapr commented on issue #2637: Loosen formdata check.
URL:
https://github.com/apache/incubator-openwhisk/pull/2637#issuecomment-323123658
@dubeejw could you review
This is an automated message from the Apache Git
markusthoemmes commented on a change in pull request #2592: Make the result of
response consistent for both Wsk CLI and REST
URL:
https://github.com/apache/incubator-openwhisk/pull/2592#discussion_r133761914
##
File path: tests/src/test/scala/common/WskTestHelpers.scala
markusthoemmes commented on a change in pull request #2592: Make the result of
response consistent for both Wsk CLI and REST
URL:
https://github.com/apache/incubator-openwhisk/pull/2592#discussion_r133761802
##
File path: tests/src/test/scala/common/rest/WskRest.scala
##
kevinlai opened a new pull request #2638: Preserve rule status on update
according to #1840.
URL: https://github.com/apache/incubator-openwhisk/pull/2638
Resolves #1840
Ran PG (#695)
This is an automated message from
rabbah opened a new pull request #2637: Loosen formdata check.
URL: https://github.com/apache/incubator-openwhisk/pull/2637
Fixes #2634.
This is an automated message from the Apache Git Service.
To respond to the message,
mrutkows opened a new pull request #19: Consolidate all unit tests that are
supposed to fail under exclude dir.
URL: https://github.com/apache/incubator-openwhisk-utilities/pull/19
This is an automated message from the
houshengbo commented on a change in pull request #2592: Make the result of
response consistent for both Wsk CLI and REST
URL:
https://github.com/apache/incubator-openwhisk/pull/2592#discussion_r133757960
##
File path: tests/src/test/scala/common/rest/WskRest.scala
##
@@
houshengbo commented on a change in pull request #2592: Make the result of
response consistent for both Wsk CLI and REST
URL:
https://github.com/apache/incubator-openwhisk/pull/2592#discussion_r133757169
##
File path: tests/src/test/scala/common/WskTestHelpers.scala
##
mrutkows commented on issue #33: REVIEW: update Travis scripts to conform to
other repos. and add scancode.
URL:
https://github.com/apache/incubator-openwhisk-deploy-kube/pull/33#issuecomment-323080496
@ddragosd dragos No, the failure is well past scancode utility running,
which if you
mrutkows closed pull request #33: REVIEW: update Travis scripts to conform to
other repos. and add scancode.
URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/33
This is an automated message from the
mrutkows commented on issue #33: REVIEW: update Travis scripts to conform to
other repos. and add scancode.
URL:
https://github.com/apache/incubator-openwhisk-deploy-kube/pull/33#issuecomment-323115825
@ddragosd started over in:
houshengbo closed pull request #363: Add missing ASF license headers to 2
integration test java files.
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/363
This is an automated message from the Apache Git
rabbah opened a new issue #2635: webaction json response no longer pretty
printed
URL: https://github.com/apache/incubator-openwhisk/issues/2635
```
> curl -k https://guest.openwhisk/default/echo.json
houshengbo closed pull request #97: bump to gradle4
URL: https://github.com/apache/incubator-openwhisk-cli/pull/97
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub
markusthoemmes closed issue #2611: trigger posting post akka port generates
errors
URL: https://github.com/apache/incubator-openwhisk/issues/2611
This is an automated message from the Apache Git Service.
To respond to the
markusthoemmes closed pull request #2613: Patch posting of trigger activation.
URL: https://github.com/apache/incubator-openwhisk/pull/2613
This is an automated message from the Apache Git Service.
To respond to the
markusthoemmes commented on a change in pull request #2592: Make the result of
response consistent for both Wsk CLI and REST
URL:
https://github.com/apache/incubator-openwhisk/pull/2592#discussion_r133751452
##
File path: tests/src/test/scala/common/WskTestHelpers.scala
houshengbo closed pull request #49: run travis files from propert directory;add
scancode again.
URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/49
This is an automated message from the Apache Git
markusthoemmes commented on a change in pull request #2592: Make the result of
response consistent for both Wsk CLI and REST
URL:
https://github.com/apache/incubator-openwhisk/pull/2592#discussion_r133750881
##
File path: tests/src/test/scala/common/rest/WskRest.scala
##
mrutkows opened a new pull request #363: Add missing ASF license headers to 2
integration test java files.
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/363
This is an automated message from the Apache
zhy989 commented on issue #210: How to use message feed
URL:
https://github.com/apache/incubator-openwhisk-package-kafka/issues/210#issuecomment-323106472
@bjustin-ibm thanks!
I have tried to set the LOCAL_DEV environment variable to true, it works
well without modify code, thanks!
mrutkows commented on issue #314: Pull the openwhisk images instead of building
them in Travis
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/pull/314#issuecomment-323104074
@houshengbo @daisy-ycguo thank you very much!
cjelger opened a new issue #2634: Form URL encoded parameters are not properly
processed by web actions
URL: https://github.com/apache/incubator-openwhisk/issues/2634
On a local vagrant machine, when POSTing some `form-urlencoded` parameters
to a web action, the parameters are NOT added
akrabat commented on issue #2630: JSON encoding is inconsistent for web actions
URL:
https://github.com/apache/incubator-openwhisk/issues/2630#issuecomment-323099171
Bluemix results for comparison (only base64 encode tests):
```php
base64_encode(json_encode(["greeting" =>
mrutkows commented on issue #49: run travis files from propert directory;add
scancode again.
URL:
https://github.com/apache/incubator-openwhisk-deploy-kube/pull/49#issuecomment-323098282
These changed were originally submitted and arpproved on
houshengbo opened a new pull request #98: Sync openwhisk
URL: https://github.com/apache/incubator-openwhisk-cli/pull/98
This is an automated message from the Apache Git Service.
To respond to the message, please log on
bjustin-ibm commented on issue #210: How to use message feed
URL:
https://github.com/apache/incubator-openwhisk-package-kafka/issues/210#issuecomment-323093897
@zhy989 ah very nice! Any easier way to handle this situation is to set the
`LOCAL_DEV` environment variable to `true` when
mrutkows closed pull request #12: Update to ASF license; adjust README.
URL: https://github.com/apache/incubator-openwhisk-package-template/pull/12
This is an automated message from the Apache Git Service.
To respond to the
mrutkows opened a new pull request #49: run travis files from propert
directory;add scancode again.
URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/49
This is an automated message from the Apache Git
zhy989 closed issue #210: How to use message feed
URL: https://github.com/apache/incubator-openwhisk-package-kafka/issues/210
This is an automated message from the Apache Git Service.
To respond to the message, please log
zhy989 commented on issue #210: How to use message feed
URL:
https://github.com/apache/incubator-openwhisk-package-kafka/issues/210#issuecomment-323091776
@bjustin-ibm thanks?
The action can be invoked successfully when I set verify to False from line
"response =
csantanapr closed pull request #27: Allow CLI to sort entities by name (#2326)
URL: https://github.com/apache/incubator-openwhisk-client-go/pull/27
This is an automated message from the Apache Git Service.
To respond to the
houshengbo commented on issue #33: REVIEW: update Travis scripts to conform to
other repos. and add scancode.
URL:
https://github.com/apache/incubator-openwhisk-deploy-kube/pull/33#issuecomment-323083387
This PR needs a rebase, since there is a conflict.
dragos commented on issue #33: REVIEW: update Travis scripts to conform to
other repos. and add scancode.
URL:
https://github.com/apache/incubator-openwhisk-deploy-kube/pull/33#issuecomment-323082584
/cc @ddragosd
This is
mrutkows commented on issue #33: REVIEW: update Travis scripts to conform to
other repos. and add scancode.
URL:
https://github.com/apache/incubator-openwhisk-deploy-kube/pull/33#issuecomment-323080496
@dragos No, the failure is well past scancode utility running, which if you
look in
bjustin-ibm commented on a change in pull request #2429: adding param for
install catalog
URL:
https://github.com/apache/incubator-openwhisk/pull/2429#discussion_r133716376
##
File path: ansible/tasks/installOpenwhiskCatalog.yml
##
@@ -5,6 +5,7 @@
bjustin-ibm commented on a change in pull request #2429: adding param for
install catalog
URL:
https://github.com/apache/incubator-openwhisk/pull/2429#discussion_r133715935
##
File path: ansible/tasks/installOpenwhiskCatalog.yml
##
@@ -29,6 +30,6 @@
version: "{{
markusthoemmes closed pull request #2616: Add retry to rate-throttle tests
becasue of the possiblility, that it?
URL: https://github.com/apache/incubator-openwhisk/pull/2616
This is an automated message from the Apache Git
bjustin-ibm commented on issue #210: How to use message feed
URL:
https://github.com/apache/incubator-openwhisk-package-kafka/issues/210#issuecomment-323073893
@zhy989 First verify that the message was indeed produced. Yes, the action
_should_ fail if the produce was not successful, but
rabbah commented on issue #2072: CacheConcurrencyTests: tie number of threads
to number of available CPUs instead of hard-coded value.
URL:
https://github.com/apache/incubator-openwhisk/pull/2072#issuecomment-323073223
leave well enough alone.
rabbah closed pull request #2072: CacheConcurrencyTests: tie number of threads
to number of available CPUs instead of hard-coded value.
URL: https://github.com/apache/incubator-openwhisk/pull/2072
This is an automated
cbickel closed pull request #2527: Allow multiple controllers and invokers in
local environments.
URL: https://github.com/apache/incubator-openwhisk/pull/2527
This is an automated message from the Apache Git Service.
To
cbickel opened a new pull request #2633: Allow multiple controllers and
invokers in local environments
URL: https://github.com/apache/incubator-openwhisk/pull/2633
Signed-off-by: Markus Thoemmes
This allows deploying multiple controllers and invokers
markusthoemmes commented on issue #2072: CacheConcurrencyTests: tie number of
threads to number of available CPUs instead of hard-coded value.
URL:
https://github.com/apache/incubator-openwhisk/pull/2072#issuecomment-323071948
@rabbah same question to you ? . Haven't seen that test fail
markusthoemmes commented on issue #2137: Make action container log reading more
robust
URL:
https://github.com/apache/incubator-openwhisk/pull/2137#issuecomment-323071511
@sven-lange-last want to port this to the new folderstructure?
markusthoemmes commented on issue #2060: allow concurrent activation processing
URL:
https://github.com/apache/incubator-openwhisk/pull/2060#issuecomment-323071330
@tysonnorris the code this is building upon got removed entirely. How do you
want to proceed with this PR?
mdeuser commented on issue #28: Remove the dependency on test cases in
openwhisk CLI
URL:
https://github.com/apache/incubator-openwhisk-client-go/pull/28#issuecomment-323050179
@houshengbo - with this change are there any CLI tests run in travis?
/cc @csantanapr @dubeejw
cbickel commented on a change in pull request #2527: Allow multiple controllers
and invokers in local environments.
URL:
https://github.com/apache/incubator-openwhisk/pull/2527#discussion_r133691667
##
File path: ansible/logs.yml
##
@@ -24,13 +24,14 @@
-
markusthoemmes commented on a change in pull request #2527: Allow multiple
controllers and invokers in local environments.
URL:
https://github.com/apache/incubator-openwhisk/pull/2527#discussion_r133686472
##
File path: ansible/logs.yml
##
@@ -24,13 +24,14 @@
-
cbickel commented on a change in pull request #2527: Allow multiple controllers
and invokers in local environments.
URL:
https://github.com/apache/incubator-openwhisk/pull/2527#discussion_r133685662
##
File path: ansible/logs.yml
##
@@ -24,13 +24,14 @@
-
markusthoemmes commented on a change in pull request #2527: Allow multiple
controllers and invokers in local environments.
URL:
https://github.com/apache/incubator-openwhisk/pull/2527#discussion_r133685440
##
File path: ansible/logs.yml
##
@@ -24,13 +24,14 @@
-
1 - 100 of 108 matches
Mail list logo