upgle commented on issue #3265: Change recently added metric key name. (very
simple change)
URL:
https://github.com/apache/incubator-openwhisk/pull/3265#issuecomment-368209072
I will close this PR because these metrics removed from #3284 PR.
upgle closed pull request #3265: Change recently added metric key name. (very
simple change)
URL: https://github.com/apache/incubator-openwhisk/pull/3265
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
upgle opened a new pull request #3343: Support time-series DB using Kamon Tag,
and optional granular metric.
URL: https://github.com/apache/incubator-openwhisk/pull/3343
This PR is for supporting time-series database using Kamon Tag like
OpenTSDB, Datadog.
and also added **optional**
rabbah closed issue #3321: __ow_body should always be a string (or null?)
URL: https://github.com/apache/incubator-openwhisk/issues/3321
This is an automated message from the Apache Git Service.
To respond to the message,
csantanapr closed pull request #3318: Remove restriction on updating a trigger
with a new feed.
URL: https://github.com/apache/incubator-openwhisk/pull/3318
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
csantanapr closed issue #3297: Allow adding a feed on trigger update.
URL: https://github.com/apache/incubator-openwhisk/issues/3297
This is an automated message from the Apache Git Service.
To respond to the message, please
csantanapr closed pull request #: Make parameters with defined values final
URL: https://github.com/apache/incubator-openwhisk/pull/
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
csantanapr closed pull request #3341: Remove view supporting now removed API
for listing assets in namespace.
URL: https://github.com/apache/incubator-openwhisk/pull/3341
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
csantanapr closed pull request #3323: Normalize a missing body for raw actions
to JsString.empty.
URL: https://github.com/apache/incubator-openwhisk/pull/3323
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
csantanapr commented on issue #3334: Cli ansible fix
URL:
https://github.com/apache/incubator-openwhisk/pull/3334#issuecomment-368200559
PG2 2858 ?
This is an automated message from the Apache Git Service.
To respond to the
dgrove-oss commented on issue #3338: implement suspend/resume for
KubernetesContainer
URL:
https://github.com/apache/incubator-openwhisk/pull/3338#issuecomment-368196944
PG3 / 1913 ?
This is an automated message from the
rabbah commented on issue #3335: Allow container tests to run without a proxy.
URL:
https://github.com/apache/incubator-openwhisk/pull/3335#issuecomment-368192234
pg5/115 ?
This is an automated message from the Apache Git
rabbah commented on issue #3323: Normalize a missing body for raw actions to
JsString.empty.
URL:
https://github.com/apache/incubator-openwhisk/pull/3323#issuecomment-368192195
pg3/1912 ?
This is an automated message from
rabbah commented on issue #3341: Remove view supporting now removed API for
listing assets in namespace.
URL:
https://github.com/apache/incubator-openwhisk/pull/3341#issuecomment-368192119
pg2/2856 ?
This is an automated
pritidesai opened a new pull request #754: Adding support for default package
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/754
Closes #681
This is an automated message from the Apache Git Service.
To
kameshsampath commented on issue #97: Added maven archetype for Java Action
URL:
https://github.com/apache/incubator-openwhisk-devtools/pull/97#issuecomment-368184984
my basd to miss `--main` in the README.md, updateed now @ddragosd
dgrove-oss commented on issue #3338: implement suspend/resume for
KubernetesContainer
URL:
https://github.com/apache/incubator-openwhisk/pull/3338#issuecomment-368184519
PG3 / 1913 ?
This is an automated message from the
mptap commented on issue #3168: Rate limit requests to download CLI, Docker and
iOS SDKs
URL:
https://github.com/apache/incubator-openwhisk/issues/3168#issuecomment-368183667
@dubeejw: Please could you review?
https://github.com/apache/incubator-openwhisk/pull/3342 Assumed key =
mptap opened a new pull request #3342: #3168 Rate limit requests to download
CLI, Docker and iOS SDKs
URL: https://github.com/apache/incubator-openwhisk/pull/3342
Nginx configured so that requests to download the CLI, Docker and iOS SDKs
are rate limited.
csantanapr closed pull request #3314: Configure Gradle build scan plugin
URL: https://github.com/apache/incubator-openwhisk/pull/3314
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
ddragosd commented on a change in pull request #97: Added maven archetype for
Java Action
URL:
https://github.com/apache/incubator-openwhisk-devtools/pull/97#discussion_r170398377
##
File path: java-action-archetype/README.md
##
@@ -0,0 +1,39 @@
+# Maven Archetype for
ddragosd commented on a change in pull request #97: Added maven archetype for
Java Action
URL:
https://github.com/apache/incubator-openwhisk-devtools/pull/97#discussion_r170398377
##
File path: java-action-archetype/README.md
##
@@ -0,0 +1,39 @@
+# Maven Archetype for
rabbah commented on issue #3341: Remove view supporting now removed API for
listing assets in namespace.
URL:
https://github.com/apache/incubator-openwhisk/pull/3341#issuecomment-368171920
So did I!
This is an automated
csantanapr commented on issue #3334: Cli ansible fix
URL:
https://github.com/apache/incubator-openwhisk/pull/3334#issuecomment-368170143
@jonpspri PG?
This is an automated message from the Apache Git Service.
To respond to
csantanapr commented on issue #3339: Add the empty NOTICE file for future
development
URL:
https://github.com/apache/incubator-openwhisk/pull/3339#issuecomment-368169993
empty file WTF?
This is an automated message from the
csantanapr commented on issue #3341: Remove view supporting now removed API for
listing assets in namespace.
URL:
https://github.com/apache/incubator-openwhisk/pull/3341#issuecomment-368169382
I thought you already did
This
dgrove-oss commented on a change in pull request #3338: implement
suspend/resume for KubernetesContainer
URL:
https://github.com/apache/incubator-openwhisk/pull/3338#discussion_r170387260
##
File path:
rabbah opened a new pull request #3341: Remove view supporting now removed API
for listing assets in namespace.
URL: https://github.com/apache/incubator-openwhisk/pull/3341
This is an automated message from the Apache Git
dgrove-oss commented on a change in pull request #3338: implement
suspend/resume for KubernetesContainer
URL:
https://github.com/apache/incubator-openwhisk/pull/3338#discussion_r170385887
##
File path:
dgrove-oss commented on a change in pull request #3338: implement
suspend/resume for KubernetesContainer
URL:
https://github.com/apache/incubator-openwhisk/pull/3338#discussion_r170385786
##
File path:
rabbah commented on issue #3318: Remove restriction on updating a trigger with
a new feed.
URL:
https://github.com/apache/incubator-openwhisk/pull/3318#issuecomment-368156182
PG3/1901 ?
This is an automated message from
csantanapr commented on issue #3335: Allow container tests to run without a
proxy.
URL:
https://github.com/apache/incubator-openwhisk/pull/3335#issuecomment-368153588
@rabbah Yeah we need to see if we can re-use this class in the runtime
repos. Can't remember why I copied the class. I
csantanapr commented on issue #3335: Allow container tests to run without a
proxy.
URL:
https://github.com/apache/incubator-openwhisk/pull/3335#issuecomment-368153588
Yeah we need to see if we can re-use this class in the runtime repos. Can't
remember why I copied the class. I guess I
csantanapr commented on a change in pull request #3335: Allow container tests
to run without a proxy.
URL:
https://github.com/apache/incubator-openwhisk/pull/3335#discussion_r170381377
##
File path: tests/src/test/scala/actionContainers/ActionContainer.scala
##
@@
rabbah commented on issue #3340: Default activation response to empty JsObject.
URL:
https://github.com/apache/incubator-openwhisk/pull/3340#issuecomment-368150368
could be... or remove the option and use an empty object. None is used on
the non-blocking path so the result doesn?t matter.
houshengbo commented on issue #3339: Add the empty NOTICE file for future
development
URL:
https://github.com/apache/incubator-openwhisk/pull/3339#issuecomment-368145751
@rabbah Apache does not define the suffix of the license and notice files. I
think we can use any suffix for them.
markusthoemmes commented on issue #3340: Default activation response to empty
JsObject.
URL:
https://github.com/apache/incubator-openwhisk/pull/3340#issuecomment-368146726
I discovered it in a loopback invoker test where I stubbed the invoker out.
Was a mere accident. I was thinking
houshengbo commented on issue #3339: Add the empty NOTICE file for future
development
URL:
https://github.com/apache/incubator-openwhisk/pull/3339#issuecomment-368145751
@rabbah Apache does not define the suffix of the license and notice files. I
need to confirm whether we can use .md
rabbah commented on issue #3340: Default activation response to empty JsObject.
URL:
https://github.com/apache/incubator-openwhisk/pull/3340#issuecomment-368144952
This is OK - where are we using the response result though incorrectly;
isn't there another bug?
markusthoemmes commented on a change in pull request #: Make parameters
with defined values final
URL:
https://github.com/apache/incubator-openwhisk/pull/#discussion_r170372376
##
File path: core/controller/src/main/scala/whisk/core/controller/Actions.scala
##
rabbah commented on issue #: Make parameters with defined values final
URL:
https://github.com/apache/incubator-openwhisk/pull/#issuecomment-368142579
pg1/2649 ?
This is an automated message from the Apache Git
dubeejw commented on issue #176: Only fetch action code when saving an action
URL:
https://github.com/apache/incubator-openwhisk-cli/pull/176#issuecomment-350515547
Example output:
```
./wsk action get test -i
ok: got action test
{
"namespace": "test@test",
dubeejw closed pull request #63: Parse new 'rules' field in 'trigger get'
response
URL: https://github.com/apache/incubator-openwhisk-client-go/pull/63
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
dubeejw closed pull request #220: WIP: test travis
URL: https://github.com/apache/incubator-openwhisk-cli/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 provenance:
As this is a foreign pull request
houshengbo opened a new pull request #65: Add the empty NOTICE file for further
development
URL: https://github.com/apache/incubator-openwhisk-client-go/pull/65
This is an automated message from the Apache Git Service.
To
dubeejw commented on issue #176: Only fetch action code when saving an action
URL:
https://github.com/apache/incubator-openwhisk-cli/pull/176#issuecomment-368137707
@mdeuser, any comments here?
This is an automated message
markusthoemmes commented on a change in pull request #3338: implement
suspend/resume for KubernetesContainer
URL:
https://github.com/apache/incubator-openwhisk/pull/3338#discussion_r170363289
##
File path:
markusthoemmes commented on a change in pull request #3338: implement
suspend/resume for KubernetesContainer
URL:
https://github.com/apache/incubator-openwhisk/pull/3338#discussion_r170363985
##
File path:
markusthoemmes commented on a change in pull request #3338: implement
suspend/resume for KubernetesContainer
URL:
https://github.com/apache/incubator-openwhisk/pull/3338#discussion_r170362912
##
File path:
markusthoemmes commented on a change in pull request #3338: implement
suspend/resume for KubernetesContainer
URL:
https://github.com/apache/incubator-openwhisk/pull/3338#discussion_r170364578
##
File path:
markusthoemmes commented on a change in pull request #3338: implement
suspend/resume for KubernetesContainer
URL:
https://github.com/apache/incubator-openwhisk/pull/3338#discussion_r170363009
##
File path:
houshengbo opened a new pull request #230: Add the LICENSE file and empty
NOTICE file
URL: https://github.com/apache/incubator-openwhisk-cli/pull/230
This is an automated message from the Apache Git Service.
To respond to
houshengbo opened a new pull request #753: Add the empty NOTICE for further
development
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/753
This is an automated message from the Apache Git Service.
To
dgrove-oss commented on issue #3338: implement suspend/resume for
KubernetesContainer
URL:
https://github.com/apache/incubator-openwhisk/pull/3338#issuecomment-368132145
@jcrossley3 @bbrowning @bwmcadams Please review proposed k8s changes.
Thanks.
houshengbo opened a new pull request #3339: Add the empty NOTICE file for
future development
URL: https://github.com/apache/incubator-openwhisk/pull/3339
This PR adds an empty NOTICE file for future use of release, since for each
release as an Apache project, we need to edit this NOTICE
dgrove-oss opened a new pull request #3338: implement suspend/resume for
KubernetesContainer
URL: https://github.com/apache/incubator-openwhisk/pull/3338
We can implement suspend and resume for KubernetesContainerPool by
delegating the operations to an invoker-agent that runs as a
houshengbo opened a new pull request #48: Add the support to check the
existence of NOTICE and LICENSE files
URL: https://github.com/apache/incubator-openwhisk-release/pull/48
This is an automated message from the Apache
rabbah commented on issue #3335: Allow container tests to run without a proxy.
URL:
https://github.com/apache/incubator-openwhisk/pull/3335#issuecomment-368124838
There may be a heisenbug with my sleep hack - several travis builds failed
in a row. Will have to investigate that.
rabbah closed pull request #3332: Add '-e' to redo for ansible variables and
change '-a' to gradle arguments only.
URL: https://github.com/apache/incubator-openwhisk/pull/3332
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below
rabbah commented on issue #3215: adding namePrefix config to ContainerArgsConfig
URL:
https://github.com/apache/incubator-openwhisk/pull/3215#issuecomment-368123776
Restarted Travis.
This is an automated message from the
rabbah closed pull request #3331: Make package test resilient against eventual
consistency.
URL: https://github.com/apache/incubator-openwhisk/pull/3331
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
markusthoemmes opened a new pull request #3337: Put active-ack consumers in
their own consumer-groups.
URL: https://github.com/apache/incubator-openwhisk/pull/3337
Just like with the invoker consumers, it doesn't make sense to have those in
one group as crash of one will cause a
markusthoemmes closed pull request #3330: Update Log Store Fetch
URL: https://github.com/apache/incubator-openwhisk/pull/3330
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
csantanapr commented on issue #3314: Configure Gradle build scan plugin
URL:
https://github.com/apache/incubator-openwhisk/pull/3314#issuecomment-368115345
Playground4/1390/ ?
This is an automated message from the Apache
rabbah opened a new pull request #3336: Remove dead field.
URL: https://github.com/apache/incubator-openwhisk/pull/3336
This is an automated message from the Apache Git Service.
To respond to the message, please log on
akrabat commented on issue #3335: Allow container tests to run without a proxy.
URL:
https://github.com/apache/incubator-openwhisk/pull/3335#issuecomment-368106033
I copied the changed `ActionContainer.scala` file to my clone of
incubator-openwhisk-runtime-php (Note that the package name
dubeejw commented on issue #3330: Update Log Store Fetch
URL:
https://github.com/apache/incubator-openwhisk/pull/3330#issuecomment-368100274
PG3 1908 ?
This is an automated message from the Apache Git Service.
To respond to
markusthoemmes closed pull request #3219: Kubernetes ContainerFactoryProvider
implementation
URL: https://github.com/apache/incubator-openwhisk/pull/3219
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
rabbah commented on issue #3335: Allow container tests to run without a proxy.
URL:
https://github.com/apache/incubator-openwhisk/pull/3335#issuecomment-368096873
@csantanapr so all the runtime repos cloned and owned this code :)
maybe the maven dep management being rolled out will
rabbah commented on issue #3335: Allow container tests to run without a proxy.
URL:
https://github.com/apache/incubator-openwhisk/pull/3335#issuecomment-368096873
@csantanapr so all the runtime repos cloned and owned this code :)
maybe the maven dep management being rolled out will
jonpspri opened a new pull request #229: Travis local cli
URL: https://github.com/apache/incubator-openwhisk-cli/pull/229
See https://github.com/apache/incubator-openwhisk/issues/3329
These changes to the travis build script should cause it to exercise the CLI
copy logic in the
rabbah commented on issue #3335: Allow container tests to run without a proxy.
URL:
https://github.com/apache/incubator-openwhisk/pull/3335#issuecomment-368096873
@csantanapr so all the runtime repos cloned and owned this code :)
kameshsampath commented on a change in pull request #97: Added maven archetype
for Java Action
URL:
https://github.com/apache/incubator-openwhisk-devtools/pull/97#discussion_r170316034
##
File path: java-action-archetype/README.md
##
@@ -0,0 +1,14 @@
+# Maven Archetype
kameshsampath commented on a change in pull request #97: Added maven archetype
for Java Action
URL:
https://github.com/apache/incubator-openwhisk-devtools/pull/97#discussion_r170315889
##
File path: java-action-archetype/README.md
##
@@ -0,0 +1,14 @@
+# Maven Archetype
jonpspri opened a new pull request #3334: Cli ansible fix
URL: https://github.com/apache/incubator-openwhisk/pull/3334
Changes to ansible build so that it will work appropriately for local CLI
build. Assumption is 'incubator-openwhisk-cli' is in the same directory as
csantanapr commented on issue #: Make parameters with defined values final
URL:
https://github.com/apache/incubator-openwhisk/pull/#issuecomment-368065978
PG?
This is an automated message from the Apache Git
dubeejw commented on issue #3330: Update Log Store Fetch
URL:
https://github.com/apache/incubator-openwhisk/pull/3330#issuecomment-368062365
@markusthoemmes, added a second commit that sends the HTTP request to the
log store instead of just the HTTP headers.
houshengbo commented on issue #3: Use Apache Tentacles to create a LICENSE and
NOTICE report
URL:
https://github.com/apache/incubator-openwhisk-release/issues/3#issuecomment-368062626
@daisy-ycguo I have tried the Tentacles to generate the report for our
staging directory, I found all
houshengbo commented on issue #3: Use Apache Tentacles to create a LICENSE and
NOTICE report
URL:
https://github.com/apache/incubator-openwhisk-release/issues/3#issuecomment-368062626
@daisy-ycguo I have tried the Tentacles to generate the report for our
staging directory, I found all
dubeejw commented on issue #3330: Update Log Store Fetch
URL:
https://github.com/apache/incubator-openwhisk/pull/3330#issuecomment-368062365
@markusthoemmes, added a second commit that send the HTTP request to the log
store instead of just the HTTP headers.
rabbah opened a new pull request #: Make parameters with defined values
final
URL: https://github.com/apache/incubator-openwhisk/pull/
Web actions can carry a final annotation which makes their parameters with
predefined values immutable. This patch makes the same behavior
ddragosd commented on a change in pull request #97: Added maven archetype for
Java Action
URL:
https://github.com/apache/incubator-openwhisk-devtools/pull/97#discussion_r170294203
##
File path: java-action-archetype/README.md
##
@@ -0,0 +1,14 @@
+# Maven Archetype for
ddragosd commented on a change in pull request #97: Added maven archetype for
Java Action
URL:
https://github.com/apache/incubator-openwhisk-devtools/pull/97#discussion_r170291220
##
File path: java-action-archetype/README.md
##
@@ -0,0 +1,14 @@
+# Maven Archetype for
lionelvillard commented on issue #98: ShortDesc name field is optional
URL:
https://github.com/apache/incubator-openwhisk-client-js/pull/98#issuecomment-368030566
At least for `packages.update` the name argument is at the top-level. The
one is `Package` is ignored.
gbdrt commented on issue #98: ShortDesc name field is optional
URL:
https://github.com/apache/incubator-openwhisk-client-js/pull/98#issuecomment-368025321
LGTM.
But what is the usecase where you don't need to provide a name?
csantanapr commented on a change in pull request #3332: Redo usage now requires
1+ components
URL:
https://github.com/apache/incubator-openwhisk/pull/3332#discussion_r170259864
##
File path: tools/build/redo
##
@@ -94,7 +94,7 @@ def getArgs():
jonpspri opened a new pull request #3332: Redo usage now requires 1+ components
URL: https://github.com/apache/incubator-openwhisk/pull/3332
Found this while testing for the CLI usage issue. Redo used to print a
'usage' with component showing as optional when it is in fact mandatory to
markusthoemmes closed pull request #3272: improve loadbalancer's schedule
algorithm
URL: https://github.com/apache/incubator-openwhisk/pull/3272
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
88 matches
Mail list logo