dubee commented on a change in pull request #319: Refactor action invoke
functions
URL:
https://github.com/apache/incubator-openwhisk-cli/pull/319#discussion_r195306011
##
File path: commands/action.go
##
@@ -172,65 +171,79 @@ var actionInvokeCmd = {
dubee closed pull request #324: Git pre-commit hook to format src.
URL: https://github.com/apache/incubator-openwhisk-cli/pull/324
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
chetanmeh closed pull request #3453: Introduce a AttachmentStore SPI
URL: https://github.com/apache/incubator-openwhisk/pull/3453
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
Ahmed-elsayed-mahmoud commented on issue #3755: Change the load balancer to
reduce invoker activation queueing
URL:
https://github.com/apache/incubator-openwhisk/issues/3755#issuecomment-397141007
@markusthoemmes Thanks for your comment! I have already finished
implementation of this
Ahmed-elsayed-mahmoud commented on issue #3755: Change the load balancer to
reduce invoker activation queueing
URL:
https://github.com/apache/incubator-openwhisk/issues/3755#issuecomment-397141007
@markusthoemmes Thanks for your comment. I have already finished
implementation of this
rabbah commented on issue #315: Refactor trigger create using new utility
methods to factor out printing of the feed action result
URL:
https://github.com/apache/incubator-openwhisk-cli/pull/315#issuecomment-396104723
@csantanapr @dubee @houshengbo @mdeuser @pritidesai may I get one of
rabbah commented on issue #324: Git pre-commit hook to format src.
URL:
https://github.com/apache/incubator-openwhisk-cli/pull/324#issuecomment-397138548
@dubee @mdeuser if you're ok with the PR can you approve and merge?
rabbah commented on issue #3763: api gateway example not consistent with
observed behavior
URL:
https://github.com/apache/incubator-openwhisk/issues/3763#issuecomment-397136640
Is creating the API more than once updating the API? I restarted the API
gateway and then followed the steps
codecov-io commented on issue #3760: Add API gateway to redo.
URL:
https://github.com/apache/incubator-openwhisk/pull/3760#issuecomment-397050317
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3760?src=pr=h1)
Report
> Merging
rabbah commented on issue #3763: api gateway example not consistent with
observed behavior
URL:
https://github.com/apache/incubator-openwhisk/issues/3763#issuecomment-397135371
continuing with the example to create an API from swagger where I exported
the swagger via `wsk api get`:
rabbah commented on issue #3760: Add API gateway to redo.
URL:
https://github.com/apache/incubator-openwhisk/pull/3760#issuecomment-397133154
@mdeuser can you take a look at this PR - I separated the routemgmt package
installation as discussed on Slack.
dgrove-oss opened a new pull request #222: replace with owdev in
minikube commands
URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/222
This is an automated message from the Apache Git Service.
To
codecov-io commented on issue #3680: Adjust invoker playbook to pull docker
images when a prefix and tag is specified.
URL:
https://github.com/apache/incubator-openwhisk/pull/3680#issuecomment-390292471
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3680?src=pr=h1)
codecov-io commented on issue #3680: Adjust invoker playbook to pull docker
images when a prefix and tag is specified.
URL:
https://github.com/apache/incubator-openwhisk/pull/3680#issuecomment-390292471
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3680?src=pr=h1)
rabbah opened a new issue #3762: postdeploy (install catalog) playbook does not
self clean
URL: https://github.com/apache/incubator-openwhisk/issues/3762
there is no `mode=clean` support for the postdeploy ansible playbook, so it
is not possible to uninstall the catalog easily.
csantanapr closed pull request #221: add retry loop to test invoke
URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/221
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
csantanapr closed pull request #219: minikube doc improvement: configuration
commands
URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/219
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
ajayk opened a new pull request #59: Node6 and express updates
URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/59
* Node version updated to 6.14.3
* Express to 4.16.3
* body-parser 1.18.3
* request 2.87.0
* use const instead of var
rabbah commented on issue #3759: routemgmt.yml playbook will fail silently if
the wsk binary is missing
URL:
https://github.com/apache/incubator-openwhisk/issues/3759#issuecomment-397081341
@mdeuser why not put it in the api gateway playbook? Logically it won't work
without it.
rabbah commented on issue #3759: routemgmt.yml playbook will fail silently if
the wsk binary is missing
URL:
https://github.com/apache/incubator-openwhisk/issues/3759#issuecomment-397076761
@mdeuser there is a postdeploy playbook - it could also go in there; that
playbook installs the
rabbah commented on issue #3759: routemgmt.yml playbook will fail silently if
the wsk binary is missing
URL:
https://github.com/apache/incubator-openwhisk/issues/3759#issuecomment-397076761
@mdeuser there is a postdeploy playbook - it could also go in there; that
playbook installs the
codecov-io commented on issue #3760: Add API gateway to redo.
URL:
https://github.com/apache/incubator-openwhisk/pull/3760#issuecomment-397050317
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3760?src=pr=h1)
Report
> Merging
jasonpet closed pull request #3619: Provide an activation store SPI
URL: https://github.com/apache/incubator-openwhisk/pull/3619
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
dgrove-oss opened a new pull request #221: add retry loop to test invoke
URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/221
Since we don't have a reliably way to verify that the
controller has registered the invoker as healthy, add
a retry loop around the invoke
codecov-io commented on issue #3760: Add API gateway to redo.
URL:
https://github.com/apache/incubator-openwhisk/pull/3760#issuecomment-397050317
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3760?src=pr=h1)
Report
> Merging
markusthoemmes commented on a change in pull request #3661: Restrict allowed
namespaces when creating action of certain kinds
URL:
https://github.com/apache/incubator-openwhisk/pull/3661#discussion_r195190902
##
File path:
markusthoemmes commented on a change in pull request #3661: Restrict allowed
namespaces when creating action of certain kinds
URL:
https://github.com/apache/incubator-openwhisk/pull/3661#discussion_r195189983
##
File path:
markusthoemmes commented on a change in pull request #3661: Restrict allowed
namespaces when creating action of certain kinds
URL:
https://github.com/apache/incubator-openwhisk/pull/3661#discussion_r195189406
##
File path:
codecov-io commented on issue #3757: Introduce configurable action blob size
limit.
URL:
https://github.com/apache/incubator-openwhisk/pull/3757#issuecomment-397028913
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3757?src=pr=h1)
Report
> Merging
mrutkows closed pull request #184: Add the explanation to the PGP key
configuration
URL: https://github.com/apache/incubator-openwhisk-release/pull/184
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
mrutkows closed issue #88: Document "release bot"
URL: https://github.com/apache/incubator-openwhisk-release/issues/88
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub
codecov-io commented on issue #3453: Introduce a AttachmentStore SPI
URL:
https://github.com/apache/incubator-openwhisk/pull/3453#issuecomment-395695075
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3453?src=pr=h1)
Report
> Merging
codecov-io commented on issue #3453: Introduce a AttachmentStore SPI
URL:
https://github.com/apache/incubator-openwhisk/pull/3453#issuecomment-395695075
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3453?src=pr=h1)
Report
> Merging
rabbah closed pull request #3515: Introduce separate Akka dispatchers for
CouchDB and Kafka Clients (#2954)
URL: https://github.com/apache/incubator-openwhisk/pull/3515
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the
chetanmeh commented on a change in pull request #3453: Introduce a
AttachmentStore SPI
URL:
https://github.com/apache/incubator-openwhisk/pull/3453#discussion_r195156648
##
File path:
common/scala/src/main/scala/whisk/core/database/AttachmentStore.scala
##
@@ -0,0 +1,70
chetanmeh commented on a change in pull request #3453: Introduce a
AttachmentStore SPI
URL:
https://github.com/apache/incubator-openwhisk/pull/3453#discussion_r195155941
##
File path:
common/scala/src/main/scala/whisk/core/database/CouchDbRestStore.scala
##
@@ -357,22
dgrove-oss closed pull request #220: Add the mini headers to Dockerfile and tpl
file
URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/220
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
chetanmeh commented on a change in pull request #3453: Introduce a
AttachmentStore SPI
URL:
https://github.com/apache/incubator-openwhisk/pull/3453#discussion_r195149057
##
File path:
common/scala/src/main/scala/whisk/core/database/CouchDbRestStore.scala
##
@@ -52,7
houshengbo opened a new issue #185: Make sure files defined to include mini
header really include the mini header
URL: https://github.com/apache/incubator-openwhisk-release/issues/185
Some files have already included full header, but we can leave them as they
are for the first release.
markusthoemmes closed pull request #3761: Revert "Mitigate missing docker 1.12
binary download. (#3758)" as the cli is downloadbale again.
URL: https://github.com/apache/incubator-openwhisk/pull/3761
This is a PR merged from a forked repository.
As GitHub hides the original diff on
codecov-io commented on issue #3761: Revert "Mitigate missing docker 1.12
binary download. (#3758)" as the cli is downloadbale again.
URL:
https://github.com/apache/incubator-openwhisk/pull/3761#issuecomment-396993256
#
houshengbo opened a new pull request #220: Add the correct headers to
Dockerfile and tpl file
URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/220
This is an automated message from the Apache Git Service.
cbickel opened a new pull request #3761: Revert "Mitigate missing docker 1.12
binary download. (#3758)" as the cli is downloadbale again.
URL: https://github.com/apache/incubator-openwhisk/pull/3761
This reverts commit 008996d099a730a2179ead97504ab631d30f1a80.
##
rabbah commented on issue #3760: Add API gateway to redo.
URL:
https://github.com/apache/incubator-openwhisk/pull/3760#issuecomment-396976677
@mdeuser can you take a look?
This is an automated message from the Apache Git
rabbah opened a new pull request #3760: Add API gateway to redo.
URL: https://github.com/apache/incubator-openwhisk/pull/3760
API gateway is missing from `redo` aka `wskdev`. Add it and also separate
the `wsk` download into its own playbook. Fix bug in script which failed
silently if
rabbah opened a new issue #3759: API gateway playbook will fail silently if the
wsk binary is missing
URL: https://github.com/apache/incubator-openwhisk/issues/3759
If you manage to run the playbook (for example with mode=clean) with a
missing `wsk` binary the playbook succeeds even
dgrove-oss opened a new pull request #219: minikube doc improvement:
configuration commands
URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/219
use minikube config set to persist minikube configuration
across start/stop operations.
rabbah commented on a change in pull request #3453: Introduce a AttachmentStore
SPI
URL:
https://github.com/apache/incubator-openwhisk/pull/3453#discussion_r195099717
##
File path:
common/scala/src/main/scala/whisk/core/database/CouchDbRestStore.scala
##
@@ -52,7 +52,8
csantanapr commented on issue #3758: Mitigate build problem of invoker.
URL:
https://github.com/apache/incubator-openwhisk/pull/3758#issuecomment-396932952
I preferred to find a copy of the tgz (or make a new tgz) and put it some
where we own and use it from there.
markusthoemmes closed pull request #3758: Mitigate build problem of invoker.
URL: https://github.com/apache/incubator-openwhisk/pull/3758
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
rabbah commented on issue #3751: Allow invoker pools to overlap for small N.
URL:
https://github.com/apache/incubator-openwhisk/pull/3751#issuecomment-396924236
@tysonnorris @bbrowning FYI I think this change is generally benign but for
small deployments changes the pool allocation.
codecov-io commented on issue #3758: Mitigate build problem of invoker.
URL:
https://github.com/apache/incubator-openwhisk/pull/3758#issuecomment-396921916
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3758?src=pr=h1)
Report
> Merging
mhenke1 commented on a change in pull request #3752: Move uuid into namespace
to decouple it from authkey
URL:
https://github.com/apache/incubator-openwhisk/pull/3752#discussion_r195050604
##
File path: core/controller/src/main/scala/whisk/core/controller/Actions.scala
##
cbickel opened a new pull request #3758: Mitigate build problem of invoker.
URL: https://github.com/apache/incubator-openwhisk/pull/3758
As it is currently not possible to download the docker-cli on building the
invoker image, this PR should unblock travis again.
## Description
mhenke1 commented on a change in pull request #3752: Move uuid into namespace
to decouple it from authkey
URL:
https://github.com/apache/incubator-openwhisk/pull/3752#discussion_r195042158
##
File path: common/scala/src/main/scala/whisk/core/entity/Identity.scala
##
@@
mhenke1 commented on a change in pull request #3752: Move uuid into namespace
to decouple it from authkey
URL:
https://github.com/apache/incubator-openwhisk/pull/3752#discussion_r195042158
##
File path: common/scala/src/main/scala/whisk/core/entity/Identity.scala
##
@@
qiuhui commented on issue #133: create sample action fail?
URL:
https://github.com/apache/incubator-openwhisk-devtools/issues/133#issuecomment-396898529
@rabbah it works,Thanks!
This is an automated message from the Apache
qiuhui closed issue #133: create sample action fail?
URL: https://github.com/apache/incubator-openwhisk-devtools/issues/133
This is an automated message from the Apache Git Service.
To respond to the message, please log on
rabbah commented on issue #133: create sample action fail?
URL:
https://github.com/apache/incubator-openwhisk-devtools/issues/133#issuecomment-396897815
Use: wsk -i ...
This is an automated message from the Apache Git
qiuhui opened a new issue #133: create sample action fail?
URL: https://github.com/apache/incubator-openwhisk-devtools/issues/133
when I try to create action using follwing cmd:
`wsk action create hello hello.js`
it failed! the output is:
error: Unable to create action 'hello': Put
vooolll commented on issue #3173: incubator-openwhisk-2882 Delete printlns
URL:
https://github.com/apache/incubator-openwhisk/pull/3173#issuecomment-396871760
@cbickel @rabbah could you please review, I could continue with refactoring,
because there is repetitions in database cleanup, or
markusthoemmes commented on issue #3756: Report metrics of kafka's queue time.
URL:
https://github.com/apache/incubator-openwhisk/pull/3756#issuecomment-396851549
PG1 3017
This is an automated message from the Apache Git
tz70s opened a new pull request #3757: Introduce larger action blob size limit
configuration.
URL: https://github.com/apache/incubator-openwhisk/pull/3757
I think this pr still needs some improvements, but I need some feedback on
current implementation. It's better label this with WIP.
markusthoemmes opened a new pull request #3756: Report metrics of kafka's queue
time.
URL: https://github.com/apache/incubator-openwhisk/pull/3756
This adds reporting of the time a message resides in kafka from being
created to being read by a consumer. This is a very helpful metric for
markusthoemmes commented on issue #3755: Change the load balancer to reduce
invoker activation queueing
URL:
https://github.com/apache/incubator-openwhisk/issues/3755#issuecomment-396828788
Thanks for the proposal . What you're suggesting has indeed been discussed
on the dev list and
markusthoemmes commented on a change in pull request #3619: Provide an
activation store SPI
URL:
https://github.com/apache/incubator-openwhisk/pull/3619#discussion_r194967915
##
File path: common/scala/src/main/scala/whisk/core/entity/ActivationStore.scala
##
@@ -0,0
66 matches
Mail list logo
