dubee closed pull request #258: Revert "Update the commit time for the the tag
"latest""
URL: https://github.com/apache/incubator-openwhisk-cli/pull/258
This is an automated message from the Apache Git Service.
To respond to
dubee opened a new pull request #258: Revert "Update the commit time for the
the tag "latest""
URL: https://github.com/apache/incubator-openwhisk-cli/pull/258
Reverts apache/incubator-openwhisk-cli#255
This is an automated me
sciabarracom commented on issue #2: Releasing the revised Golang runtime
URL:
https://github.com/apache/incubator-openwhisk-runtime-go/pull/2#issuecomment-375815024
Released the refactoring:
- reworked the "PipeExec" now called the Executor, consolidating all the
channels and go routine
dubee closed pull request #257: Switch to use Maven dependency
URL: https://github.com/apache/incubator-openwhisk-cli/pull/257
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
rabbah closed pull request #27: add webhook for slack notification of PR results
URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/27
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
rabbah closed pull request #22: add webhook for slack notification of PR results
URL: https://github.com/apache/incubator-openwhisk-runtime-docker/pull/22
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 provena
dgrove-oss opened a new pull request #12: add webhook for slack notification of
PR results
URL: https://github.com/apache/incubator-openwhisk-runtime-python/pull/12
This is an automated message from the Apache Git Service.
T
dgrove-oss opened a new pull request #22: add webhook for slack notification of
PR results
URL: https://github.com/apache/incubator-openwhisk-runtime-docker/pull/22
This is an automated message from the Apache Git Service.
T
dgrove-oss opened a new pull request #27: add webhook for slack notification of
PR results
URL: https://github.com/apache/incubator-openwhisk-runtime-java/pull/27
This is an automated message from the Apache Git Service.
To
dgrove-oss opened a new pull request #10: add webhook for slack notification of
PR results
URL: https://github.com/apache/incubator-openwhisk-runtime-php/pull/10
This is an automated message from the Apache Git Service.
To r
houshengbo opened a new pull request #257: Switch to use Maven dependency
URL: https://github.com/apache/incubator-openwhisk-cli/pull/257
This is an automated message from the Apache Git Service.
To respond to the message, pl
dubee closed pull request #255: Update the commit time for the the tag "latest"
URL: https://github.com/apache/incubator-openwhisk-cli/pull/255
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 th
KeonHee commented on a change in pull request #141: Add a membership management
using the discovery
URL:
https://github.com/apache/incubator-openwhisk-package-alarms/pull/141#discussion_r176825594
##
File path: provider/app.js
##
@@ -39,12 +41,76 @@ var monitoringInterval
tysonnorris commented on a change in pull request #3211: implement
ContainerFactory.cpuShare to fix #3110
URL:
https://github.com/apache/incubator-openwhisk/pull/3211#discussion_r176824320
##
File path:
core/invoker/src/main/scala/whisk/core/containerpool/docker/DockerContainerFac
tysonnorris commented on a change in pull request #3211: implement
ContainerFactory.cpuShare to fix #3110
URL:
https://github.com/apache/incubator-openwhisk/pull/3211#discussion_r176824102
##
File path: core/invoker/src/main/resources/application.conf
##
@@ -39,7 +49,6 @@
tysonnorris commented on a change in pull request #3211: implement
ContainerFactory.cpuShare to fix #3110
URL:
https://github.com/apache/incubator-openwhisk/pull/3211#discussion_r176824102
##
File path: core/invoker/src/main/resources/application.conf
##
@@ -39,7 +49,6 @@
dgrove-oss opened a new pull request #35: add webhook for slack notification
URL: https://github.com/apache/incubator-openwhisk-runtime-swift/pull/35
This is an automated message from the Apache Git Service.
To respond to the
dgrove-oss opened a new pull request #29: webhook for slack notification
URL: https://github.com/apache/incubator-openwhisk-runtime-nodejs/pull/29
This is an automated message from the Apache Git Service.
To respond to the me
rabbah closed pull request #263: update slack webhook to new service instance
URL: https://github.com/apache/incubator-openwhisk-catalog/pull/263
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
dgrove-oss opened a new pull request #284: add webhook for slack notification
of PR results
URL: https://github.com/apache/incubator-openwhisk-apigateway/pull/284
This is an automated message from the Apache Git Service.
To
dgrove-oss opened a new pull request #263: update slack webhook to new service
instance
URL: https://github.com/apache/incubator-openwhisk-catalog/pull/263
This is an automated message from the Apache Git Service.
To respond
dgrove-oss opened a new pull request #72: add webhook for slack notifications
of PR results
URL: https://github.com/apache/incubator-openwhisk-client-go/pull/72
This is an automated message from the Apache Git Service.
To re
dgrove-oss opened a new pull request #256: add webhook for slack notification
of PR test results
URL: https://github.com/apache/incubator-openwhisk-cli/pull/256
This is an automated message from the Apache Git Service.
To re
dgrove-oss opened a new pull request #817: add notification webhook for slack
notification of PR results.
URL: https://github.com/apache/incubator-openwhisk-wskdeploy/pull/817
This is an automated message from the Apache Git
rabbah closed pull request #167: Add webhook for slackbot notification of
TravisCI jobs
URL: https://github.com/apache/incubator-openwhisk-deploy-kube/pull/167
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
dgrove-oss commented on issue #167: Add webhook for slackbot notification of
TravisCI jobs
URL:
https://github.com/apache/incubator-openwhisk-deploy-kube/pull/167#issuecomment-375731923
@csantanapr -- please merge. encrypted URL to final version.
-
jasonpet commented on issue #141: Add a membership management using the
discovery
URL:
https://github.com/apache/incubator-openwhisk-package-alarms/pull/141#issuecomment-375719036
@KeonHee - I tried this out by putting in two different addresses for
DISCOVER_DESTINATIONS. However, both a
houshengbo opened a new pull request #69: Publish the binaries of openwhisk cli
and wskdeploy
URL: https://github.com/apache/incubator-openwhisk-release/pull/69
This PR adds the support to push the binaries of openwhisk cli and
wskdeploy into the staging url in Apache.
-
sven-lange-last commented on issue #3485: Action time limit test cleanup
URL:
https://github.com/apache/incubator-openwhisk/pull/3485#issuecomment-375653989
PG 2 / 2958 🔵.
PG 3 / 2047 completed unstable with a single failed test, i.e. the expected
test failed because there were inte
sven-lange-last commented on issue #3485: Action time limit test cleanup
URL:
https://github.com/apache/incubator-openwhisk/pull/3485#issuecomment-375653989
PG 2 / 2958 🔵.
PG 3 / 2047 completed unstable with a single failed test, i.e. the expected
test failed because there were inte
chetanmeh commented on issue #3482: Controller protocol should be https in
controller application.conf
URL:
https://github.com/apache/incubator-openwhisk/issues/3482#issuecomment-375673533
Yes. Pureconfig internally uses typesafe config which has a deterministic
[lookup order][1] where re
cbickel closed pull request #3480: Retry in feed tests to avoid race condition
URL: https://github.com/apache/incubator-openwhisk/pull/3480
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 i
houshengbo opened a new pull request #255: Update the commit time for the the
tag "latest"
URL: https://github.com/apache/incubator-openwhisk-cli/pull/255
Each time the commit is changed for the tag "latest", the commit
time remains to be the initial time when the tag is created for
t
rabbah commented on issue #3480: Retry in feed tests to avoid race condition
URL:
https://github.com/apache/incubator-openwhisk/pull/3480#issuecomment-375667962
In fact see https://github.com/apache/incubator-openwhisk/issues/3021 for
longer discusison and examples.
-
rabbah commented on issue #3480: Retry in feed tests to avoid race condition
URL:
https://github.com/apache/incubator-openwhisk/pull/3480#issuecomment-375667770
I think Mark’s point is that you may observe more of these failure in
general because the root cause is outlined in the reference
vvraskin commented on issue #3480: Retry in feed tests to avoid race condition
URL:
https://github.com/apache/incubator-openwhisk/pull/3480#issuecomment-37562
well, the test was correct for a deployment with a single controller :)
--
mdeuser commented on issue #3480: Retry in feed tests to avoid race condition
URL:
https://github.com/apache/incubator-openwhisk/pull/3480#issuecomment-375665489
@vvraskin - i understand your point, but if i understand correctly, the
reason the test failed was due to cross-controller cache
rabbah commented on issue #3032: Round-robin HA controller can lead to
concurrent modications errors
URL:
https://github.com/apache/incubator-openwhisk/issues/3032#issuecomment-375662902
Other solution suggested in
https://github.com/apache/incubator-openwhisk/issues/1815#issuecomment-277
vvraskin commented on issue #3480: Retry in feed tests to avoid race condition
URL:
https://github.com/apache/incubator-openwhisk/pull/3480#issuecomment-375662714
Agree that this failure should go away with the session stickiness or alike.
Given the change is there, I would expect a test t
mdeuser commented on issue #3480: Retry in feed tests to avoid race condition
URL:
https://github.com/apache/incubator-openwhisk/pull/3480#issuecomment-375658351
@vvraskin - it sounds like the fix for #3032 is the appropriate remedy for
the mainWhisk/8244 test failure reported by this issu
rabbah commented on issue #3480: Retry in feed tests to avoid race condition
URL:
https://github.com/apache/incubator-openwhisk/pull/3480#issuecomment-375656641
@mdeuser https://github.com/apache/incubator-openwhisk/issues/3032 the fact
there are multiple controllers shouldn’t really leak
mdeuser commented on issue #3480: Retry in feed tests to avoid race condition
URL:
https://github.com/apache/incubator-openwhisk/pull/3480#issuecomment-375655641
@vvraskin - is it possible that a customer might also experience similar
issues due the controller cache invalidation delays? i
sciabarracom commented on a change in pull request #2: Releasing the revised
Golang runtime
URL:
https://github.com/apache/incubator-openwhisk-runtime-go/pull/2#discussion_r176724615
##
File path: openwhisk/actionProxy.go
##
@@ -0,0 +1,97 @@
+/*
+ * Licensed to the Apache
rabbah commented on issue #187: support --count on wsk list
URL:
https://github.com/apache/incubator-openwhisk-cli/issues/187#issuecomment-375655114
Yes.
This is an automated message from the Apache Git Service.
To respond
mdeuser commented on issue #187: support --count on wsk list
URL:
https://github.com/apache/incubator-openwhisk-cli/issues/187#issuecomment-375654162
@rabbah - is this example still a valid reflection of the updates
> `GET /namespaces/_/activations?count=true` return `{"activations": N}
sven-lange-last commented on issue #3485: Action time limit test cleanup
URL:
https://github.com/apache/incubator-openwhisk/pull/3485#issuecomment-375653989
PG 2 / 2958 with latest changes running.
PG 3 / 2047 running as well - it will fail at test `Action limits should
reject creat
vvraskin commented on a change in pull request #2: Releasing the revised Golang
runtime
URL:
https://github.com/apache/incubator-openwhisk-runtime-go/pull/2#discussion_r176722339
##
File path: openwhisk/actionProxy.go
##
@@ -0,0 +1,97 @@
+/*
+ * Licensed to the Apache Sof
vvraskin commented on issue #3482: Controller protocol should be https in
controller application.conf
URL:
https://github.com/apache/incubator-openwhisk/issues/3482#issuecomment-375652151
So basically by moving the properties into reference.conf we assure that any
application.conf will ov
sven-lange-last commented on issue #3485: Action time limit test cleanup
URL:
https://github.com/apache/incubator-openwhisk/pull/3485#issuecomment-375599395
PG 2 / 2957 completed successfully.
This is an automated message fro
vvraskin commented on issue #3480: Retry in feed tests to avoid race condition
URL:
https://github.com/apache/incubator-openwhisk/pull/3480#issuecomment-375645892
PG1 2739 is green, @cbickel do you mind merging it?
This is an
rabbah commented on issue #3155: Support ?count on collections
URL:
https://github.com/apache/incubator-openwhisk/pull/3155#issuecomment-375644694
My oversight. We should update the swagger.
This is an automated message from
jthomas closed issue #108: can't use count when listing entities
URL: https://github.com/apache/incubator-openwhisk-client-js/issues/108
This is an automated message from the Apache Git Service.
To respond to the message, ple
jthomas closed pull request #109: add count to list of valid qs params
URL: https://github.com/apache/incubator-openwhisk-client-js/pull/109
This is an automated message from the Apache Git Service.
To respond to the message,
codecov-io commented on issue #109: add count to list of valid qs params
URL:
https://github.com/apache/incubator-openwhisk-client-js/pull/109#issuecomment-375612089
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk-client-js/pull/109?src=pr&el=h1)
Report
> Merging
[#109](h
codecov-io commented on issue #109: add count to list of valid qs params
URL:
https://github.com/apache/incubator-openwhisk-client-js/pull/109#issuecomment-375612089
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk-client-js/pull/109?src=pr&el=h1)
Report
> Merging
[#109](h
codecov-io commented on issue #109: add count to list of valid qs params
URL:
https://github.com/apache/incubator-openwhisk-client-js/pull/109#issuecomment-375612089
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk-client-js/pull/109?src=pr&el=h1)
Report
> Merging
[#109](h
csantanapr commented on issue #109: add count to list of valid qs params
URL:
https://github.com/apache/incubator-openwhisk-client-js/pull/109#issuecomment-375627914
@jthomas I updated README and unit tests
This is an automat
codecov-io commented on issue #109: add count to list of valid qs params
URL:
https://github.com/apache/incubator-openwhisk-client-js/pull/109#issuecomment-375612089
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk-client-js/pull/109?src=pr&el=h1)
Report
> Merging
[#109](h
codecov-io commented on issue #109: add count to list of valid qs params
URL:
https://github.com/apache/incubator-openwhisk-client-js/pull/109#issuecomment-375612089
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk-client-js/pull/109?src=pr&el=h1)
Report
> Merging
[#109](h
jthomas commented on issue #3155: Support ?count on collections
URL:
https://github.com/apache/incubator-openwhisk/pull/3155#issuecomment-375626186
Do we still keep the API swagger current with changes like these? For the OW
JS SDK, that file was super useful when implemeting the library.
jthomas commented on issue #109: add count to list of valid qs params
URL:
https://github.com/apache/incubator-openwhisk-client-js/pull/109#issuecomment-375623935
Unit tests for all the affected resources have a ["should list all X with
parameters"](https://github.com/apache/incubator-open
codecov-io commented on issue #109: add count to list of valid qs params
URL:
https://github.com/apache/incubator-openwhisk-client-js/pull/109#issuecomment-375612089
#
[Codecov](https://codecov.io/gh/apache/incubator-openwhisk-client-js/pull/109?src=pr&el=h1)
Report
> Merging
[#109](h
csantanapr opened a new pull request #109: add count to list of valid qs params
URL: https://github.com/apache/incubator-openwhisk-client-js/pull/109
Closes #108
This is an automated message from the Apache Git Service.
To re
sven-lange-last commented on issue #3485: Action time limit test cleanup
URL:
https://github.com/apache/incubator-openwhisk/pull/3485#issuecomment-375599467
@cbickel can you please also take a look?
This is an automated messa
sven-lange-last commented on issue #3485: Action time limit test cleanup
URL:
https://github.com/apache/incubator-openwhisk/pull/3485#issuecomment-375599395
PG 3 / 2046 running.
This is an automated message from the Apache Gi
sven-lange-last commented on issue #3461: Add action creation test with maximum
time limit
URL:
https://github.com/apache/incubator-openwhisk/pull/3461#issuecomment-375598590
Superseded by #3485. Closing this PR.
This is an
sven-lange-last closed pull request #3461: Add action creation test with
maximum time limit
URL: https://github.com/apache/incubator-openwhisk/pull/3461
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 provenan
sven-lange-last commented on issue #3485: Action time limit test cleanup
URL:
https://github.com/apache/incubator-openwhisk/pull/3485#issuecomment-375598387
@rabbah I already addressed all your comments from above. Please take a look
at the other changes.
-
sven-lange-last opened a new pull request #3485: Action time limit test cleanup
URL: https://github.com/apache/incubator-openwhisk/pull/3485
Improving tests around (maximum) action time limit
## Description
Major goals of this pull request:
* Some tests use a Node.js based a
69 matches
Mail list logo