ningyougang commented on issue #3432: Introduce jmx for kafka
URL:
https://github.com/apache/incubator-openwhisk/pull/3432#issuecomment-373277550
thanks.
regarding `controller/invoker`, when build, will copy `copyJMXFiles.sh` to
controller/invoker's image.
but regarding `kafka`, beca
ningyougang commented on issue #3442: Add -o option to overwrite target files
URL:
https://github.com/apache/incubator-openwhisk/pull/3442#issuecomment-373276248
In my local build env. the second build will be failed due to already exist
the files, the error like below
```
TASK [cli
ningyougang opened a new pull request #3442: Add -o option to overwrite target
files
URL: https://github.com/apache/incubator-openwhisk/pull/3442
If target directory already has unzip files, the unzip command will
give tip:[y]es, [n]o, [A]ll, [N]one, [r]ename, so it is better to
use t
dubee commented on a change in pull request #3421: Add ElasticSearch Log Store
URL:
https://github.com/apache/incubator-openwhisk/pull/3421#discussion_r174683082
##
File path:
common/scala/src/main/scala/whisk/core/containerpool/logging/ElasticSearchLogStore.scala
##
@@ -
dubee commented on a change in pull request #3421: Add ElasticSearch Log Store
URL:
https://github.com/apache/incubator-openwhisk/pull/3421#discussion_r174680770
##
File path:
common/scala/src/main/scala/whisk/core/containerpool/logging/ElasticSearchRestClient.scala
##
@@
dubee commented on a change in pull request #3421: Add ElasticSearch Log Store
URL:
https://github.com/apache/incubator-openwhisk/pull/3421#discussion_r174680770
##
File path:
common/scala/src/main/scala/whisk/core/containerpool/logging/ElasticSearchRestClient.scala
##
@@
csantanapr closed pull request #3441: Add swift41 settings gradle
URL: https://github.com/apache/incubator-openwhisk/pull/3441
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 p
dubee commented on a change in pull request #3421: Add ElasticSearch Log Store
URL:
https://github.com/apache/incubator-openwhisk/pull/3421#discussion_r174672462
##
File path:
common/scala/src/main/scala/whisk/core/containerpool/logging/ElasticSearchLogStore.scala
##
@@ -
dubee commented on a change in pull request #3421: Add ElasticSearch Log Store
URL:
https://github.com/apache/incubator-openwhisk/pull/3421#discussion_r174672462
##
File path:
common/scala/src/main/scala/whisk/core/containerpool/logging/ElasticSearchLogStore.scala
##
@@ -
dubee commented on a change in pull request #3421: Add ElasticSearch Log Store
URL:
https://github.com/apache/incubator-openwhisk/pull/3421#discussion_r174672435
##
File path:
common/scala/src/main/scala/whisk/core/containerpool/logging/ElasticSearchLogStore.scala
##
@@ -
jasonpet opened a new pull request #161: Add health monitoring support
URL: https://github.com/apache/incubator-openwhisk-package-cloudant/pull/161
Add optional time-triggered self monitor support to the cloudant provider.
If turned on the monitoring status can be viewed in the existing hea
csantanapr opened a new pull request #3441: Add swift41 settings gradle
URL: https://github.com/apache/incubator-openwhisk/pull/3441
include swift41 project in settings so it's picked up by distDocker
This is an automated mess
csantanapr closed pull request #3437: swagger generation update
URL: https://github.com/apache/incubator-openwhisk/pull/3437
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 pul
bwmcadams opened a new pull request #3440: Fixes Kubernetes logging issues from
#3439 re: activation logs where timestamp errors caused logging failures
URL: https://github.com/apache/incubator-openwhisk/pull/3440
## Description
- Fixed an issue where empty results from K8S log HTTP coul
codecov-io commented on issue #105: Allow string argument for action get API
URL:
https://github.com/apache/incubator-openwhisk-client-js/pull/105#issuecomment-373200813
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk-client-js/pull/105?src=pr&el=h1)
Report
> Merging
[#10
mrutkows closed issue #623: Support "blackbox" Docker images
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/623
This is an automated message from the Apache Git Service.
To respond to the message, please
mrutkows closed pull request #801: Adding Docker and Native support
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/801
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
mrutkows closed issue #279: Support new "native" (or "binary") kind
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/279
This is an automated message from the Apache Git Service.
To respond to the message,
dubeejw closed pull request #3430: Bump ansible version
URL: https://github.com/apache/incubator-openwhisk/pull/3430
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 reques
dubeejw commented on issue #105: Allow string argument for action get API
URL:
https://github.com/apache/incubator-openwhisk-client-js/pull/105#issuecomment-373207614
@jthomas, I think this is the fix you wanted. Take a look.
pritidesai commented on a change in pull request #801: Adding Docker and Native
support
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/pull/801#discussion_r174633038
##
File path: tests/src/integration/docker/actions/go/sample.go
##
@@ -0,0 +1,44 @@
+/*
R
pritidesai commented on a change in pull request #801: Adding Docker and Native
support
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/pull/801#discussion_r174633760
##
File path: parsers/manifest_parser_test.go
##
@@ -893,6 +896,42 @@ func TestComposeAction
pritidesai commented on a change in pull request #801: Adding Docker and Native
support
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/pull/801#discussion_r174633645
##
File path: tests/dat/manifest_data_compose_actions_for_docker.yaml
##
@@ -0,0 +1,39 @@
+#
codecov-io commented on issue #105: Allow string argument for action get API
URL:
https://github.com/apache/incubator-openwhisk-client-js/pull/105#issuecomment-373200813
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk-client-js/pull/105?src=pr&el=h1)
Report
> Merging
[#10
pritidesai commented on a change in pull request #801: Adding Docker and Native
support
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/pull/801#discussion_r174633038
##
File path: tests/src/integration/docker/actions/go/sample.go
##
@@ -0,0 +1,44 @@
+/*
R
dubeejw opened a new pull request #105: Allow string argument for action get API
URL: https://github.com/apache/incubator-openwhisk-client-js/pull/105
Restores original action get API that allows for an action name to be passed
to the get method as a string.
---
ddragosd commented on a change in pull request #801: Adding Docker and Native
support
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/pull/801#discussion_r174629587
##
File path: tests/dat/manifest_data_compose_actions_for_docker.yaml
##
@@ -0,0 +1,39 @@
+#
+
bwmcadams commented on issue #3439: KubernetesContainer doesn't properly track
the timestamp, leading to logging problems esp. under load
URL:
https://github.com/apache/incubator-openwhisk/issues/3439#issuecomment-373195534
I have a PR nearly read to address this issue.
--
mrutkows commented on a change in pull request #801: Adding Docker and Native
support
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/pull/801#discussion_r174624002
##
File path: tests/src/integration/docker/actions/go/sample.go
##
@@ -0,0 +1,44 @@
+/*
Rev
mrutkows closed pull request #64: Change the directory to generate the release
notes
URL: https://github.com/apache/incubator-openwhisk-release/pull/64
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
bwmcadams opened a new issue #3439: KubernetesContainer doesn't properly track
the timestamp, leading to logging problems esp. under load
URL: https://github.com/apache/incubator-openwhisk/issues/3439
## Environment details:
* OpenWhisk running on Kubernetes
* all versions
mrutkows commented on a change in pull request #801: Adding Docker and Native
support
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/pull/801#discussion_r174623013
##
File path: parsers/manifest_parser_test.go
##
@@ -893,6 +896,42 @@ func TestComposeActionsF
mrutkows commented on a change in pull request #801: Adding Docker and Native
support
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/pull/801#discussion_r174624002
##
File path: tests/src/integration/docker/actions/go/sample.go
##
@@ -0,0 +1,44 @@
+/*
Rev
mrutkows commented on a change in pull request #801: Adding Docker and Native
support
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/pull/801#discussion_r174623134
##
File path: parsers/yamlparser.go
##
@@ -90,24 +90,26 @@ type YAMLParser struct {
// Action
mrutkows commented on a change in pull request #801: Adding Docker and Native
support
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/pull/801#discussion_r174622768
##
File path: parsers/manifest_parser_test.go
##
@@ -893,6 +896,42 @@ func TestComposeActionsF
pritidesai opened a new issue #803: Report - Parameters in JSON form
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/803
Scott from watson has shown interest in spitting out JSON data structure
with all the `inputs` from every OW entities, something like this:
man
mrutkows commented on issue #790: Add support to export binary (zip and jar)
actions
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/pull/790#issuecomment-373180541
@kpavel Thanks!
This is an automated messa
pritidesai closed pull request #790: Add support to export binary (zip and jar)
actions
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/790
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 pro
dubeejw closed issue #3414: Add swift 4.1 runtime/kind to default set of
runtimes.
URL: https://github.com/apache/incubator-openwhisk/issues/3414
This is an automated message from the Apache Git Service.
To respond to the me
dubeejw closed pull request #3420: Add swift:4.1 with Codable to default set of
runtimes
URL: https://github.com/apache/incubator-openwhisk/pull/3420
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:
csantanapr commented on issue #3420: Add swift:4.1 with Codable to default set
of runtimes
URL:
https://github.com/apache/incubator-openwhisk/pull/3420#issuecomment-373159218
Playground1/2713 🔵
This is an automated message
dubeejw commented on issue #3420: Add swift:4.1 with Codable to default set of
runtimes
URL:
https://github.com/apache/incubator-openwhisk/pull/3420#issuecomment-373155006
PG1 2713 ⏳
This is an automated message from the Ap
pritidesai commented on issue #802: Resolves #730
URL:
https://github.com/apache/incubator-openwhisk-wskdeploy/pull/802#issuecomment-373153547
@kpavel please create a separate branch named `fix-for-730` instead of
proposing changes in your `master`, reviewing off of master branch is not
f
sciabarracom opened a new pull request #2: Releasing the revised Golang runtime
URL: https://github.com/apache/incubator-openwhisk-runtime-go/pull/2
notable changes:
- removed the handshake and completely revised the health check
- changed the protocol, now logs are stdout and stderr,
houshengbo opened a new pull request #64: Change the directory to generate the
release notes
URL: https://github.com/apache/incubator-openwhisk-release/pull/64
By default, the release notes file is saved under the $HOME or the WORK_DIR.
This
PR changes the directory to $OPENWHISK_SOURCE
sciabarracom closed pull request #1: Releasing the OpenWhisk Go Runtime
URL: https://github.com/apache/incubator-openwhisk-runtime-go/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
sciabarracom opened a new pull request #70: adding start library for easy
implementation of Go actions
URL: https://github.com/apache/incubator-openwhisk-client-go/pull/70
there is only one file, start.go providing the whisk.Start and
whisk.StartWithArgs functions.
Those are helpers
chetanmeh closed issue #3161: Add type property to entity documents
URL: https://github.com/apache/incubator-openwhisk/issues/3161
This is an automated message from the Apache Git Service.
To respond to the message, please lo
chetanmeh commented on issue #3161: Add type property to entity documents
URL:
https://github.com/apache/incubator-openwhisk/issues/3161#issuecomment-373130266
Closed via #3162
This is an automated message from the Apache Gi
dubeejw commented on issue #3438: Error thrown from MetricEmitter
URL:
https://github.com/apache/incubator-openwhisk/issues/3438#issuecomment-373125855
Closing as fixed.
This is an automated message from the Apache Git Servic
dubeejw closed issue #3438: Error thrown from MetricEmitter
URL: https://github.com/apache/incubator-openwhisk/issues/3438
This is an automated message from the Apache Git Service.
To respond to the message, please log on Git
dubeejw opened a new issue #3438: Error thrown from MetricEmitter
URL: https://github.com/apache/incubator-openwhisk/issues/3438
We have observed an `ArrayIndexOutOfBoundsException` being thrown from
`MetricEmitter`.
Line throwing the error:
https://github.com/apache/incubator-op
csantanapr commented on issue #26: update to pickup openwhisk@3.13.0
URL:
https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/26#issuecomment-373110230
@jthomas what issue with `3.13.0` ?
This is an automated me
csantanapr closed issue #97: ShortDesc type should declare name as optional
URL: https://github.com/apache/incubator-openwhisk-client-js/issues/97
This is an automated message from the Apache Git Service.
To respond to the
csantanapr commented on issue #97: ShortDesc type should declare name as
optional
URL:
https://github.com/apache/incubator-openwhisk-client-js/issues/97#issuecomment-373109933
Closed by #98
This is an automated message fr
mrutkows opened a new issue #63: Verify API Gateway legal NOTICE / CREDITS
files ack. licenses/contribs.
URL: https://github.com/apache/incubator-openwhisk-release/issues/63
In our Apache proposal I noted the following:
```
Adobe-API-Platform
Openresty - Licensed under the
kpavel opened a new pull request #802: Resolves #730
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/802
This PR extends wskdeploy with a capability to explicitly define
relationships between projects and assets beyond single project. More
specifically, it allows to expre
cbickel closed issue #3403: Accept tabs in go code
URL: https://github.com/apache/incubator-openwhisk/issues/3403
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and u
jthomas commented on issue #26: update to pickup openwhisk@3.13.0
URL:
https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/26#issuecomment-373065945
Better to now wait till 3.13.1 due to issue with current release.
-
mdeuser commented on issue #3437: swagger generation update
URL:
https://github.com/apache/incubator-openwhisk/pull/3437#issuecomment-373062506
@markusthoemmes - indeed. currently the apigateway tests reside in the
incubator-openwhisk-cli repo and will have additional tests
https://gith
markusthoemmes commented on issue #3437: swagger generation update
URL:
https://github.com/apache/incubator-openwhisk/pull/3437#issuecomment-373051370
Do we need additional tests to cover this?
This is an automated message fr
mdeuser commented on issue #3437: swagger generation update
URL:
https://github.com/apache/incubator-openwhisk/pull/3437#issuecomment-373050304
pg4/1478
This is an automated message from the Apache Git Service.
To respond to
mdeuser opened a new pull request #3437: swagger generation update
URL: https://github.com/apache/incubator-openwhisk/pull/3437
## Description
In support of creating APIs with secured (`require-whisk-auth` annotation
with number or string) web-actions
- swagger update causes `
cbickel commented on issue #3436: move cleanup logic of cloudant/kafka trigger
dbs to a different location
URL:
https://github.com/apache/incubator-openwhisk/pull/3436#issuecomment-373042764
@rabbah We are just moving it.
Th
mhenke1 commented on issue #3432: Introduce jmx for kafka
URL:
https://github.com/apache/incubator-openwhisk/pull/3432#issuecomment-373029498
@ningyougang
The reason for your problems is that the uid of the root user on the vm
(where Ansible does work) and the uid of the root user insi
houshengbo closed pull request #62: Generate release note
URL: https://github.com/apache/incubator-openwhisk-release/pull/62
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 pul
houshengbo commented on issue #62: Generate release note
URL:
https://github.com/apache/incubator-openwhisk-release/pull/62#issuecomment-373022718
Cool, I see the point of moving the command "rm -rf .git".
This is an automate
rabbah commented on issue #3436: move cleanup logic of cloudant/kafka trigger
dbs to a different location
URL:
https://github.com/apache/incubator-openwhisk/pull/3436#issuecomment-373021992
Why remove this?
This is an automa
steffenrost opened a new pull request #3436: move cleanup logic of
cloudant/kafka trigger dbs to a different location
URL: https://github.com/apache/incubator-openwhisk/pull/3436
Cleanup Python script is removed at this location.
jonpspri opened a new pull request #3435: CLI docs update and markdown linting.
URL: https://github.com/apache/incubator-openwhisk/pull/3435
Update CLI documentation per #3426 to acurately reflect the process for
local CLI build.
## Description
Update to `ansible/README.md` to
jthomas commented on a change in pull request #99: Add Code Query Parameter
URL:
https://github.com/apache/incubator-openwhisk-client-js/pull/99#discussion_r174441059
##
File path: test/unit/actions.test.js
##
@@ -60,7 +79,7 @@ test('should retrieve action from string iden
cbickel closed pull request #3434: Fix negative values and blocking usage in
Kafka lag monitoring.
URL: https://github.com/apache/incubator-openwhisk/pull/3434
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
markusthoemmes commented on issue #3434: Fix negative values and blocking usage
in Kafka lag monitoring.
URL:
https://github.com/apache/incubator-openwhisk/pull/3434#issuecomment-372998004
PG2 2920 👍
This is an automated me
jonpspri commented on issue #3430: Bump ansible version
URL:
https://github.com/apache/incubator-openwhisk/pull/3430#issuecomment-372985035
Had an offline with Vadim. If this change passes tests, we're GTG with it
as it is now. I have latent concerns about how old-in-the-tooth some of ou
jonpspri commented on issue #3426: CLI Builds not found in expected location
URL:
https://github.com/apache/incubator-openwhisk/issues/3426#issuecomment-372984355
For local build of the zip file, we need to execute `./gradlew release
-PnativeCompile`. (Notice the `release`, not compile).
cbickel closed pull request #3418: support cleanup of cloudant/kafka trigger dbs
URL: https://github.com/apache/incubator-openwhisk/pull/3418
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
markusthoemmes opened a new pull request #3434: Fix negative values and
blocking usage in Kafka lag monitoring.
URL: https://github.com/apache/incubator-openwhisk/pull/3434
## Description
1. `endOffsets` might be eventually consistent to the locally stored offset
ningyougang commented on issue #3432: Introduce jmx for kafka
URL:
https://github.com/apache/incubator-openwhisk/pull/3432#issuecomment-372915814
How strange, in my local building env, don't export below error(I login into
the kafka container and make sure the /root/jmxremote.password's pe
ningyougang commented on issue #3432: Introduce jmx for kafka
URL:
https://github.com/apache/incubator-openwhisk/pull/3432#issuecomment-372915814
How strange, in my local building env, don't export below error
```
/root/jmxremote.password\nError: Password file not readable:
/root/jmx
79 matches
Mail list logo