[GitHub] chetanmeh commented on issue #4140: ContainerProxyTests Heisenbug

chetanmeh commented on issue #4140: ContainerProxyTests Heisenbug URL: https://github.com/apache/incubator-openwhisk/issues/4140#issuecomment-446473708 Another one at https://scans.gradle.com/s/st23hp56clclk/tests/xaqugzlgs2zxa-4fdg7fshdyxgs

[GitHub] ningyougang edited a comment on issue #4164: Add warn log when too many requests happened

ningyougang edited a comment on issue #4164: Add warn log when too many requests happened URL: https://github.com/apache/incubator-openwhisk/pull/4164#issuecomment-446422425 @markusthoemmes , yes, you are right, just a debug level log, `this log is saved to controllerN_logs.log` have

[GitHub] ningyougang edited a comment on issue #4164: Add warn log when too many requests happened

ningyougang edited a comment on issue #4164: Add warn log when too many requests happened URL: https://github.com/apache/incubator-openwhisk/pull/4164#issuecomment-446422425 @markusthoemmes , yes, you are right, just a debug level log, `this log is saved to controllerN_logs.log` have

[GitHub] ningyougang edited a comment on issue #4164: Add warn log when too many requests happened

ningyougang edited a comment on issue #4164: Add warn log when too many requests happened URL: https://github.com/apache/incubator-openwhisk/pull/4164#issuecomment-446422425 @markusthoemmes , yes, you are right, just a debug level log, `this log is saved to controllerN_logs.log` have

[GitHub] rabbah opened a new pull request #3: Fix missing RUNTIME name.

rabbah opened a new pull request #3: Fix missing RUNTIME name. URL: https://github.com/apache/incubator-openwhisk-runtime-dotnet/pull/3 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] tysonnorris opened a new pull request #4171: mesos - expose config for setting heartbeatMaxFailures to reconnect after master failover

tysonnorris opened a new pull request #4171: mesos - expose config for setting heartbeatMaxFailures to reconnect after master failover URL: https://github.com/apache/incubator-openwhisk/pull/4171 In case of mesos master failover, we need to reconnect to master by re-subscribing.

[GitHub] ningyougang edited a comment on issue #4164: Add warn log when too many requests happened

ningyougang edited a comment on issue #4164: Add warn log when too many requests happened URL: https://github.com/apache/incubator-openwhisk/pull/4164#issuecomment-446422425 @markusthoemmes , yes, you are right, just a debug level log, `this log is saved to controllerN_logs.log` have

[GitHub] ningyougang commented on issue #4164: Add warn log when too many requests happened

ningyougang commented on issue #4164: Add warn log when too many requests happened URL: https://github.com/apache/incubator-openwhisk/pull/4164#issuecomment-446422425 @markusthoemmes , yes, you are right, just a debug level log, `this log is saved to controllerN_logs.log` have some

[GitHub] csantanapr closed pull request #2: Add travis hooks to test and publish docker images.

csantanapr closed pull request #2: Add travis hooks to test and publish docker images. URL: https://github.com/apache/incubator-openwhisk-runtime-dotnet/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

[GitHub] pritidesai closed pull request #353: Clarify GitHub nightly-interim releases tags from Apache releases.

pritidesai closed pull request #353: Clarify GitHub nightly-interim releases tags from Apache releases. URL: https://github.com/apache/incubator-openwhisk-website/pull/353 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for

[GitHub] dgrove-oss edited a comment on issue #4169: tests:testSwaggerCodegen failing

dgrove-oss edited a comment on issue #4169: tests:testSwaggerCodegen failing URL: https://github.com/apache/incubator-openwhisk/issues/4169#issuecomment-446380630 #4170 disabled the failing test. As people have time, we should do a real fix and re-enable the test. @bbrowning

[GitHub] dgrove-oss commented on issue #4169: tests:testSwaggerCodegen failing

dgrove-oss commented on issue #4169: tests:testSwaggerCodegen failing URL: https://github.com/apache/incubator-openwhisk/issues/4169#issuecomment-446380630 #4170 disables the failing test. As people have time, we should do a real fix and re-enable the test. @bbrowning

[GitHub] dgrove-oss closed pull request #4170: disable broken swaggerCodegen tests

dgrove-oss closed pull request #4170: disable broken swaggerCodegen tests URL: https://github.com/apache/incubator-openwhisk/pull/4170 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

[GitHub] codecov-io commented on issue #4170: disable broken swaggerCodegen tests

codecov-io commented on issue #4170: disable broken swaggerCodegen tests URL: https://github.com/apache/incubator-openwhisk/pull/4170#issuecomment-446378656 # [Codecov](https://codecov.io/gh/apache/incubator-openwhisk/pull/4170?src=pr=h1) Report > :exclamation: No coverage uploaded

[GitHub] macdonst commented on issue #143: Updating package parameters doesn't seem to take affect

macdonst commented on issue #143: Updating package parameters doesn't seem to take affect URL: https://github.com/apache/incubator-openwhisk-client-js/issues/143#issuecomment-446356536 Okay, well I'm going to close this a user error :)

[GitHub] macdonst closed issue #143: Updating package parameters doesn't seem to take affect

macdonst closed issue #143: Updating package parameters doesn't seem to take affect URL: https://github.com/apache/incubator-openwhisk-client-js/issues/143 This is an automated message from the Apache Git Service. To

[GitHub] starpit commented on issue #143: Updating package parameters doesn't seem to take affect

starpit commented on issue #143: Updating package parameters doesn't seem to take affect URL: https://github.com/apache/incubator-openwhisk-client-js/issues/143#issuecomment-446355312 yes, i agree that this API could be refined; it is not a reflexive API.

[GitHub] macdonst commented on issue #143: Updating package parameters doesn't seem to take affect

macdonst commented on issue #143: Updating package parameters doesn't seem to take affect URL: https://github.com/apache/incubator-openwhisk-client-js/issues/143#issuecomment-446354425 Yeah, using `ow.packages.update({ name, package})` has resolved the issue but I'm not sure why that

[GitHub] pritidesai opened a new pull request #394: Integrating wskdeploy into wsk CLI

pritidesai opened a new pull request #394: Integrating wskdeploy into wsk CLI URL: https://github.com/apache/incubator-openwhisk-cli/pull/394 This is an automated message from the Apache Git Service. To respond to the

[GitHub] macdonst removed a comment on issue #143: Updating package parameters doesn't seem to take affect

macdonst removed a comment on issue #143: Updating package parameters doesn't seem to take affect URL: https://github.com/apache/incubator-openwhisk-client-js/issues/143#issuecomment-446335481 @starpit here is a smaller example that still doesn't work that is based off the sample code

[GitHub] macdonst commented on issue #143: Updating package parameters doesn't seem to take affect

macdonst commented on issue #143: Updating package parameters doesn't seem to take affect URL: https://github.com/apache/incubator-openwhisk-client-js/issues/143#issuecomment-446335481 @starpit here is a smaller example that still doesn't work that is based off the sample code from the

[GitHub] starpit commented on issue #143: Updating package parameters doesn't seem to take affect

starpit commented on issue #143: Updating package parameters doesn't seem to take affect URL: https://github.com/apache/incubator-openwhisk-client-js/issues/143#issuecomment-446335143 you have tried `ow.packages.update({ name, package})`? your example has `ow.packages.update(pack)` which

[GitHub] macdonst commented on issue #143: Updating package parameters doesn't seem to take affect

macdonst commented on issue #143: Updating package parameters doesn't seem to take affect URL: https://github.com/apache/incubator-openwhisk-client-js/issues/143#issuecomment-446333922 @starpit Yeah, I updated the code above as that was a copy and paste from my trashing trying to figure

[GitHub] pritidesai closed issue #1005: Action has no kind set - Error decoding runtimes with Local Values

pritidesai closed issue #1005: Action has no kind set - Error decoding runtimes with Local Values URL: https://github.com/apache/incubator-openwhisk-wskdeploy/issues/1005 This is an automated message from the Apache Git

[GitHub] pritidesai commented on a change in pull request #1017: Deleting static runtimes data file, instead, adding data to go file

pritidesai commented on a change in pull request #1017: Deleting static runtimes data file, instead, adding data to go file URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/1017#discussion_r240758461 ## File path: runtimes/runtimes.go ## @@ -56,7 +54,6

[GitHub] pritidesai closed pull request #1017: Deleting static runtimes data file, instead, adding data to go file

pritidesai closed pull request #1017: Deleting static runtimes data file, instead, adding data to go file URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/1017 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below

[GitHub] starpit edited a comment on issue #143: Updating package parameters doesn't seem to take affect

starpit edited a comment on issue #143: Updating package parameters doesn't seem to take affect URL: https://github.com/apache/incubator-openwhisk-client-js/issues/143#issuecomment-446329927 hi, i think the issue is that the `parameters` field is an array i.e. `[{key, value}, ...]` of

[GitHub] starpit commented on issue #143: Updating package parameters doesn't seem to take affect

starpit commented on issue #143: Updating package parameters doesn't seem to take affect URL: https://github.com/apache/incubator-openwhisk-client-js/issues/143#issuecomment-446329927 hi, i think the issue is that the `parameters` array is an array i.e. `[{key, value}, ...]` of key-value

[GitHub] macdonst opened a new issue #143: Updating package parameters doesn't seem to take affect

macdonst opened a new issue #143: Updating package parameters doesn't seem to take affect URL: https://github.com/apache/incubator-openwhisk-client-js/issues/143 I'm attempting to update my package parameters while executing an action. Unfortunately, no matter what I try the parameters

[GitHub] mrutkows commented on a change in pull request #1017: Deleting static runtimes data file, instead, adding data to go file

mrutkows commented on a change in pull request #1017: Deleting static runtimes data file, instead, adding data to go file URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/1017#discussion_r240750610 ## File path: runtimes/runtimes.go ## @@ -56,7 +54,6 @@

[GitHub] dgrove-oss opened a new pull request #4170: disable broken swaggerCodegen tests

dgrove-oss opened a new pull request #4170: disable broken swaggerCodegen tests URL: https://github.com/apache/incubator-openwhisk/pull/4170 This is an automated message from the Apache Git Service. To respond to the

[GitHub] dgrove-oss opened a new issue #4169: tests:testSwaggerCodegen failing

dgrove-oss opened a new issue #4169: tests:testSwaggerCodegen failing URL: https://github.com/apache/incubator-openwhisk/issues/4169 Test currently failing. Appears to be breakage in upstream dependency. $ ./gradlew :tests:testSwaggerCodegen > Task :tests:testSwaggerCodegen

[GitHub] mrutkows opened a new pull request #353: Clarify GitHub nightly-interim releases tags from Apache releases.

mrutkows opened a new pull request #353: Clarify GitHub nightly-interim releases tags from Apache releases. URL: https://github.com/apache/incubator-openwhisk-website/pull/353 This is an automated message from the Apache

[GitHub] dgrove-oss commented on issue #3: add encoded URL for travis2slack webhook

dgrove-oss commented on issue #3: add encoded URL for travis2slack webhook URL: https://github.com/apache/incubator-openwhisk-composer-python/pull/3#issuecomment-446232792 Success.

[GitHub] dgrove-oss opened a new pull request #3: add encoded URL for travis2slack webhook

dgrove-oss opened a new pull request #3: add encoded URL for travis2slack webhook URL: https://github.com/apache/incubator-openwhisk-composer-python/pull/3 This is an automated message from the Apache Git Service. To

[GitHub] cbickel commented on issue #4137: Remove busy loop in case the invoker is overloaded.

cbickel commented on issue #4137: Remove busy loop in case the invoker is overloaded. URL: https://github.com/apache/incubator-openwhisk/pull/4137#issuecomment-446188127 @sven-lange-last and me decided to not merge this yet, as it is too risky currently (before everyone leaves during

[GitHub] cbickel closed pull request #4137: Remove busy loop in case the invoker is overloaded.

cbickel closed pull request #4137: Remove busy loop in case the invoker is overloaded. URL: https://github.com/apache/incubator-openwhisk/pull/4137 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:

[GitHub] markusthoemmes commented on issue #4164: Add warn log when too many requests happened

markusthoemmes commented on issue #4164: Add warn log when too many requests happened URL: https://github.com/apache/incubator-openwhisk/pull/4164#issuecomment-446160253 @ningyougang that feels like a Debug level log then, certainly not a warning since nothing is wrong with the system in

[GitHub] cbickel opened a new pull request #4168: Remove unused ansible parameters in group_vars/all file.

cbickel opened a new pull request #4168: Remove unused ansible parameters in group_vars/all file. URL: https://github.com/apache/incubator-openwhisk/pull/4168 ## Description This PR cleans up some config parameters in `group_vars/all`-file of ansible, taht are not used anymore.

[GitHub] chetanmeh commented on issue #4165: Update to Kamon 1.1.3 from 0.6 series

chetanmeh commented on issue #4165: Update to Kamon 1.1.3 from 0.6 series URL: https://github.com/apache/incubator-openwhisk/pull/4165#issuecomment-446154308 Discussed this with @ivantopo on [gitter](https://gitter.im/kamon-io/Kamon?at=5c0f880a80986419d572ba8f) and it seems we would need

[GitHub] chetanmeh edited a comment on issue #4165: Update to Kamon 1.1.3 from 0.6 series

chetanmeh edited a comment on issue #4165: Update to Kamon 1.1.3 from 0.6 series URL: https://github.com/apache/incubator-openwhisk/pull/4165#issuecomment-446138192 Seems like Kamon has changed the [metric naming

[GitHub] chetanmeh commented on issue #4165: Update to Kamon 1.1.3 from 0.6 series

chetanmeh commented on issue #4165: Update to Kamon 1.1.3 from 0.6 series URL: https://github.com/apache/incubator-openwhisk/pull/4165#issuecomment-446138192 Seems like Kamon has changed the metric naming convention. Earlier it used to include the metric type/category as part of name like

[GitHub] chetanmeh commented on a change in pull request #4165: Update to Kamon 1.1.3 from 0.6 series

chetanmeh commented on a change in pull request #4165: Update to Kamon 1.1.3 from 0.6 series URL: https://github.com/apache/incubator-openwhisk/pull/4165#discussion_r240514286 ## File path: core/invoker/src/main/scala/org/apache/openwhisk/core/invoker/Invoker.scala ##

[GitHub] chetanmeh commented on a change in pull request #4165: Update to Kamon 1.1.3 from 0.6 series

chetanmeh commented on a change in pull request #4165: Update to Kamon 1.1.3 from 0.6 series URL: https://github.com/apache/incubator-openwhisk/pull/4165#discussion_r240537006 ## File path: common/scala/src/main/resources/application.conf ## @@ -19,7 +19,11 @@ akka.http

[GitHub] cbickel commented on a change in pull request #4167: Add tid from header to RestResult in tests.

cbickel commented on a change in pull request #4167: Add tid from header to RestResult in tests. URL: https://github.com/apache/incubator-openwhisk/pull/4167#discussion_r240534351 ## File path: tests/src/test/scala/common/rest/WskRestOperations.scala ## @@ -299,20

[GitHub] cbickel opened a new pull request #4167: Add tid from header to RestResult in tests.

cbickel opened a new pull request #4167: Add tid from header to RestResult in tests. URL: https://github.com/apache/incubator-openwhisk/pull/4167 ## Description This PR adds the tid from the tid-header to the rest result. This will enhance debugging failed test cases. In

[GitHub] ningyougang edited a comment on issue #4164: Add warn log when too many requests happened

ningyougang edited a comment on issue #4164: Add warn log when too many requests happened URL: https://github.com/apache/incubator-openwhisk/pull/4164#issuecomment-446128276 Regarding `Can you elaborate on the lack of information you have with the current logs?` it is better to output

[GitHub] ningyougang edited a comment on issue #4164: Add warn log when too many requests happened

ningyougang edited a comment on issue #4164: Add warn log when too many requests happened URL: https://github.com/apache/incubator-openwhisk/pull/4164#issuecomment-446128276 Regarding `Can you elaborate on the lack of information you have with the current logs?` it is better to output

[GitHub] ningyougang edited a comment on issue #4164: Add warn log when too many requests happened

ningyougang edited a comment on issue #4164: Add warn log when too many requests happened URL: https://github.com/apache/incubator-openwhisk/pull/4164#issuecomment-446128276 Regarding `Can you elaborate on the lack of information you have with the current logs?` it is better to output

[GitHub] ningyougang edited a comment on issue #4164: Add warn log when too many requests happened

ningyougang edited a comment on issue #4164: Add warn log when too many requests happened URL: https://github.com/apache/incubator-openwhisk/pull/4164#issuecomment-446128276 Regarding `Can you elaborate on the lack of information you have with the current logs?` it is better to output

[GitHub] ningyougang commented on issue #4164: Add warn log when too many requests happened

ningyougang commented on issue #4164: Add warn log when too many requests happened URL: https://github.com/apache/incubator-openwhisk/pull/4164#issuecomment-446128276 Regarding `Can you elaborate on the lack of information you have with the current logs?` it is better to output below

[GitHub] upgle opened a new pull request #4166: Update the package name in a document

upgle opened a new pull request #4166: Update the package name in a document URL: https://github.com/apache/incubator-openwhisk/pull/4166 ## Description Update the package name in a doc. ## Related issue and scope - [ ] I opened an issue to propose and

[GitHub] ningyougang commented on a change in pull request #4164: Add warn log when too many requests happened

ningyougang commented on a change in pull request #4164: Add warn log when too many requests happened URL: https://github.com/apache/incubator-openwhisk/pull/4164#discussion_r240521991 ## File path:

[GitHub] ningyougang commented on a change in pull request #4164: Add warn log when too many requests happened

ningyougang commented on a change in pull request #4164: Add warn log when too many requests happened URL: https://github.com/apache/incubator-openwhisk/pull/4164#discussion_r240521627 ## File path:

[GitHub] chetanmeh opened a new pull request #4165: Update to Kamon 1.1.3 from 0.6 series

chetanmeh opened a new pull request #4165: Update to Kamon 1.1.3 from 0.6 series URL: https://github.com/apache/incubator-openwhisk/pull/4165 Updates Kamon to 1.x release (1.1.3) from current 0.6 version ## Description Going forward I would like to enable metrics reporting via

[GitHub] markusthoemmes commented on a change in pull request #4164: Add warn log when too many requests happened

markusthoemmes commented on a change in pull request #4164: Add warn log when too many requests happened URL: https://github.com/apache/incubator-openwhisk/pull/4164#discussion_r240504779 ## File path:

[GitHub] markusthoemmes commented on a change in pull request #4164: Add warn log when too many requests happened

markusthoemmes commented on a change in pull request #4164: Add warn log when too many requests happened URL: https://github.com/apache/incubator-openwhisk/pull/4164#discussion_r240504031 ## File path: