[GitHub] rabbah opened a new issue #4: reduce bloat in this repo (and other runtime repos)

2018-02-24 Thread GitBox
rabbah opened a new issue #4: reduce bloat in this repo (and other runtime repos) URL: https://github.com/apache/incubator-openwhisk-runtime-php/issues/4 There are some gradle and ansible commands in here that are not relevant - for example, deploying a full openwhisk stack. Was

[GitHub] rabbah opened a new issue #5: reduce clone and own in the test harness (applies to other repos also)

2018-02-24 Thread GitBox
rabbah opened a new issue #5: reduce clone and own in the test harness (applies to other repos also) URL: https://github.com/apache/incubator-openwhisk-runtime-php/issues/5 the test directory 4 files, only 1 of which is relevant to testing this runtime; the rest are cloned from openwhisk

[GitHub] rabbah commented on issue #3340: Default activation response to empty JsObject.

2018-02-24 Thread GitBox
rabbah commented on issue #3340: Default activation response to empty JsObject. URL: https://github.com/apache/incubator-openwhisk/pull/3340#issuecomment-368229792 Travis is enough - The result = None is not observable unless there's an invariant violation which the loop back likely

[GitHub] csantanapr commented on issue #176: Only fetch action code when saving an action

2018-02-24 Thread GitBox
csantanapr commented on issue #176: Only fetch action code when saving an action URL: https://github.com/apache/incubator-openwhisk-cli/pull/176#issuecomment-368220243 @mdeuser bump This is an automated message from the

[GitHub] markusthoemmes commented on issue #3340: Default activation response to empty JsObject.

2018-02-24 Thread GitBox
markusthoemmes commented on issue #3340: Default activation response to empty JsObject. URL: https://github.com/apache/incubator-openwhisk/pull/3340#issuecomment-368223394 No PG needed here I'd say? This is an automated

[GitHub] markusthoemmes commented on a change in pull request #3343: Support time-series DB using Kamon Tag, and optional granular metric.

2018-02-24 Thread GitBox
markusthoemmes commented on a change in pull request #3343: Support time-series DB using Kamon Tag, and optional granular metric. URL: https://github.com/apache/incubator-openwhisk/pull/3343#discussion_r170419868 ## File path: common/scala/src/main/scala/whisk/common/Logging.scala

[GitHub] markusthoemmes commented on a change in pull request #3343: Support time-series DB using Kamon Tag, and optional granular metric.

2018-02-24 Thread GitBox
markusthoemmes commented on a change in pull request #3343: Support time-series DB using Kamon Tag, and optional granular metric. URL: https://github.com/apache/incubator-openwhisk/pull/3343#discussion_r170420043 ## File path: common/scala/src/main/scala/whisk/common/Logging.scala

[GitHub] markusthoemmes commented on a change in pull request #3343: Support time-series DB using Kamon Tag, and optional granular metric.

2018-02-24 Thread GitBox
markusthoemmes commented on a change in pull request #3343: Support time-series DB using Kamon Tag, and optional granular metric. URL: https://github.com/apache/incubator-openwhisk/pull/3343#discussion_r170420980 ## File path: common/scala/src/main/scala/whisk/common/Logging.scala

[GitHub] markusthoemmes closed pull request #2781: Add a noOp invoker to test the controller without Invoker overhead

2018-02-24 Thread GitBox
markusthoemmes closed pull request #2781: Add a noOp invoker to test the controller without Invoker overhead URL: https://github.com/apache/incubator-openwhisk/pull/2781 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for

[GitHub] markusthoemmes closed pull request #2792: Add a controller simulator to drive invoker benchmarks.

2018-02-24 Thread GitBox
markusthoemmes closed pull request #2792: Add a controller simulator to drive invoker benchmarks. URL: https://github.com/apache/incubator-openwhisk/pull/2792 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] markusthoemmes commented on issue #2792: Add a controller simulator to drive invoker benchmarks.

2018-02-24 Thread GitBox
markusthoemmes commented on issue #2792: Add a controller simulator to drive invoker benchmarks. URL: https://github.com/apache/incubator-openwhisk/pull/2792#issuecomment-368228856 Closing as abandoned. This is an automated

[GitHub] markusthoemmes commented on issue #2781: Add a noOp invoker to test the controller without Invoker overhead

2018-02-24 Thread GitBox
markusthoemmes commented on issue #2781: Add a noOp invoker to test the controller without Invoker overhead URL: https://github.com/apache/incubator-openwhisk/pull/2781#issuecomment-368228824 Closing as abandoned. @cbickel please reopen if you want to pick this up again.

[GitHub] csantanapr commented on issue #3343: Support time-series DB using Kamon Tag, and optional granular metric.

2018-02-24 Thread GitBox
csantanapr commented on issue #3343: Support time-series DB using Kamon Tag, and optional granular metric. URL: https://github.com/apache/incubator-openwhisk/pull/3343#issuecomment-368222577 If it?s optional and and is false by default I don?t see an issue to add in the grand scheme of

[GitHub] markusthoemmes commented on a change in pull request #3343: Support time-series DB using Kamon Tag, and optional granular metric.

2018-02-24 Thread GitBox
markusthoemmes commented on a change in pull request #3343: Support time-series DB using Kamon Tag, and optional granular metric. URL: https://github.com/apache/incubator-openwhisk/pull/3343#discussion_r170419789 ## File path: common/scala/src/main/scala/whisk/common/Logging.scala

[GitHub] markusthoemmes commented on a change in pull request #3343: Support time-series DB using Kamon Tag, and optional granular metric.

2018-02-24 Thread GitBox
markusthoemmes commented on a change in pull request #3343: Support time-series DB using Kamon Tag, and optional granular metric. URL: https://github.com/apache/incubator-openwhisk/pull/3343#discussion_r170419789 ## File path: common/scala/src/main/scala/whisk/common/Logging.scala

[GitHub] upgle commented on a change in pull request #3343: Support time-series DB using Kamon Tag, and optional granular metric.

2018-02-24 Thread GitBox
upgle commented on a change in pull request #3343: Support time-series DB using Kamon Tag, and optional granular metric. URL: https://github.com/apache/incubator-openwhisk/pull/3343#discussion_r170420290 ## File path: common/scala/src/main/scala/whisk/common/Logging.scala

[GitHub] upgle commented on a change in pull request #3343: Support time-series DB using Kamon Tag, and optional granular metric.

2018-02-24 Thread GitBox
upgle commented on a change in pull request #3343: Support time-series DB using Kamon Tag, and optional granular metric. URL: https://github.com/apache/incubator-openwhisk/pull/3343#discussion_r170420907 ## File path: common/scala/src/main/scala/whisk/common/Logging.scala

[GitHub] markusthoemmes closed pull request #3044: add WhiskActivation.Outcome, hiding Either[ActivationId, WhiskActivation] impl

2018-02-24 Thread GitBox
markusthoemmes closed pull request #3044: add WhiskActivation.Outcome, hiding Either[ActivationId, WhiskActivation] impl URL: https://github.com/apache/incubator-openwhisk/pull/3044 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed

[GitHub] markusthoemmes commented on a change in pull request #3196: Fixes #3195 - create config object for docker in the invoker and get ?

2018-02-24 Thread GitBox
markusthoemmes commented on a change in pull request #3196: Fixes #3195 - create config object for docker in the invoker and get ? URL: https://github.com/apache/incubator-openwhisk/pull/3196#discussion_r170421471 ## File path:

[GitHub] markusthoemmes commented on issue #3189: Make some CLI tests more stable.

2018-02-24 Thread GitBox
markusthoemmes commented on issue #3189: Make some CLI tests more stable. URL: https://github.com/apache/incubator-openwhisk/pull/3189#issuecomment-368229036 @cbickel the testcase is fixed in the meantime I think. Please open a new PR if you want to fix the test-action names still.

[GitHub] markusthoemmes closed pull request #3189: Make some CLI tests more stable.

2018-02-24 Thread GitBox
markusthoemmes closed pull request #3189: Make some CLI tests more stable. URL: https://github.com/apache/incubator-openwhisk/pull/3189 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] rabbah commented on a change in pull request #3340: Default activation response to empty JsObject.

2018-02-24 Thread GitBox
rabbah commented on a change in pull request #3340: Default activation response to empty JsObject. URL: https://github.com/apache/incubator-openwhisk/pull/3340#discussion_r170421664 ## File path: common/scala/src/main/scala/whisk/core/entity/WhiskActivation.scala ## @@

[GitHub] rabbah commented on issue #3340: Default activation response to empty JsObject.

2018-02-24 Thread GitBox
rabbah commented on issue #3340: Default activation response to empty JsObject. URL: https://github.com/apache/incubator-openwhisk/pull/3340#issuecomment-368229792 Travis is enough - The result = None is not observable unless there's an invariant violation; returning result = None and a

[GitHub] csantanapr commented on issue #97: Added maven archetype for Java Action

2018-02-24 Thread GitBox
csantanapr commented on issue #97: Added maven archetype for Java Action URL: https://github.com/apache/incubator-openwhisk-devtools/pull/97#issuecomment-368222132 @ddragosd is this one good for merging ? This is an

[GitHub] markusthoemmes commented on a change in pull request #3343: Support time-series DB using Kamon Tag, and optional granular metric.

2018-02-24 Thread GitBox
markusthoemmes commented on a change in pull request #3343: Support time-series DB using Kamon Tag, and optional granular metric. URL: https://github.com/apache/incubator-openwhisk/pull/3343#discussion_r170420063 ## File path: common/scala/src/main/scala/whisk/common/Logging.scala

[GitHub] markusthoemmes commented on a change in pull request #3343: Support time-series DB using Kamon Tag, and optional granular metric.

2018-02-24 Thread GitBox
markusthoemmes commented on a change in pull request #3343: Support time-series DB using Kamon Tag, and optional granular metric. URL: https://github.com/apache/incubator-openwhisk/pull/3343#discussion_r170420944 ## File path: common/scala/src/main/scala/whisk/common/Logging.scala

[GitHub] markusthoemmes commented on a change in pull request #3343: Support time-series DB using Kamon Tag, and optional granular metric.

2018-02-24 Thread GitBox
markusthoemmes commented on a change in pull request #3343: Support time-series DB using Kamon Tag, and optional granular metric. URL: https://github.com/apache/incubator-openwhisk/pull/3343#discussion_r170420972 ## File path:

[GitHub] upgle commented on a change in pull request #3343: Support time-series DB using Kamon Tag, and optional granular metric.

2018-02-24 Thread GitBox
upgle commented on a change in pull request #3343: Support time-series DB using Kamon Tag, and optional granular metric. URL: https://github.com/apache/incubator-openwhisk/pull/3343#discussion_r170421346 ## File path: common/scala/src/main/scala/whisk/common/Logging.scala

[GitHub] chetanmeh commented on issue #5: reduce clone and own in the test harness (applies to other repos also)

2018-02-24 Thread GitBox
chetanmeh commented on issue #5: reduce clone and own in the test harness (applies to other repos also) URL: https://github.com/apache/incubator-openwhisk-runtime-php/issues/5#issuecomment-368256180 So far there was a gradle project dependency on core tests which is being now replaced

[GitHub] dubeejw commented on issue #3168: Rate limit requests to download CLI, Docker and iOS SDKs

2018-02-24 Thread GitBox
dubeejw commented on issue #3168: Rate limit requests to download CLI, Docker and iOS SDKs URL: https://github.com/apache/incubator-openwhisk/issues/3168#issuecomment-368281366 @mptap, I'll take a look. Thanks for working on this issue!

[GitHub] kameshsampath opened a new issue #12: [Epic] Improvements to Java Runtime

2018-02-24 Thread GitBox
kameshsampath opened a new issue #12: [Epic] Improvements to Java Runtime URL: https://github.com/apache/incubator-openwhisk-runtime-java/issues/12 The following are list of tasks that we need to work to improve the OW Java Action runtime: 1. avoid usage of com.sun.* packages -