csantanapr closed pull request #2875: Added support for configurable java
action method name.
URL: https://github.com/apache/incubator-openwhisk/pull/2875
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
csantanapr commented on issue #2875: Added support for configurable java action
method name.
URL:
https://github.com/apache/incubator-openwhisk/pull/2875#issuecomment-338359123
? PG 2 2195
This is an automated message from
rabbah closed pull request #2879: Update nodejs to 6.11.4
URL: https://github.com/apache/incubator-openwhisk/pull/2879
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
rabbah closed issue #2076: Update NodeJS 6 to latest 6.11.x LTS
URL: https://github.com/apache/incubator-openwhisk/issues/2076
This is an automated message from the Apache Git Service.
To respond to the message, please log
rabbah closed pull request #74: add publish to travis for kube-couchdb
URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/74
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
csantanapr commented on issue #2879: Update nodejs to 6.11.4
URL:
https://github.com/apache/incubator-openwhisk/pull/2879#issuecomment-338347312
? PG 4 915
This is an automated message from the Apache Git Service.
To respond
csantanapr closed pull request #2: Added support for configurable java action
method name
URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/2
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
csantanapr opened a new pull request #74: add publish to travis for kube-couchdb
URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/74
This is an automated message from the Apache Git Service.
To respond to
csantanapr opened a new pull request #2879: Update nodejs to 6.11.4
URL: https://github.com/apache/incubator-openwhisk/pull/2879
Closes #2076
This is an automated message from the Apache Git Service.
To respond to the
csantanapr commented on issue #1: test pr reviews
URL:
https://github.com/apache/incubator-openwhisk-runtime-java/pull/1#issuecomment-338344492
test i done
This is an automated message from the Apache Git Service.
To respond
csantanapr closed pull request #1: test pr reviews
URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/1
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
csantanapr closed issue #53: Expose health information to trigger owner
URL: https://github.com/apache/incubator-openwhisk-package-kafka/issues/53
This is an automated message from the Apache Git Service.
To respond to the
csantanapr commented on issue #53: Expose health information to trigger owner
URL:
https://github.com/apache/incubator-openwhisk-package-kafka/issues/53#issuecomment-338338930
This is now implemented with the new lifecycle `READ` on the feed action
mrutkows opened a new issue #615: Support "raw-http" field on Action
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/615
Please add support for this (new) field (i.e., "raw-http") as an explicit
field (with dedicated unit tests) within yamlparser.go.
Although we
csantanapr closed pull request #76: Add type support for annotations and limits
URL: https://github.com/apache/incubator-openwhisk-client-js/pull/76
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:
houshengbo opened a new pull request #614: Add the support to read env variable
for package name
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/614
Closes: #610
This is an automated message from the Apache
mrutkows commented on issue #612: Allow use of "kind" for "runtime" field
(overloaded semantic) with warning
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/issues/612#issuecomment-338282439
This would mean supporting it in "yamlparser.go" within the Action struct
and
mrutkows commented on issue #612: Allow use of "kind" for "runtime" field
(overloaded semantic) with warning
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/issues/612#issuecomment-338282439
This would mean supporting it in "yamlparser.go" within the Action struct
and
mrutkows opened a new issue #613: Test file not being used
"deployment_data_action_with_inputs_outputs.yaml"
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/613
The test file: tests/dat/deployment_data_action_with_inputs_outputs.yaml
does not parse correctly (not
rabbah commented on issue #62: added redis host for invoker
URL:
https://github.com/apache/incubator-openwhisk-devtools/pull/62#issuecomment-338279030
https://github.com/apache/incubator-openwhisk/commit/45e392e13fc7cd659e2a29b20ab070288412e200
rabbah closed pull request #2871: Invokers only require Redis if using dynamic
Id assignment
URL: https://github.com/apache/incubator-openwhisk/pull/2871
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
rabbah closed issue #79: Add tests to improve coverage of error handling
URL: https://github.com/apache/incubator-openwhisk-client-js/issues/79
This is an automated message from the Apache Git Service.
To respond to the
rabbah closed pull request #78: switch from request-promise to needle
URL: https://github.com/apache/incubator-openwhisk-client-js/pull/78
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
rabbah closed issue #77: consider switching from request-promise to needle
URL: https://github.com/apache/incubator-openwhisk-client-js/issues/77
This is an automated message from the Apache Git Service.
To respond to the
mrutkows opened a new issue #612: Allow use of "kind" for "runtime" field
(overloaded semantic) with warning
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/612
Note: in the spec., we made a decision to use the term "runtime" instead of
"kind" as used on the CLI as
dubeejw commented on issue #2873: Add documentation for save action feature
URL:
https://github.com/apache/incubator-openwhisk/pull/2873#issuecomment-338275026
@jpkessle, thanks for the feedback!
This is an automated message
dubeejw commented on issue #2873: Add documentation for save action feature
URL:
https://github.com/apache/incubator-openwhisk/pull/2873#issuecomment-338275156
@csantanapr, I made updates based on comments.
This is an
mrutkows opened a new issue #611: Action flag definitions, where are they used?
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/611
utils/flags.go
where are these flags used?
```
//action flag definition
//from go cli
action struct {
jpkessle commented on a change in pull request #2873: Add documentation for
save action feature
URL:
https://github.com/apache/incubator-openwhisk/pull/2873#discussion_r146023687
##
File path: docs/actions.md
##
@@ -286,6 +286,26 @@
jpkessle commented on a change in pull request #2873: Add documentation for
save action feature
URL:
https://github.com/apache/incubator-openwhisk/pull/2873#discussion_r146023403
##
File path: docs/actions.md
##
@@ -286,6 +286,26 @@
ddragosd commented on issue #62: added redis host for invoker
URL:
https://github.com/apache/incubator-openwhisk-devtools/pull/62#issuecomment-338272027
This is awesome @rabbah.
This is an automated message from the Apache
ddragosd commented on a change in pull request #62: added redis host for invoker
URL:
https://github.com/apache/incubator-openwhisk-devtools/pull/62#discussion_r146022911
##
File path: docker-compose/docker-compose.yml
##
@@ -81,12 +82,15 @@ services:
-
jpkessle commented on a change in pull request #2873: Add documentation for
save action feature
URL:
https://github.com/apache/incubator-openwhisk/pull/2873#discussion_r146022636
##
File path: docs/actions.md
##
@@ -286,6 +286,26 @@
ddragosd commented on a change in pull request #62: added redis host for invoker
URL:
https://github.com/apache/incubator-openwhisk-devtools/pull/62#discussion_r146022141
##
File path: docker-compose/docker-compose.yml
##
@@ -73,6 +73,7 @@ services:
links:
-
jthomas commented on issue #78: switch from request-promise to needle
URL:
https://github.com/apache/incubator-openwhisk-client-js/pull/78#issuecomment-338270903
LGTM
This is an automated message from the Apache Git Service.
dubeejw closed pull request #2589: Add the fundamental framework of REST
invocation for test cases
URL: https://github.com/apache/incubator-openwhisk/pull/2589
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
rabbah commented on issue #62: added redis host for invoker
URL:
https://github.com/apache/incubator-openwhisk-devtools/pull/62#issuecomment-338258065
I?m going to merge a PR from @dgrove-oss that makes redis optional too
(upstream).
rabbah commented on issue #62: added redis host for invoker
URL:
https://github.com/apache/incubator-openwhisk-devtools/pull/62#issuecomment-338258065
I?m going to merge a PR from @groved-oss that makes redis optional too
(upstream).
tysonnorris commented on a change in pull request #62: added redis host for
invoker
URL:
https://github.com/apache/incubator-openwhisk-devtools/pull/62#discussion_r146010333
##
File path: docker-compose/docker-compose.yml
##
@@ -73,6 +73,7 @@ services:
links:
tysonnorris commented on a change in pull request #62: added redis host for
invoker
URL:
https://github.com/apache/incubator-openwhisk-devtools/pull/62#discussion_r146009799
##
File path: docker-compose/docker-compose.yml
##
@@ -81,12 +82,15 @@ services:
-
mrutkows commented on issue #2869: CLI should use filename as action name when
one is not provided
URL:
https://github.com/apache/incubator-openwhisk/issues/2869#issuecomment-338253570
@dubeejw @rabbah as long as the CLI (API) provide the "clean contract" to
the users... I agree with
mrutkows commented on issue #2869: CLI should use filename as action name when
one is not provided
URL:
https://github.com/apache/incubator-openwhisk/issues/2869#issuecomment-338253570
@dubeejw @rabbah as long as the CLI (API) provide the "clean contract" to
the users... I agree with
mrutkows commented on issue #2869: CLI should use filename as action name when
one is not provided
URL:
https://github.com/apache/incubator-openwhisk/issues/2869#issuecomment-338253570
@dubeejw @rabbah as long as the CLI (API) provide the "clean contract" to
the users... I agree with
codecov-io commented on issue #78: switch from request-promise to needle
URL:
https://github.com/apache/incubator-openwhisk-client-js/pull/78#issuecomment-338231607
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk-client-js/pull/78?src=pr=h1)
Report
> Merging
houshengbo commented on a change in pull request #2589: Add the fundamental
framework of REST invocation for test cases
URL:
https://github.com/apache/incubator-openwhisk/pull/2589#discussion_r145977283
##
File path: tests/src/test/scala/common/rest/WskRest.scala
##
@@
houshengbo commented on a change in pull request #2589: Add the fundamental
framework of REST invocation for test cases
URL:
https://github.com/apache/incubator-openwhisk/pull/2589#discussion_r145977283
##
File path: tests/src/test/scala/common/rest/WskRest.scala
##
@@
houshengbo commented on a change in pull request #2589: Add the fundamental
framework of REST invocation for test cases
URL:
https://github.com/apache/incubator-openwhisk/pull/2589#discussion_r145977283
##
File path: tests/src/test/scala/common/rest/WskRest.scala
##
@@
houshengbo commented on a change in pull request #2589: Add the fundamental
framework of REST invocation for test cases
URL:
https://github.com/apache/incubator-openwhisk/pull/2589#discussion_r145976126
##
File path: tests/src/test/scala/common/rest/WskRest.scala
##
@@
starpit commented on issue #79: No tests to cover error handling?
URL:
https://github.com/apache/incubator-openwhisk-client-js/issues/79#issuecomment-338193133
I see. Yes, there may be a test gap for eg requesting a non existing action,
package, etc. Or non-overwriting creates against an
cbickel opened a new pull request #2878: Streamingly read user-logs.
URL: https://github.com/apache/incubator-openwhisk/pull/2878
Reading the user logs with synchronous APIs is cumbersome and not very
efficient.
This uses akka-streams and its various connectors to read a user's log
jthomas commented on issue #77: consider switching from request-promise to
needle
URL:
https://github.com/apache/incubator-openwhisk-client-js/issues/77#issuecomment-338136986
:+1: on this. I had thought about moving away from `rp` before due to the
size. Hadn't realised it had
style95 commented on issue #2810: Add couchdb clustering
URL:
https://github.com/apache/incubator-openwhisk/pull/2810#issuecomment-338127383
@rabbah
Code is updated accordingly.
Regarding `q` value, default is `8`.
I think it would be a good starting point, how do you think?
markusthoemmes commented on a change in pull request #2877: Better error
controller message for authorization failure
URL:
https://github.com/apache/incubator-openwhisk/pull/2877#discussion_r145885089
##
File path:
53 matches
Mail list logo
