chetanmeh commented on issue #3722: Scala based admin tooling
URL:
https://github.com/apache/incubator-openwhisk/pull/3722#issuecomment-400200513
> will we be able to add tab completion to wskadmin-next
Currently Scallop does not support that. Would be an interesting feature to
add
cliffjansen opened a new pull request #31: Add new AMQP package
URL: https://github.com/apache/incubator-openwhisk-external-resources/pull/31
I would like to submit this AMQP 1.0 package (connection feed and associated
Action and Web Action) as an external OpenWhisk resource. Thanks.
chetanmeh commented on issue #3722: Scala based admin tooling
URL:
https://github.com/apache/incubator-openwhisk/pull/3722#issuecomment-400199062
> We should probably prune these error logs.
Those logs were coming because Http connection pool was not getting closed.
So closing them
chetanmeh commented on issue #3722: Scala based admin tooling
URL:
https://github.com/apache/incubator-openwhisk/pull/3722#issuecomment-400198899
> wskadmin] Error: File './wskadmin.conf' not found
This was happening because the [Spring Boot Launch Script][1] changes the
base workin
chetanmeh commented on issue #3722: Scala based admin tooling
URL:
https://github.com/apache/incubator-openwhisk/pull/3722#issuecomment-400197481
> Should we suppress the stack trace - in this case the error message is
useful enough. Perhaps a -d debug flag to emit the trace.
Makes
chetanmeh commented on a change in pull request #3722: Scala based admin tooling
URL:
https://github.com/apache/incubator-openwhisk/pull/3722#discussion_r198019824
##
File path: tools/admin/README-NEXT.md
##
@@ -0,0 +1,126 @@
+
+
+## Administrative Operations
+
+The `wskad
chetanmeh opened a new issue #3807: Generate config for wskadmin scala tooling
URL: https://github.com/apache/incubator-openwhisk/issues/3807
New Scala based wskadmin tooling (#3722) needs configuration to access the
DB details.
So for python based `wskadmin` script used to source t
codecov-io edited a comment on issue #3779: S3AttachmentStore
URL:
https://github.com/apache/incubator-openwhisk/pull/3779#issuecomment-398322147
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3779?src=pr&el=h1)
Report
> Merging
[#3779](https://codecov.io/gh/apache/
codecov-io edited a comment on issue #3779: S3AttachmentStore
URL:
https://github.com/apache/incubator-openwhisk/pull/3779#issuecomment-398322147
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3779?src=pr&el=h1)
Report
> Merging
[#3779](https://codecov.io/gh/apache/
ddragosd commented on issue #136: PureConfig updates
URL:
https://github.com/apache/incubator-openwhisk-devtools/pull/136#issuecomment-400171390
@markusthoemmes , @cbickel I'm asking for your help with this review since
you've worked on the PR in the upstream. Thanks in advance !
---
ddragosd opened a new pull request #136: PureConfig updates
URL: https://github.com/apache/incubator-openwhisk-devtools/pull/136
This PR updates `docker-compose` as per the changes in
https://github.com/apache/incubator-openwhisk/pull/3617
-
codecov-io edited a comment on issue #3725: Add Ruby2.5 runtime support
URL:
https://github.com/apache/incubator-openwhisk/pull/3725#issuecomment-397487604
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3725?src=pr&el=h1)
Report
> Merging
[#3725](https://codecov.io/
rabbah commented on issue #3725: Add Ruby2.5 runtime support
URL:
https://github.com/apache/incubator-openwhisk/pull/3725#issuecomment-400163945
@remore i squashed and rebased this to master to resolve a new conflicts
with docs i updated.
@csantanapr we'll need to document the examp
csantanapr opened a new issue #956: Docs incorrect for apigateway syntax
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/956
The docs for apigateway, contains the wrong syntax to specify method.
```
apis:
hello-world:
hello:
worl
csantanapr opened a new issue #955: api basePath error when it have multiple
segments
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/955
I tried to create an api declaring an apigateway syntax
```yaml
apis:
FunctionswithDataStarterKit:
Func
codecov-io edited a comment on issue #3685: Enable test code coverage
collection for containers
URL:
https://github.com/apache/incubator-openwhisk/pull/3685#issuecomment-390943297
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3685?src=pr&el=h1)
Report
> Merging
[#
rabbah commented on issue #3760: Add API gateway to redo.
URL:
https://github.com/apache/incubator-openwhisk/pull/3760#issuecomment-400136637
Thanks the PR passed now.
This is an automated message from the Apache Git Service
ddragosd closed pull request #135: docker-compose.yml requires docker engine
1.13+
URL: https://github.com/apache/incubator-openwhisk-devtools/pull/135
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 provenanc
codecov-io edited a comment 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&el=h1)
Report
> Merging
[#3760](https://codecov.io/gh/
codecov-io edited a comment 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&el=h1)
Report
> Merging
[#3760](https://codecov.io/gh/
codecov-io edited a comment on issue #3741: Cleaning up the test related
classes
URL:
https://github.com/apache/incubator-openwhisk/pull/3741#issuecomment-395872642
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3741?src=pr&el=h1)
Report
> Merging
[#3741](https://
csantanapr commented on issue #3760: Add API gateway to redo.
URL:
https://github.com/apache/incubator-openwhisk/pull/3760#issuecomment-400124603
I fixed controller image on dockerhub today
This is an automated message from
rabbah commented on issue #3760: Add API gateway to redo.
URL:
https://github.com/apache/incubator-openwhisk/pull/3760#issuecomment-400123812
@csantanapr the API gateway build may be failing because the openwhisk image
on dockerhub is several days old (@dgrove-oss observed similar issue on
csantanapr closed pull request #3801: add a doc for securing actions
URL: https://github.com/apache/incubator-openwhisk/pull/3801
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 forei
rabbah commented on issue #3722: Scala based admin tooling
URL:
https://github.com/apache/incubator-openwhisk/pull/3722#issuecomment-400120027
will we be able to add tab completion to wskadmin-next? (the wskadmin
version supports tab completion if you configure the bash shell properly.)
-
rabbah edited a comment on issue #3722: Scala based admin tooling
URL:
https://github.com/apache/incubator-openwhisk/pull/3722#issuecomment-400118480
Suggestion:
```
> ./bin/wskadmin-next
[wskadmin] Error: Subcommand required
```
should instead print help.
perhap
rabbah edited a comment on issue #3722: Scala based admin tooling
URL:
https://github.com/apache/incubator-openwhisk/pull/3722#issuecomment-400118480
Suggestion:
```
> ./bin/wskadmin-next
[wskadmin] Error: Subcommand required
```
should instead print help.
perhap
rabbah commented on issue #3722: Scala based admin tooling
URL:
https://github.com/apache/incubator-openwhisk/pull/3722#issuecomment-400119255
We should probably prune these error logs.
```
akka.stream.StreamTcpException: Tcp command
[Connect(172.17.0.1:5984,None,List(),Some(10 s
rabbah edited a comment on issue #3722: Scala based admin tooling
URL:
https://github.com/apache/incubator-openwhisk/pull/3722#issuecomment-400119119
```
> ./bin/wskadmin-next -c wskadmin.conf user get guest
[wskadmin] Error: File './wskadmin.conf' not found
> ./bin/wskadmin-next
rabbah commented on issue #3722: Scala based admin tooling
URL:
https://github.com/apache/incubator-openwhisk/pull/3722#issuecomment-400119119
```
> ./bin/wskadmin-next -c ./wskadmin.conf user get guest
[wskadmin] Error: File './wskadmin.conf' not found
> ./bin/wskadmin-next -c `
rabbah edited a comment on issue #3722: Scala based admin tooling
URL:
https://github.com/apache/incubator-openwhisk/pull/3722#issuecomment-400118726
Should we suppress the stack trace - in this case the error message is
useful enough. Perhaps a `-d` debug flag to emit the trace.
``
rabbah commented on issue #3722: Scala based admin tooling
URL:
https://github.com/apache/incubator-openwhisk/pull/3722#issuecomment-400118726
Should we suppress the stack trace?
```
> ./bin/wskadmin-next user get guest
[wskadmin] Error: Incomplete config. Provide application.c
rabbah edited a comment on issue #3722: Scala based admin tooling
URL:
https://github.com/apache/incubator-openwhisk/pull/3722#issuecomment-400118480
Suggestion:
```
> ./bin/wskadmin-next
[wskadmin] Error: Subcommand required
```
should instead print help.
perhap
rabbah commented on issue #3722: Scala based admin tooling
URL:
https://github.com/apache/incubator-openwhisk/pull/3722#issuecomment-400118480
Suggestion:
```
> ./bin/wskadmin-next
[wskadmin] Error: Subcommand required
```
should instead print help.
---
rabbah commented on a change in pull request #3722: Scala based admin tooling
URL:
https://github.com/apache/incubator-openwhisk/pull/3722#discussion_r197964162
##
File path: tools/admin/README-NEXT.md
##
@@ -0,0 +1,126 @@
+
+
+## Administrative Operations
+
+The `wskadmin
rabbah commented on a change in pull request #3722: Scala based admin tooling
URL:
https://github.com/apache/incubator-openwhisk/pull/3722#discussion_r197964015
##
File path: tools/admin/README-NEXT.md
##
@@ -0,0 +1,126 @@
+
+
+## Administrative Operations
+
+The `wskadmin
codecov-io edited a comment on issue #3806: add register and until to ansible
invoker pull runtime images tasks
URL:
https://github.com/apache/incubator-openwhisk/pull/3806#issuecomment-400113305
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3806?src=pr&el=h1)
Report
rabbah commented on a change in pull request #3806: add register and until to
ansible invoker pull runtime images tasks
URL:
https://github.com/apache/incubator-openwhisk/pull/3806#discussion_r197960649
##
File path: ansible/roles/controller/tasks/deploy.yml
##
@@ -13,10
codecov-io commented on issue #3806: add register and until to ansible invoker
pull runtime images tasks
URL:
https://github.com/apache/incubator-openwhisk/pull/3806#issuecomment-400113305
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3806?src=pr&el=h1)
Report
> Me
csantanapr opened a new pull request #3806: add register and until to ansible
invoker pull runtime images tasks
URL: https://github.com/apache/incubator-openwhisk/pull/3806
`retries` and `delays` in ansible only applies to with `until`
They are being ignored, this PR adds the `until` an
houshengbo closed pull request #205: Add the installation instruction to 0.9.0
release
URL: https://github.com/apache/incubator-openwhisk-release/pull/205
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 proven
csantanapr commented on a change in pull request #205: Add the installation
instruction to 0.9.0 release
URL:
https://github.com/apache/incubator-openwhisk-release/pull/205#discussion_r197931349
##
File path: releases/0.9.0-incubating/INSTALL.md
##
@@ -0,0 +1,188 @@
+
+
+
csantanapr commented on a change in pull request #205: Add the installation
instruction to 0.9.0 release
URL:
https://github.com/apache/incubator-openwhisk-release/pull/205#discussion_r197931258
##
File path: releases/0.9.0-incubating/INSTALL.md
##
@@ -0,0 +1,188 @@
+
+
+
codecov-io edited a comment on issue #3722: Scala based admin tooling
URL:
https://github.com/apache/incubator-openwhisk/pull/3722#issuecomment-394275882
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3722?src=pr&el=h1)
Report
> Merging
[#3722](https://codecov.io/gh
houshengbo opened a new pull request #205: Add the installation instruction to
0.9.0 release
URL: https://github.com/apache/incubator-openwhisk-release/pull/205
This PR adds the instruction of installtion for release 0.9.0,
adds the bin directory back to the released package, and picks u
rabbah edited a comment on issue #3804: Error in installing route management
actions under Ubuntu
URL:
https://github.com/apache/incubator-openwhisk/issues/3804#issuecomment-400041393
It runs after post deploy - so naturally if the bug is because the system is
not fully ready 3760 will su
rabbah commented on issue #3804: Error in installing route management actions
under Ubuntu
URL:
https://github.com/apache/incubator-openwhisk/issues/3804#issuecomment-400041393
It runs after post deploy - so naturally if the bug is a that’s not fully
ready 3760 will surface the issue in i
houshengbo commented on issue #3804: Error in installing route management
actions under Ubuntu
URL:
https://github.com/apache/incubator-openwhisk/issues/3804#issuecomment-400040656
I am not sure if this PR
https://github.com/apache/incubator-openwhisk/pull/3760 will resolve the issue
her
dgrove-oss commented on issue #234: Update for upstream move of version info to
PureConfig
URL:
https://github.com/apache/incubator-openwhisk-deploy-kube/pull/234#issuecomment-400034203
Note. This PR will not pass Travis until a new openwhisk/controller image
is published on DockerHub
-
dgrove-oss opened a new pull request #234: Update for upstream move of version
info to PureConfig
URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/234
Update Helm chart for upstream PR 3617 that eliminated
whisk_version name and moved the version number and date
to
rabbah edited a comment on issue #3805: Add retries to install route management
actions
URL:
https://github.com/apache/incubator-openwhisk/pull/3805#issuecomment-400031435
using ` + retries: "{{ docker.pull.retries }}" ` seems like the wrong knob
here.
also the post deploy playbook wh
rabbah commented on issue #3805: Add retries to install route management actions
URL:
https://github.com/apache/incubator-openwhisk/pull/3805#issuecomment-400031435
if there's a broken deployment wont this retry for ever?
also the post deploy playbook which installs the catalog will suff
houshengbo opened a new pull request #3805: Add retries to install route
management actions
URL: https://github.com/apache/incubator-openwhisk/pull/3805
## Description
## Related issue and scope
- [ ] I opened an issue to propose and discuss this change (#)
mhenke1 commented on a change in pull request #3788: Store and pass variant
data in AuthKey
URL:
https://github.com/apache/incubator-openwhisk/pull/3788#discussion_r197836711
##
File path: common/scala/src/main/scala/whisk/core/entity/GenericAuthKey.scala
##
@@ -0,0 +1,39
mhenke1 commented on a change in pull request #3788: Store and pass variant
data in AuthKey
URL:
https://github.com/apache/incubator-openwhisk/pull/3788#discussion_r197836576
##
File path:
tests/src/test/scala/whisk/core/containerpool/test/ContainerProxyTests.scala
##
@@
mhenke1 commented on a change in pull request #3788: Store and pass variant
data in AuthKey
URL:
https://github.com/apache/incubator-openwhisk/pull/3788#discussion_r197836178
##
File path: core/controller/src/main/scala/whisk/core/controller/Triggers.scala
##
@@ -387,15 +
markusthoemmes commented on a change in pull request #3788: Store and pass
variant data in AuthKey
URL:
https://github.com/apache/incubator-openwhisk/pull/3788#discussion_r197821706
##
File path: core/controller/src/main/scala/whisk/core/controller/Triggers.scala
##
@@ -3
markusthoemmes commented on a change in pull request #3788: Store and pass
variant data in AuthKey
URL:
https://github.com/apache/incubator-openwhisk/pull/3788#discussion_r197824106
##
File path:
tests/src/test/scala/whisk/core/containerpool/test/ContainerProxyTests.scala
###
markusthoemmes commented on a change in pull request #3788: Store and pass
variant data in AuthKey
URL:
https://github.com/apache/incubator-openwhisk/pull/3788#discussion_r197821428
##
File path: common/scala/src/main/scala/whisk/core/entity/GenericAuthKey.scala
##
@@ -0,
kpavel opened a new pull request #135: docker-compose.yml requires docker
engine 1.13+
URL: https://github.com/apache/incubator-openwhisk-devtools/pull/135
yml version '3' requires docker engine 1.13+
This is an automated mes
mhenke1 commented on a change in pull request #3788: Store and pass variant
data in AuthKey
URL:
https://github.com/apache/incubator-openwhisk/pull/3788#discussion_r197803866
##
File path:
tests/src/test/scala/whisk/core/containerpool/test/ContainerProxyTests.scala
##
@@
codecov-io edited a comment on issue #3778: Add documentation to the
loadbalancer.
URL:
https://github.com/apache/incubator-openwhisk/pull/3778#issuecomment-398187244
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3778?src=pr&el=h1)
Report
> Merging
[#3778](https:/
starpit opened a new issue #120: add `debug` calls?
URL: https://github.com/apache/incubator-openwhisk-client-js/issues/120
would it make sense to sprinkle the code with `debug` calls? personally, i
would find this helpful for debugging.
starpit closed issue #119: add support for retries
URL: https://github.com/apache/incubator-openwhisk-client-js/issues/119
This is an automated message from the Apache Git Service.
To respond to the message, please log on Git
starpit opened a new issue #175: in-code docs have hrefs to non-existant pages
URL: https://github.com/apache/incubator-openwhisk-package-cloudant/issues/175
e.g.
https://github.com/apache/incubator-openwhisk-package-cloudant/blob/master/actions/account-actions/read-database.js
which
mdeuser commented on a change in pull request #3778: Add documentation to the
loadbalancer.
URL:
https://github.com/apache/incubator-openwhisk/pull/3778#discussion_r197782166
##
File path:
core/controller/src/main/scala/whisk/core/loadBalancer/ShardingContainerPoolBalancer.scala
chetanmeh closed pull request #3802: Bump gradle and scoverage version, adjust
scoverage plumbing.
URL: https://github.com/apache/incubator-openwhisk/pull/3802
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 p
rabbah commented on issue #3804: Error in installing route management actions
under Ubuntu
URL:
https://github.com/apache/incubator-openwhisk/issues/3804#issuecomment-399923244
See https://github.com/apache/incubator-openwhisk/pull/3760
markusthoemmes closed pull request #3617: Move version information to
pureconfig and remove `whisk_version_name`.
URL: https://github.com/apache/incubator-openwhisk/pull/3617
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below fo
houshengbo commented on issue #3804: Error in installing route management
actions under Ubuntu
URL:
https://github.com/apache/incubator-openwhisk/issues/3804#issuecomment-399922093
This is a known issue, which may happen when running openwhisk.yml. There
could be a race condition issue wi
cbickel commented on issue #3617: Move version information to pureconfig and
remove `whisk_version_name`.
URL:
https://github.com/apache/incubator-openwhisk/pull/3617#issuecomment-399909977
PG5#455 🔵
This is an automated me
codecov-io edited a comment on issue #3802: Bump gradle and scoverage version,
adjust scoverage plumbing.
URL:
https://github.com/apache/incubator-openwhisk/pull/3802#issuecomment-399769052
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3802?src=pr&el=h1)
Report
> M
chetanmeh commented on a change in pull request #3802: Bump gradle and
scoverage version, adjust scoverage plumbing.
URL:
https://github.com/apache/incubator-openwhisk/pull/3802#discussion_r197732198
##
File path: build.gradle
##
@@ -21,7 +21,7 @@ buildscript {
}
chetanmeh commented on issue #3802: Bump gradle and scoverage version, adjust
scoverage plumbing.
URL:
https://github.com/apache/incubator-openwhisk/pull/3802#issuecomment-399888448
With switch to Gradle 4.8 we can remove the explicit [build scan plugin][1]
introduced via #3314 with `--sc
markusthoemmes commented on issue #3802: Bump gradle and scoverage version,
adjust scoverage plumbing.
URL:
https://github.com/apache/incubator-openwhisk/pull/3802#issuecomment-399879471
PG2 3303 🔵
This is an automated mess
codecov-io edited a comment on issue #3617: Move version information to
pureconfig and remove `whisk_version_name`.
URL:
https://github.com/apache/incubator-openwhisk/pull/3617#issuecomment-387315559
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/3617?src=pr&el=h1)
Rep
76 matches
Mail list logo